search

eecs485staff / madoop

A light weight MapReduce framework for education
MIT License
9 stars 4 forks source link

Unit tests use example input #39

Closed jaredzh closed 2 years ago

jaredzh commented 2 years ago

Closes #37

codecov[bot] commented 2 years ago

Codecov Report

Merging #39 (ff887e1) into develop (1b46faf) will decrease coverage by 0.01%. The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop      #39      +/-   ##
===========================================
- Coverage    96.13%   96.11%   -0.02%     
===========================================
  Files            4        4              
  Lines          207      206       -1     
===========================================
- Hits           199      198       -1     
  Misses           8        8
Impacted Files Coverage Δ
madoop/__main__.py 93.75% <100.00%> (-0.13%) :arrow_down:

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 1b46faf...ff887e1. Read the comment docs.

jaredzh commented 2 years ago

I updated map.sh so that it's case-sensitive in order to be consistent with map.py. It causes errors in the tests if map.sh is case-insensitive because we are using group by hash which causes the outputs to be different. @awdeorio