Skip to content

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
@robstoll robstoll force-pushed the bc-compile-also-specs branch from 9b1985e to 3db8cbe Compare January 8, 2021 22:36
@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

2 participants