From b4ed187f88b78e2c40e15d3b3b68ddb0f564884a Mon Sep 17 00:00:00 2001 From: Naveen <172697+naveensrinivasan@users.noreply.github.com> Date: Mon, 24 Jul 2023 15:45:59 -0500 Subject: [PATCH] :seedling: Unit tests OSSFuzz client (#3301) * :seedling: 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> --- clients/ossfuzz/client_test.go | 186 +++++++++++++++++++++++++++++++++ 1 file changed, 186 insertions(+) diff --git a/clients/ossfuzz/client_test.go b/clients/ossfuzz/client_test.go index d7c18720afe..7a964645833 100644 --- 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) + } + } +}