Skip to content
New issue

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

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

Already on GitHub? # to your account

Added better testing of the Context Initializers and TestSuite{}.Run() #301

Merged
merged 1 commit into from
Jun 6, 2020

Conversation

lonnblad
Copy link
Member

@lonnblad lonnblad commented Jun 6, 2020

  • Copied the SuiteContext for testing with the new Context Initializers
  • Added testing with TestSuite{}.Run() to fmt_output_test.go
  • Added testing with TestSuite{}.Run() to run_test.go

@lonnblad lonnblad force-pushed the add-tests-for-the-new-initializers branch from 4321a20 to b33e213 Compare June 6, 2020 13:58
@codecov
Copy link

codecov bot commented Jun 6, 2020

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #301      +/-   ##
==========================================
+ Coverage   84.56%   84.84%   +0.27%     
==========================================
  Files          17       17              
  Lines        2521     2521              
==========================================
+ Hits         2132     2139       +7     
+ Misses        274      266       -8     
- Partials      115      116       +1     
Impacted Files Coverage Δ
run.go 72.53% <0.00%> (+2.11%) ⬆️
test_context.go 42.85% <0.00%> (+28.57%) ⬆️

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 678c3ae...6b2e720. Read the comment docs.

@lonnblad lonnblad changed the title Copied the SuiteContext for testing of TestSuite{}.Run() Added better testing of the Context Initializers and TestSuite{}.Run() Jun 6, 2020
@lonnblad lonnblad force-pushed the add-tests-for-the-new-initializers branch from b33e213 to 6b2e720 Compare June 6, 2020 14:49
@lonnblad lonnblad merged commit 7f46bb3 into master Jun 6, 2020
@lonnblad lonnblad deleted the add-tests-for-the-new-initializers branch June 13, 2020 09:01
@lonnblad lonnblad restored the add-tests-for-the-new-initializers branch October 19, 2020 17:01
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant