Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[kinesis] Disabled testGetStreamNamesWithPagination on CI due to cost #15085

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

suvodeep-pyne
Copy link
Contributor

The createStream() call in this test is incurring recurring costs in CI. The test is retained for manual use.

Note that the test only runs when AWS secrets are loaded in the env.

…cost

The createStream() call in this test is incurring recurring costs in CI. The test is retained for manual use.

Note that the test only runs when AWS secrets are loaded in the env
@codecov-commenter
Copy link

codecov-commenter commented Feb 18, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 63.38%. Comparing base (59551e4) to head (f8052af).
Report is 1732 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #15085      +/-   ##
============================================
+ Coverage     61.75%   63.38%   +1.63%     
- Complexity      207     1480    +1273     
============================================
  Files          2436     2741     +305     
  Lines        133233   154028   +20795     
  Branches      20636    23778    +3142     
============================================
+ Hits          82274    97630   +15356     
- Misses        44911    49013    +4102     
- Partials       6048     7385    +1337     
Flag Coverage Δ
custom-integration1 100.00% <ø> (+99.99%) ⬆️
integration 100.00% <ø> (+99.99%) ⬆️
integration1 100.00% <ø> (+99.99%) ⬆️
integration2 0.00% <ø> (ø)
java-11 63.33% <ø> (+1.62%) ⬆️
java-21 63.27% <ø> (+1.65%) ⬆️
skip-bytebuffers-false 63.37% <ø> (+1.62%) ⬆️
skip-bytebuffers-true 63.22% <ø> (+35.50%) ⬆️
temurin 63.38% <ø> (+1.63%) ⬆️
unittests 63.38% <ø> (+1.63%) ⬆️
unittests1 56.07% <ø> (+9.18%) ⬆️
unittests2 33.86% <ø> (+6.13%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@@ -112,7 +112,7 @@ public void testGetStreamNames() {
"Expected to find test stream " + _testStreamName + " in list of streams: " + streams);
}

@Test(dependsOnMethods = "testGetStreamNames")
@Test(dependsOnMethods = "testGetStreamNames", enabled = false)
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's add some description on why this test is disabled by default

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants