Skip to content

Commit 4f0a6a1

Browse files
committed
Add a note about tests
1 parent 127a58a commit 4f0a6a1

1 file changed

Lines changed: 12 additions & 0 deletions

File tree

docs/contributing/code-style/web/angular-migration-guide.md

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,18 @@ described under [Angular](./angular.md). This guide provides a step-by-step appr
55
migrate existing Angular components and directives to align with these practices. New code
66
**should** strive to follow these guidelines from the start.
77

8+
:::warning
9+
10+
As usual when refactoring existing code, it's generally advisable to first put the code under tests.
11+
This provides a basic safety net against regressions during the migration. If the component has
12+
significant amount of business logic extracting it to a separate service first might make it easier
13+
to test.
14+
15+
After migrating we recommend performing a full regression sweep of the components to catch anything
16+
missed any unit tests.
17+
18+
:::
19+
820
:::info
921

1022
We provide a Bitwarden specific

0 commit comments

Comments
 (0)