search

botobag / artemis

GraphQL implementation for Go
Other
10 stars 0 forks source link

[test] Fix worker_pool_executor_internal_test. #131

Closed zonr closed 5 years ago

zonr commented 5 years ago
  1. Use mutex to update numTasks in consume for simplicity.
  2. In consume, only decrement task count when a task has been polled. This potentially fixes #130.

Closes #130.

codecov[bot] commented 5 years ago

Codecov Report

Merging #131 into master will increase coverage by 0.02%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #131      +/-   ##
==========================================
+ Coverage    73.8%   73.82%   +0.02%     
==========================================
  Files          66       66              
  Lines        6981     6981              
==========================================
+ Hits         5152     5154       +2     
+ Misses       1558     1557       -1     
+ Partials      271      270       -1
Impacted Files Coverage Δ
concurrent/worker_pool_executor.go 89.69% <0%> (+0.68%) :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 6262a27...4493535. Read the comment docs.