Skip to content

refactor: test events phase 1 (@miodec)#8021

Merged
Miodec merged 114 commits into
masterfrom
test-events-2
May 29, 2026
Merged

refactor: test events phase 1 (@miodec)#8021
Miodec merged 114 commits into
masterfrom
test-events-2

Conversation

@Miodec
Copy link
Copy Markdown
Member

@Miodec Miodec commented May 28, 2026

Consolidate all data into one array of events, derive all stats from this array.

Phase 1 runs the new system in shadow mode, comparing to the original and sending mismatches to the backend.
Phase 2 will fully replace the system.

Copy link
Copy Markdown
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

Copilot reviewed 27 out of 27 changed files in this pull request and generated 8 comments.

Comment thread frontend/src/ts/test/events/data.ts
Comment thread frontend/src/ts/test/events/data.ts Outdated
Comment thread frontend/src/ts/test/test-logic.ts Outdated
Comment thread frontend/src/ts/test/test-logic.ts Outdated
Comment thread frontend/src/ts/test/test-input.ts
Comment thread frontend/src/ts/test/events/stats.ts
Comment thread frontend/__tests__/utils/strings.spec.ts Outdated
Comment thread frontend/src/ts/test/test-logic.ts Outdated
@github-actions
Copy link
Copy Markdown
Contributor

Continuous integration check(s) failed. Please review the failing check's logs and make the necessary changes.

@github-actions github-actions Bot added waiting for update Pull requests or issues that require changes/comments before continuing and removed waiting for review Pull requests that require a review before continuing labels May 29, 2026
@github-actions github-actions Bot removed the waiting for update Pull requests or issues that require changes/comments before continuing label May 29, 2026
@Miodec Miodec requested a review from Copilot May 29, 2026 07:22
@github-actions github-actions Bot added the waiting for review Pull requests that require a review before continuing label May 29, 2026
Copy link
Copy Markdown
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

Copilot reviewed 27 out of 27 changed files in this pull request and generated 7 comments.

Comment thread frontend/src/ts/test/events/data.ts
Comment thread frontend/src/ts/test/test-stats.ts
Comment thread frontend/src/ts/test/test-input.ts
Comment thread frontend/src/ts/test/test-logic.ts
Comment thread frontend/src/ts/test/test-logic.ts
Comment thread packages/contracts/src/results.ts
Comment thread backend/src/api/controllers/result.ts
Comment thread frontend/__tests__/test/events/stats.spec.ts
Comment thread frontend/__tests__/test/events/helpers.spec.ts
Comment thread frontend/src/ts/test/events/data.ts Outdated
Comment thread frontend/src/ts/test/events/data.ts Outdated
Comment thread frontend/src/ts/test/events/types.ts Outdated
@github-actions github-actions Bot removed the waiting for review Pull requests that require a review before continuing label May 29, 2026
fehmer
fehmer previously approved these changes May 29, 2026
@Miodec Miodec merged commit f9074fb into master May 29, 2026
21 of 29 checks passed
@Miodec Miodec deleted the test-events-2 branch May 29, 2026 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend Server stuff frontend User interface or web stuff packages Changes in local packages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants