Skip to content

Commit

Permalink
Update refactoring-checklist.md
Browse files Browse the repository at this point in the history
  • Loading branch information
quii authored Dec 19, 2023
1 parent e89befe commit 842bd02
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion refactoring-checklist.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ The activity often gets conflated with more significant design changes, but they

Refactoring is just improving existing code and <u>not changing behaviour</u>; therefore, tests shouldn't have to change.

This is why it's the 3rd step of the TDD cycle. Once you have added a behaviour and a test to back it up, refactoring should be an activity which requires no change to your test code. **You're doing something else** if you are "refactoring" some code but must change your tests.
This is why it's the 3rd step of the TDD cycle. Once you have added a behaviour and a test to back it up, refactoring should be an activity which requires no change to your test code. **You're doing something else** if you are "refactoring" some code and having to change tests at the same time.

Many very helpful refactorings are simple to learn and easy to do (your IDE almost entirely automates many) but, over time, become hugely impactful to the quality of our system.

Expand Down

0 comments on commit 842bd02

Please sign in to comment.