1521444 - Add webdriver unit tests for indefinite script timeout

Status: RESOLVED FIXED

(Testing :: geckodriver, enhancement)

Description

[r2hkri]

Follow-up to 1128997

> The only thing I actually miss in this patch are some more unit tests for the
> webdriver crate. I don't want to run wdspec to verify if changes in
> webdriver/geckodriver work. Running the Rust unittest is way less expensive.
> Would you mind getting those tests added in a follow-up bug?
> Just create one and upload the new patch. I would appreciate and be happy to
> review it.

Add unit tests to cover webdriver functionality that was suggested and implemented in linked bug (difference between undefined and null when serializing and de-serializing script timeout values).

Comment 1

[Andreas Tolfsen ❲:ato❳]

Can you be more specific in the description what test you are going
to add here? Otherwise I fear we risk scope creep.

Comment 2

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

Thank you for filing this bug. Do you also want to work on it, and add the remaining tests?

Comment 3

[r2hkri]

(In reply to Andreas Tolfsen ⦗:ato⦘ from comment #1)

Can you be more specific in the description what test you are going
to add here? Otherwise I fear we risk scope creep.

Test cases that check serializing/deserializing null properly like:

     fn test_json_timeouts_response_with_null_timeout() {
         let json = r#"{"value":{"script":null,"pageLoad":2,"implicit":3}}"#;
         let data = WebDriverResponse::Timeouts(TimeoutsResponse::new(None, 2, 3));
         check_serialize(&json, &data);
     }

(In reply to Henrik Skupin (:whimboo) [⌚️UTC+1] from comment #2)

Thank you for filing this bug. Do you also want to work on it, and add the remaining tests?

Yes, I can work on it.

Comment 4

[r2hkri]

Attachment: Bug 1521444 - Add webdriver unit tests for indefinite script timeout.

Add tests that check serialization and deserialization for indefinite script timeout

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/590f9a6f20a5
Add webdriver unit tests for indefinite script timeout. r=whimboo

Comment 6

[Razvan Maries]

https://hg.mozilla.org/mozilla-central/rev/590f9a6f20a5