Prefer using `project.add_developer` in specs

2 files changed, 5 insertions, 5 deletions

diff --git a/spec/models/ci/build_spec.rb b/spec/models/ci/build_spec.rb
index 9d5849eeec1..2dfca8bcfce 100644
--- a/spec/models/ci/build_spec.rb
+++ b/spec/models/ci/build_spec.rb
@@ -208,7 +208,7 @@ describe Ci::Build, :models do
     end
 
     it 'expects to have retried builds instead the original ones' do
-      project.team << [user, :developer]
+      project.add_developer(user)
 
       retried_rspec = Ci::Build.retry(rspec_test, user)
 
@@ -817,7 +817,7 @@ describe Ci::Build, :models do
     subject { build.other_actions }
 
     before do
-      project.team << [user, :developer]
+      project.add_developer(user)
     end
 
     it 'returns other actions' do
@@ -865,7 +865,7 @@ describe Ci::Build, :models do
     let(:build) { create(:ci_build, :manual, pipeline: pipeline) }
 
     before do
-      project.team << [user, :developer]
+      project.add_developer(user)
     end
 
     context 'when build is manual' do
diff --git a/spec/models/ci/pipeline_spec.rb b/spec/models/ci/pipeline_spec.rb
index 45c710b2381..10c2bfbb400 100644
--- a/spec/models/ci/pipeline_spec.rb
+++ b/spec/models/ci/pipeline_spec.rb
@@ -563,7 +563,7 @@ describe Ci::Pipeline, models: true do
     context 'on failure and build retry' do
       before do
         build.drop
-        project.team << [user, :developer]
+        project.add_developer(user)
 
         Ci::Build.retry(build, user)
       end
@@ -776,7 +776,7 @@ describe Ci::Pipeline, models: true do
     let(:latest_status) { pipeline.statuses.latest.pluck(:status) }
 
     before do
-      project.team << [user, :developer]
+      project.add_developer(user)
     end
 
     context 'when there is a failed build and failed external status' do