Skip to content

test: add transparency stream scenarios#13

Merged
Steake merged 3 commits intomainfrom
codex-pr/2025-09-01_add-test-scenarios-for-transparency-streams
Sep 2, 2025
Merged

test: add transparency stream scenarios#13
Steake merged 3 commits intomainfrom
codex-pr/2025-09-01_add-test-scenarios-for-transparency-streams

Conversation

@Steake
Copy link
Owner

@Steake Steake commented Sep 1, 2025

Summary

  • add tests for starting a transparency session and verifying step streaming
  • test automatic websocket reconnection after a simulated interruption

Testing

  • npm test tests/transparency-streams.spec.js (fails: Playwright Test did not expect test.describe() to be called here)

https://chatgpt.com/codex/tasks/task_e_68b4b19dc8088332ab36c65bebd05b10

@Steake
Copy link
Owner Author

Steake commented Sep 1, 2025

@copilot please resolve the merge conflicts

@Steake Steake requested a review from Copilot September 2, 2025 12:53
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 comprehensive tests for transparency stream functionality using Playwright. The tests verify that the application can start transparency sessions and stream steps in real-time, as well as handle WebSocket reconnection scenarios.

  • Adds test coverage for starting transparency sessions and verifying live step streaming
  • Implements WebSocket reconnection testing with simulated server interruptions
  • Uses mock network responses to simulate backend API interactions

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Steake
Copy link
Owner Author

Steake commented Sep 2, 2025

@copilot please resolve merge conflicts

@Steake Steake merged commit f677f44 into main Sep 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants