Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tests for loadController #259

Merged
merged 1 commit into from
Jul 31, 2021
Merged

Tests for loadController #259

merged 1 commit into from
Jul 31, 2021

Conversation

groenroos
Copy link
Member

This PR adds unit tests for the loadController module.

Closes #126.

@groenroos groenroos added the maintenance Keep dependencies, code and conventions fresh label Jul 31, 2021
@groenroos groenroos self-assigned this Jul 31, 2021
@codecov
Copy link

codecov bot commented Jul 31, 2021

Codecov Report

Merging #259 (c03f5e9) into master (44189b6) will increase coverage by 2.72%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #259      +/-   ##
==========================================
+ Coverage   91.82%   94.54%   +2.72%     
==========================================
  Files          35       35              
  Lines        1247     1247              
==========================================
+ Hits         1145     1179      +34     
+ Misses        102       68      -34     
Impacted Files Coverage Δ
core/loadController.js 100.00% <100.00%> (+87.17%) ⬆️

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 44189b6...c03f5e9. Read the comment docs.

@groenroos groenroos merged commit e6cded6 into master Jul 31, 2021
@groenroos groenroos deleted the test/loadController branch July 31, 2021 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Keep dependencies, code and conventions fresh
Development

Successfully merging this pull request may close these issues.

Write tests for loadController
1 participant