✨ Unit test for all_checks

Addresses #435

Signed-off-by: naveen <172697+naveensrinivasan@users.noreply.github.com>

Author: naveensrinivasan

checks/all_checks.go

@@ -15,11 +15,20 @@
 // Package checks defines all Scorecard checks.
 package checks
 
-import "github.com/ossf/scorecard/v4/checker"
+import (
+	"github.com/ossf/scorecard/v4/checker"
+)
 
 // AllChecks is the list of all security checks that will be run.
 var AllChecks = checker.CheckNameToFnMap{}
 
-func registerCheck(name string, fn checker.CheckFn) {
+func registerCheck(name string, fn checker.CheckFn) error {
+	if name == "" {
+		return errInternalNameCannotBeEmpty
+	}
+	if fn == nil {
+		return errInternalCheckFuncCannotBeNil
+	}
 	AllChecks[name] = fn
+	return nil
 }

checks/all_checks_test.go

@@ -0,0 +1,70 @@
+// Copyright 2020 Security Scorecard Authors
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//      http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+// Package checks defines all Scorecard checks.
+package checks
+
+import (
+	"testing"
+
+	"github.com/ossf/scorecard/v4/checker"
+)
+
+func Test_registerCheck(t *testing.T) {
+	t.Parallel()
+	//nolint
+	type args struct {
+		name string
+		fn   checker.CheckFn
+	}
+	//nolint
+	tests := []struct {
+		name    string
+		args    args
+		wanterr bool
+	}{
+		{
+			name: "registerCheck",
+			args: args{
+				name: "test",
+				fn:   func(x *checker.CheckRequest) checker.CheckResult { return checker.CheckResult{} },
+			},
+			wanterr: false,
+		},
+		{
+			name: "empty func",
+			args: args{
+				name: "test",
+			},
+			wanterr: true,
+		},
+		{
+			name: "empty name",
+			args: args{
+				name: "",
+				fn:   func(x *checker.CheckRequest) checker.CheckResult { return checker.CheckResult{} },
+			},
+			wanterr: true,
+		},
+	}
+	for _, tt := range tests {
+		tt := tt
+		t.Run(tt.name, func(t *testing.T) {
+			t.Parallel()
+			if err := registerCheck(tt.args.name, tt.args.fn); (err != nil) != tt.wanterr {
+				t.Errorf("registerCheck() error = %v, wantErr %v", err, tt.wanterr)
+			}
+		})
+	}
+}

checks/binary_artifact.go

@@ -26,7 +26,10 @@ const CheckBinaryArtifacts string = "Binary-Artifacts"
 
 //nolint
 func init() {
-	registerCheck(CheckBinaryArtifacts, BinaryArtifacts)
+	if err := registerCheck(CheckBinaryArtifacts, BinaryArtifacts); err != nil {
+		// this should never happen
+		panic(err)
+	}
 }
 
 // BinaryArtifacts  will check the repository contains binary artifacts.

checks/branch_protection.go

@@ -28,7 +28,10 @@ const (
 
 //nolint:gochecknoinits
 func init() {
-	registerCheck(CheckBranchProtection, BranchProtection)
+	if err := registerCheck(CheckBranchProtection, BranchProtection); err != nil {
+		// this should never happen
+		panic(err)
+	}
 }
 
 // BranchProtection runs the Branch-Protection check.

checks/ci_tests.go

@@ -31,7 +31,10 @@ const (
 
 //nolint:gochecknoinits
 func init() {
-	registerCheck(CheckCITests, CITests)
+	if err := registerCheck(CheckCITests, CITests); err != nil {
+		// this should never happen
+		panic(err)
+	}
 }
 
 // CITests runs CI-Tests check.

checks/cii_best_practices.go

@@ -32,7 +32,10 @@ const (
 
 //nolint:gochecknoinits
 func init() {
-	registerCheck(CheckCIIBestPractices, CIIBestPractices)
+	if err := registerCheck(CheckCIIBestPractices, CIIBestPractices); err != nil {
+		// this should never happen
+		panic(err)
+	}
 }
 
 // CIIBestPractices runs CII-Best-Practices check.

checks/code_review.go

@@ -27,7 +27,10 @@ const CheckCodeReview = "Code-Review"
 
 //nolint:gochecknoinits
 func init() {
-	registerCheck(CheckCodeReview, DoesCodeReview)
+	if err := registerCheck(CheckCodeReview, DoesCodeReview); err != nil {
+		// this should never happen
+		panic(err)
+	}
 }
 
 // DoesCodeReview attempts to determine whether a project requires review before code gets merged.

checks/contributors.go

@@ -31,7 +31,10 @@ const (
 
 //nolint:gochecknoinits
 func init() {
-	registerCheck(CheckContributors, Contributors)
+	if err := registerCheck(CheckContributors, Contributors); err != nil {
+		// this should never happen
+		panic(err)
+	}
 }
 
 // Contributors run Contributors check.

checks/dangerous_workflow.go

@@ -59,7 +59,10 @@ func containsUntrustedContextPattern(variable string) bool {
 
 //nolint:gochecknoinits
 func init() {
-	registerCheck(CheckDangerousWorkflow, DangerousWorkflow)
+	if err := registerCheck(CheckDangerousWorkflow, DangerousWorkflow); err != nil {
+		// this should never happen
+		panic(err)
+	}
 }
 
 // Holds stateful data to pass thru callbacks.

checks/dependency_update_tool.go

@@ -26,7 +26,10 @@ const CheckDependencyUpdateTool = "Dependency-Update-Tool"
 
 //nolint
 func init() {
-	registerCheck(CheckDependencyUpdateTool, DependencyUpdateTool)
+	if err := registerCheck(CheckDependencyUpdateTool, DependencyUpdateTool); err != nil {
+		// this should never happen
+		panic(err)
+	}
 }
 
 // DependencyUpdateTool checks if the repository uses a dependency update tool.