5 ms·

I don’t get the 100% branch coverage requirement. If the code is off of ‘master’ why would you have to go and fix it?

• hinkley 3y ago
  Conditional branches, not repository branches.
• kristiandupont 3y ago
  I think he's referring to branching in the code (if-then-else blocks).
• anotherhue 3y ago
  I was confused also but decided they meant if/else branching
  • abdusco 3y ago
    Maybe the author means something else. How else could you hit 100% of the lines if you're not covering all branches.
    • morelisp 3y ago
      if (a || b) { ... } Branch coverage means testing a = true, b = anything; and a = false, b = true; and a = false, b = false.
• hooksfordays 3y ago
  Branch coverage means something different. If you have a function with 2 if statements, there are 4 branches. 100% code coverage could be 1 test and would run the code before + after the if statements, and enter each if statement block. 100% branch coverage would test this code 4 times: once without entering either if statement, once only entering the first, once only entering the second, and once entering both.
  • morelisp 3y ago
    You can achieve branch coverage of two if statements by running it twice, taking the different result on each the second time. You are describing stronger / orthogonal types of coverage.
  • sghiassy 3y ago
    Oh, so like cyclomatic complexity coverage
• bewaretheirs 3y ago
  Branch coverage refers to branch instructions (in the underlying machine code), not a branch in whatever source control system you're using.