Skip to content

Conversation

@zannkukai
Copy link
Contributor

The acq_account API could return some translated messages. As these
messages could be tested we need to ensure that no translation is done
during unitests. Now we mock the translation function to return the same
value than input value.

Co-Authored-by: Renaud Michotte renaud.michotte@gmail.com

Code review check list

  • Commit message template compliance.
  • Commit message without typos.
  • File names.
  • Functions names.
  • Functions docstrings.
  • Unnecessary commited files?
  • Cypress tests successful?

The `acq_account` API could return some translated messages. As these
messages could be tested we need to ensure that no translation is done
during unitests. Now we mock the translation function to return the same
value than input value.

Co-Authored-by: Renaud Michotte <renaud.michotte@gmail.com>
@zannkukai zannkukai self-assigned this Apr 7, 2022
@zannkukai zannkukai merged commit e92c579 into rero:staging Apr 7, 2022
@zannkukai zannkukai deleted the zan-fix-acq-test branch April 7, 2022 11:38
@PascalRepond PascalRepond added the bug Breaks something but is not blocking label Apr 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Breaks something but is not blocking

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants