Skip to content

Commit

Permalink
post: update snapshot testing sucks
Browse files Browse the repository at this point in the history
  • Loading branch information
vernak2539 committed Apr 3, 2024
1 parent 454ae0f commit fb0c2a8
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions src/content/blog/2024/04/snapshot-testing-sucks.md
Original file line number Diff line number Diff line change
Expand Up @@ -80,9 +80,11 @@ Let's take a moment to imagine something.

First, you have nine teams (30+ people) working on a single codebase. They each own their individual areas of the codebase. The way CODEOWNERS has been set up requires approvals by each team when their files are changed.

Second, common components (think something like a design system) are used across the codebase.
Second, common components/depenenceis are used across the entire codebase. These could be something like a design system, or a framework, like React.

When a common component is updated all the snapshots across the codebase need to be updated. Now you need to get sign-off from the all teams. In my experience, it takes a long time and can result in a lot of merge conflicts if not done quickly.
When these are updated, all snapshots in the codebase need to be updated. This happened when React changed how Fragments are rendered (or Enzyme did, or whatever, you get the point).

Now you need to get sign-off from the all teams. In my experience, it takes a long time and can result in a lot of merge conflicts if not done quickly.

## /endrant

Expand Down

0 comments on commit fb0c2a8

Please sign in to comment.