Bug 960931: Remove unused variables from tests r?ato draft
authorDavid Burns <dburns@mozilla.com>
Wed, 25 May 2016 15:31:51 +0100
changeset 370918 fc059eb3716662f7f465544e40d0fc23ea332a48
parent 370917 973223348e5be16a203608a3a77480e4aadf42ed
child 521850 37d047da2805a53da9e9ef08278abb5923ef50eb
push id19169
push userdburns@mozilla.com
push dateWed, 25 May 2016 14:32:15 +0000
reviewersato
bugs960931
milestone49.0a1
Bug 960931: Remove unused variables from tests r?ato We create the variables with what is returned from the session creation but never do anything with them. MozReview-Commit-ID: LDZQrtqdniB
testing/marionette/harness/marionette/tests/unit/test_session.py
--- a/testing/marionette/harness/marionette/tests/unit/test_session.py
+++ b/testing/marionette/harness/marionette/tests/unit/test_session.py
@@ -26,24 +26,24 @@ class TestSession(MarionetteTestCase):
         # Optional capabilities we want Marionette to support
         self.assertIn("device", caps)
         self.assertIn("rotatable", caps)
         self.assertIn("takesScreenshot", caps)
         self.assertIn("version", caps)
 
     def test_we_can_get_the_session_id(self):
         # Sends newSession
-        caps = self.marionette.start_session()
+        self.marionette.start_session()
 
         self.assertTrue(self.marionette.session_id is not None)
         self.assertTrue(isinstance(self.marionette.session_id, unicode))
 
     def test_we_can_set_the_session_id(self):
         # Sends newSession
-        caps = self.marionette.start_session(session_id="ILoveCheese")
+        self.marionette.start_session(session_id="ILoveCheese")
 
         self.assertEqual(self.marionette.session_id, "ILoveCheese")
         self.assertTrue(isinstance(self.marionette.session_id, unicode))
 
     def test_we_only_support_one_active_session_at_a_time(self):
         self.marionette.start_session()
         self.assertTrue(isinstance(self.marionette.session_id, unicode))
         self.assertRaises(SessionNotCreatedException, self.marionette._send_message, "newSession", {})