Merge branch 'qa/thrid-tentative-of-remove-flakiness-from-e2e-test' into 'master'

Third tentative of removing flakiness from 'promote issue into epic' e2e test

Closes gitlab-org/quality/nightly#106

See merge request gitlab-org/gitlab-ee!14008

qa/qa/page/project/issue/show.rb

@@ -15,12 +15,12 @@ module QA
           end
 
           view 'app/assets/javascripts/notes/components/comment_form.vue' do
-            element :comment_button, required: true
-            element :comment_input, required: true
+            element :comment_button
+            element :comment_input
           end
 
           view 'app/assets/javascripts/notes/components/discussion_filter.vue' do
-            element :discussion_filter
+            element :discussion_filter, required: true
             element :filter_options
           end
 

qa/qa/specs/features/browser_ui/2_plan/ee_epic/promote_issue_to_epic_spec.rb

@@ -23,6 +23,10 @@ module QA
         end
 
         Page::Project::Issue::Show.perform do |show|
+          # Due to the randomness of tests execution, sometimes a previous test
+          # may have changed the filter, which makes the below action needed.
+          # TODO: Make this test completely independent, not requiring the below step.
+          show.select_all_activities_filter
           show.comment('/promote')
 
           expect(show).to have_content("promoted to epic")