Skip to content

Conversation

mohitpubnub
Copy link
Contributor

@mohitpubnub mohitpubnub commented Sep 2, 2025

unit tests added for input parameter validation and response parsing including edge cases.

additional integration tests for edge cases.

@pubnub-ops-terraform
Copy link

pubnub-ops-terraform commented Sep 2, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

@mohitpubnub mohitpubnub marked this pull request as ready for review September 3, 2025 06:28
@mohitpubnub mohitpubnub requested a review from parfeon as a code owner September 3, 2025 06:28
Copy link
Contributor

@parfeon parfeon left a comment

Choose a reason for hiding this comment

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

LGTM!

@parfeon parfeon added status: done This issue is considered resolved. priority: low This PR should be reviewed after all high and medium PRs. type: test This PR contains new tests for existing functionality or fixes to existing tests. labels Sep 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: low This PR should be reviewed after all high and medium PRs. status: done This issue is considered resolved. type: test This PR contains new tests for existing functionality or fixes to existing tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants