Update the state for each actual LRP when no state is given (#3953)

Author: gururajsh

lib/cloud_controller/diego/reporters/instances_stats_reporter.rb

@@ -48,9 +48,9 @@ def actual_lrp_info(process, stats=nil, quota_stats=nil, log_cache_errors=nil, i
         bbs_instances_client.lrp_instances(process).each do |actual_lrp|
           next unless actual_lrp.actual_lrp_key.index < process.instances
 
-          state ||= LrpStateTranslator.translate_lrp_state(actual_lrp)
+          lrp_state = state || LrpStateTranslator.translate_lrp_state(actual_lrp)
 
-          info = build_info(state, actual_lrp, process, stats, quota_stats, log_cache_errors)
+          info = build_info(lrp_state, actual_lrp, process, stats, quota_stats, log_cache_errors)
           info[:isolation_segment] = isolation_segment unless isolation_segment.nil?
           result[actual_lrp.actual_lrp_key.index] = info
         end

spec/unit/lib/cloud_controller/diego/reporters/instances_stats_reporter_spec.rb

@@ -120,6 +120,34 @@ def make_actual_lrp(instance_guid:, index:, state:, error:, since:)
           expect(instances_reporter.stats_for_app(process)).to eq([expected_stats_response, []])
         end
 
+        context 'when there are multiple lrps with different states' do
+          let(:bbs_actual_lrps_response) { [actual_lrp_1, actual_lrp_2] }
+          let(:actual_lrp_1) do
+            make_actual_lrp(
+              instance_guid: 'instance-a', index: 0, state: ::Diego::ActualLRPState::RUNNING, error: 'some-details', since: two_days_ago_since_epoch_ns
+            ).tap do |actual_lrp|
+              actual_lrp.actual_lrp_net_info = lrp_1_net_info
+            end
+          end
+          let(:actual_lrp_2) do
+            make_actual_lrp(
+              instance_guid: 'instance-b', index: 1, state: ::Diego::ActualLRPState::CLAIMED, error: 'some-details', since: two_days_ago_since_epoch_ns
+            ).tap do |actual_lrp|
+              actual_lrp.actual_lrp_net_info = lrp_1_net_info
+            end
+          end
+
+          before do
+            allow(bbs_instances_client).to receive_messages(lrp_instances: bbs_actual_lrps_response, desired_lrp_instance: bbs_desired_lrp_response)
+          end
+
+          it 'shows all correct state for all instances' do
+            result, = instances_reporter.stats_for_app(process)
+            expect(result[0][:state]).to eq('RUNNING')
+            expect(result[1][:state]).to eq('STARTING')
+          end
+        end
+
         context 'when a NoRunningInstances error is thrown for desired_lrp and it exists an actual_lrp' do
           let(:error) { CloudController::Errors::NoRunningInstances.new('No running instances ruh roh') }
           let(:expected_stopping_response) do