Skip to content

fix: add noise#208

Merged
khareyash05 merged 1 commit intomainfrom
update-config
Mar 2, 2026
Merged

fix: add noise#208
khareyash05 merged 1 commit intomainfrom
update-config

Conversation

@officialasishkumar
Copy link
Member

No description provided.

Signed-off-by: Asish Kumar <officialasishkumar@gmail.com>
Copilot AI review requested due to automatic review settings February 24, 2026 14:10
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a Keploy configuration file for the sse-redis sample application. The sse-redis app is a Go-based test fixture designed to exercise Keploy's streaming test-case support for SSE, NDJSON, multipart, and plain-text streaming endpoints. The configuration file includes noise settings to handle flaky runtime-generated fields (like timestamps and UUIDs) that change between test runs, ensuring test stability during replay.

Changes:

  • Added keploy.yml configuration file with globalNoise settings for dynamic fields in streaming responses

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

eventID: ""
preInstallDependencies: false

# Visit [https://keploy.io/docs/running-keploy/configuration-file/] to learn about using keploy through configration file.
Copy link

Copilot AI Feb 24, 2026

Choose a reason for hiding this comment

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

Spelling error: "configration" should be "configuration".

Suggested change
# Visit [https://keploy.io/docs/running-keploy/configuration-file/] to learn about using keploy through configration file.
# Visit [https://keploy.io/docs/running-keploy/configuration-file/] to learn about using keploy through configuration file.

Copilot uses AI. Check for mistakes.
"message_id": []
"request_id": []
"created_at": []
"delivered_at": []
Copy link

Copilot AI Feb 24, 2026

Choose a reason for hiding this comment

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

The noise configuration is missing "server_time" which is a dynamic field used in the code (see main.go lines 128 and 366). This field contains timestamps that change on every run and should be included in the globalNoise body configuration to ensure test stability during replay. Add "server_time": [] to the body section under global.

Suggested change
"delivered_at": []
"delivered_at": []
"server_time": []

Copilot uses AI. Check for mistakes.
@khareyash05 khareyash05 merged commit 1df4227 into main Mar 2, 2026
23 checks passed
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.

4 participants