-
-
Notifications
You must be signed in to change notification settings - Fork 208
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
Add infix feature extractors samples (#1568) #1733
base: main
Are you sure you want to change the base?
Add infix feature extractors samples (#1568) #1733
Conversation
// } | ||
// } | ||
|
||
data class Person2( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO: Merge with Person class
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1733 +/- ##
=========================================
Coverage 92.68% 92.68%
Complexity 116 116
=========================================
Files 442 442
Lines 4895 4895
Branches 234 234
=========================================
Hits 4537 4537
Misses 311 311
Partials 47 47
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
.../src/commonTest/kotlin/ch/tutteli/atrium/api/fluent/en_GB/samples/FeatureExtractorSamples.kt
Outdated
Show resolved
Hide resolved
fails { | ||
expect(person) its { age } toBeLessThan 20 toBeGreaterThan 30 | ||
// | | not evaluated/reported because `toBeLessThan` already fails | ||
// | | use an expectation-group block; for more details: https://github.com/robstoll/atrium#define-single-expectations-or-expectation-groups |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
update link as well
...x/src/commonTest/kotlin/ch/tutteli/atrium/api/infix/en_GB/samples/FeatureExtractorSamples.kt
Outdated
Show resolved
Hide resolved
First attempt
9f60da8
to
7d3ddf4
Compare
…trium/api/infix/en_GB/samples/FeatureExtractorSamples.kt Co-authored-by: Robert Stoll <[email protected]>
@constantine-zubrylin just as info, did not have a look again, let me know once it is ready for review |
Sure. Sorry, I have some problems with free time, but I'll return to the ticket soon. |
No stress and thanks for your contributions |
Adds infix feature extractors samples based on fluent tests
I confirm that I have read the Contributor Agreements v1.0, agree to be bound on them and confirm that my contribution is compliant.