search

smartcar / clutch-assert

The assertion library that comes in clutch.
MIT License
5 stars 0 forks source link

fix .clutchrc loader #15

Closed gurpreetatwal closed 7 years ago

gurpreetatwal commented 7 years ago

The current loader for the .clutchrc file has a flaw in that it only works if the loader.js file is required by a file that is located in same directory as the .clutchrc

For example, it would work if say test/test.js required the loader and the .clutchrc is in the test folder but would not work if the .clutchrc was in the root of the project, as described by the readme.

It also broke in the common workflow of using the require hook of test runners as the file would not be loaded in the root directory.

codecov-io commented 7 years ago

Codecov Report

Merging #15 into master will not change coverage. The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #15   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           6      6           
  Lines         126    129    +3     
=====================================
+ Hits          126    129    +3
Impacted Files Coverage Δ
loader/index.js 100% <100%> (ø) :arrow_up:
loader/helpers.js 100% <100%> (ø) :arrow_up:

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 529355b...818ae43. Read the comment docs.