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

Improve mochi errors#3647

Merged
jasonLaster merged 1 commit into
firefox-devtools:masterfrom
jasonLaster:improve-errors
Aug 14, 2017
Merged

Improve mochi errors#3647
jasonLaster merged 1 commit into
firefox-devtools:masterfrom
jasonLaster:improve-errors

Conversation

@jasonLaster
Copy link
Copy Markdown
Contributor

@jasonLaster jasonLaster commented Aug 12, 2017

Associated Issue: #3623

Summary of Changes

  • switch from module mode state to a data object so that tests can run in parallel
  • add support for handling errors
  • some other small tweaks like results

Before

screen shot 2017-08-12 at 10 47 57 am

After

screen shot 2017-08-12 at 10 47 33 am

@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 12, 2017

Codecov Report

Merging #3647 into master will increase coverage by 0.34%.
The diff coverage is 96.36%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3647      +/-   ##
==========================================
+ Coverage   54.49%   54.84%   +0.34%     
==========================================
  Files         120      120              
  Lines        4786     4812      +26     
  Branches      992     1001       +9     
==========================================
+ Hits         2608     2639      +31     
+ Misses       2178     2173       -5
Impacted Files Coverage Δ
bin/mochi.js 72.18% <96.36%> (+10.49%) ⬆️
src/actions/ast.js 78.57% <0%> (+1.78%) ⬆️

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 de46df8...2eca854. Read the comment docs.

Comment thread bin/mochi.js
}

function onGecko(line) {
function onFrame(line, data) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nit: looks like data is not needed

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

true... I'll move this file to a place where prettier works and then we'll find other terrible things.

@jasonLaster jasonLaster merged commit ce03d8f into firefox-devtools:master Aug 14, 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