• xmunk
    link
    fedilink
    arrow-up
    32
    ·
    5 months ago

    Also, all the automated tests were commented out.

    • grrgyle@slrpnk.net
      link
      fedilink
      arrow-up
      26
      arrow-down
      1
      ·
      5 months ago

      Checking the ci logs for the past week, like:

      86 test suites passed successfully
          ❌ 0 failures
          ✅ 0 success
          🤷‍♂️ 1785 skipped
      
    • TheSlad
      link
      fedilink
      arrow-up
      21
      ·
      5 months ago

      Ive heard of stories where people would have an imposed test coverage percentage requirement… and they would just have a single dummy method that printed “.” to the console thousands of times. They then have a single test for that one method, and whenever their codebase grows to big, they add more lines to it so that the dummy method has enough lines to meet the test coverage requirement.

      • xmunk
        link
        fedilink
        arrow-up
        10
        ·
        5 months ago

        Percentage of lines covered by tests is such a terrible metric.