From bb68dfc002ccfb8124095b152e6b6e533fd046d6 Mon Sep 17 00:00:00 2001 From: rwijaya Date: Mon, 25 Nov 2013 15:35:26 +0800 Subject: [PATCH] MDL-42858 Course Management: Update phpunit test for core_course_management_helper_test --- course/tests/management_helper_test.php | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/course/tests/management_helper_test.php b/course/tests/management_helper_test.php index f4cb41bc86e..ee9a952035a 100644 --- a/course/tests/management_helper_test.php +++ b/course/tests/management_helper_test.php @@ -191,8 +191,8 @@ class core_course_management_helper_test extends advanced_testcase { $parentassignment->assign(CAP_PREVENT); // Now we have capability on the category and subcategory but not the parent. - // Try to mark the subcategory as visible. This shouldn't be possible because its parent is hidden. - $this->assertFalse(\core_course\management\helper::action_category_show($subcategory)); + // Try to mark the subcategory as visible. This should be possible although its parent is set to hidden. + $this->assertTrue(\core_course\management\helper::action_category_show($subcategory)); $cat = coursecat::get($category->id); $subcat = coursecat::get($subcategory->id); $course = $DB->get_record('course', array('id' => $course->id), 'id, visible, visibleold', MUST_EXIST); @@ -348,8 +348,8 @@ class core_course_management_helper_test extends advanced_testcase { $parentassignment->assign(CAP_PREVENT); // Now we have capability on the category and subcategory but not the parent. - // Try to mark the subcategory as visible. This shouldn't be possible because its parent is hidden. - $this->assertFalse(\core_course\management\helper::action_category_show($subcategory)); + // Try to mark the subcategory as visible. This should be possible although its parent is set to hidden. + $this->assertTrue(\core_course\management\helper::action_category_show($subcategory)); $cat = coursecat::get($category->id); $subcat = coursecat::get($subcategory->id); $course = $DB->get_record('course', array('id' => $course->id), 'id, visible, visibleold', MUST_EXIST);