Skip to content

Commit

Permalink
MDL-81805 behat: remove unnecessary @javascript from tests
Browse files Browse the repository at this point in the history
  • Loading branch information
lameze committed May 7, 2024
1 parent 3fdb0f3 commit cb49940
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@ Feature: In a course administration page, navigate through report page, test for
| admin | C1 | editingteacher |
| student1 | C1 | student |

@javascript
Scenario: Selector should be available in the course participation page
Given I log in as "admin"
And I am on "Course 1" course homepage
Expand Down
3 changes: 0 additions & 3 deletions report/participation/tests/behat/filter_participation.feature
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@ Feature: In a participation report, admin can filter student actions
| name | Test forum name |
| idnumber | forum1 |

@javascript
Scenario: Filter participation report when standard log reader is enabled later
Given I log in as "admin"
And I navigate to "Plugins > Logging > Manage log stores" in site administration
Expand All @@ -51,7 +50,6 @@ Feature: In a participation report, admin can filter student actions
| -1- | All actions |
| Student 1 | Yes (1) |

@javascript
Scenario: Filter participation report when only standard log reader is enabled by default
Given I am on the "Test forum name" "forum activity" page logged in as student1
When I am on the "Course 1" course page logged in as admin
Expand All @@ -65,7 +63,6 @@ Feature: In a participation report, admin can filter student actions
| -1- | All actions |
| Student 1 | Yes (1) |

@javascript
Scenario Outline: Filter participation report by viewable roles
Given I am on the "Course 1" course page logged in as "teacher1"
When I navigate to "Reports" in current page administration
Expand Down
8 changes: 5 additions & 3 deletions report/participation/tests/behat/message_participants.feature
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
@report @report_participation @javascript
Feature: Use the particiaption report to message groups of students
@report @report_participation
Feature: Use the participation report to message groups of students
In order to engage with students based on participation
As a teacher
I need to be able to message students who have not participated in an activity
Expand Down Expand Up @@ -28,6 +28,7 @@ Feature: Use the particiaption report to message groups of students
| idnumber | book1 |
And I am on the "Test book name" "book activity" page logged in as student1

@javascript
Scenario: Message all students from the participation report
Given I am on the "Course 1" course page logged in as teacher1
And I navigate to "Reports" in current page administration
Expand All @@ -42,6 +43,7 @@ Feature: Use the particiaption report to message groups of students
And I press "Send message to 3 people"
And I should see "Message sent to 3 people"

@javascript
Scenario: Message students who have not participated in book
Given I am on the "Course 1" course page logged in as teacher1
And I navigate to "Reports" in current page administration
Expand All @@ -59,7 +61,7 @@ Feature: Use the particiaption report to message groups of students
And I press "Send message to 2 people"
And I should see "Message sent to 2 people"

Scenario: Ensure no message options when messaging is disabled
Scenario: When messaging is disabled no message options should be displayed
Given the following config values are set as admin:
| messaging | 0 |
And I am on the "Course 1" course page logged in as teacher1
Expand Down

0 comments on commit cb49940

Please sign in to comment.