Skip to content
This repository has been archived by the owner on Jan 11, 2023. It is now read-only.

update mochitests #3202

Merged
merged 1 commit into from
Jun 22, 2017
Merged

Conversation

jasonLaster
Copy link
Contributor

@jasonLaster jasonLaster commented Jun 22, 2017

Summary of Changes

  • pulling in a changed test from mc
  • dropping a debugger statement
  • prettifying

@codecov
Copy link

codecov bot commented Jun 22, 2017

Codecov Report

Merging #3202 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3202   +/-   ##
=======================================
  Coverage   47.61%   47.61%           
=======================================
  Files          98       98           
  Lines        4060     4060           
  Branches      838      838           
=======================================
  Hits         1933     1933           
  Misses       2127     2127

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 348bf6b...cdd4ff2. Read the comment docs.

@codecov
Copy link

codecov bot commented Jun 22, 2017

Codecov Report

Merging #3202 into master will decrease coverage by 0.33%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3202      +/-   ##
==========================================
- Coverage   47.94%   47.61%   -0.34%     
==========================================
  Files          98       98              
  Lines        4063     4060       -3     
  Branches      838      838              
==========================================
- Hits         1948     1933      -15     
- Misses       2115     2127      +12
Impacted Files Coverage Δ
src/actions/navigation.js 0% <0%> (-15.39%) ⬇️
src/reducers/event-listeners.js 22.22% <0%> (-11.12%) ⬇️
src/reducers/async-requests.js 91.66% <0%> (-8.34%) ⬇️
src/reducers/pause.js 29.87% <0%> (-5.2%) ⬇️
src/reducers/sources.js 75.52% <0%> (-3.5%) ⬇️
src/client/firefox.js 86.66% <0%> (-0.84%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8c93fb9...6cd24df. Read the comment docs.

Copy link
Contributor

@codehag codehag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks pretty good, a couple of comments

@@ -151,6 +151,7 @@ export function selectSourceURL(
) {
return ({ dispatch, getState }: ThunkArgs) => {
const source = getSourceByURL(getState(), url);
debugger;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this be here?

@@ -67,7 +73,7 @@ describe("Tests", () => {

it("editor gutter", async () => await editorGutter(ctx));

xit("editor highlight", async () => await editorHighlight(ctx));
it.only("editor highlight", async () => await editorHighlight(ctx));
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i think we want this to just be "it"

@@ -30,6 +30,8 @@ module.exports = async function(ctx) {
// place.
await waitForDispatch(dbg, "LOAD_SOURCE_TEXT");

debugger;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

probably this debugger statement can go

Copy link
Contributor

@codehag codehag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jasonLaster jasonLaster merged commit 5b49a57 into firefox-devtools:master Jun 22, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants