Skip to content

Commit d5b6bb7

Browse files
committed
fix: correction to workshop 4 solutions
workshop 4 incorrectly lists a "prime path" which isn't prime
1 parent 84a4b27 commit d5b6bb7

3 files changed

Lines changed: 2 additions & 6 deletions

File tree

assets/workshops/workshop04-solutions.html

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1007,9 +1007,8 @@ <h2 id="isp-and-control-flow">3. ISP and control flow</h2>
10071007
left-hand ‘if’ branch)</em></li>
10081008
<li>BCDFB, CDFBC, DFBCD <em>(these all execute the right-hand ‘if’
10091009
branch)</em></li>
1010-
<li>EFBCDF <em>(this takes one left and one right branch)</em></li>
10111010
<li>CDEFBG <em>(takes left branch)</em></li>
1012-
<li>CDFBG <em>(take rights branch)</em></li>
1011+
<li>CDFBG <em>(takes right branch)</em></li>
10131012
</ul>
10141013
<p>What proportion of the prime paths your tests cover will depend on
10151014
what tests you chose. But note that if your tests don’t have node or

assets/workshops/workshop04-solutions.md

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -639,7 +639,6 @@ Consider the following Java method for collapsing sequences of blanks, taken
639639
from the `StringUtils` class of Apache Velocity
640640
(<http://velocity.apache.org/>), version 1.3.1:
641641

642-
643642
``` { .java .numberLines }
644643

645644
/**
@@ -920,9 +919,8 @@ prefix, or represent paths through the same loop):
920919
- ABG, ABCDEF, ABCDF
921920
- BCDEFB, CDEFBC, DEFBCD, EFBCDE, FBCDEF *(these all execute the left-hand 'if' branch)*
922921
- BCDFB, CDFBC, DFBCD *(these all execute the right-hand 'if' branch)*
923-
- EFBCDF *(this takes one left and one right branch)*
924922
- CDEFBG *(takes left branch)*
925-
- CDFBG *(take rights branch)*
923+
- CDFBG *(takes right branch)*
926924

927925
What proportion of the prime paths your tests cover will
928926
depend on what tests you chose.

assets/workshops/workshop04.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -376,7 +376,6 @@ Consider the following Java method for collapsing sequences of blanks, taken
376376
from the `StringUtils` class of Apache Velocity
377377
(<http://velocity.apache.org/>), version 1.3.1:
378378

379-
380379
``` { .java .numberLines }
381380

382381
/**

0 commit comments

Comments
 (0)