Skip to content
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

Corrección de properties medline_title, medline_code en exportación d… #1280

Merged
merged 1 commit into from May 13, 2016

Conversation

swarzesherz
Copy link
Contributor

…e metadatos

@@ -437,11 +437,11 @@ def test_study_area_empty_queryset(self):
journalstd = self._makeOne(dummy_journal, dummy_issue)
self.assertEqual(journalstd.study_area, '')

def test_medline_title_is_the_journal_title(self):
Copy link
Collaborator

Choose a reason for hiding this comment

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

@robertatakenaka, você consegue confirmar que as asserções descritas nos casos de teste são falsas?
Em especial: test_medline_title_is_the_journal_title e test_medline_code_must_always_be_empty

Copy link
Member

Choose a reason for hiding this comment

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

@gustavofonseca : test_medline_title_is_the_journal_title deve ser falso, pois o título Medline é um título abreviado sempre, a menos que eles mudem a regra, e no journal title é o título completo.
test_medline_code_must_always_be_empty também não é verdadeiro, pois pode ter conteúdo. Este formulário foi feito em 1997 e baseado na LILACS e naquela época pode ser que existisse Medline Code para todos os periódicos. Consultando a base title, não existe Medline Code para todos os títulos Medline, então suponho que este campo deixou de ter função ou nunca teve no SciELO.

Copy link
Collaborator

Choose a reason for hiding this comment

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

@robertatakenaka, ok, valeu!

@gustavofonseca gustavofonseca merged commit 2a2f6c3 into scieloorg:beta May 13, 2016
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.

None yet

3 participants