Merge lp:~wallyworld/launchpad/windmill-1.3r1544 into lp:launchpad

[Ian Booth]

Upgrade windmill from r1440 to r1544 to fix an issue preventing some new launchpad tests from running

See bug 652626

The version of windmill packaged with launchpad, currently 1.3r1440, has a bug which prevents some newly written windmill tests from running. The windmill bug results in code like the following:

client.asserts.assertProperty(id=u"field.review_type", validator='disabled|false')

failing with an array index error.

The windmill trunk has already been updated to fix this bug. A new windmill package has been built from the tip revision of the windmill trunk (r1544). The package is windmill-1.3r1544.tar.gz.

Implementation

  This branch updates the buildout config so the new windmill package is used.

Tests

  The tests from branch lp:~wallyworld/launchpad/reviews-without-reviewer which failed with the older version of
  windmill were re-run and they now pass.

  bin/test -vvt TestRequestReview

  A selection of other windmill tests were also run - regression testing.

[Tim Penhey (thumper)]

Does the new r1544 package have the changes that were in Bjorn's branch?

[Ian Booth (wallyworld)]

Yes. I checked it out and his changes actually got pushed to the
windmill trunk on github. And of course the lp mirror I pulled from to
build the egg has the changes also since the mirrored branch location
was corrected. So no need to merge anything.

On 01/10/10 12:55, Tim Penhey wrote:
> Does the new r1544 package have the changes that were in Bjorn's branch?

[Tim Penhey (thumper)]

Awesome

review: Approve

=== modified file 'versions.cfg'
--- versions.cfg	2010-09-18 08:00:27 +0000
+++ versions.cfg	2010-10-01 02:38:54 +0000
@@ -76,10 +76,11 @@
 van.testing = 2.0.1
 wadllib = 1.1.5
 webunit = 1.3.8
-# r1440 of lp:~bjornt/windmill/1.3-lp. It includes our patches to make test
-# setup and tear down more robust, which didn't make it into the 1.3 release.
-# Windmill 1.5 will include our patches.
-windmill = 1.3beta3-lp-r1440
+# r1544 of lp:windmill (the tip revision at the time of packaging).
+# We need to use this revison rather than the official 1.3 release since 
+# there is a bug fix for WindmillTestClient.asserts.assertProperty() which 
+# makes this api call work property on html disabled and readonly attributes.
+windmill = 1.3r1544
 wsgi-fileserver = 0.2.7
 wsgi-intercept = 0.4
 wsgi-jsonrpc = 0.2.8