Skip to content

refactor: improve null safety using Scala Option #1186

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.

Sign up for GitHub

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

Open
pallakartheekreddy wants to merge 13 commits into
base: cloud-store-refactoring
Choose a base branch
from

Merge branch 'develop' into refactor/null-safety-option

4feea45
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Open

refactor: improve null safety using Scala Option #1186

Merge branch 'develop' into refactor/null-safety-option
4feea45
Select commit
Loading
Failed to load commit list.
GitHub Actions / Content Test Results succeeded Feb 27, 2026 in 0s

110 passed, 0 failed and 0 skipped

Tests passed successfully

Report Passed Failed Skipped Time
content-api/content-service/target/surefire-reports/TEST-controllers.v3.BadRequestSpec.xml 1✅ 379ms
content-api/content-service/target/surefire-reports/TEST-controllers.v3.ChannelSpec.xml 4✅ 165ms
content-api/content-service/target/surefire-reports/TEST-controllers.v3.ContentSpec.xml 27✅ 305ms
content-api/content-service/target/surefire-reports/TEST-controllers.v3.HealthSpec.xml 1✅ 157ms
content-api/content-service/target/surefire-reports/TEST-controllers.v3.LicenseSpec.xml 4✅ 169ms
content-api/content-service/target/surefire-reports/TEST-controllers.v4.AppSpec.xml 3✅ 164ms
content-api/content-service/target/surefire-reports/TEST-controllers.v4.AssetSpec.xml 11✅ 259ms
content-api/content-service/target/surefire-reports/TEST-controllers.v4.CollectionSpec.xml 22✅ 715ms
content-api/content-service/target/surefire-reports/TEST-controllers.v4.ContentSpec.xml 23✅ 490ms
content-api/content-service/target/surefire-reports/TEST-controllers.v4.EventSetSpec.xml 8✅ 203ms
content-api/content-service/target/surefire-reports/TEST-controllers.v4.EventSpec.xml 5✅ 5s
content-api/content-service/target/surefire-reports/TEST-controllers.v4.ObjectSpec.xml 1✅ 136ms

✅ content-api/content-service/target/surefire-reports/TEST-controllers.v3.BadRequestSpec.xml

1 tests were completed in 379ms with 1 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
controllers.v3.BadRequestSpec 1✅ 379ms

✅ controllers.v3.BadRequestSpec

✅ Application should::send 404 on a bad request - /boum

✅ content-api/content-service/target/surefire-reports/TEST-controllers.v3.ChannelSpec.xml

4 tests were completed in 165ms with 4 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
controllers.v3.ChannelSpec 4✅ 165ms

✅ controllers.v3.ChannelSpec

✅ Channel Controller  should::return success response for create API
✅ return success response for update API
✅ return success response for read API
✅ return success response for retire API

✅ content-api/content-service/target/surefire-reports/TEST-controllers.v3.ContentSpec.xml

27 tests were completed in 305ms with 27 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
controllers.v3.ContentSpec 27✅ 305ms

✅ controllers.v3.ContentSpec

✅ Content Controller  should::return success response for create API
✅ Content Controller  should::return success response for update API
✅ Content Controller  should::return success response for read API
✅ Content Controller  should::return success response for hierarchy add API
✅ Content Controller  should::return success response for hierarchy remove API
✅ Content Controller  should::return success response for hierarchy get API
✅ Content Controller  should::return success response for flag API
✅ Content Controller  should::return success response for acceptFlag API
✅ Content Controller  should::return success response for rejectFlag API
✅ Content Controller  should::return success response for bundle API
✅ Content Controller  should::return success response for publish API
✅ Content Controller  should::return success response for review API
✅ Content Controller  should::return success response for discard API
✅ Content Controller  should::return success response for retire API
✅ Content Controller  should::return success response for linkDialCode API
✅ Content Controller  should::return success response for collectionLinkDialCode API
✅ Content Controller  should::return success response for reserveDialCode API
✅ Content Controller  should::return success response for releaseDialcodes API
✅ Content Controller  should::return success response for publishUnlisted API
✅ Content Controller  should::return success response for presignedUrl upload API
✅ return success response for upload API with file
✅ return success response for upload API with fileUrl
✅ return success response for upload API with fileUrl and fileFormat
✅ return success response for importContent API
✅ return success response for hierarchy update API
✅ return success response for bookmark hierarchy API
✅ return success response for copy API

✅ content-api/content-service/target/surefire-reports/TEST-controllers.v3.HealthSpec.xml

1 tests were completed in 157ms with 1 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
controllers.v3.HealthSpec 1✅ 157ms

✅ controllers.v3.HealthSpec

✅ Application should::return api health status report - successful

✅ content-api/content-service/target/surefire-reports/TEST-controllers.v3.LicenseSpec.xml

4 tests were completed in 169ms with 4 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
controllers.v3.LicenseSpec 4✅ 169ms

✅ controllers.v3.LicenseSpec

✅ License Controller  should::return success response for create API
✅ License Controller  should::return success response for update API
✅ License Controller  should::return success response for read API
✅ License Controller  should::return success response for retire API

✅ content-api/content-service/target/surefire-reports/TEST-controllers.v4.AppSpec.xml

3 tests were completed in 164ms with 3 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
controllers.v4.AppSpec 3✅ 164ms

✅ controllers.v4.AppSpec

✅ AppController  should::return success response for register API
✅ AppController  should::return success response for update API
✅ AppController  should::return success response for read API

✅ content-api/content-service/target/surefire-reports/TEST-controllers.v4.AssetSpec.xml

11 tests were completed in 259ms with 11 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
controllers.v4.AssetSpec 11✅ 259ms

✅ controllers.v4.AssetSpec

✅ AssetController  should::return success response for create API
✅ AssetController  should::return success response for update API
✅ AssetController  should::return success response for read API
✅ AssetController  should::return success response for upload API with file
✅ AssetController  should::return success response for upload API with fileUrl
✅ AssetController  should::return success response for upload API with fileUrl and fileFormat
✅ AssetController  should::return success response for pre signed Url upload API
✅ AssetController  should::return success response for copy API
✅ AssetController  should::return success response for licenceValidate API
✅ Asset controller with invalid request  should::return client error response for create API
✅ Asset controller with invalid request  should::return client error response for create API

✅ content-api/content-service/target/surefire-reports/TEST-controllers.v4.CollectionSpec.xml

22 tests were completed in 715ms with 22 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
controllers.v4.CollectionSpec 22✅ 715ms

✅ controllers.v4.CollectionSpec

✅ Collection Controller  should::return success response for create API
✅ Collection Controller  should::return success response for update API
✅ Collection Controller  should::return success response for read API
✅ Collection Controller  should::return success response for private read API
✅ Collection Controller  should::return success response for hierarchy add API
✅ Collection Controller  should::return success response for hierarchy remove API
✅ Collection Controller  should::return success response for hierarchy get API
✅ Collection Controller  should::return success response for flag API
✅ Collection Controller  should::return success response for acceptFlag API
✅ Collection Controller  should::return success response for discard API
✅ Collection Controller  should::return success response for retire API
✅ Collection Controller  should::return success response for collectionLinkDialCode API
✅ Collection Controller  should::return success response for collection review reject API
✅ Collection Controller  should::return success response for collection review API
✅ return success response for hierarchy update API
✅ return success response for bookmark hierarchy API
✅ return success response for copy API
✅ Collection Controller with invalid request  should::return client error response for create API
✅ Collection Controller with invalid request  should::return client error response for create API
✅ Collection Controller with valid request  should::return success response for systemUpdate API
✅ Collection Controller with valid request  should::return success response for dialcode reserve API
✅ Collection Controller with valid request  should::return success response for dialcode release API

✅ content-api/content-service/target/surefire-reports/TEST-controllers.v4.ContentSpec.xml

23 tests were completed in 490ms with 23 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
controllers.v4.ContentSpec 23✅ 490ms

✅ controllers.v4.ContentSpec

✅ Content Controller  should::return success response for create API
✅ Content Controller  should::return success response for update API
✅ Content Controller  should::return success response for read API
✅ Content Controller  should::return success response for private read API
✅ Content Controller  should::return success response for flag API
✅ Content Controller  should::return success response for acceptFlag API
✅ Content Controller  should::return success response for discard API
✅ Content Controller  should::return success response for retire API
✅ Content Controller  should::return success response for linkDialCode API
✅ Content Controller  should::return success response for presignedUrl upload API
✅ Content Controller  should::return success response for rejectContent API
✅ Content Controller  should::return success response for review API
✅ Content Controller  should::return success response for reviewReject API
✅ return success response for upload API with file
✅ return success response for upload API with fileUrl
✅ return success response for upload API with fileUrl and fileFormat
✅ return success response for importContent API
✅ return success response for copy API
✅ Content Controller with invalid request  should::return client error response for create API
✅ Content Controller with invalid request  should::return client error response for create API
✅ Content Controller with valid request  should::return success response for systemUpdate API
✅ Content Controller with valid request  should::return success response for dialcode reserve API
✅ Content Controller with valid request  should::return success response for dialcode release API

✅ content-api/content-service/target/surefire-reports/TEST-controllers.v4.EventSetSpec.xml

8 tests were completed in 203ms with 8 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
controllers.v4.EventSetSpec 8✅ 203ms

✅ controllers.v4.EventSetSpec

✅ EventSet Controller  should::return success response for create API
✅ EventSet Controller  should::return success response for update API
✅ EventSet Controller  should::return success response for read API
✅ EventSet Controller  should::return success response for hierarchy get API
✅ EventSet Controller  should::return success response for discard API
✅ EventSet Controller  should::return success response for retire API
✅ EventSet Controller  should::return error response when updating status using update API
✅ EventSet Controller  should::return success response for publish API

✅ content-api/content-service/target/surefire-reports/TEST-controllers.v4.EventSpec.xml

5 tests were completed in 5s with 5 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
controllers.v4.EventSpec 5✅ 5s

✅ controllers.v4.EventSpec

✅ Event Controller  should::return success response for create API
✅ Event Controller  should::return success response for read API
✅ Event Controller  should::return success response for update API
✅ Event Controller  should::return error response when updating status using update API
✅ Event Controller  should::return success response for publish API

✅ content-api/content-service/target/surefire-reports/TEST-controllers.v4.ObjectSpec.xml

1 tests were completed in 136ms with 1 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
controllers.v4.ObjectSpec 1✅ 136ms

✅ controllers.v4.ObjectSpec

✅ Object controller should::return success response for read API
View more details on GitHub Actions