mailRe: r21927 - /trunk/test_suite/gui_tests/relax_disp.py


Others Months | Index by Date | Thread Index
>>   [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Header


Content

Posted by Edward d'Auvergne on December 10, 2013 - 11:23:
Hi Troels,

Please take a note of this change.  It fixes the GUI tests in relax,
as the Relax_disp.test_bug_21076_multi_col_peak_list GUI test you
introduced broke subsequent GUI tests.  Note that each GUI test must
close or terminate each GUI element that it uses.  This concept does
not exist for the system or unit tests.  The tearDown() method for the
GUI tests cannot detect which GUI elements are open and automatically
close them, unfortunately.  This may change in the future but, for
now, you need to keep track of what a GUI test opens while you are
writing it.

Cheers,

Edward



On 10 December 2013 11:17,  <edward@xxxxxxxxxxxxx> wrote:
Author: bugman
Date: Tue Dec 10 11:17:10 2013
New Revision: 21927

URL: http://svn.gna.org/viewcvs/relax?rev=21927&view=rev
Log:
Fix for the Relax_disp.test_bug_21076_multi_col_peak_list GUI test.

The peak intensity wizard is now closed at the end of the test so that 
subsequent tests can cleanly
operate.  Without closing this wizard, launching it a second time in 
another test will always fail.


Modified:
    trunk/test_suite/gui_tests/relax_disp.py

Modified: trunk/test_suite/gui_tests/relax_disp.py
URL: 
http://svn.gna.org/viewcvs/relax/trunk/test_suite/gui_tests/relax_disp.py?rev=21927&r1=21926&r2=21927&view=diff
==============================================================================
--- trunk/test_suite/gui_tests/relax_disp.py (original)
+++ trunk/test_suite/gui_tests/relax_disp.py Tue Dec 10 11:17:10 2013
@@ -201,6 +201,9 @@

         # Now check that the first is set to 'Z_A0', since the keyword 
'auto' was used for spectrum_id.
         self.assertEqual(cur_id, 'Z_A0')
+
+        # Finally close the wizard to allow subsequent tests to be able to 
operate cleanly.
+        wizard.Close()


     def test_hansen_trunc_data(self):


_______________________________________________
relax (http://www.nmr-relax.com)

This is the relax-commits mailing list
relax-commits@xxxxxxx

To unsubscribe from this list, get a password
reminder, or change your subscription options,
visit the list information page at
https://mail.gna.org/listinfo/relax-commits



Related Messages


Powered by MHonArc, Updated Tue Dec 10 14:00:07 2013