[MDL-73054] Problem running reportbuilder/tests/external/conditions unit tests alone - Moodle Tracker

XML

Word

Printable

Details

Type: Bug
Status: Closed
Priority: Minor
Resolution: Fixed
Affects Version/s: 3.11.4, 4.0
Fix Version/s: 3.11.5
Component/s: Filters, Report builder, Repositories, Unit tests
Labels:
- ci
- triaged

Affected Branches:
MOODLE_311_STABLE, MOODLE_400_STABLE
Fixed Branches:
MOODLE_311_STABLE
Epic Link:
Report builder 4.0
Pull from Repository:
git://github.com/paulholden/moodle.git
Pull 3.11 Branch:
~~MDL-73054~~-311
Pull 3.11 Diff URL:
https://github.com/moodle/moodle/compare/MOODLE_311_STABLE...paulholden:MDL-73054-311
Pull Main Branch:
~~MDL-73054~~
Pull Main Diff URL:
https://github.com/moodle/moodle/compare/master...paulholden:MDL-73054
Testing Instructions:
Hide

Run the following test cases, confirm they still pass:

$ vendor/bin/phpunit --testsuite filter_emoticon_testsuite,repository_googledocs_testsuite

Master only: confirm this testcase now passes:

$ vendor/bin/phpunit --filter='core_reportbuilder\\external\\conditions\\add_test::test_execute'
Show
Run the following test cases, confirm they still pass: $ vendor/bin/phpunit --testsuite filter_emoticon_testsuite,repository_googledocs_testsuite Master only: confirm this testcase now passes: $ vendor/bin/phpunit --filter='core_reportbuilder\\external\\conditions\\add_test::test_execute'

Description

Working on MDL-72769 we realized that tests on reportbuilder/tests/external/conditions are failing when they are running alone:

coding_exception: Coding error detected, it must be fixed by a programmer: $OUTPUT->header() needs to be called before collecting JavaScript requirements.

Attachments

- Sort By Name
- Sort By Date
- Ascending
- Descending
- Thumbnails
- List
- Download All

MDL-73054_master.png
25/Nov/21 10:42 AM
83 kB
Angelia Dela Cruz
MDL-73054_v311.png
25/Nov/21 10:42 AM
45 kB
Angelia Dela Cruz

Activity

People

Assignee:: Paul Holden

Reporter:: Amaia Anabitarte

Peer reviewer:: Odei Alba

Integrator:: Eloy Lafuente (stronk7)

Tester:: Angelia Dela Cruz

Participants:: Amaia Anabitarte, Angelia Dela Cruz, CiBoT, Eloy Lafuente (stronk7), Odei Alba, Paul Holden

Votes:: 1 Vote for this issue

Watchers:: 6 Start watching this issue

Dates

Created:: 11/Nov/21 7:41 PM

Updated:: 26/Nov/21 10:49 PM

Resolved:: 26/Nov/21 10:49 PM

Fix Release Date:: 17/Jan/22

Time Tracking

Estimated:

Not Specified

Remaining:

Logged:

4h 50m