Fix CI workflow to run modern and legacy tests with proper isolation

[Copilot]

CI build failing because modern (JUnit 5) and legacy (JUnit 4) tests run together without proper isolation, causing Spring context conflicts. The make script handles this correctly by running tests in separate phases.

Changes

• Phase 1 - Modern tests: Build with -Pmodern-tests -DskipLegacyTests=true and modern lock file
• Phase 2 - Legacy tests: Clean, rebuild without modern profile, run with -DskipModernTests=true
• Added -T 1C for multi-threaded builds matching make script behavior

Before

- name: Run unit tests & build JSP files
  run: docker exec openo-tomcat-dev mvn package -Pjspc,modern-tests -Ddependency.lock.filename=dependencies-lock-modern.json

After

# Phase 1: Modern tests
- name: Build for modern tests
  run: docker exec openo-tomcat-dev mvn clean -DskipModernTests=true -DskipLegacyTests=true -T 1C package war:exploded -Pjspc,modern-tests -Ddependency.lock.filename=dependencies-lock-modern.json

- name: Run modern tests
  run: docker exec openo-tomcat-dev mvn test -Pmodern-tests -DskipLegacyTests=true -Ddependency.lock.filename=dependencies-lock-modern.json

- name: Clean up between test phases
  run: docker exec openo-tomcat-dev mvn clean -q

# Phase 2: Legacy tests  
- name: Build for legacy tests
  run: docker exec openo-tomcat-dev mvn clean -DskipModernTests=true -DskipLegacyTests=true -T 1C package war:exploded -Pjspc

- name: Run legacy tests
  run: docker exec openo-tomcat-dev mvn test -DskipModernTests=true

Mirrors the isolation approach in .devcontainer/development/scripts/make when running make install --run-tests.

Original prompt

Currently the build test for the github pipeline is failing because the scripts to run it are not properly running the seperate test and legacy tests with the right scope, as the "make" script in the dev container does. Fix this so does not fail because of this issue.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[github-actions]

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

[keploy]

To generate Unit Tests for this PR, please click here.

[yingbull]

Unable to reassign; closing. Can be reopened by openosp. @kk-chung

[yingbull]

Unable to reassign; requesting close. Can be reopened by openosp. @kk-chung

[sebastian-j-ibanez]

Closing - creating different builds for the modern and legacy tests was addressed in PR #1792.