Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
test: smoke tests with testscript (#305)
* test: playing with testscript * fix: multiple servers * fix: -update flag * test: user management tests * test: fix config * fix: ensure perms after clone * fix: race condition * fix: shutdown * test: set-username * test: repo collab * test: repo mirror and other ops * test: import repo * test: repo create * test: disable hooks on testscript * test: random port: prevent port reuse * test: wait for server * fix: git with no user info * test: no idea whats going on on windows Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com> * test: create keys on runtime Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com> * chore: organizing repo Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com> * fix: remove if for windows * Revert "fix: remove if for windows" This reverts commit 5776fde. * chore: trying something out Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com> * refactor: use crypto/ssh instead of ssh binary * fix: neg only the actual cmd * fix: unix2dos on windows tests * fix: unix2dos * fix: skip hooks on tests * fix: trainling whitespace * chore: editorconfig and gitattributes * test: maybe its not really needed? Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com> * Revert "test: maybe its not really needed?" This reverts commit a704c5f. * fix: improve \r\n handling Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com> * fix: crlf * chore: trigger Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com> * fix: disable unix2docs Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com> * test: validate repo * chore: debug * fix(test): convert dos crlf to lf (#311) * fix(test): convert dos crlf to lf * use temp files * chore: log * fix: ssh config Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com> * fix: config Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com> * fix(ci): use build workflow from main * fix: editorconfig * fix: editorconfig Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com> * feat(test): add SanitizeRepo tests * fix(test): sanitizerepo test --------- Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com> Co-authored-by: Ayman Bagabas <ayman.bagabas@gmail.com>
- Loading branch information
1 parent
1c781fe
commit 6d6e112
Showing
21 changed files
with
744 additions
and
23 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| root = true | ||
|
|
||
| [*] | ||
| charset=utf-8 | ||
| end_of_line=lf | ||
| insert_final_newline=true | ||
| trim_trailing_whitespace=true | ||
| indent_size=2 | ||
| indent_style=space | ||
|
|
||
| [*.go] | ||
| indent_size=4 | ||
| indent_style=tab |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,2 @@ | ||
| # To prevent CRLF breakages on Windows for fragile files, like testdata. | ||
| * -text |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,11 +1,29 @@ | ||
| package test | ||
|
|
||
| import "net" | ||
| import ( | ||
| "net" | ||
| "sync" | ||
| ) | ||
|
|
||
| var ( | ||
| used = map[int]struct{}{} | ||
| lock sync.Mutex | ||
| ) | ||
|
|
||
| // RandomPort returns a random port number. | ||
| // This is mainly used for testing. | ||
| func RandomPort() int { | ||
| addr, _ := net.Listen("tcp", ":0") //nolint:gosec | ||
| _ = addr.Close() | ||
| return addr.Addr().(*net.TCPAddr).Port | ||
| port := addr.Addr().(*net.TCPAddr).Port | ||
| lock.Lock() | ||
|
|
||
| if _, ok := used[port]; ok { | ||
| lock.Unlock() | ||
| return RandomPort() | ||
| } | ||
|
|
||
| used[port] = struct{}{} | ||
| lock.Unlock() | ||
| return port | ||
| } |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,56 @@ | ||
| package utils | ||
|
|
||
| import "testing" | ||
|
|
||
| func TestValidateRepo(t *testing.T) { | ||
| t.Run("valid", func(t *testing.T) { | ||
| for _, repo := range []string{ | ||
| "lower", | ||
| "Upper", | ||
| "with-dash", | ||
| "with/slash", | ||
| "withnumb3r5", | ||
| "with.dot", | ||
| "with_underline", | ||
| } { | ||
| t.Run(repo, func(t *testing.T) { | ||
| if err := ValidateRepo(repo); err != nil { | ||
| t.Errorf("expected no error, got %v", err) | ||
| } | ||
| }) | ||
| } | ||
| }) | ||
| t.Run("invalid", func(t *testing.T) { | ||
| for _, repo := range []string{ | ||
| "with$", | ||
| "with@", | ||
| "with!", | ||
| } { | ||
| t.Run(repo, func(t *testing.T) { | ||
| if err := ValidateRepo(repo); err == nil { | ||
| t.Error("expected an error, got nil") | ||
| } | ||
| }) | ||
| } | ||
| }) | ||
| } | ||
|
|
||
| func TestSanitizeRepo(t *testing.T) { | ||
| cases := []struct { | ||
| in, out string | ||
| }{ | ||
| {"lower", "lower"}, | ||
| {"Upper", "Upper"}, | ||
| {"with/slash", "with/slash"}, | ||
| {"with.dot", "with.dot"}, | ||
| {"/with_forward_slash", "with_forward_slash"}, | ||
| {"withgitsuffix.git", "withgitsuffix"}, | ||
| } | ||
| for _, c := range cases { | ||
| t.Run(c.in, func(t *testing.T) { | ||
| if got := SanitizeRepo(c.in); got != c.out { | ||
| t.Errorf("expected %q, got %q", c.out, got) | ||
| } | ||
| }) | ||
| } | ||
| } |
Oops, something went wrong.