fix: fall back to path when no repo URL provided

thisislawatts · thisislawatts · commit ff1eb040d448 · 2024-05-14T11:21:49.000+02:00
diff --git a/scan/target.go b/scan/target.go
@@ -42,7 +42,7 @@ func NewRepositoryTarget(path string, options ...TargetOptions) (Target, error)
 		var err error
 		result.repositoryUrl, err = util.GetRepositoryUrl(path)
 		if err != nil {
-			return &RepositoryTarget{}, err
+			return result, err
 		}
 	}
 
diff --git a/scan/target_test.go b/scan/target_test.go
@@ -23,7 +23,7 @@ func TestTarget_pathToNonRepo(t *testing.T) {
 	repoTarget, ok := target.(*RepositoryTarget)
 	assert.True(t, ok)
 	assert.Empty(t, repoTarget.GetRepositoryUrl())
-	assert.Empty(t, repoTarget.GetPath())
+	assert.Equal(t, expectedPath, repoTarget.GetPath())
 }
 
 func TestTarget_withRepoUrl(t *testing.T) {

Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,7 @@ func NewRepositoryTarget(path string, options ...TargetOptions) (Target, error)`
`42`	`42`	`var err error`
`43`	`43`	`result.repositoryUrl, err = util.GetRepositoryUrl(path)`
`44`	`44`	`if err != nil {`
`45`		`- return &RepositoryTarget{}, err`
	`45`	`+ return result, err`
`46`	`46`	`}`
`47`	`47`	`}`
`48`	`48`
Original file line number	Diff line number	Diff line change
`@@ -23,7 +23,7 @@ func TestTarget_pathToNonRepo(t *testing.T) {`
`23`	`23`	`repoTarget, ok := target.(*RepositoryTarget)`
`24`	`24`	`assert.True(t, ok)`
`25`	`25`	`assert.Empty(t, repoTarget.GetRepositoryUrl())`
`26`		`- assert.Empty(t, repoTarget.GetPath())`
	`26`	`+ assert.Equal(t, expectedPath, repoTarget.GetPath())`
`27`	`27`	`}`
`28`	`28`
`29`	`29`	`func TestTarget_withRepoUrl(t *testing.T) {`