Merge branch 'qa-e2e-improve-review-merge-request-spec' into 'master'

Improvements to make review_merge_request_spec more reliable

See merge request gitlab-org/gitlab!28974

qa/qa/page/project/web_ide/edit.rb

@@ -120,10 +120,12 @@ module QA
 
           def add_to_modified_content(content)
             finished_loading?
+            modified_text_area.click
             modified_text_area.set content
           end
 
           def modified_text_area
+            wait_for_animated_element(:editor_container)
             within_element(:editor_container) do
               find('.modified textarea.inputarea')
             end

qa/qa/specs/features/browser_ui/3_create/web_ide/review_merge_request_spec.rb

@@ -4,12 +4,13 @@ module QA
   context 'Create', quarantine: { type: :new } do
     describe 'Review a merge request in Web IDE' do
       let(:new_file) { 'awesome_new_file.txt' }
+      let(:original_text) { 'Text' }
       let(:review_text) { 'Reviewed ' }
 
       let(:merge_request) do
         Resource::MergeRequest.fabricate_via_api! do |mr|
           mr.file_name = new_file
-          mr.file_content = 'Text'
+          mr.file_content = original_text
         end
       end