繰り返しグローバル変数とし、Python単体テスト以外で試してみよう

Question

私はPython Seleniumを使い始めて、以下に示すスクリプトを書いています。

これは、失敗したテスト（test01、test02、test03、..）にリンクされたリターンコードを出力します。

各テストで同じことがチェックされているのを無視します。

私は、それぞれがglobal resを繰り返し宣言し、次にtry/exceptというブロックを持っているので、よりクリーンな方法でテストを書くかどうかを理解しようとしています。

誰でもこの改善方法に関するアドバイスをいただけますか？

# global variable for return code. Zero is success. 
res=0 

@atexit.register 
def send_health(): 
    print ("res=%s") % res 

class Login(unittest2.TestCase): 
    @classmethod 
    def setUpClass(inst): 
     binary = FirefoxBinary('C:\\Program Files (x86)\\Mozilla Firefox\\firefox.exe') 
     inst.driver = webdriver.Firefox(firefox_binary=binary) 
     inst.base_url = "https://stackoverflow.com" 

    def test01(self): 
     global res 
     driver = self.driver 
     try: 
      self.assertEqual("Ask a Question", driver.title) 
     except Exception,e: 
      print ("Exception: %s" % e) 
      driver.save_screenshot('screenshot_test01.png') 
      res=1 
      return 

    def test02(self): 
     global res 
     driver = self.driver 
     try: 
      self.assertEqual("Ask a Question", driver.title) 
     except Exception,e: 
      print ("Exception: %s" % e) 
      driver.save_screenshot('screenshot_test02.png') 
      res=2 
      return 

    def test03(self): 
     global res 
     driver = self.driver 
     try: 
      self.assertEqual("Ask a Question", driver.title) 
     except Exception,e: 
      print ("Exception: %s" % e) 
      driver.save_screenshot('screenshot_test03.png') 
      res=3 
      return 

if __name__ == "__main__": 
    unittest2.main() 

Answer 1

上記のヒントをありがとう。私はコードをこのバージョンにまで縮小しましたが、これはうまくいけば清潔でより標準的です。

class Login(unittest2.TestCase): 

    @classmethod 
    def handleError(self, e, res): 
     print ("Test failed with exception: %s" % e) 
     self.result = res 
     sys.exit() 

    @classmethod 
    def setUpClass(self): 
     binary = FirefoxBinary('C:\\Program Files (x86)\\Mozilla Firefox\\firefox.exe') 
     self.driver = webdriver.Firefox(firefox_binary=binary) 
     self.base_url = "https://stackoverflow.com" 
     self.result = 0 

    def test01(self): 
     driver = self.driver 
     try: 
      self.assertEqual("Ask a Question", driver.title) 
     except Exception,e: 
      self.handleError(e, 1) 

    def test02(self): 
     driver = self.driver 
     try: 
      self.assertEqual("Ask a Question", driver.title) 
     except Exception,e: 
      self.handleError(e, 2) 

    def test03(self): 
     driver = self.driver 
     try: 
      self.assertEqual("Ask a Question", driver.title) 
     except Exception,e: 
      self.handleError(e, 3) 

    @classmethod 
    def tearDownClass(self): 
     self.driver.quit() 
     print ("res=%s") % self.result 

if __name__ == "__main__": 
    unittest2.main() 

Answer 2

ここではグローバル変数はまったく必要ありません。あなたはクラスの中にいます。全体を通してself.resを使用してください。

Answer 3

これはまさにsetUpインスタンスメソッドのためのもので、テストクラスごとに1回実行されるsetUpClassメソッドと非常によく似ています。

def setUp(self): 
    # this code will be executed before each and every test 
    self.res = 0 

def tearDown(self): 
    # this code will be executed post each and every test. 

ところで、なぜグローバル変数を使用していますか？それらの必要はありません。実際、グローバルを使用するための正当な根拠はほとんどありません。さらに、テストは独立して独立している必要があり、グローバル変数を使用するとそのルールに違反します。