Skip to content

Commit ab327d1

Browse files
more-informative-analysis
Summary: - Log affrimative parameterisations. - Cleaned up dummy keys analysis.
1 parent e839dd6 commit ab327d1

File tree

1 file changed

+8
-9
lines changed

1 file changed

+8
-9
lines changed

public/discovery/static_analyzer.go

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -550,13 +550,6 @@ func (asa *standardMethodPreparedAnalyzer) Analyze() error {
550550
}
551551
addressSpace := addressSpaceFormulator.GetAddressSpace()
552552
asa.method.SetAddressSpace(addressSpace)
553-
// asa.fullHierarchy = &standardAnalyzedHierarchy{
554-
// provider: prov,
555-
// service: providerService,
556-
// resource: resource,
557-
// method: method,
558-
// registryAPI: registryAPI,
559-
// }
560553
dummyParameters, dummyParametersErr := asa.generateDummyRequiredMap(asa.method)
561554
if dummyParametersErr != nil {
562555
return fmt.Errorf("static analysis failed: could not generate dummy parameters for method '%s' on resource '%s': %w", asa.method.GetName(), asa.resource.GetName(), dummyParametersErr)
@@ -578,13 +571,19 @@ func (asa *standardMethodPreparedAnalyzer) Analyze() error {
578571
nil,
579572
logging.GetLogger(),
580573
)
574+
var dummyKeys []string
575+
for k := range dummyParameters {
576+
dummyKeys = append(dummyKeys, k)
577+
}
578+
dummyKeysStr := strings.Join(dummyKeys, ", ")
581579
armoury, armouryErr := preparator.BuildHTTPRequestCtx(anysdk.NewHTTPPreparatorConfig(false))
582580
if armouryErr != nil {
583-
asa.errors = append(asa.errors, fmt.Errorf("warning: could not build HTTP request context for method '%s' on resource '%s': %w", asa.method.GetName(), asa.resource.GetName(), armouryErr))
581+
asa.errors = append(asa.errors, fmt.Errorf("warning: could not build HTTP request context for method '%s' on resource '%s' using dummy parameter keys: '%s': %w", asa.method.GetName(), asa.resource.GetName(), dummyKeysStr, armouryErr))
584582
} else {
585583
if armoury == nil {
586-
asa.errors = append(asa.errors, fmt.Errorf("warning: could not build HTTP request context for method '%s' on resource '%s': got nil armoury", asa.method.GetName(), asa.resource.GetName()))
584+
asa.errors = append(asa.errors, fmt.Errorf("warning: could not build HTTP request context for method '%s' on resource '%s' using dummy parameter keys: '%s': got nil armoury", asa.method.GetName(), asa.resource.GetName(), dummyKeysStr))
587585
}
586+
asa.affirmatives = append(asa.affirmatives, fmt.Sprintf("successfully built HTTP request context for method '%s' on resource '%s' using dummy parameter keys: '%s'", asa.method.GetName(), asa.resource.GetName(), dummyKeysStr))
588587
}
589588
default:
590589
}

0 commit comments

Comments
 (0)