update runner limits spec to use group membership

spec/factories/groups.rb

 FactoryGirl.define do
-  factory :group do
+  factory :group, class: Group, parent: :namespace do
     sequence(:name) { |n| "group#{n}" }
     path { name.downcase.gsub(/\s/, '_') }
     type 'Group'

spec/features/ci_shared_runner_limits_spec.rb

@@ -2,8 +2,8 @@ require 'spec_helper'
 
 feature 'CI shared runner limits', feature: true do
   let(:user) { create(:user) }
-  let(:project) { create(:project, :public, namespace: namespace, shared_runners_enabled: true) }
-  let(:namespace) { create(:namespace) }
+  let!(:project) { create(:project, :public, namespace: group, shared_runners_enabled: true) }
+  let(:group) { create(:group) }
 
   before do
     login_as(user)
@@ -11,7 +11,7 @@ feature 'CI shared runner limits', feature: true do
 
   context 'when project member' do
     before do
-      project.team << [user, :developer]
+      group.add_developer(user)
     end
 
     context 'without limit' do
@@ -23,16 +23,16 @@ feature 'CI shared runner limits', feature: true do
 
     context 'when limit is defined' do
       context 'when limit is exceeded' do
-        let(:namespace) { create(:namespace, :with_used_build_minutes_limit) }
+        let(:group) { create(:group, :with_used_build_minutes_limit) }
 
         scenario 'it displays a warning message on project homepage' do
           visit namespace_project_path(project.namespace, project)
-          expect_quota_exceeded_alert("#{namespace.name} has exceeded their build minutes quota.")
+          expect_quota_exceeded_alert("#{group.name} has exceeded their build minutes quota.")
         end
       end
 
       context 'when limit not yet exceeded' do
-        let(:namespace) { create(:namespace, :with_not_used_build_minutes_limit) }
+        let(:group) { create(:group, :with_not_used_build_minutes_limit) }
 
         scenario 'it does not display a warning message on project homepage' do
           visit namespace_project_path(project.namespace, project)
@@ -41,7 +41,7 @@ feature 'CI shared runner limits', feature: true do
       end
 
       context 'when minutes are not yet set' do
-        let(:namespace) { create(:namespace, :with_build_minutes_limit) }
+        let(:group) { create(:group, :with_build_minutes_limit) }
 
         scenario 'it does not display a warning message on project homepage' do
           visit namespace_project_path(project.namespace, project)
@@ -52,7 +52,7 @@ feature 'CI shared runner limits', feature: true do
   end
 
   context 'when not a project member' do
-    let(:namespace) { create(:namespace, :with_used_build_minutes_limit) }
+    let(:group) { create(:group, :with_used_build_minutes_limit) }
 
     context 'when limit is defined and limit is exceeded' do
       scenario 'it does not display a warning message on project homepage' do