Merge lp://staging/~sinzui/juju-ci-tools/gate-win into lp://staging/juju-ci-tools

Proposed by Curtis Hovey
Status: Merged
Merged at revision: 1508
Proposed branch: lp://staging/~sinzui/juju-ci-tools/gate-win
Merge into: lp://staging/juju-ci-tools
Diff against target: 246 lines (+128/-25)
4 files modified
gotesttarfile.py (+10/-5)
gotestwin.py (+25/-17)
tests/test_gotesttarfile.py (+25/-3)
tests/test_gotestwin.py (+68/-0)
To merge this branch: bzr merge lp://staging/~sinzui/juju-ci-tools/gate-win
Reviewer Review Type Date Requested Status
Martin Packman (community) Approve
Review via email: mp+300219@code.staging.launchpad.net

Description of the change

Run windows unit test from a local tarfile.

This branch gives us the option of running gated windows tests. There are two general changes.

1. We can pass a tarfile instead of a revision build number to gotestwin.py to test the revision we are gating. I added tests for this to ensure the old behaviour works.
2. I noticed that "go test -i ./..." wasn't called in gotesttarfile.py as it is in run-unit-test.

I have run this commands like this several times. I think we can recommend developers to test windows fixes like so:
    ../../juju-ci-tools/gotestwin.py \
        developer-win-unit-tester.vapour.ws ./juju-core_1.25.6.tar.gz github.com/juju/juju/cmd

To post a comment you must log in.
Revision history for this message
Martin Packman (gz) wrote :

Couple of minor things to fix and some misc comments.

review: Approve
1508. By Curtis Hovey

Revisions per review.

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