test(alluxio): strengthen reconcile assertion to err+RequeueAfter checks

When Dataset is absent, ReconcileInternal reaches the 'no dataset bound'
branch which returns RequeueAfterInterval(5s) with nil error. Replace the
weak panic-safety check with precise assertions:
  Expect(err).ToNot(HaveOccurred())
  Expect(result.RequeueAfter).ToNot(BeZero())

Note: Gemini's suggestion used result.Requeue which would be wrong here;
RequeueAfterInterval sets RequeueAfter, not Requeue.

Signed-off-by: Harsh <harshmastic@gmail.com>

Author: hxrshxz

pkg/controllers/v1alpha1/alluxio/alluxio_runtime_controller_test.go

@@ -104,11 +104,11 @@ var _ = Describe("AlluxioRuntimeController", func() {
 					Namespace: testNamespace,
 				},
 			}
-			// Reconcile will proceed past getRuntime (runtime exists) and call
-			// ReconcileInternal. We cannot assert on the internal result because
-			// ReconcileInternal may fail on missing Dataset or other resources in
-			// unit-test conditions. We only assert that the call does not panic.
-			Expect(func() { r.Reconcile(context.TODO(), req) }).ToNot(Panic()) //nolint:errcheck
+			// Reconcile proceeds past getRuntime, builds the engine, then hits the
+			// "no dataset bound" branch which returns RequeueAfter(5s) with no error.
+			result, err := r.Reconcile(context.TODO(), req)
+			Expect(err).ToNot(HaveOccurred())
+			Expect(result.RequeueAfter).ToNot(BeZero())
 		})
 	})
 })