1275288 - Add harness unit tests for exceptions raised by _load_testvars

Status: RESOLVED FIXED

(Remote Protocol :: Marionette, defect)

Description

[Anjana Vakil (:vakila)]

Test that MarionetteTestRunner._load_testvars raises
the expected exceptions when testvars filepaths are 
nonexistent or have improperly formatted JSON data

Comment 1

[Anjana Vakil (:vakila)]

Attachment: MozReview Request: Bug 1275288 - Test exception raising in _load_testvars; r?maja_zf

Test that MarionetteTestRunner._load_testvars raises
the expected exceptions when testvars filepaths are
nonexistent or have improperly formatted JSON data

Review commit: https://reviewboard.mozilla.org/r/54900/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/54900/

Comment 2

[Anjana Vakil (:vakila)]

Maja, per your comment on Bug 1275269, I'll remove the comments from this patch as they're rather redundant. I'll just wait for your review in case anything else needs to be changed.

Comment 3

[Maja Frydrychowicz :impossibus (was :maja_zf) (needinfo me)]

Comment on attachment 8755958 [details]
MozReview Request: Bug 1275288 - Test exception raising in _load_testvars; r?maja_zf

https://reviewboard.mozilla.org/r/54900/#review51594

Looks good.

::: testing/marionette/harness/marionette/tests/harness_unit/test_marionette_runner.py:291
(Diff revision 1)
>          assert runner.record_crash() == has_crashed
>          _check_crash_counts(has_crashed, runner, runner.marionette)
>  
>  
> +def test_load_testvars_throws_expected_errors(mach_parsed_kwargs):
> +    mach_parsed_kwargs

Extra line to be deleted?

nit: Please also move your new test to be next to the other "test_vars" test in the module. (Aside: we'll probably want to reorganize these tests eventually to not all be in one module. We can revisit that in the future.)

Comment 4

[Anjana Vakil (:vakila)]

Comment on attachment 8755958 [details]
MozReview Request: Bug 1275288 - Test exception raising in _load_testvars; r?maja_zf

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/54900/diff/1-2/

Comment 5

[Maja Frydrychowicz :impossibus (was :maja_zf) (needinfo me)]

Comment on attachment 8755958 [details]
MozReview Request: Bug 1275288 - Test exception raising in _load_testvars; r?maja_zf

https://reviewboard.mozilla.org/r/54900/#review51752

Great!

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/fd985ed46ae4

Comment 7

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

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