MDL-40552 course: fixup unit test

Previously there was an assertion which was never used, now
we ensure to test the course themes functionality each time.
This commit is contained in:
Dan Poltawski 2014-11-29 12:19:56 +00:00
parent d87bcfb325
commit 141e7d8747

View file

@ -335,6 +335,8 @@ class core_course_externallib_testcase extends externallib_advanced_testcase {
// Enable course completion. // Enable course completion.
set_config('enablecompletion', 1); set_config('enablecompletion', 1);
// Enable course themes.
set_config('allowcoursethemes', 1);
// Set the required capabilities by the external function // Set the required capabilities by the external function
$contextid = context_system::instance()->id; $contextid = context_system::instance()->id;
@ -415,10 +417,7 @@ class core_course_externallib_testcase extends externallib_advanced_testcase {
$this->assertEquals($courseinfo->defaultgroupingid, $course2['defaultgroupingid']); $this->assertEquals($courseinfo->defaultgroupingid, $course2['defaultgroupingid']);
$this->assertEquals($courseinfo->completionnotify, $course2['completionnotify']); $this->assertEquals($courseinfo->completionnotify, $course2['completionnotify']);
$this->assertEquals($courseinfo->lang, $course2['lang']); $this->assertEquals($courseinfo->lang, $course2['lang']);
if (!empty($CFG->allowcoursethemes)) {
$this->assertEquals($courseinfo->theme, $course2['forcetheme']); $this->assertEquals($courseinfo->theme, $course2['forcetheme']);
}
// We enabled completion at the beginning of the test. // We enabled completion at the beginning of the test.
$this->assertEquals($courseinfo->enablecompletion, $course2['enablecompletion']); $this->assertEquals($courseinfo->enablecompletion, $course2['enablecompletion']);