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

Update test json with lotIdentifier #24

Merged

Conversation

oleksiyVeretiuk
Copy link
Contributor

@oleksiyVeretiuk oleksiyVeretiuk commented Nov 27, 2017

тести не проходили через те, що по-перше порядок елементів в списках result й expected_result не співпадав, а також через те що дані в expected_result не мали ключа "value" заповненого даними для тесту.
Виправлено тут a77ad5b

@yarsanich
Copy link
Collaborator

Тести не проходять.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 70.823% when pulling a77ad5b on oleksiyVeretiuk:483469047886510_update_test_data into 2092fd1 on openprocurement:master.

# Make sorting because get_revision_changes can return
# proper results but ordering will be differ from expected result
# and test will fail
result = sorted(result, key=lambda r: r['path'])
Copy link
Collaborator

Choose a reason for hiding this comment

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

краще .sort() швидше буде виконуватись

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 3870f8d on oleksiyVeretiuk:483469047886510_update_test_data into ** on openprocurement:master**.

@leits leits merged commit dc970ca into openprocurement:master Dec 4, 2017
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

4 participants