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

lib/tests/ajaxlib_test.php are causing output in unit tests

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide

      manually execute phpunit test lib/tests/ajaxlib_test.php and verify there is nothing printed to screen and error log - do not rely on integration server here, the output is not reported as error or failure

      Show
      manually execute phpunit test lib/tests/ajaxlib_test.php and verify there is nothing printed to screen and error log - do not rely on integration server here, the output is not reported as error or failure
    • Affected Branches:
      MOODLE_25_STABLE, MOODLE_26_STABLE
    • Fixed Branches:
      MOODLE_25_STABLE, MOODLE_26_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      w12_MDL-44635_m27_ajaxlogs

      Description

      MDL-44061 caused a regression with the introduction of output to tests. We need to eliminate it. Should really have been picked up in testing.

      phpunit lib/tests/ajaxlib_test.php
      Moodle 2.7dev (Build: 20140314), pgsql, c0e88129d16f4ab0f7687990b892ec3f0949623b
      PHPUnit 3.7.32 by Sebastian Bergmann.
       
      Configuration read from /Users/danp/moodles/im/moodle/phpunit.xml
       
      ....Potential coding error: Unexpected output whilst processing AJAX request. This could be caused by trailing whitespace. Output received: 'Some example content'
      .Potential coding error: Unexpected output whilst processing AJAX request. This could be caused by trailing whitespace. Output received: 'Some example content'
      .Potential coding error: Unexpected output whilst processing AJAX request. This could be caused by trailing whitespace. Output received: 'Some example content'
      ..
       
      Time: 1.51 seconds, Memory: 53.75Mb
       
      OK (8 tests, 16 assertions)
      

        Attachments

          Issue Links

            Activity

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  12/May/14