[MDL-70725] Fix PHPUnit "test_create_and_delete_standard_issuers" - Moodle Tracker

XML

Word

Printable

Details

Type: Bug
Status: Closed
Priority: Minor
Resolution: Fixed
Affects Version/s: 3.9.4
Fix Version/s: 3.9.5, 3.10.2
Component/s: Libraries
Labels:

Affected Branches:
MOODLE_39_STABLE
Fixed Branches:
MOODLE_310_STABLE, MOODLE_39_STABLE
Pull from Repository:
git://github.com/sarjona/moodle.git
Pull 3.9 Branch:
~~MDL-70725~~-39
Pull 3.9 Diff URL:
https://github.com/sarjona/moodle/compare/db6d34630f...MDL-70725-39
Pull 3.11 Branch:
~~MDL-70725~~-311
Pull 3.11 Diff URL:
https://github.com/sarjona/moodle/compare/500bee1af0...MDL-70725-311
Pull Master Branch:
~~MDL-70725~~-master
Pull Master Diff URL:
https://github.com/sarjona/moodle/compare/fd840ab59c...MDL-70725-master
Difficulty:
Easy
Testing Instructions:
Hide

Run phpunit tests for lib/tests/oauth2_test.php

mdk phpunit -r -u lib/tests/oauth2_test.php

Confirm they pass with "OK (7 tests, 69 assertions)"
Show
Run phpunit tests for lib/tests/oauth2_test.php mdk phpunit -r -u lib/tests/oauth2_test.php Confirm they pass with "OK (7 tests, 69 assertions)"

Sprint:
Moppies Kanban

Description

While working on ~~MDL-70689~~, I realised part of the code in test_create_and_delete_standard_issuers in lib/tests/oauth2_test.php is never executed (because has been added after a "expectException").

This issue is for fixing that and separe both tests in order to run all them.

Attachments

Activity

People

Assignee:: Sara Arjona (@sarjona)

Reporter:: Sara Arjona (@sarjona)

Peer reviewer:: Carlos Escobedo

Integrator:: Andrew Lyons

Tester:: CiBoT

Participants:: Andrew Lyons, Carlos Escobedo, CiBoT, Sara Arjona (@sarjona)

Component watchers:: Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Laurent David, Raquel Ortega, Sara Arjona (@sarjona)

Votes:: 0 Vote for this issue

Watchers:: 3 Start watching this issue

Dates

Created:: 28/Jan/21 12:44 AM

Updated:: 11/Feb/21 10:31 PM

Resolved:: 11/Feb/21 10:31 PM

Fix Release Date:: 8/Mar/21

Time Tracking

Estimated:

Remaining:

Logged:

1h 5m