test_versions.py is currently breaking pep8 in master

[Henry Nash]

Somehow a set of bad aligned '}' has got into master in test_versions.py, which is causing every patch to fail. This fixes it.

[Henry Nash (henry-nash)]

So it seems that there is a formatting issue, but it is more subtle that it fails in some (maybe older?) versions of flake, but no long the one being run by Jenkins. Should still try and fix it so it passes in all cases.

[Lance Bragstad (lbragstad)]

Henry,

do you have some logs where this is happening?

[Lance Bragstad (lbragstad)]

tried recreating this pep8 1.5.0+ and flake8 1.7.0+ and wasn't able to recreate

[Henry Nash (henry-nash)]

Here's what I get:

./keystone/tests/test_versions.py:320:5: E123 closing bracket does not match indentation of opening bracket's line
    }),
    ^
./keystone/tests/test_versions.py:331:5: E123 closing bracket does not match indentation of opening bracket's line
    }),
    ^
./keystone/tests/test_versions.py:341:5: E123 closing bracket does not match indentation of opening bracket's line
    }),
    ^
./keystone/tests/test_versions.py:351:5: E123 closing bracket does not match indentation of opening bracket's line
    }),
    ^

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/118894

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/118894
Committed: https://git.openstack.org/cgit/openstack/keystone/commit/?id=5157bc62bf3b7aa3792f8148c42bc7213b607068
Submitter: Jenkins
Branch: master

commit 5157bc62bf3b7aa3792f8148c42bc7213b607068
Author: Dolph Mathews <email address hidden>
Date: Wed Sep 3 15:53:20 2014 -0500

    use one indentation style

    There are two (or three, depending on how you count) indentations styles
    mixed together in this one logical line of code, which are further mixed
    with two methods of closing parens and curly braces.

    Besides being difficult to read, the mixture resulted in an off-by-one
    error that isn't caught by the version of hacking that Jenkins is using,
    but is caught by the latest.

    Change-Id: Idab3c80c720e0bfd5741dc9e07f841b5ac270960
    Closes-Bug: 1363019

[OpenStack Infra (hudson-openstack)]

Change abandoned by Dolph Mathews (<email address hidden>) on branch: master
Review: https://review.openstack.org/117732
Reason: fixed in https://review.openstack.org/#/c/118894/