-
Notifications
You must be signed in to change notification settings - Fork 0
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
Refactor manufacturer tests #313
Conversation
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.
Only looked through the repo tests so far, as I went through I made changes to my own branch as well (66c0248) so in theory they should all be very similar now. Just a couple of minor comments for things that could be done here.
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.
Just going through the service tests, I made corresponding changes in 361b7a8.
23b8793
to
6fd92b4
Compare
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.
Just some very minor picky comments, feel free to ignore, just though it was worth being consistent while we are here.
6fd92b4
to
38edd54
Compare
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.
Just to address #313 (comment)
Co-authored-by: Joel Davies <90245114+joelvdavies@users.noreply.github.com>
Description
Refactors the manufacturer tests in the same format as in #302 as part of #90.
Removed the tests:
test_partial_update_address
(repo unit and e2e) as the standard update is enoughtest_get_with_non_existent_id
as not neededTesting instructions
Agile board tracking
closes #312