Skip to content

Add option to skip unit tests in reusable workflow - #3768

Merged
a-TODO-rov merged 2 commits into
mainfrom
skip-unit-tests
Jun 1, 2026
Merged

a-TODO-rov merged 2 commits into
mainfrom
skip-unit-tests

Conversation

@a-TODO-rov

@a-TODO-rov a-TODO-rov commented May 29, 2026 •

Copy link
Copy Markdown
Collaborator

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_tests path through reusable CI (run-tests.yml, integration.yml) so custom client-libs image runs can execute integration tests only via make test MVN_EXTRA_ARGS="-DskipUnitTests=true" with coverage upload disabled. Maven gains skipUnitTests (defaults from skipTests) and Surefire skipTests binding; the Makefile forwards MVN_EXTRA_ARGS on test / test-coverage.

Integration tests are adjusted for per-method Redis keys: TestSupport.key is no longer a shared static "key" but is set in @BeforeEach to key-<methodName>, with follow-on fixes (hash-tagged secondary keys, ClusterTestSettings.KEY_A for replica scenarios, scan/match prefixes, stream assertions using key).

Reviewed by Cursor Bugbot for commit 2079b6a. Bugbot is set up for automated code reviews on this repo. Configure here.

@jit-ci

jit-ci Bot commented May 29, 2026

Copy link
Copy Markdown

πŸ›‘οΈ Jit Security Scan Results

CRITICAL HIGH MEDIUM

βœ… No security findings were detected in this PR


Security scan by Jit

@jit-ci

jit-ci Bot commented May 29, 2026

Copy link
Copy Markdown

❌ Jit Scanner failed - Our team is investigating

Jit 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 @sera bypass to override.

1 similar comment
@jit-ci

jit-ci Bot commented May 29, 2026

Copy link
Copy Markdown

❌ Jit Scanner failed - Our team is investigating

Jit 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 @sera bypass to override.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ 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.

Comment thread pom.xml
@jit-ci

jit-ci Bot commented May 29, 2026

Copy link
Copy Markdown

❌ Jit Scanner failed - Our team is investigating

Jit 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 @sera bypass to override.

4 similar comments
@jit-ci

jit-ci Bot commented May 29, 2026

Copy link
Copy Markdown

❌ Jit Scanner failed - Our team is investigating

Jit 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 @sera bypass to override.

@jit-ci

jit-ci Bot commented May 29, 2026

Copy link
Copy Markdown

❌ Jit Scanner failed - Our team is investigating

Jit 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 @sera bypass to override.

@jit-ci

jit-ci Bot commented May 29, 2026

Copy link
Copy Markdown

❌ Jit Scanner failed - Our team is investigating

Jit 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 @sera bypass to override.

@jit-ci

jit-ci Bot commented May 29, 2026

Copy link
Copy Markdown

❌ Jit Scanner failed - Our team is investigating

Jit 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 @sera bypass to override.

@jit-ci

jit-ci Bot commented May 29, 2026

Copy link
Copy Markdown

❌ Jit Scanner failed - Our team is investigating

Jit 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 @sera bypass to override.

1 similar comment
@jit-ci

jit-ci Bot commented May 31, 2026

Copy link
Copy Markdown

❌ Jit Scanner failed - Our team is investigating

Jit 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 @sera bypass to override.

@a-TODO-rov
a-TODO-rov requested a review from uglide June 1, 2026 07:15
@a-TODO-rov
a-TODO-rov merged commit d8823f7 into main Jun 1, 2026
18 of 21 checks passed
@a-TODO-rov
a-TODO-rov deleted the skip-unit-tests branch June 1, 2026 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant