fix: set emitter and logger to nil

Since unit tests don't have a real gRPC endpoint, any leftover AppLogEmitter.emitter from a previous spec will raise a GRPC connection error when AppLogEmitter.emit is called, which gets rescued and logged — interfering with specs that assert on logger.error. Resetting AppLogEmitter.emitter and AppLogEmitter.logger to nil after each spec prevents this leak.

Author: serdarozerr

spec/unit/lib/cloud_controller/runner_spec.rb

@@ -37,6 +37,11 @@ module VCAP::CloudController
       allow(Puma::Server).to receive(:new).and_return(puma_server_double)
     end
 
+    after do
+      VCAP::AppLogEmitter.emitter = nil
+      VCAP::AppLogEmitter.logger = nil
+    end
+
     subject do
       Runner.new(argv + ['-c', config_file.path])
     end

spec/unit/lib/delayed_job/delayed_worker_spec.rb

@@ -17,6 +17,11 @@
     allow(threaded_worker).to receive(:name=).with(anything)
   end
 
+  after do
+    VCAP::AppLogEmitter.emitter = nil
+    VCAP::AppLogEmitter.logger = nil
+  end
+
   describe '#initialize' do
     it 'sets the correct default queue options' do
       worker_instance = CloudController::DelayedWorker.new(options)