Merge pull request #87 from DataDog/anmarchenko/do_not_validate_envir…

…onment_when_ci_disabled

do not collect environment tags when CI is not enabled

lib/datadog/ci/test_visibility/recorder.rb

@@ -31,7 +31,7 @@ def initialize(enabled: true, test_suite_level_visibility_enabled: false)
           @enabled = enabled
           @test_suite_level_visibility_enabled = enabled && test_suite_level_visibility_enabled
 
-          @environment_tags = Ext::Environment.tags(ENV).freeze
+          @environment_tags = @enabled ? Ext::Environment.tags(ENV).freeze : {}
           @local_context = Context::Local.new
           @global_context = Context::Global.new
         end

spec/datadog/ci/test_visibility/recorder_spec.rb

@@ -50,6 +50,21 @@
       let(:ci_enabled) { false }
     end
 
+    describe "#initialize" do
+      subject do
+        described_class.new(
+          enabled: ci_enabled,
+          test_suite_level_visibility_enabled: experimental_test_suite_level_visibility_enabled
+        )
+      end
+
+      it "doesn't collect environment tags" do
+        expect(Datadog::CI::Ext::Environment).not_to receive(:tags)
+
+        subject
+      end
+    end
+
     describe "#trace_test" do
       context "when given a block" do
         let(:spy_under_test) { spy("spy") }