Fix tests for adding environment_url

app/services/ci/retry_build_service.rb

@@ -2,8 +2,8 @@ module Ci
   class RetryBuildService < ::BaseService
     CLONE_ACCESSORS = %i[pipeline project ref tag options commands name
                          allow_failure stage stage_idx trigger_request
-                         yaml_variables when environment coverage_regex
-                         description tag_list].freeze
+                         yaml_variables when environment environment_url
+                         coverage_regex description tag_list].freeze
     def execute(build)
       reprocess!(build).tap do |new_build|

spec/factories/ci/builds.rb

@@ -63,8 +63,10 @@ FactoryGirl.define do
     trait :teardown_environment do
       environment 'staging'
+      environment_url 'http://staging.example.com/$CI_JOB_NAME'
       options environment: { name: 'staging',
-                             action: 'stop' }
+                             action: 'stop',
+                             url: 'http://staging.example.com/$CI_JOB_NAME' }
     end
     trait :allowed_to_fail do

spec/lib/ci/gitlab_ci_yaml_processor_spec.rb

@@ -105,6 +105,7 @@ module Ci
           allow_failure: false,
           when: "on_success",
           environment: nil,
+          environment_url: nil,
           yaml_variables: []
         })
       end
@@ -523,6 +524,7 @@ module Ci
           allow_failure: false,
           when: "on_success",
           environment: nil,
+          environment_url: nil,
           yaml_variables: []
         })
       end
@@ -552,6 +554,7 @@ module Ci
           allow_failure: false,
           when: "on_success",
           environment: nil,
+          environment_url: nil,
           yaml_variables: []
         })
       end
@@ -798,6 +801,7 @@ module Ci
           when: "on_success",
           allow_failure: false,
           environment: nil,
+          environment_url: nil,
           yaml_variables: []
         })
       end
@@ -1001,6 +1005,7 @@ module Ci
             when: "on_success",
             allow_failure: false,
             environment: nil,
+            environment_url: nil,
             yaml_variables: []
           })
         end
@@ -1047,6 +1052,7 @@ module Ci
             when: "on_success",
             allow_failure: false,
             environment: nil,
+            environment_url: nil,
             yaml_variables: []
           })
           expect(subject.second).to eq({
@@ -1060,6 +1066,7 @@ module Ci
             when: "on_success",
             allow_failure: false,
             environment: nil,
+            environment_url: nil,
             yaml_variables: []
           })
         end

spec/lib/gitlab/import_export/safe_model_attributes.yml

@@ -224,6 +224,7 @@ CommitStatus:
 - erased_at
 - artifacts_expire_at
 - environment
+- environment_url
 - artifacts_size
 - when
 - yaml_variables