Skip to content

Move MOG models to :core:domain - #3897

Open
shobhitagarwal1612 wants to merge 3 commits into
masterfrom
fresh
Open

Move MOG models to :core:domain#3897
shobhitagarwal1612 wants to merge 3 commits into
masterfrom
fresh

Conversation

@shobhitagarwal1612

@shobhitagarwal1612shobhitagarwal1612 commented Aug 25, 2026

Copy link
Copy Markdown
Member

Towards #3633

This PR moves the MOG models, and all of the classes it uses, to the KMP shared module :core:domain.

Most of the changes in this PR are just import changes. Also adds missing test coverage for the model classes.

@andreia-ferreira PTAL?

@codecov

codecovBot commented Aug 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.02%. Comparing base (3213cb5) to head (6ca64ce).

Additional details and impacted files
@@ Coverage Diff @@## master #3897 +/- ##
============================================
+ Coverage 68.82% 69.02% +0.20% - Complexity 1933 1941 +8 
============================================
Files 411 411 Lines 10968 10968 Branches 1428 1428 ============================================
+ Hits 7549 7571 +22 + Misses 2674 2652 -22 
Partials 745 745 
Files with missing linesCoverage Δ
...atform/android/repository/OfflineAreaRepository.kt67.56% <ø> (ø)
...groundplatform/android/ui/map/gms/mog/MogClient.kt48.10% <ø> (ø)
...atform/android/ui/map/gms/mog/MogSourceProvider.kt100.00% <ø> (ø)
...g/groundplatform/android/ui/map/gms/mog/MogTile.kt90.32% <ø> (ø)
...atform/android/ui/map/gms/mog/MogTileDownloader.kt23.07% <ø> (ø)
...ndplatform/android/ui/map/gms/mog/MogTileReader.kt66.66% <ø> (ø)
...oundplatform/domain/model/imagery/MogCollection.kt100.00% <ø> (ø)
...dplatform/domain/model/imagery/MogImageMetadata.kt77.55% <ø> (ø)
...groundplatform/domain/model/imagery/MogMetadata.kt85.71% <ø> (ø)
...g/groundplatform/domain/model/imagery/MogSource.kt84.61% <ø> (ø)
... and 2 more

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@shobhitagarwal1612