Fix invalid behavior for the cloud build option for the test and run commands

[gayaldassanayake]

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues.

Fixes #43940

Approach

1. Passed hard-coded empty string for the cloud build option for the run command. This overrides any value passed for the cloud option in the toml.

2. Instead of checking if cloud option is present, checked if the cloud option is available, before skipping tests.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

• Read the Contributing Guide
• Updated Change Log
• Checked Tooling Support (#)
• Added necessary tests
  • Unit Tests
  • Spec Conformance Tests
  • Integration Tests
  • Ballerina By Example Tests
• Increased Test Coverage
• Added necessary documentation
  • API documentation
  • Module documentation in Module.md files
  • Ballerina By Examples

[Copilot]

Pull Request Overview

This PR fixes the invalid behavior of the cloud build option for the test and run commands by updating the conditions that govern test execution delegation and ensuring cloud-related flags are handled correctly. Key changes include:

• In RunCommand, setting the cloud option to an empty string to bypass cloud import.
• In TestCommand and TestProcessor, refactoring the logic to delegate test execution when the cloud flag is set to "docker".
• Updating test cases in TestCommandTest to validate behavior under different cloud options.

Reviewed Changes

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

Show a summary per file

File	Description
cli/ballerina-cli/src/test/java/io/ballerina/cli/cmd/TestCommandTest.java	Updates tests to use the USER_HOME constant and validates new cloud option behavior.
cli/ballerina-cli/src/main/java/io/ballerina/cli/cmd/RunCommand.java	Sets the cloud option to an empty string for the run command, bypassing cloud-related imports.
misc/testerina/modules/testerina-core/src/main/java/org/ballerinalang/testerina/core/TestProcessor.java	Refactors conditions to delegate tests based on a new helper method that checks for "docker" cloud option.
cli/ballerina-cli/src/main/java/io/ballerina/cli/cmd/TestCommand.java	Refactors test task delegation by introducing an isTestingDelegated flag for enhanced clarity.
cli/ballerina-cli/src/main/java/io/ballerina/cli/task/RunTestsTask.java	Removes the early exit for cloud builds to accommodate the updated test execution delegation logic.

Comments suppressed due to low confidence (2)

cli/ballerina-cli/src/main/java/io/ballerina/cli/cmd/TestCommand.java:364

• [nitpick] The use of a hardcoded "docker" value in the isTestingDelegated flag assumes that only "docker" should delegate test execution. Consider verifying if other cloud values should be handled to avoid unintended behavior.

boolean isTestingDelegated = project.buildOptions().cloud().equals("docker");

cli/ballerina-cli/src/main/java/io/ballerina/cli/task/RunTestsTask.java:142

• The removal of the early return for non-empty cloud options in RunTestsTask means tests will now execute even for cloud builds. Ensure this change aligns with the intended behavior of delegating test execution appropriately.

//do not execute if cloud option is given, we only use the object to use the properties and methods in it

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.06%. Comparing base (f3b07a4) to head (1c3a4f0).
Report is 13 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff            @@
##             master   #43941   +/-   ##
=========================================
  Coverage     75.05%   75.06%           
- Complexity    57512    57523   +11     
=========================================
  Files          3540     3540           
  Lines        222373   222398   +25     
  Branches      28835    28836    +1     
=========================================
+ Hits         166908   166938   +30     
+ Misses        46201    46197    -4     
+ Partials       9264     9263    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.