Add tests

Author: kyoshidajp

cmd/diagnose_test.go

@@ -1,6 +1,7 @@
 package cmd
 
 import (
+	"errors"
 	"os"
 	"testing"
 
@@ -73,6 +74,78 @@ func TestDiagnose(t *testing.T) {
 	})
 }
 
+func TestDiagnosis_ErrorMessage(t *testing.T) {
+	tests := []struct {
+		name      string
+		diagnosis Diagnosis
+	}{
+		{
+			name: "has Error",
+			diagnosis: Diagnosis{
+				Error: errors.New("unknown error"),
+			},
+		},
+		{
+			name: "has no Error",
+			diagnosis: Diagnosis{
+				Error: nil,
+			},
+		},
+	}
+	expects := []struct {
+		name    string
+		message string
+	}{
+		{
+			name:    "has Error",
+			message: "unknown error",
+		},
+		{
+			name:    "has no Error",
+			message: "",
+		},
+	}
+
+	for i, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			actual := tt.diagnosis.ErrorMessage()
+			expect := expects[i].message
+			assert.Equal(t, expect, actual)
+		})
+	}
+}
+
+func TestOptions_Ignores(t *testing.T) {
+	tests := []struct {
+		name    string
+		options Options
+	}{
+		{
+			name: "ignores",
+			options: Options{
+				ignores: "package1 package2 package3",
+			},
+		},
+	}
+	expects := []struct {
+		name    string
+		ignores []string
+	}{
+		{
+			name:    "ignores",
+			ignores: []string{"package1", "package2", "package3"},
+		},
+	}
+
+	for i, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			actual := tt.options.Ignores()
+			expect := expects[i].ignores
+			assert.Equal(t, expect, actual)
+		})
+	}
+}
+
 func TestDoctors_PackageManagers(t *testing.T) {
 	tests := []struct {
 		name    string

cmd/python/pypi_test.go

@@ -13,18 +13,26 @@ func TestPyPi_fetchURLFromRegistry(t *testing.T) {
 		lib_name string
 	}{
 		{
-			name:     "source_code_uri exists",
+			name:     "info.project_urls.Source_Code exists",
 			lib_name: "pip",
 		},
+		{
+			name:     "not found",
+			lib_name: "not-found-xxxx",
+		},
 	}
 	expects := []struct {
 		name string
 		url  string
 	}{
 		{
-			name: "source_code_uri exists",
+			name: "info.project_urls.Source_Code exists",
 			url:  "https://github.com/pypa/pip",
 		},
+		{
+			name: "not found",
+			url:  "",
+		},
 	}
 
 	for i, tt := range tests {

cmd/rust/crates_test.go

@@ -0,0 +1,49 @@
+package rust
+
+import (
+	"net/http"
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+)
+
+func TestCrate_fetchURLFromRegistry(t *testing.T) {
+	tests := []struct {
+		name  string
+		crate Crate
+	}{
+		{
+			name: "has Crate.Repository",
+			crate: Crate{
+				name: "libc",
+			},
+		},
+		{
+			name: "not found",
+			crate: Crate{
+				name: "not-found-crate",
+			},
+		},
+	}
+	expects := []struct {
+		name string
+		url  string
+	}{
+		{
+			name: "has Source.git",
+			url:  "https://github.com/rust-lang/libc",
+		},
+		{
+			name: "not found",
+			url:  "",
+		},
+	}
+
+	for i, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			actual, _ := tt.crate.fetchURLFromRegistry(http.Client{})
+			expected := expects[i].url
+			assert.Equal(t, expected, actual)
+		})
+	}
+}

cmd/swift/cocoapods_test.go

@@ -1,6 +1,7 @@
 package swift
 
 import (
+	"net/http"
 	"testing"
 
 	"github.com/stretchr/testify/assert"
@@ -12,7 +13,7 @@ func TestCocoaPod_PodspecPath(t *testing.T) {
 		pod  CocoaPod
 	}{
 		{
-			name: "KeychanAccess",
+			name: "KeychainAccess",
 			pod: CocoaPod{
 				name:    "KeychainAccess",
 				version: "4.2.2",
@@ -48,3 +49,57 @@ func TestCocoaPod_PodspecPath(t *testing.T) {
 		})
 	}
 }
+
+func TestCocoapod_fetchURLFromRegistry(t *testing.T) {
+	tests := []struct {
+		name string
+		pod  CocoaPod
+	}{
+		{
+			name: "has Source.git",
+			pod: CocoaPod{
+				name:    "GoogleUtilities/Environment",
+				version: "7.10.0",
+			},
+		},
+		{
+			name: "don't have Source.Git",
+			pod: CocoaPod{
+				name:    "not_exists",
+				version: "7.10.0",
+			},
+		},
+		{
+			name: "don't have Source.Git, but have Homepage",
+			pod: CocoaPod{
+				name:    "AppCenter",
+				version: "4.2.0",
+			},
+		},
+	}
+	expects := []struct {
+		name string
+		url  string
+	}{
+		{
+			name: "has Source.git",
+			url:  "https://github.com/google/GoogleUtilities.git",
+		},
+		{
+			name: "don't have Source.Git",
+			url:  "",
+		},
+		{
+			name: "don't have Source.Git, but have Homepage",
+			url:  "https://appcenter.ms",
+		},
+	}
+
+	for i, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			actual, _ := tt.pod.fetchURLFromRegistry(http.Client{})
+			expected := expects[i].url
+			assert.Equal(t, expected, actual)
+		})
+	}
+}