From 14254008fdc7ffcd3cf0156026634aa49a88cde3 Mon Sep 17 00:00:00 2001 From: Dan Marsden Date: Tue, 1 Oct 2024 19:57:46 +1300 Subject: [PATCH] MDL-83310 course: remove hard-coded list of course formats from test. --- course/tests/courselib_test.php | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/course/tests/courselib_test.php b/course/tests/courselib_test.php index 0523913ada8..1b66e9dda1c 100644 --- a/course/tests/courselib_test.php +++ b/course/tests/courselib_test.php @@ -7504,7 +7504,7 @@ class courselib_test extends advanced_testcase { } /** - * Tests get_sorted_course_formats returns all plugins in cases where plugins are installed now, + * Tests get_sorted_course_formats returns plugins in cases where plugins are * installed previously but no longer exist, or not installed yet. * * @covers ::get_sorted_course_formats() @@ -7514,10 +7514,6 @@ class courselib_test extends advanced_testcase { $this->resetAfterTest(); - // By default returns all course formats. - $formats = get_sorted_course_formats(); - $this->assertEquals(['topics', 'weeks', 'singleactivity', 'social'], $formats); - // If there is an extra format installed that no longer exists, include in list (at end). $DB->insert_record('config_plugins', [ 'plugin' => 'format_frogs', @@ -7526,12 +7522,12 @@ class courselib_test extends advanced_testcase { ]); \core\plugin_manager::reset_caches(); $formats = get_sorted_course_formats(); - $this->assertEquals(['topics', 'weeks', 'singleactivity', 'social', 'frogs'], $formats); + $this->assertContains('frogs', $formats); // If one of the formats is not installed yet, we still return it. $DB->delete_records('config_plugins', ['plugin' => 'format_weeks']); \core\plugin_manager::reset_caches(); $formats = get_sorted_course_formats(); - $this->assertEquals(['topics', 'weeks', 'singleactivity', 'social', 'frogs'], $formats); + $this->assertContains('weeks', $formats); } }