MDL-52530 unittest: more robust capability assignments for guest
authorEugene Venter <eugene@catalyst.net.nz>
Wed, 16 Dec 2015 04:16:27 +0000 (17:16 +1300)
committerEugene Venter <eugene@catalyst.net.nz>
Tue, 22 Dec 2015 20:24:53 +0000 (09:24 +1300)
for core_files_file_storage_testcase and core_repositorylib_testcase

lib/filestorage/tests/file_storage_test.php
repository/tests/repositorylib_test.php

index 95d6065..f453f34 100644 (file)
@@ -327,8 +327,8 @@ class core_files_file_storage_testcase extends advanced_testcase {
         $repositorypluginname = 'user';
         // Override repository permission.
         $capability = 'repository/' . $repositorypluginname . ':view';
-        $allroles = $DB->get_records_menu('role', array(), 'id', 'archetype, id');
-        assign_capability($capability, CAP_ALLOW, $allroles['guest'], $syscontext->id, true);
+        $guestroleid = $DB->get_field('role', 'id', array('shortname' => 'guest'));
+        assign_capability($capability, CAP_ALLOW, $guestroleid, $syscontext->id, true);
 
         $args = array();
         $args['type'] = $repositorypluginname;
index ac161d1..a65c5bb 100644 (file)
@@ -44,8 +44,8 @@ class core_repositorylib_testcase extends advanced_testcase {
         $repositorypluginname = 'boxnet';
         // override repository permission
         $capability = 'repository/' . $repositorypluginname . ':view';
-        $allroles = $DB->get_records_menu('role', array(), 'id', 'archetype, id');
-        assign_capability($capability, CAP_ALLOW, $allroles['guest'], $syscontext->id, true);
+        $guestroleid = $DB->get_field('role', 'id', array('shortname' => 'guest'));
+        assign_capability($capability, CAP_ALLOW, $guestroleid, $syscontext->id, true);
 
         $plugintype = new repository_type($repositorypluginname);
         $pluginid = $plugintype->create(false);