search

MozillaSecurity / lithium

Line-based testcase reducer
Mozilla Public License 2.0
95 stars 25 forks source link

Revert "Fix linter import-related issues by moving entirely to absolu… #27

Closed nth10sd closed 7 years ago

nth10sd commented 7 years ago

…te_import", then rebased.

This reverts commit 713dfe0d8c916b02a52f521a496ae67a1ff6399e, then rebased to master.

We encountered some problems importing lithium as a library module via pip local install, then using the modules in interestingness, so this PR reverts the shift to absolute_import until we figure out the cause.

codecov-io commented 7 years ago

Codecov Report

Merging #27 into master will decrease coverage by 0.45%. The diff coverage is 4.16%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #27      +/-   ##
==========================================
- Coverage   69.26%   68.81%   -0.46%     
==========================================
  Files          15       15              
  Lines        1487     1491       +4     
==========================================
- Hits         1030     1026       -4     
- Misses        457      465       +8
Impacted Files Coverage Δ
lithium/examples/arithmetic/product_divides.py 100% <ø> (ø) :arrow_up:
lithium/lithium.py 79.74% <ø> (-0.03%) :arrow_down:
setup.py 0% <ø> (ø) :arrow_up:
interestingness/ximport.py 69.56% <ø> (-1.27%) :arrow_down:
interestingness/timedRun.py 0% <0%> (ø) :arrow_up:
interestingness/diffTest.py 0% <0%> (ø) :arrow_up:
interestingness/fileIngredients.py 0% <0%> (ø) :arrow_up:
interestingness/outputs.py 0% <0%> (ø) :arrow_up:
interestingness/crashes.py 0% <0%> (ø) :arrow_up:
interestingness/envVars.py 0% <0%> (ø) :arrow_up:
... and 3 more

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 6659b77...dbca73f. Read the comment docs.

nth10sd commented 7 years ago

See #28 for a potentially better solution.