vzhikserg commented 5 years ago

The tests were refactored according to the recommendations from the IDE and there were a couple of optimization to make the tests smaller and more readable. Please let me know if you have any problems with the way how I refactored the code.

codecov-io commented 5 years ago

Codecov Report

Merging #5 into master will increase coverage by 0.01%. The diff coverage is 100%.

@@            Coverage Diff             @@
##           master       #5      +/-   ##
==========================================
+ Coverage    77.8%   77.81%   +0.01%     
==========================================
  Files          20       20              
  Lines         865      861       -4     
  Branches       82       82              
==========================================
- Hits          673      670       -3     
  Misses        172      172              
+ Partials       20       19       -1

Impacted Files	Coverage Δ
...n/java/com/github/myzhan/locust4j/stats/Stats.java	`95.23% <ø> (+0.74%)`	:arrow_up:
...a/com/github/myzhan/locust4j/stats/StatsEntry.java	`72.72% <100%> (-0.55%)`	:arrow_down:
...a/com/github/myzhan/locust4j/stats/StatsError.java	`100% <100%> (ø)`	:arrow_up:
...n/java/com/github/myzhan/locust4j/utils/Utils.java	`79.31% <100%> (-0.69%)`	:arrow_down:
...ithub/myzhan/locust4j/taskset/WeighingTaskSet.java	`56% <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 15d6007...d1bacc6. Read the comment docs.

myzhan commented 5 years ago

Looks good to me, thanks!

myzhan / locust4j

Refactoring of the tests #5

Codecov Report