Merge pull request #15 from ucl-isd/CTP-3560-coursework-tests-review-…

…phpunit

CTP-3560 PHPUnit errors

classes/framework/table_base.php

@@ -298,7 +298,13 @@ public static final function get_table_name() {
             $class_name = get_called_class(); // 'mod_coursework\models\deadline_extension'
             $pieces = explode('\\', $class_name); // 'mod_coursework', 'models', 'deadline_extension'
             $table_name = end($pieces); // 'deadline_extension'
-            $table_name .= 's'; // 'deadline_extensions'
+            //todo this may need further attention but fixing the one issue identified so far.
+            if ($table_name == 'personal_deadline') {
+                // Table personal_deadline does not exist but coursework_person_deadline does.
+                $table_name = 'coursework_person_deadline';
+            } else {
+                $table_name .= 's'; // 'deadline_extensions'
+            }
         } else {
             $table_name = static::$table_name;
         }

classes/stages/base.php

@@ -83,7 +83,7 @@ public function __construct($coursework, $stageidentifier) {
      */
     private function get_allocation_strategy() {
         $strategyname = $this->strategy_name();
-        $classname = "\\modcoursework\\allocation\\strategy\\{$strategyname}";
+        $classname = "\\mod_coursework\\allocation\\strategy\\{$strategyname}";
         return new $classname($this->get_coursework(), $this);
     }