feat: Update countNode explanation to be consistent. (#600)

Resolves #599 

- Description: Rename `sourceProperty` to `fieldName` to be consistent with `sumNode` and more in harmony with the post document restructure changes.

query/graphql/planner/count.go

@@ -72,13 +72,14 @@ func (n *countNode) Explain() (map[string]interface{}, error) {
 			explainerMap[filterLabel] = source.Filter.ExternalConditions
 		}
 
-		// Add the source property.
-		explainerMap["sourceProperty"] = source.Field.Name
+		// Add the main field name.
+		explainerMap[fieldNameLabel] = source.Field.Name
+
 		sourceExplanations[i] = explainerMap
 	}
 
 	return map[string]interface{}{
-		"sources": sourceExplanations,
+		sourcesLabel: sourceExplanations,
 	}, nil
 }
 

query/graphql/planner/explain.go

@@ -37,11 +37,14 @@ var (
 )
 
 const (
+	childFieldNameLabel = "childFieldName"
 	collectionIDLabel   = "collectionID"
 	collectionNameLabel = "collectionName"
 	dataLabel           = "data"
+	fieldNameLabel      = "fieldName"
 	filterLabel         = "filter"
 	idsLabel            = "ids"
+	sourcesLabel        = "sources"
 	spansLabel          = "spans"
 )
 

query/graphql/planner/sum.go

@@ -166,20 +166,20 @@ func (n *sumNode) Explain() (map[string]interface{}, error) {
 		}
 
 		// Add the main field name.
-		explainerMap["fieldName"] = source.Field.Name
+		explainerMap[fieldNameLabel] = source.Field.Name
 
 		// Add the child field name if it exists.
 		if source.ChildTarget.HasValue {
-			explainerMap["childFieldName"] = source.ChildTarget.Name
+			explainerMap[childFieldNameLabel] = source.ChildTarget.Name
 		} else {
-			explainerMap["childFieldName"] = nil
+			explainerMap[childFieldNameLabel] = nil
 		}
 
 		sourceExplanations[i] = explainerMap
 	}
 
 	return map[string]interface{}{
-		"sources": sourceExplanations,
+		sourcesLabel: sourceExplanations,
 	}, nil
 }
 

tests/integration/query/explain/with_count_test.go

@@ -91,9 +91,9 @@ func TestExplainQueryOneToManyWithACount(t *testing.T) {
 			},
 		},
 
-		// ----> selectTopNode                    (explainable but no-attributes)
-		//    ----> countNode                (explainable)
-		//        ----> selectNode           (explainable)
+		// ----> selectTopNode                (explainable but no-attributes)
+		//    ----> countNode                 (explainable)
+		//        ----> selectNode            (explainable)
 		//             ----> typeIndexJoin    (explainable)
 		//                 ----> typeJoinMany (non-explainable)
 		//                     ----> scanNode (explainable)
@@ -104,8 +104,8 @@ func TestExplainQueryOneToManyWithACount(t *testing.T) {
 						"countNode": dataMap{
 							"sources": []dataMap{
 								{
-									"filter":         nil,
-									"sourceProperty": "books",
+									"filter":    nil,
+									"fieldName": "books",
 								},
 							},
 							"selectNode": dataMap{