From c2e3cf0f8bf5e07432d96981c82606a538b99212 Mon Sep 17 00:00:00 2001
From: Jared O'Connell <joconnel@redhat.com>
Date: Fri, 5 Apr 2024 14:38:22 -0400
Subject: [PATCH 1/3] Exposed more info from Ref

---
 schema/ref.go        | 18 ++++++++++++++++--
 schema/scope_test.go |  2 ++
 2 files changed, 18 insertions(+), 2 deletions(-)

diff --git a/schema/ref.go b/schema/ref.go
index 5ca30ec..a97252e 100644
--- a/schema/ref.go
+++ b/schema/ref.go
@@ -7,13 +7,15 @@ import (
 
 // Ref holds the definition of a reference to a scope-wide object. The ref must always be inside a scope,
 // either directly or indirectly. If several scopes are embedded within each other, the Ref references the object
-// in the current scope.
+// in the scope specified. DEFAULT_NAMESPACE for current scope.
 type Ref interface {
 	Object
 
 	ID() string
+	Namespace() string
 	Display() Display
 	GetObject() Object
+	ObjectReady() bool
 }
 
 // NewRefSchema creates a new reference to an object in a wrapping Scope by ID.
@@ -87,10 +89,18 @@ func (r *RefSchema) ReflectedType() reflect.Type {
 	return r.referencedObjectCache.ReflectedType()
 }
 
+func (r *RefSchema) ObjectReady() bool {
+	return r.referencedObjectCache != nil
+}
+
 func (r *RefSchema) ID() string {
 	return r.IDValue
 }
 
+func (r *RefSchema) Namespace() string {
+	return r.ObjectNamespace
+}
+
 func (r *RefSchema) Display() Display {
 	return r.DisplayValue
 }
@@ -104,8 +114,12 @@ func (r *RefSchema) ApplyScope(scope Scope, namespace string) {
 	objects := scope.Objects()
 	referencedObject, ok := objects[r.IDValue]
 	if !ok {
+		availableObjects := ""
+		for objectID, _ := range objects {
+			availableObjects += objectID + "\n"
+		}
 		panic(BadArgumentError{
-			Message: fmt.Sprintf("Referenced object '%s' not found in scope with namespace %q", r.IDValue, namespace),
+			Message: fmt.Sprintf("Referenced object '%s' not found in scope with namespace %q; available:\n%s", r.IDValue, namespace, availableObjects),
 		})
 	}
 	r.referencedObjectCache = referencedObject
diff --git a/schema/scope_test.go b/schema/scope_test.go
index e0f840a..399cafc 100644
--- a/schema/scope_test.go
+++ b/schema/scope_test.go
@@ -463,7 +463,9 @@ func TestApplyingExternalNamespace(t *testing.T) {
 			err = testData.ref.ValidateReferences()
 			assert.Error(t, err)
 			assert.Contains(t, err.Error(), "missing its link")
+			assert.Equals(t, testData.ref.ObjectReady(), false)
 			testData.scope.ApplyScope(externalScope, "test-namespace")
+			assert.Equals(t, testData.ref.ObjectReady(), true)
 			// Now it's applied, so the error should be resolved.
 			// Outermost
 			assert.NoError(t, testData.scope.ValidateReferences())

From 23eebaa238052ac9a42184945c172848500b3887 Mon Sep 17 00:00:00 2001
From: Jared O'Connell <joconnel@redhat.com>
Date: Fri, 5 Apr 2024 14:51:20 -0400
Subject: [PATCH 2/3] Fix linting error

---
 schema/ref.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/schema/ref.go b/schema/ref.go
index a97252e..24fd616 100644
--- a/schema/ref.go
+++ b/schema/ref.go
@@ -115,7 +115,7 @@ func (r *RefSchema) ApplyScope(scope Scope, namespace string) {
 	referencedObject, ok := objects[r.IDValue]
 	if !ok {
 		availableObjects := ""
-		for objectID, _ := range objects {
+		for objectID := range objects {
 			availableObjects += objectID + "\n"
 		}
 		panic(BadArgumentError{

From 6d5fa011eab12f2fbc96c42dbf1c100ae109e7c1 Mon Sep 17 00:00:00 2001
From: Jared O'Connell <joconnel@redhat.com>
Date: Fri, 5 Apr 2024 17:22:43 -0400
Subject: [PATCH 3/3] Added line to test Namespace method

---
 schema/scope_test.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/schema/scope_test.go b/schema/scope_test.go
index 399cafc..7f8eebe 100644
--- a/schema/scope_test.go
+++ b/schema/scope_test.go
@@ -321,7 +321,7 @@ func TestApplyingExternalNamespace(t *testing.T) {
 	// The applied scope must be passed down to all of those types, validating
 	// that the scope gets applied down and that errors are propagated up.
 	refRefSchema := schema.NewNamespacedRefSchema("scopeTestObjectB", "test-namespace", nil)
-
+	assert.Equals(t, refRefSchema.Namespace(), "test-namespace")
 	refProperty := schema.NewPropertySchema(
 		refRefSchema,
 		nil,