search

eclipse / kapua

Eclipse Public License 2.0
222 stars 160 forks source link

Break job engine to rest api dependency #4073

Open dseurotech opened 1 month ago

dseurotech commented 1 month ago

This PR breaks the dependency that job-engine had to the rest-api, reorganising the commons-rest modules into a more structured commons-jersey modules (splitted in to rest and web submodules), that contain all the necessary components to handle jersey-based services (be they web apps or rest apis).

Breaking changes!

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 35.57692% with 67 lines in your changes missing coverage. Please review.

Project coverage is 16.56%. Comparing base (ca5615b) to head (37406a1).

Files Patch % Lines
...ommons/jersey/rest/model/IsJobRunningResponse.java 0.00% 12 Missing :warning:
...ersey/rest/model/IsJobRunningMultipleResponse.java 0.00% 9 Missing :warning:
...ommons/jersey/rest/filters/CORSResponseFilter.java 0.00% 7 Missing :warning:
...ommons/jersey/rest/model/MultipleJobIdRequest.java 0.00% 5 Missing :warning:
...apua/job/engine/client/JobEngineServiceClient.java 0.00% 5 Missing :warning:
...mmons/jersey/rest/errors/KapuaExceptionMapper.java 0.00% 4 Missing :warning:
...ersey/rest/errors/KapuaRuntimeExceptionMapper.java 0.00% 4 Missing :warning:
...esources/v1/resources/DeviceManagementBundles.java 0.00% 4 Missing :warning:
.../resources/v1/resources/ServiceConfigurations.java 0.00% 3 Missing :warning:
...sources/v1/resources/DeviceManagementPackages.java 0.00% 2 Missing :warning:
... and 9 more
Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/eclipse/kapua/pull/4073/graphs/tree.svg?width=650&height=150&src=pr&token=1P4N4CApH8&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse)](https://app.codecov.io/gh/eclipse/kapua/pull/4073?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse) ```diff @@ Coverage Diff @@ ## develop #4073 +/- ## ============================================= - Coverage 16.75% 16.56% -0.19% Complexity 22 22 ============================================= Files 2016 2020 +4 Lines 52357 52362 +5 Branches 4417 4413 -4 ============================================= - Hits 8773 8676 -97 - Misses 43184 43285 +101 - Partials 400 401 +1 ``` | [Files](https://app.codecov.io/gh/eclipse/kapua/pull/4073?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse) | Coverage Δ | | |---|---|---| | [...clipse/kapua/client/security/bean/AuthContext.java](https://app.codecov.io/gh/eclipse/kapua/pull/4073?src=pr&el=tree&filepath=client%2Fsecurity%2Fsrc%2Fmain%2Fjava%2Forg%2Feclipse%2Fkapua%2Fclient%2Fsecurity%2Fbean%2FAuthContext.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse#diff-Y2xpZW50L3NlY3VyaXR5L3NyYy9tYWluL2phdmEvb3JnL2VjbGlwc2Uva2FwdWEvY2xpZW50L3NlY3VyaXR5L2JlYW4vQXV0aENvbnRleHQuamF2YQ==) | `0.00% <ø> (ø)` | | | [...ersey/rest/ExceptionConfigurationProviderImpl.java](https://app.codecov.io/gh/eclipse/kapua/pull/4073?src=pr&el=tree&filepath=commons-jersey%2Frest%2Fsrc%2Fmain%2Fjava%2Forg%2Feclipse%2Fkapua%2Fcommons%2Fjersey%2Frest%2FExceptionConfigurationProviderImpl.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse#diff-Y29tbW9ucy1qZXJzZXkvcmVzdC9zcmMvbWFpbi9qYXZhL29yZy9lY2xpcHNlL2thcHVhL2NvbW1vbnMvamVyc2V5L3Jlc3QvRXhjZXB0aW9uQ29uZmlndXJhdGlvblByb3ZpZGVySW1wbC5qYXZh) | `0.00% <ø> (ø)` | | | [...kapua/commons/jersey/rest/JaxbContextResolver.java](https://app.codecov.io/gh/eclipse/kapua/pull/4073?src=pr&el=tree&filepath=commons-jersey%2Frest%2Fsrc%2Fmain%2Fjava%2Forg%2Feclipse%2Fkapua%2Fcommons%2Fjersey%2Frest%2FJaxbContextResolver.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse#diff-Y29tbW9ucy1qZXJzZXkvcmVzdC9zcmMvbWFpbi9qYXZhL29yZy9lY2xpcHNlL2thcHVhL2NvbW1vbnMvamVyc2V5L3Jlc3QvSmF4YkNvbnRleHRSZXNvbHZlci5qYXZh) | `0.00% <ø> (ø)` | | | [...commons/jersey/rest/KapuaCommonApiCoreSetting.java](https://app.codecov.io/gh/eclipse/kapua/pull/4073?src=pr&el=tree&filepath=commons-jersey%2Frest%2Fsrc%2Fmain%2Fjava%2Forg%2Feclipse%2Fkapua%2Fcommons%2Fjersey%2Frest%2FKapuaCommonApiCoreSetting.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse#diff-Y29tbW9ucy1qZXJzZXkvcmVzdC9zcmMvbWFpbi9qYXZhL29yZy9lY2xpcHNlL2thcHVhL2NvbW1vbnMvamVyc2V5L3Jlc3QvS2FwdWFDb21tb25BcGlDb3JlU2V0dGluZy5qYXZh) | `100.00% <100.00%> (ø)` | | | [...ons/jersey/rest/KapuaCommonApiCoreSettingKeys.java](https://app.codecov.io/gh/eclipse/kapua/pull/4073?src=pr&el=tree&filepath=commons-jersey%2Frest%2Fsrc%2Fmain%2Fjava%2Forg%2Feclipse%2Fkapua%2Fcommons%2Fjersey%2Frest%2FKapuaCommonApiCoreSettingKeys.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse#diff-Y29tbW9ucy1qZXJzZXkvcmVzdC9zcmMvbWFpbi9qYXZhL29yZy9lY2xpcHNlL2thcHVhL2NvbW1vbnMvamVyc2V5L3Jlc3QvS2FwdWFDb21tb25BcGlDb3JlU2V0dGluZ0tleXMuamF2YQ==) | `100.00% <100.00%> (ø)` | | | [...mmons/jersey/rest/KapuaSerializableBodyWriter.java](https://app.codecov.io/gh/eclipse/kapua/pull/4073?src=pr&el=tree&filepath=commons-jersey%2Frest%2Fsrc%2Fmain%2Fjava%2Forg%2Feclipse%2Fkapua%2Fcommons%2Fjersey%2Frest%2FKapuaSerializableBodyWriter.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse#diff-Y29tbW9ucy1qZXJzZXkvcmVzdC9zcmMvbWFpbi9qYXZhL29yZy9lY2xpcHNlL2thcHVhL2NvbW1vbnMvamVyc2V5L3Jlc3QvS2FwdWFTZXJpYWxpemFibGVCb2R5V3JpdGVyLmphdmE=) | `100.00% <ø> (ø)` | | | [...ipse/kapua/commons/jersey/rest/ListBodyWriter.java](https://app.codecov.io/gh/eclipse/kapua/pull/4073?src=pr&el=tree&filepath=commons-jersey%2Frest%2Fsrc%2Fmain%2Fjava%2Forg%2Feclipse%2Fkapua%2Fcommons%2Fjersey%2Frest%2FListBodyWriter.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse#diff-Y29tbW9ucy1qZXJzZXkvcmVzdC9zcmMvbWFpbi9qYXZhL29yZy9lY2xpcHNlL2thcHVhL2NvbW1vbnMvamVyc2V5L3Jlc3QvTGlzdEJvZHlXcml0ZXIuamF2YQ==) | `100.00% <ø> (ø)` | | | [...ons/jersey/rest/MoxyJsonConfigContextResolver.java](https://app.codecov.io/gh/eclipse/kapua/pull/4073?src=pr&el=tree&filepath=commons-jersey%2Frest%2Fsrc%2Fmain%2Fjava%2Forg%2Feclipse%2Fkapua%2Fcommons%2Fjersey%2Frest%2FMoxyJsonConfigContextResolver.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse#diff-Y29tbW9ucy1qZXJzZXkvcmVzdC9zcmMvbWFpbi9qYXZhL29yZy9lY2xpcHNlL2thcHVhL2NvbW1vbnMvamVyc2V5L3Jlc3QvTW94eUpzb25Db25maWdDb250ZXh0UmVzb2x2ZXIuamF2YQ==) | `100.00% <ø> (ø)` | | | [...e/kapua/commons/jersey/rest/RestApiErrorCodes.java](https://app.codecov.io/gh/eclipse/kapua/pull/4073?src=pr&el=tree&filepath=commons-jersey%2Frest%2Fsrc%2Fmain%2Fjava%2Forg%2Feclipse%2Fkapua%2Fcommons%2Fjersey%2Frest%2FRestApiErrorCodes.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse#diff-Y29tbW9ucy1qZXJzZXkvcmVzdC9zcmMvbWFpbi9qYXZhL29yZy9lY2xpcHNlL2thcHVhL2NvbW1vbnMvamVyc2V5L3Jlc3QvUmVzdEFwaUVycm9yQ29kZXMuamF2YQ==) | `100.00% <ø> (ø)` | | | [...a/commons/jersey/rest/RestApiRuntimeException.java](https://app.codecov.io/gh/eclipse/kapua/pull/4073?src=pr&el=tree&filepath=commons-jersey%2Frest%2Fsrc%2Fmain%2Fjava%2Forg%2Feclipse%2Fkapua%2Fcommons%2Fjersey%2Frest%2FRestApiRuntimeException.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse#diff-Y29tbW9ucy1qZXJzZXkvcmVzdC9zcmMvbWFpbi9qYXZhL29yZy9lY2xpcHNlL2thcHVhL2NvbW1vbnMvamVyc2V5L3Jlc3QvUmVzdEFwaVJ1bnRpbWVFeGNlcHRpb24uamF2YQ==) | `100.00% <ø> (ø)` | | | ... and [155 more](https://app.codecov.io/gh/eclipse/kapua/pull/4073?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse) | | ... and [3 files with indirect coverage changes](https://app.codecov.io/gh/eclipse/kapua/pull/4073/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse)