Unit test for TranslationBranchApprover not very easy to understand
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Fix Released
|
Medium
|
Henning Eggers |
Bug Description
The unit test TestTranslation
These methods should be split up so that the steps to set up each individual test method become clearer. For example, the reader of test_replace_
1. A potemplate entry is created in the database with domain A and path YB
2. An upload is simulated by putting a file with path C into the import queue.
3. The approver is created with path D as its parameter.
Changed in rosetta: | |
importance: | Undecided → Medium |
status: | New → Triaged |
tags: | added: cleanup |
Changed in rosetta: | |
milestone: | none → 2.2.4 |
Changed in rosetta: | |
status: | Fix Committed → Fix Released |
Fixing this while implementing "import translatoins from branches".