991242 - AttributeError: 'TestGaiaLaunch' object has no attribute 'launch_gaia_app

Status: RESOLVED FIXED

(Remote Protocol :: Marionette, defect)

Description

[David Burns :automatedtester]

I noticed this when cleaning up the Marionette tests


Details at 

11:24:17     INFO -  TEST-START test_switch_frame_b2g.py
11:24:18     INFO -  test_launch_app (test_switch_frame_b2g.TestGaiaLaunch) ... ERROR
11:24:19     INFO -  ======================================================================
11:24:19     INFO -  ERROR: None
11:24:19     INFO -  ----------------------------------------------------------------------
11:24:19     INFO -  Traceback (most recent call last):
11:24:19     INFO -    File "/builds/slave/test/build/tests/marionette/marionette/marionette_test.py", line 163, in run
11:24:19     INFO -      testMethod()
11:24:19     INFO -    File "/builds/slave/test/build/tests/marionette/tests/testing/marionette/client/marionette/tests/unit/test_switch_frame_b2g.py", line 16, in test_launch_app
11:24:19     INFO -      app_frame = self.launch_gaia_app('../sms/sms.html')
11:24:19     INFO -  TEST-UNEXPECTED-FAIL | test_switch_frame_b2g.py test_switch_frame_b2g.TestGaiaLaunch.test_launch_app | AttributeError: 'TestGaiaLaunch' object has no attribute 'launch_gaia_app'

Comment 1

[Jonathan Griffin (:jgriffin)]

I think we've moved away from trying to test Gaia within the Marionette framework itself; relevant unit tests should live in gaia-ui-tests.  Probably we should just remove this test.

Comment 2

[David Burns :automatedtester]

Attachment: Remove Marionette unit test as this is covered by Gaia testing

Comment 3

[David Burns :automatedtester]

I added the file to the manifest in bug 811796 and have removed it from that patch

Comment 4

[Jonathan Griffin (:jgriffin)]

Comment on attachment 8400842 [details] [diff] [review]
Remove Marionette unit test as this is covered by Gaia testing

Review of attachment 8400842 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks.

Comment 5

[David Burns :automatedtester]

landed in https://hg.mozilla.org/integration/mozilla-inbound/rev/f4fcae008c89

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/f4fcae008c89