Skip to content

Feature/CSPL-4397 Eliminate reliance on time in tests#1735

Draft
kasiakoziol wants to merge 10 commits intodevelopfrom
feature/CSPL-4397-eliminate-reliance-on-time-in-tests
Draft

Feature/CSPL-4397 Eliminate reliance on time in tests#1735
kasiakoziol wants to merge 10 commits intodevelopfrom
feature/CSPL-4397-eliminate-reliance-on-time-in-tests

Conversation

@kasiakoziol
Copy link
Collaborator

@kasiakoziol kasiakoziol commented Feb 26, 2026

Description

It removes time dependencies from the the tests (integration, smoke).

Key Changes

  • Replacing hard time.Sleep(...) with condition-based Eventually(...)
  • Replacing long "debug sleeps" in App Framework tests with explicit "wait until phase/event happens"
  • Introducing/standardizing polling helpers
  • Defining separate poll intervals and introducing a specific timeout for app installation
  • Making scale event emission depend on "ready replica convergence"
  • Replacing Monitoring Console config map checks “sleep then assert” with “wait until config map contains expected entries”

Testing and Verification

Automated tests

Related Issues

Jira: https://splunk.atlassian.net/browse/CSPL-4397

PR Checklist

  • Code changes adhere to the project's coding standards.
  • Relevant unit and integration tests are included.
  • Documentation has been updated accordingly.
  • All tests pass locally.
  • The PR description follows the project's guidelines.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 26, 2026

CLA Assistant Lite bot CLA Assistant Lite bot All contributors have signed the COC ✍️ ✅

@kasiakoziol kasiakoziol changed the base branch from main to develop February 26, 2026 09:43
@patryk
Copy link

patryk commented Feb 26, 2026

Please configure your bot to stop tagging me in the PRs to this project. Thanks.

@patrykw-splunk
Copy link
Collaborator

@patryk Our apologies for that. We'll fix it asap.
PS. great GH alias :)

@patryk
Copy link

patryk commented Feb 26, 2026

PS. great GH alias :)

Heh, I suppose the perk of signing up for niche (in 2009) sites early.

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