-
Notifications
You must be signed in to change notification settings - Fork 75
Equipe 3 - Atividade 4 #307
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
Kamillakrdoso
wants to merge
56
commits into
spgroup:master
Choose a base branch
from
thaisabr:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
56 commits
Select commit
Hold shift + click to select a range
75e9c4e
Implementação de parte dos testes da feature XMLImport, incluindo tam…
thaisabr 54c7368
Exclusão de métodos desnecessários, que foram indevidamente incluídos.
thaisabr 9601eb2
Atividade 4
Kamillakrdoso ed1e75a
Correções da atividade 4: Ajuste na escrita dos cenários, definição d…
thaisabr 59e653b
Atividade 5: Correção do código para os testes implementados por Thaí…
thaisabr 8d3dbfb
Ajuste em código referente à feature Funder
thaisabr d347cca
Ajuste de testes de XMLImport: inclusão de assert após upload de arqu…
thaisabr 96d93d6
Correção no tratamento ao atributo "members" das publicações importadas.
thaisabr a7cbf4a
Correção em mensagem usada no título da tela.
thaisabr 58498e1
Mudança para tornar opcional atributos obrigatórios que não existem n…
thaisabr a50fd9d
Desenvolvimento da feature XMLImport e consequente atualização dos do…
thaisabr f566959
Exclusão de imports adicionados indevidamente.
thaisabr 07d81d7
Correção das diretivas de pré-processamento e da lógica de definição …
thaisabr c81ba74
Atividade 5
Kamillakrdoso ced452c
Atividade 5
Kamillakrdoso 4cb0198
Correção na persistência de publicações importadas no geral e nos dad…
thaisabr b599f33
Ajustes quanto ao uso de internacionalização em XMLImport.
thaisabr dab2010
Refatoração: Renomeação de métodos
thaisabr 7f55a3e
Correção para salvar publicações importadas em telas específicas do t…
thaisabr c131a69
Refatoração: Renomeação de método de savePublication para createPubli…
thaisabr 5ecef4b
Ajuste em label de botão
thaisabr 3a36120
Correção: Exclusão de etapa manual para execução de testes através da…
thaisabr 43e4e7f
Exclusão de chamadas de println
thaisabr fcff6c8
Refatoração: Inclusão da feature Funder e exclusão de alterações inde…
thaisabr 92fbef3
Inclusão da feature Funder
thaisabr f97e7f2
Identificação de artefatos relacionados à feature XMLImport.
thaisabr 0842010
Atividade 5 - Correção relacionadas a duplicação e modularidade
Kamillakrdoso 0821da3
Alterações e ajustes decorrentes de atualizações no repositório central.
thaisabr 718540e
Alterações e ajustes decorrentes de atualizações no repositório central.
thaisabr 0e471d8
Inclusão de mensagem de alerta no caso de não haver publicações para …
thaisabr 0dbf706
Inclusão de mensagem de XMLImport.
thaisabr 785900b
Refatoração: Exclusão de método redundante e limpeza de código simples.
thaisabr edc1091
Refatoração: Reuso de código através de chamada de método; remoção de…
thaisabr 270fc95
Refatoração: Exclusão de métodos redundantes; inclusão de parâmetros …
thaisabr 493c9d4
Ajuste no uso de diretivas de pré-processamento.
thaisabr ddc8365
Correção em importação de research project (ainda há problema)
thaisabr cfd51aa
Correção simples em mensagem de alerta ao importar publicação.
thaisabr 823e437
Refatoração: excluindo ResearchTestDataAndOperations, removendo redun…
Kamillakrdoso c531873
Merge remote-tracking branch 'origin/master'
Kamillakrdoso 85fa255
Ajuste por conta de refatoração realizada por Kamilla: exclusão de co…
thaisabr bf9c560
Refatoração: Criação de método para encapsular trechos de código repe…
thaisabr c5293f9
Correção em refatoração realizada por Kamilla: comando import inválido.
thaisabr feb97ae
Ajuste em refatoração realizada por Kamilla
thaisabr f288f39
Refatoração: Extração de método para evitar replicação de código.
thaisabr f022303
Ajuste em alterações realizadas por Kamilla.
thaisabr db92292
Correção em testes devido às mudanças realizadas em XMLController.gro…
thaisabr 81a64ff
Correção Metodo de Busca Linha de Pesquisa por Member
Kamillakrdoso 49b667d
Correção: removendo redundância
Kamillakrdoso 5443388
Correção: Cenário web, upload de arquivo XML invalido (upload de arqu…
Kamillakrdoso 4827477
Correção: metodo configureFileName(filename), e aumentando o escopo d…
Kamillakrdoso fb8bb71
Refatoração: nome método
Kamillakrdoso d8e1a06
Correção da refatoração nome método
Kamillakrdoso 52a2095
Refatoração: removendo métodos redundantes
Kamillakrdoso 1f496bd
Correção: removendo ResearchLineController de alguns passos de XMLImp…
Kamillakrdoso 14cc233
Correção: removendo ResearchLineController de alguns passos de XMLImp…
Kamillakrdoso 088b972
Correção: cenário research line
Kamillakrdoso File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -60,8 +60,6 @@ class ResearchLineController { | |
| [researchLineInstanceList: lista] | ||
|
|
||
| } | ||
|
|
||
|
|
||
| } | ||
| //#end | ||
|
|
||
|
|
@@ -238,5 +236,80 @@ class ResearchLineController { | |
|
|
||
| researchLineInstance | ||
| } | ||
|
|
||
| def findAllResearchLine(){ | ||
| HashMap<String,String> lista = new HashMap<String, String>() | ||
| for(researchline in ResearchLine.getAll()) | ||
| { | ||
| if(!researchline.getDescription().equals("stable")) { | ||
| researchline.setDescription("stable") | ||
| } | ||
| lista.put(researchline.getName(),researchline.getDescription()) | ||
| } | ||
| [researchLineInstanceList: lista] | ||
| } | ||
|
|
||
| def findResearchByMember(member){ | ||
| ArrayList<String> lista = new ArrayList<String>() | ||
| Member actor = new Member() | ||
| for(research in ResearchLine.getAll()) | ||
| { | ||
| for(currentMember in research.getMembers()){ | ||
| if(member.equals(currentMember.getName())){ | ||
| actor = currentMember | ||
| } | ||
| } | ||
| } | ||
| return actor.getId() | ||
| } | ||
|
|
||
| def findAllResearchByMember(member, research){ | ||
| HashMap<String, String> listagem = new HashMap<String, String>() | ||
| for(researchL in ResearchLine.getAll()){ | ||
| if((researchL.getName().equals(research)) && (researchL.getMembers().contains(member))){ | ||
| listagem.put(member, research) | ||
| } | ||
| } | ||
| return listagem | ||
| } | ||
|
|
||
| def checkIfResearchLineExists(researchName, list){ | ||
| for (research in list) { | ||
| if (research.getName().equals(researchName)) { | ||
| return true | ||
| } | ||
| } | ||
| return false | ||
| } | ||
|
|
||
| def checkSavedResearchByDescription(nameOfResearch, status){ | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. problemas similares aos acima. reescrever este e o próximo método |
||
| HashMap<String,String> lista = findAllResearchLine() | ||
| for(int i; i< lista.size(); i++){ | ||
|
|
||
| if(lista.containsKey(nameOfResearch)) { | ||
| if(lista.containsValue(status)){ | ||
|
|
||
| return true | ||
| } | ||
| } | ||
| } | ||
| return false | ||
|
|
||
| } | ||
|
|
||
| def checkDeletedResearchByDescription(nameOfResearch, status){ | ||
| List<ResearchLine> lista = ResearchLine.findAll() | ||
| boolean exist = checkSavedResearchByDescription(nameOfResearch, status) | ||
| for(research in lista){ | ||
| if((exist) && (research.getName().equals(nameOfResearch))){ | ||
| if(research.getDescription().equals(status)){ | ||
| lista.remove(research) | ||
| return true | ||
| } | ||
|
|
||
| } | ||
| } | ||
| return false | ||
| } | ||
| } | ||
| //#end | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
todas estas funções parecem não pertencer a um controller. elas não são ações de um controller. deve ter algum lugar mais apropriado para elas.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Neste caso teria que adicionar todos os métodos de checagem e verificação,
para testDataAndOperations de ResearchLine?
2014-07-23 15:31 GMT-03:00 Paulo Borba notifications@github.com:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alguns devem ficar em testDataAndOperations, outros talvez em Pages, ou em arquivos de steps. Tem que ser analisado caso a caso. Tome como base o código implementado em features como Article.
On Jul 29, 2014, at 8:41 PM, Kamilla Cardoso notifications@github.com wrote: