Skip to content

Conversation

@AnujRNair
Copy link
Contributor

Summary

This PR:

  • Removes the Jasmine testing framework and replaces it with Jest.
  • Updates the tests to use the new framework / runner
  • Updates the tests to test in memory file system rather than writing to the physical filesystem
  • Refactors the tests to organize them better and make them easier to work with.
  • Updates the npm scripts to reflect the new changes

Requirements (place an x in each [ ])

@codecov
Copy link

codecov bot commented Dec 20, 2018

Codecov Report

Merging #24 into master will decrease coverage by 3.57%.
The diff coverage is 100%.

Impacted file tree graph

@@ Coverage Diff @@ ## master #24 +/- ## ========================================== - Coverage 97.32% 93.75% -3.58%  ========================================== Files 2 2 Lines 187 80 -107 Branches 14 11 -3 ========================================== - Hits 182 75 -107  + Misses 5 4 -1  - Partials 0 1 +1
Impacted Files Coverage Δ
plugin.js 91.52% <100%> (-1.54%) ⬇️
test-utils/webpack-helpers.js 100% <100%> (ø)

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 77da922...8cdf784. Read the comment docs.

@AnujRNair AnujRNair merged commit 16bc24c into master Dec 20, 2018
@AnujRNair AnujRNair deleted the an-tests-to-jest branch December 20, 2018 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants