(fix) cli get change waits on 'in progress' and 'unspecified' (#1042)

re #1041

![image](https://github.com/user-attachments/assets/639b239b-9efe-4545-b070-649f07b800e2)
with the changes in where we do processing for a change. when we first
ask the status of the risk analysis, each step will be 0, ie unknown. We
should wait on this status of ChangeAnalysisStatus_STATUS_UNSPECIFIED
similar to ChangeAnalysisStatus_STATUS_INPROGRESS.

longer term GetChangeRisks will be removed, and we will ask the
change_timeline_v2 what is happening and this logic will be different.

GitOrigin-RevId: 295095de4be0675e8abfaab9006797f2e6163ca4

Author: tphoney

cmd/changes_get_change.go

@@ -102,7 +102,14 @@ fetch:
 			}
 		}
 
-		if riskRes.Msg.GetChangeRiskMetadata().GetChangeAnalysisStatus().GetStatus() == sdp.ChangeAnalysisStatus_STATUS_INPROGRESS {
+		currentStatus := riskRes.Msg.GetChangeRiskMetadata().GetChangeAnalysisStatus().GetStatus()
+		// if everything is pending or in progress we need to wait.
+		// - ChangeAnalysisStatus_STATUS_UNSPECIFIED: When all are pending
+		// - ChangeAnalysisStatus_STATUS_SKIPPED: When all are skipped
+		// - ChangeAnalysisStatus_STATUS_INPROGRESS: When >1 is in progress
+		// - ChangeAnalysisStatus_STATUS_ERROR: When any of the steps are errored
+		// - ChangeAnalysisStatus_STATUS_DONE: Any other situation
+		if currentStatus == sdp.ChangeAnalysisStatus_STATUS_INPROGRESS || currentStatus == sdp.ChangeAnalysisStatus_STATUS_UNSPECIFIED {
 			// Extract the currently running milestone if you can
 			milestones := riskRes.Msg.GetChangeRiskMetadata().GetChangeAnalysisStatus().GetProgressMilestones()
 			var currentMilestone string