Merge branch 'qa-ml-fix-view-commit-patch-test' into 'master'

Fabricate a file via the API

Closes gitlab-org/quality/nightly#42

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

3 files changed, 43 insertions, 16 deletions

diff --git a/qa/qa/resource/file.rb b/qa/qa/resource/file.rb
index 57e82ac19ad..ca74654bf90 100644
--- a/qa/qa/resource/file.rb
+++ b/qa/qa/resource/file.rb
@@ -3,9 +3,12 @@
 module QA
   module Resource
     class File < Base
-      attr_accessor :name,
+      attr_accessor :author_email,
+                    :author_name,
+                    :branch,
                     :content,
-                    :commit_message
+                    :commit_message,
+                    :name
 
       attribute :project do
         Project.fabricate! do |resource|
@@ -31,6 +34,30 @@ module QA
           page.commit_changes
         end
       end
+
+      def resource_web_url(resource)
+        super
+      rescue ResourceURLMissingError
+        # this particular resource does not expose a web_url property
+      end
+
+      def api_get_path
+        "/projects/#{CGI.escape(project.path_with_namespace)}/repository/files/#{CGI.escape(@name)}"
+      end
+
+      def api_post_path
+        api_get_path
+      end
+
+      def api_post_body
+        {
+          branch: @branch || "master",
+          author_email: @author_email || Runtime::User.default_email,
+          author_name: @author_name || Runtime::User.username,
+          content: content,
+          commit_message: commit_message
+        }
+      end
     end
   end
 end
diff --git a/qa/qa/specs/features/browser_ui/3_create/repository/create_edit_delete_file_via_web_spec.rb b/qa/qa/specs/features/browser_ui/3_create/repository/create_edit_delete_file_via_web_spec.rb
index 46346d1b984..d345fbfe995 100644
--- a/qa/qa/specs/features/browser_ui/3_create/repository/create_edit_delete_file_via_web_spec.rb
+++ b/qa/qa/specs/features/browser_ui/3_create/repository/create_edit_delete_file_via_web_spec.rb
@@ -12,7 +12,7 @@ module QA
         file_content = 'QA Test - File content'
         commit_message_for_create = 'QA Test - Create new file'
 
-        Resource::File.fabricate! do |file|
+        Resource::File.fabricate_via_browser_ui! do |file|
           file.name = file_name
           file.content = file_content
           file.commit_message = commit_message_for_create
diff --git a/qa/qa/specs/features/browser_ui/3_create/repository/user_views_commit_diff_patch_spec.rb b/qa/qa/specs/features/browser_ui/3_create/repository/user_views_commit_diff_patch_spec.rb
index b7400cdca97..680c5e21fa4 100644
--- a/qa/qa/specs/features/browser_ui/3_create/repository/user_views_commit_diff_patch_spec.rb
+++ b/qa/qa/specs/features/browser_ui/3_create/repository/user_views_commit_diff_patch_spec.rb
@@ -3,13 +3,16 @@
 module QA
   context 'Create' do
     # failure reported: https://gitlab.com/gitlab-org/quality/nightly/issues/42
-    # also failing in staging until the fix is picked into the next release:
-    #  https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/24533
     describe 'Commit data', :quarantine do
       before(:context) do
         Runtime::Browser.visit(:gitlab, Page::Main::Login)
         Page::Main::Login.perform(&:sign_in_using_credentials)
 
+        # Get the user's details to confirm they're included in the email patch
+        @user = Resource::User.fabricate_via_api! do |user|
+          user.username = Runtime::User.username
+        end
+
         project_push = Resource::Repository::ProjectPush.fabricate! do |push|
           push.file_name = 'README.md'
           push.file_content = '# This is a test project'
@@ -21,12 +24,13 @@ module QA
         # add second file to repo to enable diff from initial commit
         @commit_message = 'Add second file'
 
-        Page::Project::Show.perform(&:create_new_file!)
-        Page::File::Form.perform do |f|
-          f.add_name('second')
-          f.add_content('second file content')
-          f.add_commit_message(@commit_message)
-          f.commit_changes
+        Resource::File.fabricate_via_api! do |file|
+          file.project = @project
+          file.name = 'second'
+          file.content = 'second file content'
+          file.commit_message = @commit_message
+          file.author_name = @user.name
+          file.author_email = @user.public_email
         end
       end
 
@@ -42,15 +46,11 @@ module QA
       end
 
       it 'user views raw email patch' do
-        user = Resource::User.fabricate_via_api! do |user|
-          user.username = Runtime::User.username
-        end
-
         view_commit
 
         Page::Project::Commit::Show.perform(&:select_email_patches)
 
-        expect(page).to have_content("From: #{user.name} <#{user.public_email}>")
+        expect(page).to have_content("From: #{@user.name} <#{@user.public_email}>")
         expect(page).to have_content('Subject: [PATCH] Add second file')
         expect(page).to have_content('diff --git a/second b/second')
       end