Skip to content
This repository has been archived by the owner on Apr 18, 2023. It is now read-only.

Commit

Permalink
Add git Resolver test using fake controller
Browse files Browse the repository at this point in the history
Signed-off-by: Andrew Bayer <andrew.bayer@gmail.com>
  • Loading branch information
abayer committed May 24, 2022
1 parent 6cc6bcd commit cb3e55e
Show file tree
Hide file tree
Showing 5 changed files with 404 additions and 204 deletions.
4 changes: 2 additions & 2 deletions gitresolver/pkg/git/params.go
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,8 @@ package git
// URLParam is the git repo url
const URLParam string = "url"

// PathParam is the path into the git repo where a file is located
const PathParam string = "path"
// PathParam is the pathInRepo into the git repo where a file is located
const PathParam string = "pathInRepo"

// CommitParam is the commit hash that a file should be fetched from
const CommitParam string = "commit"
Expand Down
2 changes: 1 addition & 1 deletion gitresolver/pkg/git/resolver.go
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ func (r *Resolver) ValidateParams(_ context.Context, params map[string]string) e
}

// TODO(sbwsg): validate repo url is well-formed, git:// or https://
// TODO(sbwsg): validate path is valid relative path
// TODO(sbwsg): validate pathInRepo is valid relative pathInRepo

return nil
}
Expand Down
Loading

0 comments on commit cb3e55e

Please sign in to comment.