Merge lp:~sileht/testscenarios/testscenarios into lp:~testtools-committers/testscenarios/trunk

[Mehdi Abaakouk]

[Zygmunt Krynicki (zyga)]

Apart from pointless whitespace changes, this looks good to me

[lp:~sileht/testscenarios/testscenarios]

22. By Mehdi Abaakouk on 2014-03-14

Allow use scenario with nose

This patch informs nose that test generated and orignal test are the same.

Closes bug: #872887

[Mehdi Abaakouk (sileht)]

I have cleaned the commit.

[Victor Stinner (vstinner)]

What is the status of this issue? I still need it for a patch on OpenStack Oslo Incubator:
https://review.openstack.org/#/c/80534/2/tests/unit/test_gettext.py

[Zygmunt Krynicki (zyga)]

Any of the people listed here can commit this: https://launchpad.net/~testtools-committers (I cannot)

[Chmouel Boudjnah (chmouel)]

I'd love to have that!

[Robert Collins (lifeless)]

What does this do precisely? The idea that they are the same test case seems wrong, since they *were* but are *no longer* identical.

review: Needs Information

[Robert Collins (lifeless)]

This still needs the question asked answered

[Robert Collins (lifeless)]

Also the source for testscenarios is now ongithub, please submit a PR there.

Unmerged revisions

22. By Mehdi Abaakouk on 2014-03-14

Allow use scenario with nose

This patch informs nose that test generated and orignal test are the same.

Closes bug: #872887

=== modified file 'lib/testscenarios/scenarios.py'
--- lib/testscenarios/scenarios.py	2012-04-04 10:02:46 +0000
+++ lib/testscenarios/scenarios.py	2014-03-14 14:18:40 +0000
@@ -52,6 +52,8 @@
         newtest.shortDescription = (lambda: newtest_desc)
     for key, value in parameters.iteritems():
         setattr(newtest, key, value)
+    # Tell nose that test and newtest are the same testcase
+    setattr(test, '_nose_case', newtest)
     return newtest