MDL-56511 Theme: behat fixes for theme clean
authorBas Brands <bas@moodle.com>
Thu, 19 Apr 2018 18:51:13 +0000 (20:51 +0200)
committerDavid Monllao <davidm@moodle.com>
Fri, 20 Apr 2018 04:18:21 +0000 (06:18 +0200)
blocks/tests/behat/behat_blocks.php
course/classes/management_renderer.php
course/tests/behat/category_change_visibility.feature
course/tests/behat/category_management.feature
course/tests/behat/course_change_visibility.feature
lib/templates/action_menu_trigger.mustache
lib/templates/initials_bar.mustache
theme/boost/classes/output/core_course/management/renderer.php
theme/boost/templates/core/action_menu_trigger.mustache

index 9f459f9..b05d84c 100644 (file)
@@ -108,7 +108,7 @@ class behat_blocks extends behat_base {
         }
 
         $this->execute('behat_general::i_click_on_in_the',
-            array("a[role='button']", "css_element", $this->escape($blockname), "block")
+            array(get_string('actions'), "link", $this->escape($blockname), "block")
         );
     }
 
index 13b9e59..56e47e9 100644 (file)
@@ -528,7 +528,7 @@ class core_course_management_renderer extends plugin_renderer_base {
             array('id' => 'course-listing-title', 'tabindex' => '0'));
         $html .= $this->course_listing_actions($category, $course, $perpage);
         $html .= $this->listing_pagination($category, $page, $perpage);
-        $html .= html_writer::start_tag('ul', array('class' => 'ml-1', 'role' => 'group'));
+        $html .= html_writer::start_tag('ul', array('class' => 'ml-1 course-list', 'role' => 'group'));
         foreach ($category->get_courses($options) as $listitem) {
             $html .= $this->course_listitem($category, $listitem, $courseid);
         }
index a59850c..55087be 100644 (file)
@@ -134,9 +134,9 @@ Feature: We can change the visibility of categories in the management interface.
     And I should see "Cat 2" in the "#category-listing ul" "css_element"
     And I should see "Cat 3" in the "#category-listing ul" "css_element"
     And I should see "Cat 4" in the "#category-listing ul" "css_element"
-    And I should see "Course 1" in the "#course-listing ul" "css_element"
-    And I should see "Course 2" in the "#course-listing ul" "css_element"
-    And I should see "Course 3" in the "#course-listing ul" "css_element"
+    And I should see "Course 1" in the "#course-listing ul.course-list" "css_element"
+    And I should see "Course 2" in the "#course-listing ul.course-list" "css_element"
+    And I should see "Course 3" in the "#course-listing ul.course-list" "css_element"
     And category in management listing should be visible "CAT1"
     And category in management listing should be visible "CAT2"
     And category in management listing should be visible "CAT3"
@@ -211,9 +211,9 @@ Feature: We can change the visibility of categories in the management interface.
     And I should see "Cat 2" in the "#category-listing ul" "css_element"
     And I should see "Cat 3" in the "#category-listing ul" "css_element"
     And I should see "Cat 4" in the "#category-listing ul" "css_element"
-    And I should see "Course 1" in the "#course-listing ul" "css_element"
-    And I should see "Course 2" in the "#course-listing ul" "css_element"
-    And I should see "Course 3" in the "#course-listing ul" "css_element"
+    And I should see "Course 1" in the "#course-listing ul.course-list" "css_element"
+    And I should see "Course 2" in the "#course-listing ul.course-list" "css_element"
+    And I should see "Course 3" in the "#course-listing ul.course-list" "css_element"
     And category in management listing should be visible "CAT1"
     And category in management listing should be visible "CAT2"
     And category in management listing should be visible "CAT3"
index 151cd7e..969fcc2 100644 (file)
@@ -101,7 +101,7 @@ Feature: Test category management actions
     And I should see "Cat 1" in the "#category-listing ul" "css_element"
     And I should not see "Cat 2" in the "#category-listing ul" "css_element"
     And I should not see "Cat 3" in the "#category-listing ul" "css_element"
-    And I should see "Course 1" in the "#course-listing ul" "css_element"
+    And I should see "Course 1" in the "#course-listing ul.course-list" "css_element"
 
   Scenario: Test I can assign roles for a category through the management interface.
     Given the following "categories" exist:
@@ -298,8 +298,8 @@ Feature: Test category management actions
     And I click on category "Cat 3" in the management interface
     #Redirect
     And I should see the "Course categories and courses" management page
-    And I should see "Course 1" in the "#course-listing ul" "css_element"
-    And I should see "Course 2" in the "#course-listing ul" "css_element"
+    And I should see "Course 1" in the "#course-listing ul.course-list" "css_element"
+    And I should see "Course 2" in the "#course-listing ul.course-list" "css_element"
     And the "movecategoriesto" "select" should be disabled
     And the "resortcategoriesby" "select" should be disabled
     And the "resortcoursesby" "select" should be disabled
index 2c41116..25c5014 100644 (file)
@@ -20,7 +20,7 @@ Feature: We can change the visibility of courses in the management interface.
     # Redirect.
     And I should see the "Course categories and courses" management page
     And I should see "Cat 1" in the "#category-listing ul" "css_element"
-    And I should see "Course 1" in the "#course-listing ul" "css_element"
+    And I should see "Course 1" in the "#course-listing ul.course-list" "css_element"
     And category in management listing should be visible "CAT1"
     And course in management listing should be visible "C1"
     And I toggle visibility of course "C1" in management listing
@@ -69,7 +69,7 @@ Feature: We can change the visibility of courses in the management interface.
     And I start watching to see if a new page loads
     And I should see the "Course categories and courses" management page
     And I should see "Cat 1" in the "#category-listing ul" "css_element"
-    And I should see "Course 1" in the "#course-listing ul" "css_element"
+    And I should see "Course 1" in the "#course-listing ul.course-list" "css_element"
     And category in management listing should be visible "CAT1"
     And course in management listing should be visible "C1"
     And I toggle visibility of course "C1" in management listing
@@ -98,7 +98,7 @@ Feature: We can change the visibility of courses in the management interface.
     And I click on "Courses" "link" in the ".view-mode-selector" "css_element"
     And a new page should have loaded since I started watching
     And I start watching to see if a new page loads
-    And I should see "Course 1" in the "#course-listing ul" "css_element"
+    And I should see "Course 1" in the "#course-listing ul.course-list" "css_element"
     And I toggle visibility of course "C1" in management listing
     And a new page should not have loaded since I started watching
     And course in management listing should be dimmed "C1"
index fce1a8b..2a79115 100644 (file)
@@ -27,4 +27,4 @@
         "triggerextraclasses": ""
     }
 }}
-<a href="#" class="{{triggerextraclasses}} toggle-display {{#menutrigger}}textmenu{{/menutrigger}}" id="action-menu-toggle-{{instance}}" title="{{title}}" role="menuitem">{{{actiontext}}}{{{menutrigger}}}{{#icon}}{{#pix}}{{key}}, {{component}}, {{title}}{{/pix}}{{/icon}}{{#rawicon}}{{{.}}}{{/rawicon}}{{#menutrigger}}<b class="caret"></b>{{/menutrigger}}</a>
+<a href="#" class="{{triggerextraclasses}} toggle-display {{#menutrigger}}textmenu{{/menutrigger}}" id="action-menu-toggle-{{instance}}" title="{{title}}">{{{actiontext}}}{{{menutrigger}}}{{#icon}}{{#pix}}{{key}}, {{component}}, {{title}}{{/pix}}{{/icon}}{{#rawicon}}{{{.}}}{{/rawicon}}{{#menutrigger}}<b class="caret"></b>{{/menutrigger}}</a>
index dcb7001..2fe1331 100644 (file)
 <div class="initialbar {{class}}">
     <span class="initialbarlabel">{{title}}</span>
     {{#current}}
-        <a class="initialbarall letter" href="{{url}}">{{all}}</a>
+        <a class="initialbarall page-item letter" href="{{url}}">{{all}}</a>
     {{/current}}
     {{^current}}
-        <div class="initialbarall letter active">{{all}}</div>
+        <div class="initialbarall letter page-item active">{{all}}</div>
     {{/current}}
 
     <div class="initialbargroups">
             <ul class="initialbargroup">
                 {{#letter}}
                     {{#selected}}
-                        <li><span class="letter active {{name}}">{{name}}</span></li>
+                        <li><span class="letter page-item active {{name}}">{{name}}</span></li>
                     {{/selected}}
                     {{^selected}}
-                        <li><a class="letter {{name}}" href="{{url}}">{{name}}</a></li>
+                        <li><a class="letter page-item {{name}}" href="{{url}}">{{name}}</a></li>
                     {{/selected}}
                 {{/letter}}
             </ul>
index d626a66..236de1c 100644 (file)
@@ -461,7 +461,7 @@ class renderer extends \core_course_management_renderer {
         $html .= html_writer::start_div('card-body');
         $html .= $this->course_listing_actions($category, $course, $perpage);
         $html .= $this->listing_pagination($category, $page, $perpage);
-        $html .= html_writer::start_tag('ul', array('class' => 'ml', 'role' => 'group'));
+        $html .= html_writer::start_tag('ul', array('class' => 'ml course-list', 'role' => 'group'));
         foreach ($category->get_courses($options) as $listitem) {
             $html .= $this->course_listitem($category, $listitem, $courseid);
         }
index 93995e1..f5078f9 100644 (file)
@@ -78,7 +78,7 @@
     }
 }}
 <div class="dropdown">
-    <a href="#" class="{{triggerextraclasses}} dropdown-toggle" id="dropdown-{{instance}}" title="{{title}}" role="button" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">
+    <a href="#" class="{{triggerextraclasses}} dropdown-toggle" id="dropdown-{{instance}}" title="{{title}}" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">
         {{{actiontext}}}
         {{{menutrigger}}}
             {{#icon}}