Skip to content

Commit e30b87e

Browse files
committed
Gadgets/Stdrev: Reuse same browser instance for all tests
1 parent b7dc30b commit e30b87e

File tree

1 file changed

+16
-9
lines changed
  • gadgets/standard_revisions-tests

1 file changed

+16
-9
lines changed

gadgets/standard_revisions-tests/base.py

Lines changed: 16 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -25,16 +25,14 @@
2525
from selenium.common.exceptions import NoAlertPresentException
2626
import unittest, time, re
2727

28-
class CppTestCase(unittest.TestCase):
29-
@classmethod
30-
def setUpClass(self):
31-
self.base_url = "http://en.cppreference.com/"
32-
28+
class Driver:
29+
def __init__(self):
30+
base_url = "http://en.cppreference.com/"
3331
driver = webdriver.Firefox()
3432
driver.implicitly_wait(30)
3533
try:
36-
driver.get(self.base_url + "/mwiki/index.php?title=Special:UserLogout&returnto=Main+Page")
37-
driver.get(self.base_url + "/mwiki/index.php?title=Special:UserLogin&returnto=Main+Page")
34+
driver.get(base_url + "/mwiki/index.php?title=Special:UserLogout&returnto=Main+Page")
35+
driver.get(base_url + "/mwiki/index.php?title=Special:UserLogin&returnto=Main+Page")
3836
driver.find_element_by_id("wpName1").clear()
3937
driver.find_element_by_id("wpName1").send_keys("test5")
4038
driver.find_element_by_id("wpPassword1").clear()
@@ -46,11 +44,20 @@ def setUpClass(self):
4644
driver.quit()
4745
raise
4846
self.driver = driver
47+
self.base_url = base_url
4948

50-
@classmethod
51-
def tearDownClass(self):
49+
50+
def __del__(self):
5251
self.driver.quit()
5352

53+
driver_instance = Driver()
54+
55+
class CppTestCase(unittest.TestCase):
56+
@classmethod
57+
def setUpClass(self):
58+
self.base_url = driver_instance.base_url
59+
self.driver = driver_instance.driver
60+
5461
def get_page(self, title):
5562
self.driver.get(self.base_url + "/w/" + title)
5663

0 commit comments

Comments
 (0)