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

Fail on debugging output anywhere in a test chain backtrace and include it in the failure report

    XMLWordPrintable

Details

    • Improvement
    • Resolution: Fixed
    • Minor
    • 3.0.7, 3.1.3
    • 3.0.7, 3.1, 3.2
    • Unit tests
    • MOODLE_30_STABLE, MOODLE_31_STABLE, MOODLE_32_STABLE
    • MOODLE_30_STABLE, MOODLE_31_STABLE
    • Hide

      Aside from unit test runs, ensure debugging does appear in the right spot when errors happen, especially on tests using @test annotation.

      1. Add my repository if you don't have it. See the repo link.
      2. cherry-pick MDL-56354_test

      vendor/bin/phpunit lib/phpunit/tests/advanced_test.php
      

      3. Ensure there is are errors and failures. The message appears with the failure reports, not inline with the .EFFF....

      Show
      Aside from unit test runs, ensure debugging does appear in the right spot when errors happen, especially on tests using @test annotation. 1. Add my repository if you don't have it. See the repo link. 2. cherry-pick MDL-56354 _test vendor/bin/phpunit lib/phpunit/tests/advanced_test.php 3. Ensure there is are errors and failures. The message appears with the failure reports, not inline with the .EFFF....

    Description

      While looking at MDL-56347 I noticed that when the message returned there was a debugging, it did not fail the test and that the debugging information was returned inline with the test results.

      1. All debugging when a test is running should be included in a failure report and not inline.
      2. removing the requirement for test to appear in the function name allows tests with @test annotation to be used and get the correct debugging results.

      Attachments

        Issue Links

          Activity

            People

              mr-russ Russell Smith
              mr-russ Russell Smith
              Ryan Wyllie Ryan Wyllie
              Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
              Jake Dallimore Jake Dallimore
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Clockify

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