1439711 - Refactor debugger server stepping tests

Status: RESOLVED FIXED

(DevTools :: Debugger, enhancement)

Description

[Jason Laster [:jlast]]

It would be nice if the stepping tests used async/await.

Comment 1

[Jason Laster [:jlast]]

Attachment: ref-tests.patch

Comment 2

[Tom Tromey :tromey]

Comment on attachment 8952498 [details] [diff] [review]
ref-tests.patch

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

Thank you for the patch.  I think this makes the code much clearer.

I found a few nits but nothing serious.

::: devtools/server/tests/unit/test_stepping-01.js
@@ +13,4 @@
>  var gCallback;
>  
>  function run_test() {
> +  do_test_pending();

I don't know the rationale for moving this.  It's fine if it all works though.  I noticed this is done in some files but not all of them...

::: devtools/server/tests/unit/test_stepping-02.js
@@ +43,3 @@
>  
> +  dumpn("Step Over to line 3");
> +  const step1 = await stepIn(gClient, threadClient);

The dump says "Step Over" but the code says step in.  This appears in a few spots in this file.

::: devtools/server/tests/unit/test_stepping-03.js
@@ +55,5 @@
> +      this.a = 1;                       // 4
> +      this.b = 2;                       // 5
> +    }                                   // 6
> +    f();                                // 7
> +    `,                                  // 4

The "// 4" on this line is mysterious.

::: devtools/server/tests/unit/test_stepping-05.js
@@ +57,5 @@
>  
> +  dumpn("Step In to line 5");
> +  const step3 = await stepIn(gClient, threadClient);
> +  equal(step3.type, "paused");
> +  equal(step3.frame.where.line, 4);

The dump here says "line 5", but according to the code, there is no line 5 and the expected line is actually 4.  And, the previous step went to line 4.  This seems strange to me, I think at least the dump should be updated, and perhaps a comment explaining what is going on.

Comment 3

[Jason Laster [:jlast]]

Thanks for the feedback!

https://treeherder.mozilla.org/#/jobs?repo=try&revision=34db41515eb139b6fcceb4f37c6d38b393cd436f

Comment 4

[Jason Laster [:jlast]]

Attachment: ref-tests-2.patch

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/d3084609beef
Refactor debugger server stepping tests. r=tromey

Comment 6

[Cosmin Sabou [:CosminS]]

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