From 31fa03861f21b9d890a86e2120512d893d93003c Mon Sep 17 00:00:00 2001
From: skodak <skodak>
Date: Sun, 29 Jul 2007 13:58:57 +0000
Subject: [PATCH] fixed grade category unit tests

---
 lib/grade/grade_category.php               |  8 --------
 lib/grade/simpletest/testgradecategory.php | 12 ++----------
 2 files changed, 2 insertions(+), 18 deletions(-)

diff --git a/lib/grade/grade_category.php b/lib/grade/grade_category.php
index 6f4a512851..fe9e327667 100644
--- a/lib/grade/grade_category.php
+++ b/lib/grade/grade_category.php
@@ -611,14 +611,6 @@ class grade_category extends grade_object {
 
     }
 
-    /**
-     * Returns true if this category has any child grade_category or grade_item.
-     * @return int number of direct children, or false if none found.
-     */
-    function has_children() {
-        return count_records('grade_categories', 'parent', $this->id) + count_records('grade_items', 'categoryid', $this->id);
-    }
-
     /**
      * Returns tree with all grade_items and categories as elements
      * @static
diff --git a/lib/grade/simpletest/testgradecategory.php b/lib/grade/simpletest/testgradecategory.php
index 77f4721df7..f233013fbf 100755
--- a/lib/grade/simpletest/testgradecategory.php
+++ b/lib/grade/simpletest/testgradecategory.php
@@ -118,7 +118,7 @@ class grade_category_test extends grade_test {
         $grade_category->fullname = 'Updated info for this unittest grade_category';
         $grade_category->path = null; // path must be recalculated if missing
         $grade_category->depth = null;
-        $grade_category->aggregation = GRADE_AGGREGATE_MAX; // should force regrading
+        $grade_category->aggregation = GRADE_AGGREGATE_MAX_ALL; // should force regrading
 
         $grade_item = $grade_category->get_grade_item();
         $this->assertEqual(0, $grade_item->needsupdate);
@@ -144,7 +144,7 @@ class grade_category_test extends grade_test {
 
         $this->assertFalse($grade_category->qualifies_for_regrading());
 
-        $grade_category->aggregation = GRADE_AGGREGATE_MAX;
+        $grade_category->aggregation = GRADE_AGGREGATE_MAX_ALL;
         $this->assertTrue($grade_category->qualifies_for_regrading());
 
         $grade_category = new grade_category($this->grade_categories[0]);
@@ -307,14 +307,6 @@ class grade_category_test extends grade_test {
         $this->assertEqual(true, $category->grade_item->is_hidden());
     }
 
-    function test_grade_category_has_children() {
-        $category = new grade_category($this->grade_categories[0]);
-        $this->assertTrue(method_exists($category, 'has_children'));
-        $this->assertTrue($category->has_children());
-        $category = new grade_category();
-        $this->assertFalse($category->has_children());
-    }
-
     function test_grade_category_generate_grades() {
         $category = new grade_category($this->grade_categories[3]);
         $this->assertTrue(method_exists($category, 'generate_grades'));
-- 
2.39.5