Write unapproval integration test.

spec/features/merge_requests/approvals_spec.rb

@@ -162,8 +162,6 @@ feature 'Merge request approvals', js: true, feature: true do
     end
 
     context 'when group is assigned to a project' do
-      it 'I am able to unapprove' do
-      end
       it 'I am able to approve' do
         create :approver_group, group: group, target: project
 
@@ -175,12 +173,10 @@ feature 'Merge request approvals', js: true, feature: true do
 
         expect(page).to have_content("Approved by")
       end
-    end
 
-    context 'when group is assigned to a merge request' do
-      it 'I am able to unapprove' do
     end
 
+    context 'when group is assigned to a merge request' do
       it 'I am able to approve' do
         create :approver_group, group: group, target: merge_request
 
@@ -194,4 +190,39 @@ feature 'Merge request approvals', js: true, feature: true do
       end
     end
   end
+
+  context 'Unapproving by approvers from groups' do
+    let(:other_user) { create(:user) }
+    let(:merge_request) { create(:merge_request, source_project: project) }
+    let(:group) { create :group }
+
+    before do
+      project.team << [user, :developer]
+
+      group.add_developer(other_user)
+      group.add_developer(user)
+
+      login_as(user)
+
+      create :approver_group, group: group, target: project
+
+      visit namespace_project_merge_request_path(project.namespace, project, merge_request)
+
+      page.within '.mr-state-widget' do
+        click_button 'Approve Merge Request'
+      end
+
+      wait_for_ajax
+    end
+
+    it 'I am able to unapprove' do
+
+      page.within '.mr-state-widget' do
+        click_button 'Remove your approval'
+      end
+
+      expect(page).not_to have_content("Approved by")
+
+    end
+  end
 end