feat: check app enclave configuration in preflight checks

[jbern0rd]

No description provided.

[sonarqubecloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

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

See analysis details on SonarQube Cloud

[Copilot]

Pull Request Overview

This PR refactors error handling in TEE services configuration by moving validation logic from PreComputeService to TeeServicesPropertiesService. The key change is that retrieveTeeServicesProperties now returns a list of WorkflowError instead of throwing exceptions, enabling better error propagation and handling.

• Enclave configuration validation (null checks, validity checks, heap size validation) is moved from PreComputeService.runTeePreCompute() to TeeServicesPropertiesService.retrieveTeeServicesProperties()
• Method getTeeServicesProperties now returns cached properties or null, while retrieveTeeServicesProperties performs validation and caching
• Exception-based error handling replaced with WorkflowError return values for better error flow control

Reviewed Changes

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

Show a summary per file

File	Description
TeeServicesPropertiesService.java	Refactored to perform enclave validation and return WorkflowError list instead of throwing exceptions; properties now cached only on success
TeeService.java	Updated to call retrieveTeeServicesProperties and return its error list directly instead of catching exceptions
PreComputeService.java	Removed enclave configuration validation logic that was moved to TeeServicesPropertiesService
TeeSconeServiceTests.java	Updated tests to match new API (returns List<WorkflowError> instead of throwing exceptions); consolidated two test cases into one parameterized test
TeeServicesPropertiesServiceTests.java	Updated tests to validate new error-returning behavior instead of exception-throwing behavior; added new test cases for heap size validation
PreComputeServiceTests.java	Removed tests for enclave validation logic that was moved to TeeServicesPropertiesService

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.