Skip to content

test(core): add getZosmfUrl unit tests#483

Merged
frankgiordano merged 1 commit intozowe:mainfrom
AHS0003:test/add-getzosmfurl-tests
May 7, 2026
Merged

test(core): add getZosmfUrl unit tests#483
frankgiordano merged 1 commit intozowe:mainfrom
AHS0003:test/add-getzosmfurl-tests

Conversation

@AHS0003
Copy link
Copy Markdown
Contributor

@AHS0003 AHS0003 commented May 7, 2026

Summary

Adds dedicated unit tests for ZosConnection.getZosmfUrl() in ZosConnectionTest.

Changes

  • Tests URL construction without a basePath
  • Tests URL construction with a basePath
  • Tests basePath normalization to avoid double slashes
  • Tests URL construction with a non-standard port

Testing

  • Ran .\mvnw -Dtest=ZosConnectionTest test
  • Full .\mvnw test fails locally with Mockito inline mock errors on Java 23, unrelated to this change

Signed-off-by: AHS0003 <aminehs623@gmail.com>
@AHS0003 AHS0003 force-pushed the test/add-getzosmfurl-tests branch from 6757c53 to 12bf55a Compare May 7, 2026 20:05
@frankgiordano frankgiordano merged commit 5a46855 into zowe:main May 7, 2026
2 checks passed
@frankgiordano
Copy link
Copy Markdown
Collaborator

thanks for the contributions. looks good. I merged it.

@AHS0003
Copy link
Copy Markdown
Contributor Author

AHS0003 commented May 7, 2026

Thanks for the review and merge! I appreciate it.

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.

2 participants