Remove blocks from Runtime::Browser.visit calls within Mattermost tests

Use .perform instead of .act in Mattermost test
Signed-off-by: ddavison <ddavison@gitlab.com>

qa/qa/specs/features/browser_ui/1_manage/login/log_into_mattermost_via_gitlab_spec.rb

@@ -4,11 +4,11 @@ module QA
   context 'Manage', :orchestrated, :mattermost do
     describe 'Mattermost login' do
       it 'user logs into Mattermost using GitLab OAuth' do
-        Runtime::Browser.visit(:gitlab, Page::Main::Login) do
-          Page::Main::Login.act { sign_in_using_credentials }
+        Runtime::Browser.visit(:gitlab, Page::Main::Login)
+        Page::Main::Login.perform(&:sign_in_using_credentials)
 
-          Runtime::Browser.visit(:mattermost, Page::Mattermost::Login) do
-            Page::Mattermost::Login.act { sign_in_using_oauth }
+        Runtime::Browser.visit(:mattermost, Page::Mattermost::Login)
+        Page::Mattermost::Login.perform(&:sign_in_using_oauth)
 
         Page::Mattermost::Main.perform do |page|
           expect(page).to have_content(/(Welcome to: Mattermost|Logout GitLab Mattermost)/)
@@ -16,6 +16,4 @@ module QA
       end
     end
   end
-    end
-  end
 end