Skip to content

Maintenance: Split Rest Event Handler BaseRouter Tests Into Multiple Files #4438

@svozza

Description

@svozza

Summary

The packages/event-handler/tests/unit/rest/BaseRouter.test.ts file is nearly 1900 lines in size and will only get bigger as we add more functionality to this event handler. We should break this monolith up in multiple files that test different areas of functionality. We can use the describe blocks as a guide on how to split the tests up.

Why is this needed?

The cognitive load of such a large file slows development and makes the suite harder to extend.

Which area does this relate to?

Tests

Solution

We should break this monolith up in multiple files that test different areas of functionailty. We can use the describe blocks as a guide on how to split the tests up.

Acknowledgment

Future readers

Please react with 👍 and your use case to help us understand customer demand.

Metadata

Metadata

Assignees

Labels

completedThis item is complete and has been merged/shippedinternalPRs that introduce changes in governance, tech debt and chores (linting setup, baseline, etc.)testsPRs that add or change tests

Type

No type

Projects

Status

Shipped

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions