🌱 Unit tests OSSFuzz client (#3301)

* 🌱 Unit tests OSSFuzz client - Included tests for IsArchived, LocalPath, ListFiles, GetFileContent, GetBranch, GetDefaultBranch, GetOrgRepoClient, GetDefaultBranchName, ListCommits, ListIssues, ListReleases, ListContributors, ListSuccessfulWorkflowRuns, ListCheckRunsForRef, ListStatuses, ListWebhooks, SearchCommits, Close, ListProgrammingLanguages, Signed-off-by: naveensrinivasan <172697+naveensrinivasan@users.noreply.github.com> * Improve OSSFuzz client tests [clients/ossfuzz/client_test.go] - Add a test for the `GetCreatedAt` method - Fix the `URI` method to return the correct value Signed-off-by: naveensrinivasan <172697+naveensrinivasan@users.noreply.github.com> --------- Signed-off-by: naveensrinivasan <172697+naveensrinivasan@users.noreply.github.com>
ossf · Jul 24, 2023 · b4ed187 · b4ed187
1 parent 41a18ff
commit b4ed187
Showing 1 changed file with 186 additions and 0 deletions.
diff --git a/clients/ossfuzz/client_test.go b/clients/ossfuzz/client_test.go
@@ -15,6 +15,8 @@
 package ossfuzz
 
 import (
+	"context"
+	"errors"
 	"fmt"
 	"net/http"
 	"net/http/httptest"
@@ -180,3 +182,187 @@ func setupServer(t *testing.T) string {
 	t.Cleanup(server.Close)
 	return server.URL
 }
+
+func TestAllClientMethods(t *testing.T) {
+	c := CreateOSSFuzzClient("testURL")
+
+	// Test InitRepo
+	{
+		err := c.InitRepo(nil, "", 0)
+		if !errors.Is(err, clients.ErrUnsupportedFeature) {
+			t.Errorf("InitRepo: Expected %v, but got %v", clients.ErrUnsupportedFeature, err)
+		}
+	}
+
+	// Test IsArchived
+	{
+		_, err := c.IsArchived()
+		if !errors.Is(err, clients.ErrUnsupportedFeature) {
+			t.Errorf("IsArchived: Expected %v, but got %v", clients.ErrUnsupportedFeature, err)
+		}
+	}
+
+	// Test LocalPath
+	{
+		_, err := c.LocalPath()
+		if !errors.Is(err, clients.ErrUnsupportedFeature) {
+			t.Errorf("LocalPath: Expected %v, but got %v", clients.ErrUnsupportedFeature, err)
+		}
+	}
+
+	// Test ListFiles
+	{
+		_, err := c.ListFiles(nil)
+		if !errors.Is(err, clients.ErrUnsupportedFeature) {
+			t.Errorf("ListFiles: Expected %v, but got %v", clients.ErrUnsupportedFeature, err)
+		}
+	}
+
+	// Test GetFileContent
+	{
+		_, err := c.GetFileContent("")
+		if !errors.Is(err, clients.ErrUnsupportedFeature) {
+			t.Errorf("GetFileContent: Expected %v, but got %v", clients.ErrUnsupportedFeature, err)
+		}
+	}
+
+	// Test GetBranch
+	{
+		_, err := c.GetBranch("")
+		if !errors.Is(err, clients.ErrUnsupportedFeature) {
+			t.Errorf("GetBranch: Expected %v, but got %v", clients.ErrUnsupportedFeature, err)
+		}
+	}
+
+	// Test GetDefaultBranch
+	{
+		_, err := c.GetDefaultBranch()
+		if !errors.Is(err, clients.ErrUnsupportedFeature) {
+			t.Errorf("GetDefaultBranch: Expected %v, but got %v", clients.ErrUnsupportedFeature, err)
+		}
+	}
+
+	// Test GetOrgRepoClient
+	{
+		_, err := c.GetOrgRepoClient(context.Background())
+		if !errors.Is(err, clients.ErrUnsupportedFeature) {
+			t.Errorf("GetOrgRepoClient: Expected %v, but got %v", clients.ErrUnsupportedFeature, err)
+		}
+	}
+
+	// Test GetDefaultBranchName
+	{
+		_, err := c.GetDefaultBranchName()
+		if !errors.Is(err, clients.ErrUnsupportedFeature) {
+			t.Errorf("GetDefaultBranchName: Expected %v, but got %v", clients.ErrUnsupportedFeature, err)
+		}
+	}
+
+	// Test ListCommits
+	{
+		_, err := c.ListCommits()
+		if !errors.Is(err, clients.ErrUnsupportedFeature) {
+			t.Errorf("ListCommits: Expected %v, but got %v", clients.ErrUnsupportedFeature, err)
+		}
+	}
+
+	// Test ListIssues
+	{
+		_, err := c.ListIssues()
+		if !errors.Is(err, clients.ErrUnsupportedFeature) {
+			t.Errorf("ListIssues: Expected %v, but got %v", clients.ErrUnsupportedFeature, err)
+		}
+	}
+
+	// Test ListReleases
+	{
+		_, err := c.ListReleases()
+		if !errors.Is(err, clients.ErrUnsupportedFeature) {
+			t.Errorf("ListReleases: Expected %v, but got %v", clients.ErrUnsupportedFeature, err)
+		}
+	}
+
+	// Test ListContributors
+	{
+		_, err := c.ListContributors()
+		if !errors.Is(err, clients.ErrUnsupportedFeature) {
+			t.Errorf("ListContributors: Expected %v, but got %v", clients.ErrUnsupportedFeature, err)
+		}
+	}
+
+	// Test ListSuccessfulWorkflowRuns
+	{
+		_, err := c.ListSuccessfulWorkflowRuns("")
+		if !errors.Is(err, clients.ErrUnsupportedFeature) {
+			t.Errorf("ListSuccessfulWorkflowRuns: Expected %v, but got %v", clients.ErrUnsupportedFeature, err)
+		}
+	}
+
+	// Test ListCheckRunsForRef
+	{
+		_, err := c.ListCheckRunsForRef("")
+		if !errors.Is(err, clients.ErrUnsupportedFeature) {
+			t.Errorf("ListCheckRunsForRef: Expected %v, but got %v", clients.ErrUnsupportedFeature, err)
+		}
+	}
+
+	// Test ListStatuses
+	{
+		_, err := c.ListStatuses("")
+		if !errors.Is(err, clients.ErrUnsupportedFeature) {
+			t.Errorf("ListStatuses: Expected %v, but got %v", clients.ErrUnsupportedFeature, err)
+		}
+	}
+
+	// Test ListWebhooks
+	{
+		_, err := c.ListWebhooks()
+		if !errors.Is(err, clients.ErrUnsupportedFeature) {
+			t.Errorf("ListWebhooks: Expected %v, but got %v", clients.ErrUnsupportedFeature, err)
+		}
+	}
+
+	// Test SearchCommits
+	{
+		_, err := c.SearchCommits(clients.SearchCommitsOptions{})
+		if !errors.Is(err, clients.ErrUnsupportedFeature) {
+			t.Errorf("SearchCommits: Expected %v, but got %v", clients.ErrUnsupportedFeature, err)
+		}
+	}
+
+	// Test Close
+	{
+		err := c.Close()
+		if err != nil {
+			t.Errorf("Close: Expected no error, but got %v", err)
+		}
+	}
+
+	// Test ListProgrammingLanguages
+	{
+		_, err := c.ListProgrammingLanguages()
+		if !errors.Is(err, clients.ErrUnsupportedFeature) {
+			t.Errorf("ListProgrammingLanguages: Expected %v, but got %v", clients.ErrUnsupportedFeature, err)
+		}
+	}
+
+	// Test ListLicenses
+	{
+		_, err := c.ListLicenses()
+		if !errors.Is(err, clients.ErrUnsupportedFeature) {
+			t.Errorf("ListLicenses: Expected %v, but got %v", clients.ErrUnsupportedFeature, err)
+		}
+	}
+	{
+		_, err := c.GetCreatedAt()
+		if !errors.Is(err, clients.ErrUnsupportedFeature) {
+			t.Errorf("GetCreatedAt: Expected %v, but got %v", clients.ErrUnsupportedFeature, err)
+		}
+	}
+	{
+		uri := c.URI()
+		if uri != "testURL" {
+			t.Errorf("URI: Expected %v, but got %v", "testURL", uri)
+		}
+	}
+}