Repository navigation
Add option to skip unit tests in reusable workflow - #3768
Conversation
π‘οΈ Jit Security Scan Resultsβ No security findings were detected in this PR
Security scan by Jit
|
β Jit Scanner failed - Our team is investigatingJit Scanner failed - Our team has been notified and is working to resolve the issue. Please contact support if you have any questions. π‘ Need to bypass this check? Comment |
1 similar comment
β Jit Scanner failed - Our team is investigatingJit Scanner failed - Our team has been notified and is working to resolve the issue. Please contact support if you have any questions. π‘ Need to bypass this check? Comment |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
β Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit d3fd112. Configure here.
β Jit Scanner failed - Our team is investigatingJit Scanner failed - Our team has been notified and is working to resolve the issue. Please contact support if you have any questions. π‘ Need to bypass this check? Comment |
4 similar comments
β Jit Scanner failed - Our team is investigatingJit Scanner failed - Our team has been notified and is working to resolve the issue. Please contact support if you have any questions. π‘ Need to bypass this check? Comment |
β Jit Scanner failed - Our team is investigatingJit Scanner failed - Our team has been notified and is working to resolve the issue. Please contact support if you have any questions. π‘ Need to bypass this check? Comment |
β Jit Scanner failed - Our team is investigatingJit Scanner failed - Our team has been notified and is working to resolve the issue. Please contact support if you have any questions. π‘ Need to bypass this check? Comment |
β Jit Scanner failed - Our team is investigatingJit Scanner failed - Our team has been notified and is working to resolve the issue. Please contact support if you have any questions. π‘ Need to bypass this check? Comment |
50d9eda to
2079b6a
Compare
β Jit Scanner failed - Our team is investigatingJit Scanner failed - Our team has been notified and is working to resolve the issue. Please contact support if you have any questions. π‘ Need to bypass this check? Comment |
1 similar comment
β Jit Scanner failed - Our team is investigatingJit Scanner failed - Our team has been notified and is working to resolve the issue. Please contact support if you have any questions. π‘ Need to bypass this check? Comment |

Adjust the integration infra to support ENV_VAR for skipping unit tests on demand.
Note
Low Risk
Changes are limited to CI wiring and test isolation; no production client or security behavior is modified.
Overview
Adds a
skip_unit_testspath through reusable CI (run-tests.yml,integration.yml) so custom client-libs image runs can execute integration tests only viamake test MVN_EXTRA_ARGS="-DskipUnitTests=true"with coverage upload disabled. Maven gainsskipUnitTests(defaults fromskipTests) and SurefireskipTestsbinding; the Makefile forwardsMVN_EXTRA_ARGSontest/test-coverage.Integration tests are adjusted for per-method Redis keys:
TestSupport.keyis no longer a shared static"key"but is set in@BeforeEachtokey-<methodName>, with follow-on fixes (hash-tagged secondary keys,ClusterTestSettings.KEY_Afor replica scenarios, scan/match prefixes, stream assertions usingkey).Reviewed by Cursor Bugbot for commit 2079b6a. Bugbot is set up for automated code reviews on this repo. Configure here.