Closed Bug 1256848 Opened 8 years ago Closed 8 years ago

[ESLint] Fix ESLint issues in devtools/client/webconsole/test/browser_webconsole_bug_603750_websocket.js

Categories

(DevTools :: Console, defect, P3)

defect

Tracking

(firefox48 fixed, firefox49 fixed)

RESOLVED FIXED
Firefox 49
Tracking Status
firefox48 --- fixed
firefox49 --- fixed

People

(Reporter: linclark, Assigned: wldcordeiro, Mentored)

References

Details

(Whiteboard: [btpp-backlog])

Attachments

(1 file)

If you haven’t contributed to Firefox before, follow the steps here to set up your environment: https://developer.mozilla.org/en-US/docs/Tools/Contributing#Getting_set_up

Then, automatically configure ESLint to work with the Firefox specific rules by following the instructions here: https://wiki.mozilla.org/DevTools/CodingStandards

Then you can see the issues that need to be fixed by running

> eslint --no-ignore devtools/client/webconsole/test/browser_webconsole_bug_603750_websocket.js
Blocks: 1256948
Priority: -- → P3
Whiteboard: [btpp-backlog]
Attached patch Bug1256848.patchSplinter Review
This patch fixes the eslint issues for `devtools/client/webconsole/test/browser_webconsole_bug_603750_websocket.js`
Attachment #8749347 - Flags: review+
Attachment #8749347 - Flags: feedback+
Comment on attachment 8749347 [details] [diff] [review]
Bug1256848.patch

Thanks Wellington.  Setting the review flag to '?' and filling in a reviewer (in this case jlast).
Attachment #8749347 - Flags: review?(jlaster)
Attachment #8749347 - Flags: review+
Attachment #8749347 - Flags: feedback+
Doh, I thought I did the right thing in the review/feedback column as those were the choices I saw when I clicked. Thanks Brian, I was actually just talking to jlast as I noticed linclark was on leave.
Wellington, I'm not familiar with `BrowserTestUtils.loadURI`, but if the tests run successfully, i believe it's okay.

Do you know how to run the tests with try? Here's an overview, https://wiki.mozilla.org/ReleaseEngineering/TryServer#Getting_access_to_the_Try_Server. Let me know if you have any questions.
Comment on attachment 8749347 [details] [diff] [review]
Bug1256848.patch

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

::: devtools/client/webconsole/test/browser_webconsole_bug_603750_websocket.js
@@ +14,5 @@
>    yield loadTab(TEST_URI2);
>  
>    let hud = yield openConsole();
>  
> +  yield BrowserTestUtils.loadURI(gBrowser.selectedBrowser, TEST_URI);

No need to yield here, since it wasn't before
Assignee: nobody → wellington
Status: NEW → ASSIGNED
(In reply to Jason Laster [:jlast] from comment #4)
> Wellington, I'm not familiar with `BrowserTestUtils.loadURI`, but if the
> tests run successfully, i believe it's okay.
> 
> Do you know how to run the tests with try? Here's an overview,
> https://wiki.mozilla.org/ReleaseEngineering/
> TryServer#Getting_access_to_the_Try_Server. Let me know if you have any
> questions.

I went ahead and pushed to try (with my suggested change in Comment 5 included): https://treeherder.mozilla.org/#/jobs?repo=try&revision=8f6c558224c6
Wellington, assuming that try push looks good (usually takes an hour or two for it to finish), I can go ahead and push the patch without any extra work from you
https://hg.mozilla.org/mozilla-central/rev/8fd42cc5e8c4
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
Attachment #8749347 - Flags: review?(jlaster)
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.