tests: test fix and add for shorthand_fields handling in gateway dump#1810
tests: test fix and add for shorthand_fields handling in gateway dump#1810Prashansa-K merged 4 commits intomainfrom
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1810 +/- ##
=======================================
Coverage 32.79% 32.79%
=======================================
Files 73 73
Lines 6550 6550
=======================================
Hits 2148 2148
Misses 4236 4236
Partials 166 166 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| issuer: http://example-issuer.com | ||
| redis: | ||
| timeout: 2000 | ||
| session_redis_cluster_max_redirections: 5 |
There was a problem hiding this comment.
These were shorthand_fields. Hence, with the current change, they would no longer show up.
There was a problem hiding this comment.
Do we have an instance in tests where shorthand field was configured with a non-default value?
|
Konnect tests are failing due to flakiness. Specific dump --skip-defaults tests are passing. Skipped a flaky test for now. This will be updated later; post the dev environment incident is resolved. |
|
Konnect tests are failing with 5xx. I will retry once after gdr version update here. Verified the changes on local. |
3 participants
Corresponding GDR PR: Kong/go-database-reconciler#357
For #1808