cedar-go: add code coverage enforcement to CI

Also, add some dummy statements to otherwise empty functions so that they're counted for code coverage purposes.

Signed-off-by: Patrick Jakubowski <patrick.jakubowski@strongdm.com>

Author: patjakdev

.github/workflows/build_and_test.yml

@@ -19,10 +19,13 @@ jobs:
         run: go build -v ./...
 
       - name: Test
-        run: go test ./...
+        run: go test -coverprofile=coverage.out ./...
 
       - name: Lint
         uses: golangci/golangci-lint-action@v7
 
       - name: Fuzz
         run: mkdir -p testdata && go test -fuzz=FuzzParse -fuzztime 60s && go test -fuzz=FuzzTokenize -fuzztime 60s
+
+      - name: Coverage check
+        run: go tool cover -func=coverage.out | sed 's/%$//' | awk '{ if ($3 < 100.0) { printf "Insufficient code coverage for %s\n", $0; failed=1 } } END { exit failed }'

.gitignore

@@ -1,3 +1,4 @@
 .idea/
 tmp/
 .DS_Store
+coverage.out

x/exp/ast/node.go

@@ -21,19 +21,19 @@ type StrOpNode struct {
 	Value types.String
 }
 
-func (n StrOpNode) isNode() {}
+func (n StrOpNode) isNode() { _ = 0 } // No-op statement injected for code coverage instrumentation{}
 
 type BinaryNode struct {
 	Left, Right IsNode
 }
 
-func (n BinaryNode) isNode() {}
+func (n BinaryNode) isNode() { _ = 0 } // No-op statement injected for code coverage instrumentation{}
 
 type NodeTypeIfThenElse struct {
 	If, Then, Else IsNode
 }
 
-func (n NodeTypeIfThenElse) isNode() {}
+func (n NodeTypeIfThenElse) isNode() { _ = 0 } // No-op statement injected for code coverage instrumentation
 
 type NodeTypeOr struct{ BinaryNode }
 
@@ -74,14 +74,14 @@ type NodeTypeLike struct {
 	Value types.Pattern
 }
 
-func (n NodeTypeLike) isNode() {}
+func (n NodeTypeLike) isNode() { _ = 0 } // No-op statement injected for code coverage instrumentation{}
 
 type NodeTypeIs struct {
 	Left       IsNode
 	EntityType types.EntityType
 }
 
-func (n NodeTypeIs) isNode() {}
+func (n NodeTypeIs) isNode() { _ = 0 } // No-op statement injected for code coverage instrumentation{}
 
 type NodeTypeIsIn struct {
 	NodeTypeIs
@@ -106,7 +106,7 @@ type UnaryNode struct {
 	Arg IsNode
 }
 
-func (n UnaryNode) isNode() {}
+func (n UnaryNode) isNode() { _ = 0 } // No-op statement injected for code coverage instrumentation{}
 
 type NodeTypeNegate struct{ UnaryNode }
 type NodeTypeNot struct{ UnaryNode }
@@ -120,7 +120,7 @@ type NodeTypeExtensionCall struct {
 	Args []IsNode
 }
 
-func (n NodeTypeExtensionCall) isNode() {}
+func (n NodeTypeExtensionCall) isNode() { _ = 0 } // No-op statement injected for code coverage instrumentation{}
 
 func stripNodes(args []Node) []IsNode {
 	if args == nil {
@@ -170,7 +170,7 @@ type NodeValue struct {
 	Value types.Value
 }
 
-func (n NodeValue) isNode() {}
+func (n NodeValue) isNode() { _ = 0 } // No-op statement injected for code coverage instrumentation{}
 
 type RecordElementNode struct {
 	Key   types.String
@@ -181,19 +181,19 @@ type NodeTypeRecord struct {
 	Elements []RecordElementNode
 }
 
-func (n NodeTypeRecord) isNode() {}
+func (n NodeTypeRecord) isNode() { _ = 0 } // No-op statement injected for code coverage instrumentation{}
 
 type NodeTypeSet struct {
 	Elements []IsNode
 }
 
-func (n NodeTypeSet) isNode() {}
+func (n NodeTypeSet) isNode() { _ = 0 } // No-op statement injected for code coverage instrumentation{}
 
 type NodeTypeVariable struct {
 	Name types.String
 }
 
-func (n NodeTypeVariable) isNode() {}
+func (n NodeTypeVariable) isNode() { _ = 0 } // No-op statement injected for code coverage instrumentation{}
 
 type IsNode interface {
 	isNode()

x/exp/ast/scope.go

@@ -106,19 +106,19 @@ type IsResourceScopeNode interface {
 
 type ScopeNode struct{}
 
-func (n ScopeNode) isScope() {}
+func (n ScopeNode) isScope() { _ = 0 } // No-op statement injected for code coverage instrumentation
 
 type PrincipalScopeNode struct{}
 
-func (n PrincipalScopeNode) isPrincipalScope() {}
+func (n PrincipalScopeNode) isPrincipalScope() { _ = 0 } // No-op statement injected for code coverage instrumentation
 
 type ActionScopeNode struct{}
 
-func (n ActionScopeNode) isActionScope() {}
+func (n ActionScopeNode) isActionScope() { _ = 0 } // No-op statement injected for code coverage instrumentation
 
 type ResourceScopeNode struct{}
 
-func (n ResourceScopeNode) isResourceScope() {}
+func (n ResourceScopeNode) isResourceScope() { _ = 0 } // No-op statement injected for code coverage instrumentation
 
 type ScopeTypeAll struct {
 	ScopeNode