Merge lp://staging/~gz/bzr-builddeb/merge_changelog_invalid_version_tweak_testing into lp://staging/bzr-builddeb

Proposed by Martin Packman
Status: Superseded
Proposed branch: lp://staging/~gz/bzr-builddeb/merge_changelog_invalid_version_tweak_testing
Merge into: lp://staging/bzr-builddeb
Diff against target: 175 lines (+68/-13) (has conflicts)
1 file modified
tests/test_merge_changelog.py (+68/-13)
Text conflict in tests/test_merge_changelog.py
To merge this branch: bzr merge lp://staging/~gz/bzr-builddeb/merge_changelog_invalid_version_tweak_testing
Reviewer Review Type Date Requested Status
James Westby Approve
Review via email: mp+85638@code.staging.launchpad.net

This proposal has been superseded by a proposal from 2011-12-14.

Description of the change

Follow up to previous mp, tidying up the merge_changelog testing a little:

<https://code.launchpad.net/~gz/bzr-builddeb/merge_changelog_invalid_version_893495/+merge/85572>

Original way of spelling the test meant it would fail when selftest was run with LANG=de_DE.UTF-8 for example. Have rearranged things a bit to make it easy to add another test for the original style of invalid version as well.

To post a comment you must log in.
669. By Martin Packman

Slightly strengthen log output assertion on error

670. By Martin Packman

Merge trunk to resolve conflict in test_merge_changelog.py

Revision history for this message
James Westby (james-w) wrote :

Hi,

Thanks for cleaning this up. It looks good to me, assuming the
conflicts are easily resolvable.

Thanks,

James

review: Approve

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches