test: fix readyToUse flag assertions in static infra provider tests

Change the assertions to work with the new, changed behavior of the `infraMachineStatus.readyToUse` flag.

Signed-off-by: Utku Ozdemir <[email protected]>

Author: utkuozdemir

client/api/omni/specs/infra.pb.go

@@ -63,6 +63,8 @@ message InfraMachineStatusSpec {
   }
 
   MachinePowerState power_state = 1;
+
+  // ReadyToUse is set to true when the machine is not dirty, i.e., there is no pending wipe operation.
   bool ready_to_use = 2;
   string last_reboot_id = 3;
   google.protobuf.Timestamp last_reboot_timestamp = 4;

client/api/omni/specs/infra.proto

@@ -299,10 +299,10 @@ func AssertInfraMachinesAreAllocated(testCtx context.Context, omniState state.St
 				assertion.Equal(extensions, res.TypedSpec().Value.Extensions)
 			})
 
-			// The machine is allocated, so the ReadyToUse field is set to false
+			// The machine is allocated, so it will be powered on and be ready to use
 			rtestutils.AssertResource[*infra.MachineStatus](ctx, t, omniState, id, func(res *infra.MachineStatus, assertion *assert.Assertions) {
 				assertion.Equal(specs.InfraMachineStatusSpec_POWER_STATE_ON, res.TypedSpec().Value.PowerState)
-				assertion.False(res.TypedSpec().Value.ReadyToUse)
+				assertion.True(res.TypedSpec().Value.ReadyToUse)
 			})
 
 			// Omni receives a SequenceEvent from the SideroLink event sink and sets the Installed field to true
@@ -338,15 +338,15 @@ func AssertAllInfraMachinesAreUnallocated(testCtx context.Context, omniState sta
 				}
 			})
 
-			// machine is unallocated, so the ReadyToUse field will be set to true
-			rtestutils.AssertResource[*infra.MachineStatus](ctx, t, omniState, id, func(res *infra.MachineStatus, assertion *assert.Assertions) {
-				assertion.True(res.TypedSpec().Value.ReadyToUse)
-			})
-
 			// provider wipes the machine and sets the Installed field to false
 			rtestutils.AssertResource[*infra.MachineState](ctx, t, omniState, id, func(res *infra.MachineState, assertion *assert.Assertions) {
 				assertion.False(res.TypedSpec().Value.Installed)
 			})
+
+			// after the machine is wiped, ReadyToUse field will be set to true
+			rtestutils.AssertResource[*infra.MachineStatus](ctx, t, omniState, id, func(res *infra.MachineStatus, assertion *assert.Assertions) {
+				assertion.True(res.TypedSpec().Value.ReadyToUse)
+			})
 		}
 	}
 }

cmd/integration-test/pkg/tests/infra.go

@@ -1 +1 @@
-v0.46.0-beta.0
+v0.46.0-beta.0