831306 - Update RemoteTestRun class to support non-restart tests

Status: RESOLVED FIXED

(Mozilla QA Graveyard :: Mozmill Automation, defect)

Description

[mario garbi]

Will upload a patch as soon as possible.

Comment 1

[mario garbi]

Attachment: patch v1.0

Linux testrun_remote reports:
http://mozmill-crowd.blargon7.com/#/remote/report/9e41582ed5e806fa373351d9d316a043

Comment 2

[Andreea Matei [:AndreeaMatei]]

Comment on attachment 703208 [details] [diff] [review]
patch v1.0

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

Tested and works as expected.
Thanks Mario

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Comment on attachment 703208 [details] [diff] [review]
patch v1.0

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

r- given the comment below. Also before we can get this landed the appropriate mozmill test has to be checked-in first.

::: libs/testrun.py
@@ +763,5 @@
>          """ Execute the normal and restart tests in sequence. """
>  
> +        try:
> +            self.restart_tests = False
> +            self.test_path = os.path.join('tests', 'remote', 'addonsTests')

Not sure why you have added addonsTests here. But that certainly breaks the testrun for future tests in other areas. Just comment in those lines and nothing more.

Comment 4

[mario garbi]

Attachment: patch v1.1

Fixed as requested
Linux - mozilla-default report :
http://mozmill-crowd.blargon7.com/#/remote/report/9e41582ed5e806fa373351d9d3185936

Comment 5

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Comment on attachment 703281 [details] [diff] [review]
patch v1.1

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

Looks good. Now we only have to wait for the test landed.

Comment 6

[Henrik Skupin [:whimboo][⌚️UTC+2]]

http://hg.mozilla.org/qa/mozmill-automation/rev/b6229a0e5320