Add cache changes for successful cache upload and cache hit, Add support for COMRPC L2Tests plugin

[smanes0213]

Reason for change: Resolve intermittent failures while running tests on testframework repo
Test Procedure: Run L1, L2, L2-OOP(If applicable)
Risks: Low
Priority: P0
Version:minor

[Copilot]

Pull request overview

This pull request modifies the L1-tests workflow to improve caching behavior and updates several repository references. The changes aim to resolve intermittent test failures by adjusting cache paths, improving cache key specificity with a new REPO_NAME variable, and updating branch references for testing purposes.

Key changes:

• Updated cache paths from thunder/ prefix to direct paths (build/, install/)
• Added REPO_NAME environment variable to make cache keys more specific
• Updated repository branch references to feature/test branches for entservices-testframework, entservices-casting, and entservices-apis
• Added cache-hit conditionals to patch application steps
• Removed redundant Thunder/Source include paths from build commands

[Copilot]

The entservices-testframework repository is being checked out from a feature branch 'feature/workflow-test' instead of the production branch. This should use a stable branch reference like 'develop' or a specific release tag to ensure consistent and reliable CI/CD pipeline execution.

Suggested change

	ref: feature/workflow-test
	ref: develop

[Copilot]

The entservices-casting repository is being checked out from a feature branch 'feature/workflowtest_L1-L2' instead of a stable branch. This should reference a production-ready branch like 'develop' or a specific release tag for stable CI/CD execution.

Suggested change

	ref: feature/workflowtest_L1-L2
	ref: develop

[Copilot]

The entservices-apis repository is being checked out from a branch named 'Test' instead of using the environment variable INTERFACES_REF which points to 'develop'. This hardcoded test branch reference should be reverted to use the INTERFACES_REF variable for consistency and stability.

Suggested change

	ref: Test
	ref: ${{ env.INTERFACES_REF }}

[Copilot]

Pull request overview

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

[Copilot]

The 'Checkout Thunder' step no longer has the cache-hit condition guard. Since Thunder is always checked out regardless of cache status, but only built when cache misses (line 184), this could lead to wasted API calls and disk usage when the cache is hit. Consider adding back 'if: steps.cache.outputs.cache-hit != true' unless Thunder checkout is needed for other purposes when cache hits.

Suggested change

	- name: Checkout Thunder
	- name: Checkout Thunder
	if: steps.cache.outputs.cache-hit != 'true'

[Copilot]

The 'Checkout entservices-apis' step removed the cache-hit condition guard, but the 'Build entservices-apis' step (line 217) does not have a cache-hit guard either. This means entservices-apis will be checked out and rebuilt every time, even when cached. Since the cache includes 'build/entservices-apis', this checkout should either have a cache-hit guard or the build should be conditional to avoid unnecessary work.

[Copilot]

The cache key now includes REPO_NAME but removes it from the INTERFACES_REF which was changed to a hardcoded 'Test' branch (line 207). This breaks the cache key's ability to track interface changes. The key should continue to track the actual interfaces reference being used, not the hardcoded 'Test' branch.