chemlab 0.0.1 → 0.1.0

data/qa/specs/features/ee/browser_ui/3_create/merge_request/add_batch_comments_in_merge_request_spec.rb

@@ -1,80 +0,0 @@
-# frozen_string_literal: true
-
-module QA
-  context 'Create' do
-    describe 'batch comments in merge request' do
-      let(:project) do
-        Resource::Project.fabricate_via_api! do |project|
-          project.name = 'project-with-merge-request'
-        end
-      end
-      let(:merge_request) do
-        Resource::MergeRequest.fabricate_via_api! do |merge_request|
-          merge_request.title = 'This is a merge request'
-          merge_request.description = 'Great feature'
-          merge_request.project = project
-        end
-      end
-
-      it 'user submits a non-diff review' do
-        Flow::Login.sign_in
-
-        merge_request.visit!
-
-        Page::MergeRequest::Show.perform do |show|
-          show.click_discussions_tab
-
-          # You can't start a review immediately, so we have to add a
-          # comment (or start a thread) first
-          show.start_discussion("I'm starting a new discussion")
-          show.type_reply_to_discussion(1, "Could you please check this?")
-          show.start_review
-          show.submit_pending_reviews
-
-          expect(show).to have_content("I'm starting a new discussion")
-          expect(show).to have_content("Could you please check this?")
-          expect(show).to have_content("1 unresolved thread")
-        end
-      end
-
-      it 'user submits a diff review' do
-        Flow::Login.sign_in
-
-        merge_request.visit!
-
-        Page::MergeRequest::Show.perform do |show|
-          show.click_diffs_tab
-          show.add_comment_to_diff("Can you check this line of code?")
-          show.start_review
-          show.submit_pending_reviews
-        end
-
-        # Overwrite the added file to create a system note as required to
-        # trigger the bug described here: https://gitlab.com/gitlab-org/gitlab/issues/32157
-        commit_message = 'Update file'
-        Resource::Repository::Commit.fabricate_via_api! do |commit|
-          commit.project = project
-          commit.commit_message = commit_message
-          commit.branch = merge_request.source_branch
-          commit.update_files(
-            [
-              {
-                file_path: merge_request.file_name,
-                content: "File updated"
-              }
-            ]
-          )
-        end
-        project.wait_for_push(commit_message)
-
-        Page::MergeRequest::Show.perform do |show|
-          show.click_discussions_tab
-          show.resolve_discussion_at_index(0)
-
-          expect(show).to have_content("Can you check this line of code?")
-          expect(show).to have_content("All threads resolved")
-        end
-      end
-    end
-  end
-end

data/qa/specs/features/ee/browser_ui/3_create/merge_request/approval_rules_spec.rb

@@ -1,97 +0,0 @@
-# frozen_string_literal: true
-
-module QA
-  context 'Create' do
-    describe 'Approval rules' do
-      let(:approver1) { Resource::User.fabricate_or_use(Runtime::Env.gitlab_qa_username_1, Runtime::Env.gitlab_qa_password_1) }
-      let(:approver2) { Resource::User.fabricate_or_use(Runtime::Env.gitlab_qa_username_2, Runtime::Env.gitlab_qa_password_2) }
-      let(:project) do
-        Resource::Project.fabricate_via_api! { |project| project.name = "approval-rules" }
-      end
-
-      def login(user = nil)
-        Runtime::Browser.visit(:gitlab, Page::Main::Login)
-        Page::Main::Login.perform { |login| login.sign_in_using_credentials(user: user) }
-      end
-
-      before do
-        project.add_member(approver1)
-        project.group.add_member(approver2)
-
-        Page::Main::Menu.perform(&:sign_out_if_signed_in)
-
-        login
-      end
-
-      it 'allows multiple approval rules with users and groups' do
-        # Create a merge request with 2 rules
-        merge_request = Resource::MergeRequest.fabricate_via_browser_ui! do |resource|
-          resource.title = 'Add a new feature'
-          resource.description = 'Great feature, much approval'
-          resource.project = project
-          resource.approval_rules = [
-            {
-              name: "user",
-              approvals_required: 1,
-              users: [approver1]
-            },
-            {
-              name: "group",
-              approvals_required: 1,
-              groups: [project.group]
-            }
-          ]
-        end
-
-        Page::MergeRequest::Show.perform do |show|
-          expect(show.num_approvals_required).to eq(2)
-          expect(show.approvals_required_from).to include("user", "group")
-        end
-
-        # As approver1, approve the MR
-        Page::Main::Menu.perform(&:sign_out)
-        login(approver1)
-
-        merge_request.visit!
-
-        Page::MergeRequest::Show.perform do |show|
-          show.click_approve
-        end
-
-        # Confirm that an approval was granted but it is not yet fully approved
-        Page::MergeRequest::Show.perform do |show|
-          expect(show).not_to be_approved
-          expect(show.approvals_required_from).to include("group")
-          expect(show.approvals_required_from).not_to include("user")
-        end
-
-        # As approver2, approve the MR
-        Page::Main::Menu.perform(&:sign_out)
-        login(approver2)
-
-        merge_request.visit!
-
-        Page::MergeRequest::Show.perform do |show|
-          show.click_approve
-        end
-
-        # Confirm that the MR is fully approved
-        Page::MergeRequest::Show.perform do |show|
-          expect(show).to be_approved
-        end
-
-        # Merge the MR as the original user
-        Page::Main::Menu.perform(&:sign_out)
-        login
-
-        merge_request.visit!
-
-        Page::MergeRequest::Show.perform do |show|
-          show.merge!
-        end
-
-        expect(page).to have_content('The changes were merged')
-      end
-    end
-  end
-end

data/qa/specs/features/ee/browser_ui/3_create/repository/assign_code_owners_spec.rb

@@ -1,83 +0,0 @@
-# frozen_string_literal: true
-
-module QA
-  context 'Create' do
-    describe 'Codeowners' do
-      # Create one user to be the assigned approver and another user who will not be an approver
-      let(:approver) { Resource::User.fabricate_or_use(Runtime::Env.gitlab_qa_username_1, Runtime::Env.gitlab_qa_password_1) }
-      let(:non_approver) { Resource::User.fabricate_or_use(Runtime::Env.gitlab_qa_username_2, Runtime::Env.gitlab_qa_password_2) }
-
-      let(:project) do
-        Resource::Project.fabricate_via_api! do |project|
-          project.name = "assign-approvers"
-          project.initialize_with_readme = true
-        end
-      end
-      let(:branch_name) { 'protected-branch' }
-
-      before do
-        project.add_member(approver, Resource::Members::AccessLevel::DEVELOPER)
-        project.add_member(non_approver, Resource::Members::AccessLevel::DEVELOPER)
-
-        Flow::Login.sign_in
-
-        project.visit!
-      end
-
-      it 'merge request assigns code owners as approvers' do
-        # Commit CODEOWNERS to master
-        Resource::Repository::Commit.fabricate_via_api! do |commit|
-          commit.project = project
-          commit.commit_message = 'Add CODEOWNERS and test files'
-          commit.add_files(
-            [
-              {
-                file_path: 'CODEOWNERS',
-                content: <<~CONTENT
-                  CODEOWNERS @#{approver.username}
-                CONTENT
-              }
-            ]
-          )
-        end
-
-        # Create a projected branch that requires approval from code owners
-        Resource::ProtectedBranch.fabricate! do |protected_branch|
-          protected_branch.branch_name = branch_name
-          protected_branch.project = project
-        end
-
-        # Push a new CODEOWNERS file
-        Resource::Repository::Push.fabricate! do |push|
-          push.repository_http_uri = project.repository_http_location.uri
-          push.branch_name = branch_name + '-patch'
-          push.file_name = 'CODEOWNERS'
-          push.file_content = <<~CONTENT
-            CODEOWNERS @#{non_approver.username}
-          CONTENT
-        end
-
-        # Create a merge request
-        Resource::MergeRequest.fabricate! do |merge_request|
-          merge_request.project = project
-          merge_request.target_new_branch = false
-          merge_request.source_branch = branch_name + '-patch'
-          merge_request.target_branch = branch_name
-          merge_request.no_preparation = true
-        end.visit!
-
-        # Check that the merge request assigns the original code owner as an
-        # approver (because the current CODEOWNERS file in the master branch
-        # doesn't have the new owner yet)
-        Page::MergeRequest::Show.perform do |show|
-          show.edit!
-          approvers = show.approvers
-
-          expect(approvers.size).to eq(1)
-          expect(approvers).to include(approver.name)
-          expect(approvers).not_to include(non_approver.name)
-        end
-      end
-    end
-  end
-end

data/qa/specs/features/ee/browser_ui/3_create/repository/code_owners_spec.rb

@@ -1,75 +0,0 @@
-# frozen_string_literal: true
-
-module QA
-  context 'Create' do
-    describe 'Codeowners' do
-      let(:files) do
-        [
-          {
-            name: 'file.txt',
-            content: 'foo'
-          },
-          {
-            name: 'README.md',
-            content: 'bar'
-          }
-        ]
-      end
-
-      before do
-        # Add two new users to a project as members
-        Flow::Login.sign_in
-
-        @user = Resource::User.fabricate_or_use(Runtime::Env.gitlab_qa_username_1, Runtime::Env.gitlab_qa_password_1)
-        @user2 = Resource::User.fabricate_or_use(Runtime::Env.gitlab_qa_username_2, Runtime::Env.gitlab_qa_password_2)
-
-        @project = Resource::Project.fabricate_via_api! do |project|
-          project.name = "codeowners"
-        end
-        @project.visit!
-
-        Page::Project::Menu.perform(&:go_to_members_settings)
-        Page::Project::Settings::Members.perform do |members_page|
-          members_page.add_member(@user.username)
-          members_page.add_member(@user2.username)
-        end
-      end
-
-      it 'displays owners specified in CODEOWNERS file' do
-        codeowners_file_content =
-          <<-CONTENT
-            * @#{@user2.username}
-            *.txt @#{@user.username}
-          CONTENT
-        files << {
-          name: 'CODEOWNERS',
-          content: codeowners_file_content
-        }
-
-        # Push CODEOWNERS and test files to the project
-        Resource::Repository::ProjectPush.fabricate! do |push|
-          push.project = @project
-          push.files = files
-          push.commit_message = 'Add CODEOWNERS and test files'
-        end
-        @project.visit!
-
-        # Check the files and code owners
-        Page::Project::Show.perform do |project_page|
-          project_page.click_file 'file.txt'
-        end
-
-        expect(page).to have_content(@user.name)
-        expect(page).not_to have_content(@user2.name)
-
-        @project.visit!
-        Page::Project::Show.perform do |project_page|
-          project_page.click_file 'README.md'
-        end
-
-        expect(page).to have_content(@user2.name)
-        expect(page).not_to have_content(@user.name)
-      end
-    end
-  end
-end

data/qa/specs/features/ee/browser_ui/3_create/repository/file_locking_spec.rb

@@ -1,161 +0,0 @@
-# frozen_string_literal: true
-
-module QA
-  context 'Create' do
-    describe 'File Locking' do
-      before do
-        Flow::Login.sign_in
-
-        @user_one = Resource::User.fabricate_or_use(Runtime::Env.gitlab_qa_username_1, Runtime::Env.gitlab_qa_password_1)
-        @user_two = Resource::User.fabricate_or_use(Runtime::Env.gitlab_qa_username_2, Runtime::Env.gitlab_qa_password_2)
-
-        @project = Resource::Project.fabricate_via_api! do |project|
-          project.name = 'file_locking'
-        end
-
-        Resource::Repository::ProjectPush.fabricate! do |push|
-          push.project = @project
-          push.file_name = 'file'
-          push.file_content = SecureRandom.hex(100000)
-        end
-
-        add_to_project user: @user_one
-        add_to_project user: @user_two
-
-        Resource::ProtectedBranch.unprotect_via_api! do |branch|
-          branch.project = @project
-          branch.branch_name = 'master'
-        end
-      end
-
-      it 'locks a directory and tries to push as a second user' do
-        push branch: 'master', file: 'directory/file', as_user: @user_one
-
-        sign_out_and_sign_in_as user: @user_one
-        go_to_directory
-        click_lock
-
-        expect_error_on_push for_file: 'directory/file', as_user: @user_two
-        expect_no_error_on_push for_file: 'directory/file', as_user: @user_one
-      end
-
-      it 'locks a file and tries to push as a second user' do
-        sign_out_and_sign_in_as user: @user_one
-        go_to_file
-        click_lock
-
-        expect_error_on_push as_user: @user_two
-        expect_no_error_on_push as_user: @user_one
-      end
-
-      it 'checks file locked by other user to be disabled' do
-        go_to_file
-        click_lock
-        sign_out_and_sign_in_as user: @user_one
-        go_to_file
-
-        Page::File::Show.perform do |show|
-          expect(show).to have_lock_button_disabled
-        end
-      end
-
-      it 'creates a merge request and fails to merge', quarantine: { issue: 'https://gitlab.com/gitlab-org/gitlab/issues/40125', type: :bug } do
-        push branch: 'test', as_user: @user_one
-
-        merge_request = Resource::MergeRequest.fabricate_via_api! do |merge_request|
-          merge_request.project = @project
-          merge_request.source_branch = 'test'
-          merge_request.target_branch = 'master'
-          merge_request.no_preparation = true
-        end
-
-        go_to_file
-        click_lock
-        sign_out_and_sign_in_as user: @user_one
-        try_to_merge merge_request: merge_request
-        Page::MergeRequest::Show.perform(&:wait_for_merge_request_error_message)
-        expect(page).to have_text("locked by #{admin_username}")
-      end
-
-      it 'locks a file and unlocks in list' do
-        sign_out_and_sign_in_as user: @user_one
-        go_to_file
-        click_lock
-        @project.visit!
-
-        Page::Project::Menu.perform(&:go_to_repository_locked_files)
-        EE::Page::Project::PathLocks::Index.perform do |list|
-          expect(list).to have_file_with_title 'file'
-          list.unlock_file 'file'
-        end
-
-        expect_no_error_on_push as_user: @user_two
-      end
-
-      def try_to_merge(merge_request:)
-        merge_request.visit!
-        Page::MergeRequest::Show.perform do |show|
-          show.try_to_merge!
-        end
-      end
-
-      def sign_out_and_sign_in_as(user:)
-        Page::Main::Login.perform do |login|
-          login.sign_out_and_sign_in_as user: user
-        end
-      end
-
-      def go_to_file
-        @project.visit!
-        Page::Project::Show.perform do |project_page|
-          project_page.click_file 'file'
-        end
-      end
-
-      def go_to_directory
-        @project.visit!
-        Page::Project::Show.perform do |project_page|
-          project_page.click_file 'directory'
-        end
-      end
-
-      def click_lock
-        Page::File::Show.perform(&:lock)
-      end
-
-      def add_to_project(user:)
-        Resource::ProjectMember.fabricate_via_api! do |member|
-          member.user = user
-          member.project = @project
-          member.access_level = member.level[:developer]
-        end
-      end
-
-      def push(branch: 'master', file: 'file', as_user:)
-        Resource::Repository::ProjectPush.fabricate! do |push|
-          push.project = @project
-          push.new_branch = false unless branch != 'master'
-          push.file_name = file
-          push.file_content = SecureRandom.hex(100000)
-          push.user = as_user
-          push.branch_name = branch
-        end
-      end
-
-      def expect_error_on_push(for_file: 'file', as_user:)
-        expect { push branch: 'master', file: for_file, as_user: as_user }.to raise_error(
-          QA::Git::Repository::RepositoryCommandError)
-      end
-
-      def expect_no_error_on_push(for_file: 'file', as_user:)
-        expect { push branch: 'master', file: for_file, as_user: as_user }.not_to raise_error
-      end
-
-      def admin_username
-        Resource::User.fabricate_via_api! do |user|
-          user.username = Runtime::User.username
-        end.name
-      end
-    end
-  end
-end