Circular ref check was not happening before model build.

This was causing a stack overflow on certain specs. Provides resolution for: pb33f/libopenapi#229

Signed-off-by: quobix <dave@quobix.com>

motor/rule_applicator.go

@@ -106,10 +106,10 @@ func ApplyRulesToRuleSet(execution *RuleSetExecution) *RuleSetExecutionResult {
 
 	// avoid building the index, we don't need it to run yet.
 	indexConfig.AvoidBuildIndex = true
-	indexConfig.AvoidCircularReferenceCheck = true
+	//indexConfig.AvoidCircularReferenceCheck = true
 
 	docConfig := datamodel.NewDocumentConfiguration()
-	docConfig.SkipCircularReferenceCheck = true
+	//docConfig.SkipCircularReferenceCheck = true
 
 	// add new pretty logger.
 	if execution.Logger == nil {
@@ -337,7 +337,7 @@ func ApplyRulesToRuleSet(execution *RuleSetExecution) *RuleSetExecutionResult {
 			specUnresolved = rolodexUnresolved.GetRootIndex().GetRootNode()
 
 			if rolodexResolved != nil && rolodexResolved.GetRootIndex() != nil {
-				resolvingErrors = rolodexResolved.GetRootIndex().GetResolver().GetResolvingErrors()
+				//resolvingErrors = rolodexResolved.GetRootIndex().GetResolver().GetResolvingErrors()
 				circularReferences = rolodexResolved.GetRootIndex().GetResolver().GetCircularReferences()
 			}
 

motor/rule_applicator_test.go

@@ -1882,7 +1882,7 @@ components:
 	assert.Len(t, results.Errors, 0)
 
 	assert.NotNil(t, results)
-	assert.Equal(t, "infinite circular reference detected: #/components/schemas/one: one -> two -> one [14:7]",
+	assert.Equal(t, "infinite circular reference detected: one: one -> two -> one [14:7]",
 		results.Results[0].Message)
 	assert.Equal(t, "resolving-references", results.Results[0].RuleId)
 }