Don't require connection for event sync if CommCare downsync source is configured

[BurningAXE]

JIRA ticket
Will be released in: 2025.3.0

Root cause analysis (for bugfixes only)

First known affected version: 2025.3.0-dev

• In WorkManager.ext we have defaultWorkerConstraints that set NetworkType.CONNECTED
• Unless explicitly specified, these default constraints apply to all periodic and one-time worker requests
• So far connection has indeed been required by all workers
• With the introduction of CommCare downsync source that's no longer the case as syncing happens locally on the device
• CommCareEventSyncDownloaderWorker is indeed built with no network connection constraint
• However, EventSyncMasterWorker requires connection so it never runs to build CommCareEventSyncDownloaderWorker

Notable changes

• When scheduling event sync in SyncOrchestratorImpl (either periodic or one time) we check if CommCare downsync source is configured and set connection constraints accordingly

Testing guidance

• New test case was added to verify this important condition

Additional work checklist

• Effect on other features and security has been considered
• Design document marked as "In development" (if applicable)
• External (Gitbook) and internal (Confluence) Documentation is up to date (or ticket created)
• Test cases in Testiny are up to date (or ticket created)
• Other teams notified about the changes (if applicable)

[Copilot]

Pull Request Overview

This PR fixes an issue where the EventSyncMasterWorker was preventing CommCare downsync from functioning due to unnecessary network connection requirements.

Key changes:

• Modified event sync scheduling to conditionally set network constraints based on whether CommCare downsync is configured
• Made event sync methods suspend functions to allow async configuration retrieval
• Updated debug UI to handle the new suspend function calls

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
SyncOrchestratorImpl.kt	Added dynamic network constraint logic and made event sync methods suspend
SyncOrchestrator.kt	Updated interface to make event sync methods suspend functions
DebugFragment.kt	Added coroutine scope launches to handle suspend function calls

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
94.1% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud