You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
expect(Date()).asLocalDate().toBeAfter(...)
//instead of
expect(Date().toInstant().atZone(ZoneId.systemDefault()).toLocalDate()).toBeAfter(...);
Following the things you need to do:
api-fluent
provide a val which returns Expect<LocalDate> in dateSubjectChangers.kt (see fileSubjectChangers.kt as a guideline)
provide a fun which expects an assertionCreator-lambda and returns Expect<Date> in dateSubjectChangers.kt (see fileSubjectChangers.kt as a guideline)
add @since 0.19.0 (adapt to current milestone) to KDOC
extend or write a separate Spec named DateAsLocalDateExpectationsSpec in specs -> jvmMain (see for instance FileAsPathExpectationsSpec) and extend it in atrium-api-fluent-en_GB -> jvmTest
api-infix
provide a val which returns Expect<Date> in dateSubjectChangers.kt(see fileSubjectChangers.kt as a guideline)
provide a fun which expects an assertionCreator-lambda and returns Expect<AB> in dateSubjectChangers.kt (see fileSubjectChangers.kt as a guideline)
add @since 0.19.0 (adapt to current milestone) to KDOC
extend or write a separate Spec named DateAsLocalDateExpectationsSpec in specs -> commonMain (see for instance FileAsPathExpectationsSpec) and extend it in atrium-api-infix-en_GB -> jvmTest
Your first contribution?
Write a comment I'll work on this if you would like to take this issue over.
This way we get the chance to revise the description in case things have changed in the meantime, we might give you additional hints and we can assign the task to you, so that others do not start as well.
Do not hesitate to ask questions here or to contact us via Atrium's slack channel if you need help
(Invite yourself in case you do not have an account yet).
The text was updated successfully, but these errors were encountered:
@robstoll I made a typo in the branch name 😅, I hope that isn't a big problem, I'll delete the branch after merge.
Anyway, I've opened a PR as I think my changes are ready for review.
Platform (all, jvm, js): jvm
Extension (none, kotlin 1.3): none
Code related feature
Following the things you need to do:
api-fluent
Expect<LocalDate>
in dateSubjectChangers.kt (see fileSubjectChangers.kt as a guideline)assertionCreator
-lambda and returnsExpect<Date>
in dateSubjectChangers.kt (see fileSubjectChangers.kt as a guideline)@since 0.19.0
(adapt to current milestone) to KDOCapi-infix
Expect<Date>
in dateSubjectChangers.kt(see fileSubjectChangers.kt as a guideline)assertionCreator
-lambda and returnsExpect<AB>
in dateSubjectChangers.kt (see fileSubjectChangers.kt as a guideline)@since 0.19.0
(adapt to current milestone) to KDOCYour first contribution?
I'll work on this
if you would like to take this issue over.This way we get the chance to revise the description in case things have changed in the meantime, we might give you additional hints and we can assign the task to you, so that others do not start as well.
(Invite yourself in case you do not have an account yet).
The text was updated successfully, but these errors were encountered: