Prior to always generating secondary instances, this scenario would have not output the duplicate choices, even if allowed. Now, itext are identified by choice position so it's not a problem. This test adds to the existing suite by using translations with the duplicates.
Why is this the best possible solution? Were any other approaches considered?
Since no code changes are needed, this test is just to improve coverage and verify the fix.
What are the regression risks?
None
Does this change require updates to documentation? If so, please file an issue here and include the link below.
No
Before submitting this PR, please make sure you have:
[x] included test cases for core behavior and edge cases in tests
[x] run nosetests and verified all tests pass
[x] run black pyxform tests to format code
[x] verified that any code or assets from external sources are properly credited in comments
Closes #585
Prior to always generating secondary instances, this scenario would have not output the duplicate choices, even if allowed. Now, itext are identified by choice position so it's not a problem. This test adds to the existing suite by using translations with the duplicates.
Why is this the best possible solution? Were any other approaches considered?
Since no code changes are needed, this test is just to improve coverage and verify the fix.
What are the regression risks?
None
Does this change require updates to documentation? If so, please file an issue here and include the link below.
No
Before submitting this PR, please make sure you have:
testsnosetestsand verified all tests passblack pyxform teststo format code