mirror of
https://github.com/0xJacky/nginx-ui.git
synced 2026-05-07 22:41:41 +08:00
918642f41c
* Refactor nginx path resolution with improved regex and fallback Updated regex patterns for extracting nginx configuration paths and added fallback mechanisms for determining paths on different operating systems. Improved error handling and logging for better debugging. * enhance: nginx path parsing and add tests #1412, #1414 --------- Co-authored-by: 0xJacky <me@jackyu.cn>
102 lines
2.4 KiB
Go
102 lines
2.4 KiB
Go
package nginx
|
|
|
|
import (
|
|
"fmt"
|
|
"path/filepath"
|
|
"testing"
|
|
|
|
"github.com/0xJacky/Nginx-UI/settings"
|
|
)
|
|
|
|
func TestExtractConfigureArg(t *testing.T) {
|
|
t.Parallel()
|
|
|
|
output := `
|
|
nginx version: nginx/1.25.2
|
|
configure arguments: --prefix="/Program Files/Nginx" --conf-path='/Program Files/Nginx/conf/nginx.conf' --pid-path=/var/run/nginx.pid
|
|
`
|
|
|
|
tests := []struct {
|
|
name string
|
|
flag string
|
|
want string
|
|
}{
|
|
{
|
|
name: "double quoted conf path",
|
|
flag: "--conf-path",
|
|
want: "/Program Files/Nginx/conf/nginx.conf",
|
|
},
|
|
{
|
|
name: "single quoted conf path alias",
|
|
flag: "conf-path",
|
|
want: "/Program Files/Nginx/conf/nginx.conf",
|
|
},
|
|
{
|
|
name: "unquoted pid path",
|
|
flag: "pid-path",
|
|
want: "/var/run/nginx.pid",
|
|
},
|
|
{
|
|
name: "missing flag",
|
|
flag: "--http-log-path",
|
|
want: "",
|
|
},
|
|
{
|
|
name: "prefix parsing",
|
|
flag: "prefix",
|
|
want: "/Program Files/Nginx",
|
|
},
|
|
}
|
|
|
|
for _, tt := range tests {
|
|
tt := tt
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
if got := extractConfigureArg(output, tt.flag); got != tt.want {
|
|
t.Fatalf("extractConfigureArg(%q) = %q, want %q", tt.flag, got, tt.want)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestGetConfAndPidPathsHandleSpaces(t *testing.T) {
|
|
originalConfigDir := settings.NginxSettings.ConfigDir
|
|
originalConfigPath := settings.NginxSettings.ConfigPath
|
|
originalPIDPath := settings.NginxSettings.PIDPath
|
|
originalNginxVOutput := nginxVOutput
|
|
|
|
t.Cleanup(func() {
|
|
settings.NginxSettings.ConfigDir = originalConfigDir
|
|
settings.NginxSettings.ConfigPath = originalConfigPath
|
|
settings.NginxSettings.PIDPath = originalPIDPath
|
|
nginxVOutput = originalNginxVOutput
|
|
})
|
|
|
|
settings.NginxSettings.ConfigDir = ""
|
|
settings.NginxSettings.ConfigPath = ""
|
|
settings.NginxSettings.PIDPath = ""
|
|
|
|
sampleConf := "/Program Files/nginx/conf/nginx.conf"
|
|
samplePID := "/Program Files/nginx/logs/nginx.pid"
|
|
|
|
nginxVOutput = fmt.Sprintf(`
|
|
nginx version: nginx/1.25.2
|
|
configure arguments: --conf-path="%s" --pid-path="%s"
|
|
`, sampleConf, samplePID)
|
|
|
|
confDir := GetConfPath()
|
|
expectedConfDir := filepath.Dir(sampleConf)
|
|
if confDir != expectedConfDir {
|
|
t.Fatalf("GetConfPath() = %q, want %q", confDir, expectedConfDir)
|
|
}
|
|
|
|
confEntry := GetConfEntryPath()
|
|
if confEntry != sampleConf {
|
|
t.Fatalf("GetConfEntryPath() = %q, want %q", confEntry, sampleConf)
|
|
}
|
|
|
|
pidPath := GetPIDPath()
|
|
if pidPath != samplePID {
|
|
t.Fatalf("GetPIDPath() = %q, want %q", pidPath, samplePID)
|
|
}
|
|
}
|