diff options
author | Valery Piashchynski <[email protected]> | 2021-06-09 10:46:39 +0300 |
---|---|---|
committer | Valery Piashchynski <[email protected]> | 2021-06-09 10:46:39 +0300 |
commit | 8020cbb83afbaaf4ea4e0321b1b6c24d38332887 (patch) | |
tree | 024c9e3540b2f12e940b6b8cc88f8b2a0359a5e9 /plugins/server | |
parent | 0d89115ffed997cc0e3e30681369bf3d2296f377 (diff) |
- Add more tests
Signed-off-by: Valery Piashchynski <[email protected]>
Diffstat (limited to 'plugins/server')
-rw-r--r-- | plugins/server/command.go | 2 | ||||
-rw-r--r-- | plugins/server/command_test.go | 22 |
2 files changed, 23 insertions, 1 deletions
diff --git a/plugins/server/command.go b/plugins/server/command.go index 527cc224..a3752e7b 100644 --- a/plugins/server/command.go +++ b/plugins/server/command.go @@ -22,7 +22,7 @@ func (server *Plugin) scanCommand(cmd []string) error { // try to stat _, err := os.Stat(cmd[i]) if err != nil { - return errors.E(op, err) + return errors.E(op, errors.FileNotFound, err) } // stat successful diff --git a/plugins/server/command_test.go b/plugins/server/command_test.go index 24051fba..d226ae9c 100644 --- a/plugins/server/command_test.go +++ b/plugins/server/command_test.go @@ -4,6 +4,7 @@ import ( "strings" "testing" + "github.com/spiral/errors" "github.com/stretchr/testify/assert" ) @@ -11,4 +12,25 @@ func TestServerCommandChecker(t *testing.T) { s := &Plugin{} cmd1 := "php ../../tests/client.php" assert.NoError(t, s.scanCommand(strings.Split(cmd1, " "))) + + cmd2 := "C:/../../abcdef/client.php" + assert.Error(t, s.scanCommand(strings.Split(cmd2, " "))) + + cmd3 := "sh ./script.sh" + err := s.scanCommand(strings.Split(cmd3, " ")) + assert.Error(t, err) + if !errors.Is(errors.FileNotFound, err) { + t.Fatal("should be of filenotfound type") + } + + cmd4 := "php ../../tests/client.php --option1 --option2" + err = s.scanCommand(strings.Split(cmd4, " ")) + assert.NoError(t, err) + + cmd5 := "php ../../tests/cluent.php --option1 --option2" + err = s.scanCommand(strings.Split(cmd5, " ")) + assert.Error(t, err) + if !errors.Is(errors.FileNotFound, err) { + t.Fatal("should be of filenotfound type") + } } |