Merge branch 'qa-fix-deploy-key-test' into 'master'

Fix add_deploy_key_spec.rb

Closes gitlab-org/quality/nightly#29

See merge request gitlab-org/gitlab-ce!24311

app/assets/javascripts/deploy_keys/components/key.vue

@@ -113,7 +113,7 @@ export default {
   <div class="gl-responsive-table-row deploy-key">
     <div class="table-section section-40">
       <div role="rowheader" class="table-mobile-header">{{ s__('DeployKeys|Deploy key') }}</div>
-      <div class="table-mobile-content">
+      <div class="table-mobile-content qa-key">
         <strong class="title qa-key-title"> {{ deployKey.title }} </strong>
         <div class="fingerprint qa-key-fingerprint">{{ deployKey.fingerprint }}</div>
       </div>

qa/qa/page/project/settings/deploy_keys.rb

@@ -14,8 +14,13 @@ class DeployKeys < Page::Base
           end
           view 'app/assets/javascripts/deploy_keys/components/key.vue' do
-            element :key_title, /class=".*qa-key-title.*"/ # rubocop:disable QA/ElementWithPattern
-            element :key_fingerprint, /class=".*qa-key-fingerprint.*"/ # rubocop:disable QA/ElementWithPattern
+            element :key
+            element :key_title
+            element :key_fingerprint
+          end
+
+          def add_key
+            click_on 'Add key'
           end
           def fill_key_title(title)
@@ -26,31 +31,29 @@ def fill_key_value(key)
             fill_in 'deploy_key_key', with: key
           end
-          def add_key
-            click_on 'Add key'
-          end
-
-          def key_title
+          def find_fingerprint(title)
             within_project_deploy_keys do
-              find_element(:key_title).text
+              find_element(:key, title)
+                .find(element_selector_css(:key_fingerprint)).text
             end
           end
-          def key_fingerprint
+          def has_key?(title, fingerprint)
             within_project_deploy_keys do
-              find_element(:key_fingerprint).text
+              find_element(:key, title)
+                .has_css?(element_selector_css(:key_fingerprint), text: fingerprint)
             end
           end
-          def key_titles
+          def key_title
             within_project_deploy_keys do
-              all_elements(:key_title)
+              find_element(:key_title).text
             end
           end
-          def key_fingerprints
+          def key_fingerprint
             within_project_deploy_keys do
-              all_elements(:key_fingerprint)
+              find_element(:key_fingerprint).text
             end
           end
@@ -58,7 +61,7 @@ def key_fingerprints
           def within_project_deploy_keys
             wait(reload: false) do
-              has_css?(element_selector_css(:project_deploy_keys))
+              has_element?(:project_deploy_keys)
             end
             within_element(:project_deploy_keys) do

qa/qa/resource/deploy_key.rb

@@ -8,11 +8,7 @@ class DeployKey < Base
       attribute :fingerprint do
         Page::Project::Settings::Repository.perform do |setting|
           setting.expand_deploy_keys do |key|
-            key_offset = key.key_titles.index do |key_title|
-              key_title.text == title
-            end
-
-            key.key_fingerprints[key_offset].text
+            key.find_fingerprint(title)
           end
         end
       end

qa/qa/specs/features/browser_ui/6_release/deploy_key/add_deploy_key_spec.rb

@@ -5,7 +5,7 @@ module QA
     describe 'Deploy key creation' do
       it 'user adds a deploy key' do
         Runtime::Browser.visit(:gitlab, Page::Main::Login)
-        Page::Main::Login.act { sign_in_using_credentials }
+        Page::Main::Login.perform(&:sign_in_using_credentials)
         key = Runtime::Key::RSA.new
         deploy_key_title = 'deploy key title'
@@ -16,7 +16,13 @@ module QA
           resource.key = deploy_key_value
         end
-        expect(deploy_key.fingerprint).to eq(key.fingerprint)
+        expect(deploy_key.fingerprint).to eq key.fingerprint
+
+        Page::Project::Settings::Repository.perform do |setting|
+          setting.expand_deploy_keys do |keys|
+            expect(keys).to have_key(deploy_key_title, key.fingerprint)
+          end
+        end
       end
     end
   end