Conversation
- Replace deprecated node10 moduleResolution with NodeNext - Update module to NodeNext to match requirement - Resolves TypeScript 7.0 deprecation warning
…ine-structure Refactor(pipeline)/clean data pipeline structure
…onents Feat(pipeline)/revise missing components
feat(api): initial schema baseline and docker setup
chore(ci): separate migration workflow and add push triggers
…for watermelondb version 3 migration
…ansion Feat(mobile)/watermelondb sync expansion
…ols-and-guide feat(pipeline): add extracted demo dataset and usage guide
chore(shared): setup docker postgres, ci, mobile auth, and schema upd…
Feat(mobile)/offline expenses sync
Chore(shared)/coverage and e2e
…kend-integration-flows ## Description This PR introduces comprehensive test coverage across both the `api` and `mobile` apps to achieve our >70% coverage threshold, while specifically focusing on the mobile client's backend-integration flows and pipeline stabilization. ### Key Changes - **API Coverage**: Created missing `spec.ts` test suites for core backend services (`auth`, `warranty`, `commissioning`, `jobs`, `scheduling`, `service-logs`, `sync-conflicts`, and `roles`). - **Mobile Integration Tests**: Added robust backend-integration tests for `HomeScreen` data fetching, `unified-sync` workflows, and the `Analytics` screen (specifically mocking and verifying the "Add Expense" modal). - **Sync Fixes**: Resolved case-sensitivity issues in `unified-sync.service` (changed `PENDING` to `pending`) to ensure queue items are successfully picked up and processed. - **Currency Standards**: Enforced Bosnian `BAM` / `KM` as the default monetary currency across expense tracking and analytics summaries. - **CI Pipeline Stabilisation**: Appended the `--forceExit` flag to our Jest test runner in `.github/workflows/ci.yml`. This fixes a critical pipeline bug where the CI would hang indefinitely waiting on active autosave timers spawned by WatermelonDB's `LokiJS` test adapter. - **Memory & Handle Leaks**: Addressed various leaking `fetch` mock promises and asynchronous tear-downs to appease `--detectOpenHandles`.
chore(api): add api e2e tests
|
View your CI Pipeline Execution ↗ for commit ae65798
☁️ Nx Cloud last updated this comment at |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Version 1.1