MDL-66553 question: Behat fixes
authorJun Pataleta <jun@moodle.com>
Tue, 15 Oct 2019 12:22:12 +0000 (20:22 +0800)
committerJun Pataleta <jun@moodle.com>
Tue, 15 Oct 2019 12:22:12 +0000 (20:22 +0800)
1. Fix sortable key for the qname/idnumber/tags column
2. Fix the default sort
3. Fix Behat test for sorting by idnumber. Added ID numbers for both
question A and question C in order for us to get consistent sorting
across DBs.

question/classes/bank/question_name_idnumber_tags_column.php
question/classes/bank/view.php
question/tests/behat/sort_questions.feature

index b9dc3f8..872e1d8 100644 (file)
@@ -78,7 +78,7 @@ class question_name_idnumber_tags_column extends question_name_column {
     public function is_sortable() {
         return [
             'name' => ['field' => 'q.name', 'title' => get_string('questionname', 'question')],
-            'lastname' => ['field' => 'q.idnumber', 'title' => get_string('idnumber', 'question')],
+            'idnumber' => ['field' => 'q.idnumber', 'title' => get_string('idnumber', 'question')],
         ];
     }
 
index 850ce2b..5bfe5d3 100644 (file)
@@ -347,7 +347,10 @@ class view {
     }
 
     protected function default_sort() {
-        return array('core_question\bank\question_type_column' => 1, 'core_question\bank\question_name_column' => 1);
+        return array(
+            'core_question\bank\question_type_column' => 1,
+            'core_question\bank\question_name_idnumber_tags_column-name' => 1
+        );
     }
 
     /**
index 7a5c84d..25a103e 100644 (file)
@@ -18,10 +18,10 @@ Feature: The questions in the question bank can be sorted in various ways
       | contextlevel | reference | name           |
       | Course       | C1        | Test questions |
     And the following "questions" exist:
-      | questioncategory | qtype     | name              | user     | questiontext    | idnumber |
-      | Test questions   | essay     | A question 1 name | admin    | Question 1 text |          |
-      | Test questions   | essay     | B question 2 name | teacher1 | Question 2 text |          |
-      | Test questions   | numerical | C question 3 name | teacher1 | Question 3 text | numidnum |
+      | questioncategory | qtype     | name              | user     | questiontext    | idnumber  |
+      | Test questions   | essay     | A question 1 name | admin    | Question 1 text | numidnuma |
+      | Test questions   | essay     | B question 2 name | teacher1 | Question 2 text |           |
+      | Test questions   | numerical | C question 3 name | teacher1 | Question 3 text | numidnumc |
     And I log in as "teacher1"
     And I am on "Course 1" course homepage
     And I navigate to "Question bank > Questions" in current page administration
@@ -33,8 +33,10 @@ Feature: The questions in the question bank can be sorted in various ways
   @javascript
   Scenario: The questions can be sorted by idnumber
     When I follow "Sort by ID number ascending"
-    Then "C question 3 name" "checkbox" should appear before "A question 1 name" "checkbox"
-    And I should see "numidnum" in the "C question 3 name" "table_row"
+    Then "C question 3 name" "checkbox" should appear after "A question 1 name" "checkbox"
+    And I should see "numidnumc" in the "C question 3 name" "table_row"
+    And I follow "Sort by ID number descending"
+    And "C question 3 name" "checkbox" should appear before "A question 1 name" "checkbox"
 
   @javascript
   Scenario: The questions can be sorted in reverse order by type