[MDL-63811] Typo in new config setting in phpunit allow list - Moodle Tracker

XML

Word

Printable

Details

Type: Bug
Status: Closed
Priority: Minor
Resolution: Fixed
Affects Version/s: 3.6
Fix Version/s: 3.6
Component/s: Unit tests
Labels:
- ci

Testing Instructions:

Hide

This is a pretty trivial fix - and there wasn't really any phpunit specific testing in the origin ~~MDL-60514~~.

Show
This is a pretty trivial fix - and there wasn't really any phpunit specific testing in the origin MDL-60514 .
Affected Branches:
MOODLE_36_STABLE
Fixed Branches:
MOODLE_36_STABLE
Pull from Repository:
git://github.com/aolley/moodle.git
Pull Master Branch:
~~MDL-63811~~
Pull Master Diff URL:
https://github.com/aolley/moodle/compare/448bd578d8e...MDL-63811

Description

~~MDL-60514~~ added a new config setting, `pathtophp`. This was added to the list of allowed config settings for behat and phpunit - however the entry for phpunit has a typo and is mistakenly listed as `pathtphp` (missing the `o`).

Attachments

Issue Links

Discovered while testing

MDL-60514 Scheduled task mod_forum\task\cron_task fails when executed as "Run now"

Closed

Activity

People

Assignee:: Adam Olley

Reporter:: Adam Olley

Peer reviewer:: Peter Dias

Integrator:: Jun Pataleta

Tester:: CiBoT

Participants:: Adam Olley, CiBoT, Jun Pataleta, Peter Dias

Component watchers:: Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze

Votes:: 0 Vote for this issue

Watchers:: 3 Start watching this issue

Dates

Created:: 30/Oct/18 6:43 AM

Updated:: 17/Nov/18 12:42 AM

Resolved:: 17/Nov/18 12:42 AM

Fix Release Date:: 3/Dec/18