Skip to content

Commit ca3fcd4

Browse files
authored
Merge pull request #242 from ritaf-ORM/patch-7
Rita's edits to ch 12
2 parents 4605394 + c52376c commit ca3fcd4

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

chapter_12_organising_test_files.asciidoc

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
[[chapter_12_organising_test_files]]
22
== Splitting Our Tests into Multiple Files, and a Generic Wait Helper
33

4+
// RITA: Reword/remove this warning since you have reviewed the chapter text in detail?
45
.🚧 Warning, chapter update in progress
56
*******************************************************************************
67
This chapter is currently in the process of being rewritten for the 3e.
@@ -871,7 +872,9 @@ Don't forget the "Refactor" in "Red, Green, Refactor"::
871872
((("Red/Green/Refactor")))
872873
873874
Don't refactor against failing tests::
875+
// RITA: Why shouldn't you refactor against failing tests in general? A brief explanation would be nice here for clarity.
874876
* In general!
877+
// RITA: Please clarify the next line. It's somewhat ambiguous. Are you saying that you should refactor against failing tests in the FT you're currently working on?
875878
* But the FT you're currently working on doesn't count.
876879
* You can occasionally put a skip on a test which is testing something you
877880
haven't written yet.

0 commit comments

Comments
 (0)