Add the FIXME reminder about edge comparison

RyanSiu1995 · RyanSiu1995 · commit 26d3a2669b67 · 2020-05-30T22:13:13.000+08:00
diff --git a/internal/utils/cloudbuild_test.go b/internal/utils/cloudbuild_test.go
@@ -75,6 +75,7 @@ func isExpectedGraph(t *testing.T, expected *result, actual *cgraph.Graph) {
 		assert.NotEmptyf(t, n, "Expected Node %s but not found in generated graph", node)
 	}
 	assert.Equalf(t, len(expected.Edges), actual.NumberEdges(), "Should have the same number of edges.")
+	// FIXME This may not be too good to compare the edge. We may need a better way to compare them programmatically
 	var buf bytes.Buffer
 	g := graphviz.New()
 	g.Render(actual, "dot", &buf)

Original file line number	Diff line number	Diff line change
`@@ -75,6 +75,7 @@ func isExpectedGraph(t testing.T, expected result, actual *cgraph.Graph) {`
`75`	`75`	`assert.NotEmptyf(t, n, "Expected Node %s but not found in generated graph", node)`
`76`	`76`	`}`
`77`	`77`	`assert.Equalf(t, len(expected.Edges), actual.NumberEdges(), "Should have the same number of edges.")`
	`78`	`+ // FIXME This may not be too good to compare the edge. We may need a better way to compare them programmatically`
`78`	`79`	`var buf bytes.Buffer`
`79`	`80`	`g := graphviz.New()`
`80`	`81`	`g.Render(actual, "dot", &buf)`