diff --git a/admin/tool/log/store/database/tests/store_test.php b/admin/tool/log/store/database/tests/store_test.php index 9d1b4d81a5a..b497ce2b9b4 100644 --- a/admin/tool/log/store/database/tests/store_test.php +++ b/admin/tool/log/store/database/tests/store_test.php @@ -309,17 +309,16 @@ class logstore_database_store_testcase extends advanced_testcase { $logmanager = get_log_manager(); $allreports = \core_component::get_plugin_list('report'); - $supportedreports = array( - 'report_log' => '/report/log', - 'report_loglive' => '/report/loglive' - ); - // Make sure all supported reports are installed. - $expectedreports = array_keys(array_intersect_key($allreports, $supportedreports)); - $reports = $logmanager->get_supported_reports('logstore_database'); - $reports = array_keys($reports); + $expectedreports = array_intersect_key([ + 'log' => 'report_log', + 'loglive' => 'report_loglive', + ], $allreports); + + $supportedreports = $logmanager->get_supported_reports('logstore_database'); + foreach ($expectedreports as $expectedreport) { - $this->assertContains($expectedreport, $reports); + $this->assertArrayHasKey($expectedreport, $supportedreports); } } } diff --git a/admin/tool/log/store/legacy/tests/store_test.php b/admin/tool/log/store/legacy/tests/store_test.php index a2aeb982b10..af3457750ed 100644 --- a/admin/tool/log/store/legacy/tests/store_test.php +++ b/admin/tool/log/store/legacy/tests/store_test.php @@ -257,20 +257,19 @@ class logstore_legacy_store_testcase extends advanced_testcase { $logmanager = get_log_manager(); $allreports = \core_component::get_plugin_list('report'); - $supportedreports = array( - 'report_log' => '/report/log', - 'report_loglive' => '/report/loglive', - 'report_outline' => '/report/outline', - 'report_participation' => '/report/participation', - 'report_stats' => '/report/stats' - ); - // Make sure all supported reports are installed. - $expectedreports = array_keys(array_intersect_key($allreports, $supportedreports)); - $reports = $logmanager->get_supported_reports('logstore_legacy'); - $reports = array_keys($reports); + $expectedreports = array_intersect_key([ + 'log' => 'report_log', + 'loglive' => 'report_loglive', + 'outline' => 'report_outline', + 'participation' => 'report_participation', + 'stats' => 'report_stats' + ], $allreports); + + $supportedreports = $logmanager->get_supported_reports('logstore_legacy'); + foreach ($expectedreports as $expectedreport) { - $this->assertContains($expectedreport, $reports); + $this->assertArrayHasKey($expectedreport, $supportedreports); } } diff --git a/admin/tool/log/store/standard/tests/store_test.php b/admin/tool/log/store/standard/tests/store_test.php index 4170d32f7ee..6076debcd32 100644 --- a/admin/tool/log/store/standard/tests/store_test.php +++ b/admin/tool/log/store/standard/tests/store_test.php @@ -237,20 +237,19 @@ class logstore_standard_store_testcase extends advanced_testcase { $logmanager = get_log_manager(); $allreports = \core_component::get_plugin_list('report'); - $supportedreports = array( - 'report_log' => '/report/log', - 'report_loglive' => '/report/loglive', - 'report_outline' => '/report/outline', - 'report_participation' => '/report/participation', - 'report_stats' => '/report/stats' - ); - // Make sure all supported reports are installed. - $expectedreports = array_keys(array_intersect_key($allreports, $supportedreports)); - $reports = $logmanager->get_supported_reports('logstore_standard'); - $reports = array_keys($reports); + $expectedreports = array_intersect_key([ + 'log' => 'report_log', + 'loglive' => 'report_loglive', + 'outline' => 'report_outline', + 'participation' => 'report_participation', + 'stats' => 'report_stats' + ], $allreports); + + $supportedreports = $logmanager->get_supported_reports('logstore_standard'); + foreach ($expectedreports as $expectedreport) { - $this->assertContains($expectedreport, $reports); + $this->assertArrayHasKey($expectedreport, $supportedreports); } }