Skip to content

test: fix ValkeyStandalone qualifier typo#89

Merged
jeremyprime merged 3 commits into
valkey-io:mainfrom
charles-openclaw:fix-valkey-standalone-typo
Jun 2, 2026
Merged

test: fix ValkeyStandalone qualifier typo#89
jeremyprime merged 3 commits into
valkey-io:mainfrom
charles-openclaw:fix-valkey-standalone-typo

Conversation

@charles-openclaw
Copy link
Copy Markdown
Contributor

Summary

  • rename the test qualifier annotation from ValkeyStanalone to ValkeyStandalone
  • update imports, references, and related Javadoc/test names to use the corrected spelling

Verification

  • rg -n 'ValkeyStanalone|Stanalone' spring-data-valkey/src/test/java
  • JAVA_HOME=/home/brad/.openclaw/workspace/.build-tools/jdk-21 ./mvnw -pl spring-data-valkey -DskipTests test-compile
  • git diff --check

Issue reference

Closes #88

@jeremyprime
Copy link
Copy Markdown
Collaborator

@charles-openclaw Thanks for the PR! The DCO check failed, as all commits need to be signed off. You can fix this by amending your commit, for example:

git commit --amend --signoff
git push --force-with-lease

Signed-off-by: Charles <charles-openclaw@9bcfae.inboxapi.ai>
@charles-openclaw charles-openclaw force-pushed the fix-valkey-standalone-typo branch from facbcd9 to 55e3283 Compare June 1, 2026 15:59
@jeremyprime
Copy link
Copy Markdown
Collaborator

Just noticed 1 more file needs updating (added since this PR was opened), ValkeyGlideZSetOrderingIntegrationTests still references ValkeyStanalone.

…ne-typo

Signed-off-by: Charles <charles-openclaw@9bcfae.inboxapi.ai>
Signed-off-by: Charles <charles-openclaw@9bcfae.inboxapi.ai>
@charles-openclaw
Copy link
Copy Markdown
Contributor Author

Thanks for the catch. I updated the branch for the new ValkeyGlideZSetOrderingIntegrationTests file and fixed its remaining ValkeyStanalone references to ValkeyStandalone.

Verification:

  • rg -n 'ValkeyStanalone|Stanalone' spring-data-valkey/src/test/java/io/valkey/springframework/data/valkey/connection/valkeyglide/ValkeyGlideZSetOrderingIntegrationTests.java || true returned no matches
  • JAVA_HOME=/home/brad/.openclaw/workspace/.build-tools/jdk-21 ./mvnw -pl spring-data-valkey -DskipTests test-compile passed
  • git diff --check -- spring-data-valkey/src/test/java/io/valkey/springframework/data/valkey/connection/valkeyglide/ValkeyGlideZSetOrderingIntegrationTests.java passed

I also signed off the upstream merge commit and the follow-up commit for DCO.

@jeremyprime jeremyprime merged commit c6d21c9 into valkey-io:main Jun 2, 2026
22 checks passed
@jeremyprime
Copy link
Copy Markdown
Collaborator

@charles-openclaw Thanks for your contribution!

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.

Fix test interface ValkeyStanalone typo

3 participants