-
Notifications
You must be signed in to change notification settings - Fork 10
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
feat: add support for the format parameter #1299
Conversation
Test Results 71 files + 1 71 suites +1 1m 25s ⏱️ -1s Results for commit fc7966c. ± Comparison against base commit a6b8b37. This pull request removes 190 and adds 50 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
shared/src/main/kotlin/com/egm/stellio/shared/model/NgsiLdDataRepresentation.kt
Show resolved
Hide resolved
shared/src/main/kotlin/com/egm/stellio/shared/queryparameter/FormatValue.kt
Show resolved
Hide resolved
shared/src/main/kotlin/com/egm/stellio/shared/queryparameter/FormatValue.kt
Outdated
Show resolved
Hide resolved
search-service/src/test/kotlin/com/egm/stellio/search/entity/web/EntityHandlerTests.kt
Show resolved
Hide resolved
search-service/src/main/kotlin/com/egm/stellio/search/temporal/util/TemporalQueryUtils.kt
Outdated
Show resolved
Hide resolved
search-service/src/main/kotlin/com/egm/stellio/search/temporal/util/TemporalQueryUtils.kt
Outdated
Show resolved
Hide resolved
search-service/src/main/kotlin/com/egm/stellio/search/temporal/util/TemporalQueryUtils.kt
Outdated
Show resolved
Hide resolved
...ch-service/src/test/kotlin/com/egm/stellio/search/temporal/web/TemporalEntityHandlerTests.kt
Outdated
Show resolved
Hide resolved
search-service/src/main/kotlin/com/egm/stellio/search/temporal/util/TemporalQueryUtils.kt
Outdated
Show resolved
Hide resolved
search-service/src/main/kotlin/com/egm/stellio/search/temporal/util/TemporalQueryUtils.kt
Outdated
Show resolved
Hide resolved
search-service/src/main/kotlin/com/egm/stellio/search/temporal/util/TemporalQueryUtils.kt
Outdated
Show resolved
Hide resolved
search-service/src/test/kotlin/com/egm/stellio/search/support/BusinessObjectsFactory.kt
Outdated
Show resolved
Hide resolved
search-service/src/main/kotlin/com/egm/stellio/search/temporal/util/TemporalQueryUtils.kt
Outdated
Show resolved
Hide resolved
search-service/src/main/kotlin/com/egm/stellio/search/temporal/util/TemporalQueryUtils.kt
Outdated
Show resolved
Hide resolved
search-service/src/main/kotlin/com/egm/stellio/search/temporal/util/TemporalQueryUtils.kt
Outdated
Show resolved
Hide resolved
search-service/src/main/kotlin/com/egm/stellio/search/temporal/util/TemporalQueryUtils.kt
Outdated
Show resolved
Hide resolved
don't know why you merged develop in your branch (we never do that, we rebase on develop), but it is a bit a mess. there is at least one commit from develop which is included in the branch... |
...ice/src/test/kotlin/com/egm/stellio/search/temporal/service/AttributeInstanceServiceTests.kt
Outdated
Show resolved
Hide resolved
shared/src/main/kotlin/com/egm/stellio/shared/queryparameter/OptionsValue.kt
Outdated
Show resolved
Hide resolved
shared/src/test/kotlin/com/egm/stellio/shared/model/NgsiLdDataRepresentationTests.kt
Outdated
Show resolved
Hide resolved
search-service/src/main/kotlin/com/egm/stellio/search/temporal/util/TemporalQueryUtils.kt
Outdated
Show resolved
Hide resolved
search-service/src/main/kotlin/com/egm/stellio/search/temporal/util/TemporalQueryUtils.kt
Outdated
Show resolved
Hide resolved
shared/src/main/kotlin/com/egm/stellio/shared/model/NgsiLdDataRepresentation.kt
Outdated
Show resolved
Hide resolved
fun fromString(key: String): FormatValue? = | ||
FormatValue.entries.find { it.value == key } |
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.
fun fromString(key: String): FormatValue? = | |
FormatValue.entries.find { it.value == key } | |
fun fromString(key: String): Either<APIException, FormatValue> = either { | |
FormatValue.entries.find { it.value == key } | |
?: return InvalidRequestException("'$key' is not a valid value for the format query parameter").left() | |
} |
There were conflicts with develop that I fixed on github, I clicked on commit after and it did a merge. Can it be fixed? |
…/util/TemporalQueryUtils.kt Co-authored-by: Thomas Bousselin <[email protected]>
…/util/TemporalQueryUtils.kt Co-authored-by: Thomas Bousselin <[email protected]>
…/web/TemporalEntityHandlerTests.kt Co-authored-by: Benoit Orihuela <[email protected]>
…/util/TemporalQueryUtils.kt Co-authored-by: Benoit Orihuela <[email protected]>
…RepresentationTests.kt Co-authored-by: Benoit Orihuela <[email protected]>
be2ac6c
to
5e9842e
Compare
Quality Gate passedIssues Measures |
No description provided.