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

              Assignee:
              skodak Petr Skoda
              Reporter:
              poltawski Dan Poltawski
              Peer reviewer:
              Dan Poltawski
              Integrator:
              Marina Glancy
              Tester:
              Adrian Greeve
              Participants:
              Component watchers:
              Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze, Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

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