[Pkg-mozext-commits] [requestpolicy] 221/257: [tst][fix] multiple installations: close all tabs on tearDown

[David Prévot]

This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to branch master
in repository requestpolicy.

commit 00050b7b1ea9d90a0e0695e6fd4272b8549c6941
Author: Martin Kimmerle <dev at 256k.de>
Date:   Thu Dec 17 11:30:38 2015 +0100

    [tst][fix] multiple installations: close all tabs on tearDown
    
    Close all tabs after the browser restart in the tearDown
    function, not only notice tabs. (The tests shouldn't leak tabs.)
---
 .../tests/addon_install_and_upgrade/test_multiple_installations.py     | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/tests/marionette/tests/addon_install_and_upgrade/test_multiple_installations.py b/tests/marionette/tests/addon_install_and_upgrade/test_multiple_installations.py
index 524dab4..9b144ec 100644
--- a/tests/marionette/tests/addon_install_and_upgrade/test_multiple_installations.py
+++ b/tests/marionette/tests/addon_install_and_upgrade/test_multiple_installations.py
@@ -41,7 +41,8 @@ class MultipleInstallationsTestCase(RequestPolicyTestCase):
             # might have broken RequestPolicy's functionality.
             self.restart()
         finally:
-            self._close_notice_tabs()
+            self.browser.tabbar.close_all_tabs(
+                exceptions=[self.browser.tabbar.tabs[0]])
             self.rp_addon.ignore_errors = False
 
             # It's highly probable that errors occur. However, the tests

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-mozext/requestpolicy.git