Merge branch 'negatable-predicate-methods' into 'master'

Add negatable predicate methods

See merge request gitlab-org/gitlab!48242

qa/.gitignore

 tmp/
 .ruby-version
+.tool-versions
 .ruby-gemset
 urls.yml

qa/qa/ee/page/group/iteration/show.rb

@@ -16,6 +16,12 @@ module QA
                 has_element?(:iteration_issue_link, issue_title: issue.title)
               end
             end
+
+            def has_no_issue?(issue)
+              within_element(:iteration_issues_container) do
+                has_no_element?(:iteration_issue_link, issue_title: issue.title)
+              end
+            end
           end
         end
       end

qa/qa/page/component/design_management.rb

@@ -108,6 +108,10 @@ module QA
           has_element?(:design_file_name, text: filename)
         end
 
+        def has_no_design?(filename)
+          has_no_element?(:design_file_name, text: filename)
+        end
+
         def has_created_icon?
           has_element?(:design_status_icon, status: 'file-addition-solid')
         end

qa/qa/page/component/issuable/sidebar.rb

@@ -69,7 +69,7 @@ module QA
             end
           end
 
-          def has_no_assignee_named?(username)
+          def has_no_assignee?(username)
             within_element(:assignee_block) do
               has_no_text?(username, wait: 120)
             end

qa/qa/page/file/show.rb

@@ -46,8 +46,16 @@ module QA
           has_no_element?(:file_name_content, text: name)
         end
 
-        def has_file_content?(text)
-          has_element?(:file_content, text: text)
+        def has_file_content?(file_content, file_number = nil)
+          if file_number
+            within_element_by_index(:file_content, file_number - 1) do
+              has_text?(file_content)
+            end
+          else
+            within_element(:file_content) do
+              has_text?(file_content)
+            end
+          end
         end
       end
     end

qa/qa/page/project/issue/index.rb

@@ -79,6 +79,10 @@ module QA
           def has_issue?(issue)
             has_element? :issue_container, issue_title: issue.title
           end
+
+          def has_no_issue?(issue)
+            has_no_element? :issue_container, issue_title: issue.title
+          end
         end
       end
     end

qa/qa/page/project/pipeline/index.rb

@@ -49,6 +49,10 @@ module QA
             has_element? :pipeline_url_link
           end
 
+          def has_no_pipeline?
+            has_no_element? :pipeline_url_link
+          end
+
           def click_run_pipeline_button
             click_element :run_pipeline_button, Page::Project::Pipeline::New
           end

qa/qa/page/project/pipeline/show.rb

@@ -72,6 +72,10 @@ module QA
             has_element? :child_pipeline
           end
 
+          def has_no_child_pipeline?
+            has_no_element? :child_pipeline
+          end
+
           def click_job(job_name)
             click_element(:job_link, text: job_name)
           end

qa/qa/page/project/wiki/show.rb

@@ -59,6 +59,10 @@ module QA
             has_element?(:wiki_page_content, content)
           end
 
+          def has_no_content?(content)
+            has_no_element?(:wiki_page_content, content)
+          end
+
           def has_no_page?
             has_element? :create_first_page_link
           end

qa/qa/specs/features/browser_ui/2_plan/issue/real_time_assignee_spec.rb

@@ -40,12 +40,12 @@ module QA
           issue.set_issue_assignees(assignee_ids: [user2.id])
 
           expect(show).to have_assignee(user2.name)
-          expect(show).to have_no_assignee_named(user1.name)
+          expect(show).not_to have_assignee(user1.name)
 
           issue.set_issue_assignees(assignee_ids: [])
 
-          expect(show).to have_no_assignee_named(user1.name)
-          expect(show).to have_no_assignee_named(user2.name)
+          expect(show).not_to have_assignee(user1.name)
+          expect(show).not_to have_assignee(user2.name)
         end
       end
     end

qa/qa/specs/features/browser_ui/2_plan/related_issues/related_issues_spec.rb

@@ -37,7 +37,7 @@ module QA
 
           show.click_remove_related_issue_button
 
-          expect(show).to have_no_text(issue_2.title, wait: max_wait)
+          expect(show).not_to have_text(issue_2.title, wait: max_wait)
         end
       end
     end

qa/qa/specs/features/browser_ui/3_create/snippet/create_project_snippet_spec.rb

@@ -16,13 +16,13 @@ module QA
 
         Page::Dashboard::Snippet::Show.perform do |snippet|
           expect(snippet).to have_snippet_title('Project snippet')
-          expect(snippet).to have_no_snippet_description
+          expect(snippet).not_to have_snippet_description
           expect(snippet).to have_visibility_type(/private/i)
           expect(snippet).to have_file_name('markdown_file.md')
           expect(snippet).to have_file_content('Snippet heading')
           expect(snippet).to have_file_content('Gitlab link')
-          expect(snippet).to have_no_file_content('###')
-          expect(snippet).to have_no_file_content('https://gitlab.com/')
+          expect(snippet).not_to have_file_content('###')
+          expect(snippet).not_to have_file_content('https://gitlab.com/')
         end
       end
     end

qa/qa/specs/features/browser_ui/3_create/snippet/delete_file_from_snippet_spec.rb

@@ -46,8 +46,8 @@ module QA
             aggregate_failures 'file names and contents' do
               expect(snippet).to have_file_name('Original file name')
               expect(snippet).to have_file_content('Original file content')
-              expect(snippet).to have_no_file_name('Second file name')
-              expect(snippet).to have_no_file_content('Second file content')
+              expect(snippet).not_to have_file_name('Second file name')
+              expect(snippet).not_to have_file_content('Second file content')
             end
           end
         end

qa/qa/specs/features/browser_ui/4_verify/ci_variable/add_remove_ci_variable_spec.rb

@@ -19,7 +19,7 @@ module QA
       it 'user adds a CI variable', testcase: 'https://gitlab.com/gitlab-org/quality/testcases/-/issues/395' do
         Page::Project::Settings::CiVariables.perform do |ci_variable|
           expect(ci_variable).to have_text('VARIABLE_KEY')
-          expect(ci_variable).to have_no_text('some_CI_variable')
+          expect(ci_variable).not_to have_text('some_CI_variable')
 
           ci_variable.click_reveal_ci_variable_value_button
 

qa/qa/specs/features/browser_ui/4_verify/pipeline/include_multiple_files_from_a_project_spec.rb

@@ -54,7 +54,7 @@ module QA
 
         Page::Project::Job::Show.perform do |job|
           aggregate_failures 'main CI is not overridden' do
-            expect(job.output).to have_no_content("#{unexpected_text}")
+            expect(job.output).not_to have_content("#{unexpected_text}")
             expect(job.output).to have_content("#{expected_text}")
           end
         end

qa/qa/specs/features/browser_ui/5_package/composer_registry_spec.rb

@@ -112,7 +112,7 @@ module QA
         Page::Project::Packages::Index.perform do |index|
           aggregate_failures 'package deletion' do
             expect(index).to have_content("Package deleted successfully")
-            expect(index).to have_no_package(package_name)
+            expect(index).not_to have_package(package_name)
           end
         end
       end

qa/qa/specs/features/browser_ui/5_package/conan_repository_spec.rb

@@ -80,7 +80,7 @@ module QA
 
         Page::Project::Packages::Index.perform do |index|
           expect(index).to have_content("Package deleted successfully")
-          expect(index).to have_no_package(package_name)
+          expect(index).not_to have_package(package_name)
         end
       end
     end

qa/qa/specs/features/browser_ui/5_package/maven_gradle_repository_spec.rb

@@ -118,7 +118,7 @@ module QA
 
         Page::Project::Packages::Index.perform do |index|
           expect(index).to have_content("Package deleted successfully")
-          expect(index).to have_no_package(package_name)
+          expect(index).not_to have_package(package_name)
         end
       end
     end

qa/qa/specs/features/browser_ui/5_package/maven_repository_spec.rb

@@ -96,7 +96,7 @@ module QA
 
         Page::Project::Packages::Index.perform do |index|
           expect(index).to have_content("Package deleted successfully")
-          expect(index).to have_no_package(package_name)
+          expect(index).not_to have_package(package_name)
         end
       end
     end

qa/qa/specs/features/browser_ui/5_package/npm_registry_spec.rb

@@ -69,7 +69,7 @@ module QA
 
         Page::Project::Packages::Index.perform do |index|
           expect(index).to have_content("Package deleted successfully")
-          expect(index).to have_no_package(package_name)
+          expect(index).not_to have_package(package_name)
         end
       end
     end

qa/qa/specs/features/browser_ui/5_package/nuget_repository_spec.rb

@@ -84,7 +84,7 @@ module QA
 
         Page::Project::Packages::Index.perform do |index|
           expect(index).to have_content("Package deleted successfully")
-          expect(index).to have_no_package(package_name)
+          expect(index).not_to have_package(package_name)
         end
       end
     end

qa/qa/specs/features/browser_ui/5_package/pypi_repository_spec.rb

@@ -104,7 +104,7 @@ module QA
         Page::Project::Packages::Index.perform do |index|
           aggregate_failures do
             expect(index).to have_content("Package deleted successfully")
-            expect(index).to have_no_package(package_name)
+            expect(index).not_to have_package(package_name)
           end
         end
       end

qa/qa/specs/features/ee/browser_ui/6_release/multi-project_pipelines_spec.rb

@@ -88,7 +88,7 @@ module QA
 
         Page::Project::Pipeline::Show.perform do |show|
           expect(show).to have_passed
-          expect(show).to have_no_job("downstream_job")
+          expect(show).not_to have_job("downstream_job")
 
           show.expand_child_pipeline
 

qa/qa/specs/features/ee/browser_ui/geo/geo_replication_project_snippets_spec.rb

@@ -50,8 +50,8 @@ module QA
               expect(snippet).to have_file_name(file_name)
               expect(snippet).to have_file_content('Geo snippet heading')
               expect(snippet).to have_file_content('GitLab link')
-              expect(snippet).to have_no_file_content('###')
-              expect(snippet).to have_no_file_content('https://gitlab.com/')
+              expect(snippet).not_to have_file_content('###')
+              expect(snippet).not_to have_file_content('https://gitlab.com/')
             end
           end
         end

qa/spec/support/matchers/have_assignee.rb

+# frozen_string_literal: true
+
+module Matchers
+  module HaveAssignee
+    RSpec::Matchers.define :have_assignee do |assignee|
+      match do |page_object|
+        page_object.has_assignee?(assignee)
+      end
+
+      match_when_negated do |page_object|
+        page_object.has_no_assignee?(assignee)
+      end
+    end
+  end
+end

qa/spec/support/matchers/have_child_pipeline.rb

+# frozen_string_literal: true
+
+module Matchers
+  module HaveChildPipeline
+    RSpec::Matchers.define :have_child_pipeline do
+      match do |page_object|
+        page_object.has_child_pipeline?
+      end
+
+      match_when_negated do |page_object|
+        page_object.has_no_child_pipeline?
+      end
+    end
+  end
+end

qa/spec/support/matchers/have_content.rb

+# frozen_string_literal: true
+
+module Matchers
+  module HaveContent
+    RSpec::Matchers.define :have_content do |content|
+      match do |page_object|
+        page_object.has_content?(content)
+      end
+
+      match_when_negated do |page_object|
+        page_object.has_no_content?(content)
+      end
+    end
+  end
+end

qa/spec/support/matchers/have_design.rb

+# frozen_string_literal: true
+
+module Matchers
+  module HaveDesign
+    RSpec::Matchers.define :have_design do |design|
+      match do |page_object|
+        page_object.has_design?(design)
+      end
+
+      match_when_negated do |page_object|
+        page_object.has_no_design?(design)
+      end
+    end
+  end
+end

qa/spec/support/matchers/have_element.rb

+# frozen_string_literal: true
+
+module Matchers
+  module HaveElement
+    RSpec::Matchers.define :have_element do |element, **kwargs|
+      match do |page_object|
+        page_object.has_element?(element, **kwargs)
+      end
+
+      match_when_negated do |page_object|
+        page_object.has_no_element?(element, **kwargs)
+      end
+    end
+  end
+end

qa/spec/support/matchers/have_file_content.rb

+# frozen_string_literal: true
+
+module Matchers
+  module HaveFileContent
+    RSpec::Matchers.define :have_file_content do |file_content, file_number|
+      match do |page_object|
+        page_object.has_file_content?(file_content, file_number)
+      end
+
+      match_when_negated do |page_object|
+        page_object.has_no_file_content?(file_content, file_number)
+      end
+    end
+  end
+end

qa/spec/support/matchers/have_issue.rb

+# frozen_string_literal: true
+
+module Matchers
+  module HaveIssue
+    RSpec::Matchers.define :have_issue do |issue|
+      match do |page_object|
+        page_object.has_issue?(issue)
+      end
+
+      match_when_negated do |page_object|
+        page_object.has_no_issue?(issue)
+      end
+    end
+  end
+end

qa/spec/support/matchers/have_job.rb

+# frozen_string_literal: true
+
+module Matchers
+  module HaveJob
+    RSpec::Matchers.define :have_job do |job|
+      match do |page_object|
+        page_object.has_job?(job)
+      end
+
+      match_when_negated do |page_object|
+        page_object.has_no_job?(job)
+      end
+    end
+  end
+end

qa/spec/support/matchers/have_package.rb

+# frozen_string_literal: true
+
+module Matchers
+  module HavePackage
+    RSpec::Matchers.define :have_package do |package|
+      match do |page_object|
+        page_object.has_package?(package)
+      end
+
+      match_when_negated do |page_object|
+        page_object.has_no_package?(package)
+      end
+    end
+  end
+end

qa/spec/support/matchers/have_pipeline.rb

+# frozen_string_literal: true
+
+module Matchers
+  module HavePipeline
+    RSpec::Matchers.define :have_pipeline do
+      match do |page_object|
+        page_object.has_pipeline?
+      end
+
+      match_when_negated do |page_object|
+        page_object.has_no_pipeline?
+      end
+    end
+  end
+end

qa/spec/support/matchers/have_related_issue_item.rb

+# frozen_string_literal: true
+
+module Matchers
+  module HaveRelatedIssueItem
+    RSpec::Matchers.define :have_related_issue_item do
+      match do |page_object|
+        page_object.has_related_issue_item?
+      end
+
+      match_when_negated do |page_object|
+        page_object.has_no_related_issue_item?
+      end
+    end
+  end
+end

qa/spec/support/matchers/have_snippet_description.rb

+# frozen_string_literal: true
+
+module Matchers
+  module HaveSnippetDescription
+    RSpec::Matchers.define :have_snippet_description do |description|
+      match do |page_object|
+        page_object.has_snippet_description?(description)
+      end
+
+      match_when_negated do |page_object|
+        page_object.has_no_snippet_description?
+      end
+    end
+  end
+end