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

compile also specs in source backward compatibility tests #772

Merged
merged 8 commits into from
Jan 8, 2021

Conversation

robstoll
Copy link
Owner

@robstoll robstoll commented Jan 8, 2021

and fix issues which were revealed due to this.
Fixes #771


I confirm that I have read the Contributor Agreements v1.0, agree to be bound on them and confirm that my contribution is compliant.

moreover:
- rename DeprecationSpek2TestEngine to Spek2ForgivingTestEngine

bcTests will fail with this commit due to:
- missing `contains all` function in api-infix (a bug)
- StaticName which is internal and not accessible from the new project
moreover:
- rename StaticName to StaticNames
  - don't use it anymore in tests as this causes troubles in bc-tests
- remove specs in 0.14.0 about AssertionPlant and co.
moreover:
- rename TestEngine id from spek2-deprecation to spek2-forgiving
@robstoll robstoll force-pushed the bc-compile-also-specs branch 2 times, most recently from 19baaaa to 9b1985e Compare January 8, 2021 22:26
and fix dependency to translations for infix, should also use de_CH in
bc-tests
@codecov
Copy link

codecov bot commented Jan 8, 2021

Codecov Report

Merging #772 (a681a49) into master (79460ae) will increase coverage by 2.97%.
The diff coverage is 96.22%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #772      +/-   ##
============================================
+ Coverage     81.86%   84.84%   +2.97%     
============================================
  Files           420      424       +4     
  Lines          3998     4024      +26     
  Branches        213      214       +1     
============================================
+ Hits           3273     3414     +141     
+ Misses          673      563     -110     
+ Partials         52       47       -5     
Flag Coverage Δ Complexity Δ
bc 80.81% <96.22%> (?) 0.00 <0.00> (?)
current 81.86% <96.22%> (ø) 0.00 <0.00> (ø)
current_windows 80.40% <96.22%> (ø) 0.00 <0.00> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ Complexity Δ
...ch/tutteli/atrium/api/infix/en_GB/mapAssertions.kt 100.00% <ø> (ø) 0.00 <0.00> (ø)
...m/api/fluent/en_GB/charSequenceContainsCheckers.kt 91.66% <90.90%> (ø) 0.00 <0.00> (ø)
...um/api/infix/en_GB/charSequenceContainsCheckers.kt 91.66% <90.90%> (ø) 0.00 <0.00> (ø)
...creating/charsequence/contains/impl/StaticNames.kt 100.00% <100.00%> (ø) 0.00 <0.00> (?)
..._GB/creating/iterable/contains/impl/StaticNames.kt 100.00% <100.00%> (ø) 0.00 <0.00> (?)
...m/api/fluent/en_GB/iterableLikeContainsCheckers.kt 100.00% <100.00%> (ø) 0.00 <0.00> (ø)
...creating/charsequence/contains/impl/StaticNames.kt 100.00% <100.00%> (ø) 0.00 <0.00> (?)
..._GB/creating/iterable/contains/impl/StaticNames.kt 100.00% <100.00%> (ø) 0.00 <0.00> (?)
...um/api/infix/en_GB/iterableLikeContainsCheckers.kt 100.00% <100.00%> (ø) 0.00 <0.00> (ø)
...m/domain/creating/collectors/AssertionCollector.kt 50.00% <0.00%> (ø) 0.00% <0.00%> (?%)
... and 36 more

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 79460ae...4de570d. Read the comment docs.

@robstoll robstoll merged commit 90ba50f into master Jan 8, 2021
@robstoll robstoll deleted the bc-compile-also-specs branch January 8, 2021 23:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

contains all no longer working in infix
1 participant