MDL-66818 mod_forum: remove unnecessary behat steps
authorSimey Lameze <simey@moodle.com>
Wed, 19 Aug 2020 23:43:18 +0000 (07:43 +0800)
committerSimey Lameze <simey@moodle.com>
Thu, 20 Aug 2020 00:05:00 +0000 (08:05 +0800)
lib/tests/behat/behat_forms.php
mod/forum/tests/behat/portfolio_export.feature

index 25a1071..2532f94 100644 (file)
@@ -56,26 +56,6 @@ class behat_forms extends behat_base {
         $this->execute('behat_general::i_click_on', [$button, 'button']);
     }
 
         $this->execute('behat_general::i_click_on', [$button, 'button']);
     }
 
-    /**
-     * Ensure a button exists.
-     * @Then the button :button exists
-     * @param string $button
-     */
-    public function button_exists(string $button) {
-        $buttonnode = $this->find_button($button);
-        $this->ensure_node_is_visible($buttonnode);
-    }
-
-    /**
-     * Ensure that a button does not exist.
-     * @Then the button :button does not exist
-     * @param string $button
-     */
-    public function button_does_not_exist(string $button) {
-        $general = behat_context_helper::get('behat_general');
-        $general->should_not_exist($button, 'button');
-    }
-
     /**
      * Press button with specified id|name|title|alt|value and switch to main window.
      *
     /**
      * Press button with specified id|name|title|alt|value and switch to main window.
      *
index f6e60d2..0bf6df2 100644 (file)
@@ -13,7 +13,7 @@ Feature: Ensure only users with appropriate permissions can export forum discuss
       | user | course | role |
       | teacher1 | C1 | editingteacher |
       | student1 | C1 | student |
       | user | course | role |
       | teacher1 | C1 | editingteacher |
       | student1 | C1 | student |
-  @javascript
+
   Scenario: A teacher can export discussions to a portfolio.
     Given the following "activities" exist:
       | activity   | name                   | intro             | course | idnumber     | groupmode |
   Scenario: A teacher can export discussions to a portfolio.
     Given the following "activities" exist:
       | activity   | name                   | intro             | course | idnumber     | groupmode |
@@ -25,7 +25,7 @@ Feature: Ensure only users with appropriate permissions can export forum discuss
     And I set portfolio instance "File download" to "Enabled and visible"
     And I click on "Save" "button"
     And I log out
     And I set portfolio instance "File download" to "Enabled and visible"
     And I click on "Save" "button"
     And I log out
-    And I log in as "student1"
+    When I log in as "student1"
     And I am on "Course 1" course homepage
     And I follow "Test forum 1"
     And I add a new discussion to "Test forum 1" forum with:
     And I am on "Course 1" course homepage
     And I follow "Test forum 1"
     And I add a new discussion to "Test forum 1" forum with:
@@ -33,12 +33,12 @@ Feature: Ensure only users with appropriate permissions can export forum discuss
       | Message | Test post message |
     And I reload the page
     And I follow "Discussion 1"
       | Message | Test post message |
     And I reload the page
     And I follow "Discussion 1"
-    And the button "Export whole discussion to portfolio" does not exist
+    Then "Export whole discussion to portfolio" "button" should not exist
     And I log out
     And I log out
-    And I log in as "admin"
+    And I log in as "teacher1"
     And I am on "Course 1" course homepage
     And I follow "Test forum 1"
     And I follow "Discussion 1"
     And I am on "Course 1" course homepage
     And I follow "Test forum 1"
     And I follow "Discussion 1"
-    And the button "Export whole discussion to portfolio" exists
+    And "Export whole discussion to portfolio" "button" should exist
     And I press "Export whole discussion to portfolio"
     And I press "Export whole discussion to portfolio"
-    Then I should see "Exporting to portfolio"
+    And I should see "Exporting to portfolio"