Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-69273

travis builds do not report phpunit failures/errors anymore

XMLWordPrintable

    • MOODLE_310_STABLE, MOODLE_35_STABLE, MOODLE_37_STABLE, MOODLE_38_STABLE, MOODLE_39_STABLE, MOODLE_400_STABLE
    • MOODLE_35_STABLE, MOODLE_37_STABLE, MOODLE_38_STABLE, MOODLE_39_STABLE
    • MDL-69273-master-2
    • Hide
      1. Apply the attached commit:

        git am MDL-69273-fail.patch
        

      2. Push to your Travis-enabled Github
        1. Confirm that the PHPUnit tests fail
      Show
      Apply the attached commit: git am MDL-69273-fail.patch Push to your Travis-enabled Github Confirm that the PHPUnit tests fail

      This seems to be a recent regression caused by MDL-66322

      Uh, oh... I think I've detected a regression of this.

      1) Do any tiny change forcing a unit test to fail (change 1 assertion, throw an exception...).
      2) Push to github and get travis testing it.
      3a) Expected result: the PHPUNIT jobs fails.
      3b) Current result: The PHPUNIT jobs end ok.

      Real example, I've a branch automatically run weekly, and known to be failing... and suddenly I got a mail from travis it had passed, I could not believe it! Looking to the logs you'll see it fails, errors... and not mater of that... it passes!

      Link: https://travis-ci.org/github/stronk7/moodle/builds/707981008

      Surely it's because the new line added after the phpunit execution "eats" its exit code, we need to fix that ASAP.

            dobedobedoh Andrew Lyons
            stronk7 Eloy Lafuente (stronk7)
            Matteo Scaramuccia Matteo Scaramuccia
            Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
            CiBoT CiBoT
            Votes:
            1 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved:

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 1 hour, 25 minutes
                1h 25m

                  Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.