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,  <[email protected]> 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
> [email protected]
>
> 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

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

This is the relax-devel mailing list
[email protected]

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-devel

Reply via email to