Skip to content

tests: fixed konnect tests - redis.ssl_verify defaults to true now#1940

Merged
Prashansa-K merged 2 commits intomainfrom
tests/fix-konnect-test
Mar 20, 2026
Merged

tests: fixed konnect tests - redis.ssl_verify defaults to true now#1940
Prashansa-K merged 2 commits intomainfrom
tests/fix-konnect-test

Conversation

@Prashansa-K
Copy link
Contributor

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Mar 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 33.58%. Comparing base (9f16682) to head (cb13916).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1940      +/-   ##
==========================================
+ Coverage   33.54%   33.58%   +0.04%     
==========================================
  Files          76       76              
  Lines        6711     6711              
==========================================
+ Hits         2251     2254       +3     
+ Misses       4280     4278       -2     
+ Partials      180      179       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Prashansa-K
Copy link
Contributor Author

Will check this later - some tests' ssl_verify key come up as "true" in one run and "false" in the next leading to failures.
Need to determine if this is because of dual testing for konnect - default-fill vs no default-fill.

@Prashansa-K Prashansa-K force-pushed the tests/fix-konnect-test branch from ab4847d to 1abfbcd Compare March 18, 2026 12:36
@Prashansa-K
Copy link
Contributor Author

redis-ee partial schema shows ssl_verify default as false. If we create one, it defaults to true. We are blocked on koko for a fix here.
https://kongstrong.slack.com/archives/C02KEASTTRC/p1773837199800549

@Prashansa-K Prashansa-K merged commit 959c522 into main Mar 20, 2026
40 of 41 checks passed
@Prashansa-K Prashansa-K deleted the tests/fix-konnect-test branch March 20, 2026 07:33
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.

3 participants