Skip to content

Conversation

@Kamillakrdoso
Copy link

Atividade 4

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

aqui deveria ter também outros /#if, certo?

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sim, desde que o modelo de features seja atualizado (o que foi marcado aqui foi com base nele). Mas se for para corrigir logo, tudo bem. Atualmente há o problema com a feature de Funder, que tem relacionamento com ResearchProject e também a feature de Member. Eu não tinha visão de tais relacionamentos antes e, infelizmente, não foi possível resolver isso à tempo da entrega.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, pode deixar assim.

thaisabr added 5 commits July 5, 2014 12:20
…e métodos para melhorar a leitura do código em XMLImportSteps, ajuste de código repetido.
…ivo, reorganização de asserts em alguns steps, correção de bug referente ao acesso ao atributo publicationDate das publicações importadas
Kamillakrdoso and others added 30 commits July 30, 2014 13:37
…serem importadas a partir do arquivo selecionado.
…dância das features TestDataAndOperations, definição de um padrão para as features do tipo TestDataAndOperations (NomeFeatureTestDataAndOperations)
Conflicts:
	test/cucumber/steps/ArticleSteps.groovy
…tidos; quebra de método em métodos menores para facilitar o entendimento. Correção de problema na verificação de total de publicações existentes.
…ovy, TeseOrDissertacao.groovy e Ferramenta.groovy
Nome do método teve que ser modificado para tornar a compreensão do
código mais claro.
…ortStep, e definindo alguns métodos de checagem no proprio step.
…ortStep, e definindo alguns métodos de checagem no proprio step.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants