1364211 - Correct Variable scope in TestSize class for start_size variable

Status: RESOLVED FIXED

(Remote Protocol :: Marionette, enhancement)

Description

[David Burns :automatedtester]

The variable is scoped against `self` and not in the `setup()` method in https://dxr.mozilla.org/mozilla-central/source/testing/marionette/harness/marionette_harness/tests/unit/test_window_rect.py#124-125

These need to have the `self.` removed

To get started, follow the details in https://wiki.mozilla.org/User:Mjzffr/New_Contributors

Comment 1

[Santiago Paez [:tiago]]

Attachment: Bug 1364211 - Correct variable scope in TestSize class for start_size variable.

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

Comment 2

[David Burns :automatedtester]

Comment on attachment 8867577 [details]
Bug 1364211 - Correct variable scope in TestSize class for start_size variable.

https://reviewboard.mozilla.org/r/139122/#review142484

Comment 3

[David Burns :automatedtester]

Thanks for the patch!

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ec72e5d8e6c8
Correct variable scope in TestSize class for start_size variable. r=automatedtester

Comment 5

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

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