Refactor end-to-end test

- to be explicit when fabricating resources via API or browser UI - and to improve test speed by fabricating some resources via API instead of via the GUI.

Refactor end-to-end test
- to be explicit when fabricating resources via API or browser UI - and to improve test speed by fabricating some resources via API instead of via the GUI.
7c8ce1fe · Walmyr Lima · 10b30708 · 7c8ce1fe
--- a/qa/qa/specs/features/browser_ui/2_plan/issue/check_mentions_for_xss_spec.rb
+++ b/qa/qa/specs/features/browser_ui/2_plan/issue/check_mentions_for_xss_spec.rb
@@ -9,12 +9,12 @@ module QA
        Runtime::Browser.visit(:gitlab, Page::Main::Login)
        Page::Main::Login.perform(&:sign_in_using_credentials)

-        user = Resource::User.fabricate! do |user|
+        user = Resource::User.fabricate_via_api! do |user|
          user.name = "eve <img src=x onerror=alert(2)&lt;img src=x onerror=alert(1)&gt;"
          user.password = "test1234"
        end

-        project = Resource::Project.fabricate! do |resource|
+        project = Resource::Project.fabricate_via_api! do |resource|
          resource.name = 'xss-test-for-mentions-project'
        end
        project.visit!
@@ -24,10 +24,11 @@ module QA
          page.add_member(user.username)
        end

-        Resource::Issue.fabricate_via_browser_ui! do |issue|
+        issue = Resource::Issue.fabricate_via_api! do |issue|
          issue.title = issue_title
          issue.project = project
        end
+        issue.visit!

        Page::Project::Issue::Show.perform do |show_page|
          show_page.select_all_activities_filter