mirror of
https://github.com/caddyserver/caddy.git
synced 2024-12-23 22:27:38 -05:00
a798e0c951
- Server types no longer need to store their own contexts; they are stored on the caddy.Instance, which means each context will be properly GC'ed when the instance is stopped. Server types should use type assertions to convert from caddy.Context to their concrete context type when they need to use it. - Pass the entire context into httpserver.GetConfig instead of only the Key field. - caddy.NewTestController now requires a server type string so it can create a controller with the proper concrete context associated with that server type. Tests still need more attention so that we can test the proper creation of startup functions, etc.
173 lines
5.7 KiB
Go
173 lines
5.7 KiB
Go
package log
|
|
|
|
import (
|
|
"testing"
|
|
|
|
"github.com/mholt/caddy"
|
|
"github.com/mholt/caddy/caddyhttp/httpserver"
|
|
)
|
|
|
|
func TestSetup(t *testing.T) {
|
|
c := caddy.NewTestController("http", `log`)
|
|
err := setup(c)
|
|
if err != nil {
|
|
t.Errorf("Expected no errors, got: %v", err)
|
|
}
|
|
cfg := httpserver.GetConfig(c)
|
|
mids := cfg.Middleware()
|
|
if mids == nil {
|
|
t.Fatal("Expected middleware, was nil instead")
|
|
}
|
|
|
|
handler := mids[0](httpserver.EmptyNext)
|
|
myHandler, ok := handler.(Logger)
|
|
|
|
if !ok {
|
|
t.Fatalf("Expected handler to be type Logger, got: %#v", handler)
|
|
}
|
|
|
|
if myHandler.Rules[0].PathScope != "/" {
|
|
t.Errorf("Expected / as the default PathScope")
|
|
}
|
|
if myHandler.Rules[0].OutputFile != DefaultLogFilename {
|
|
t.Errorf("Expected %s as the default OutputFile", DefaultLogFilename)
|
|
}
|
|
if myHandler.Rules[0].Format != DefaultLogFormat {
|
|
t.Errorf("Expected %s as the default Log Format", DefaultLogFormat)
|
|
}
|
|
if myHandler.Rules[0].Roller != nil {
|
|
t.Errorf("Expected Roller to be nil, got: %v", *myHandler.Rules[0].Roller)
|
|
}
|
|
if !httpserver.SameNext(myHandler.Next, httpserver.EmptyNext) {
|
|
t.Error("'Next' field of handler was not set properly")
|
|
}
|
|
|
|
}
|
|
|
|
func TestLogParse(t *testing.T) {
|
|
tests := []struct {
|
|
inputLogRules string
|
|
shouldErr bool
|
|
expectedLogRules []Rule
|
|
}{
|
|
{`log`, false, []Rule{{
|
|
PathScope: "/",
|
|
OutputFile: DefaultLogFilename,
|
|
Format: DefaultLogFormat,
|
|
}}},
|
|
{`log log.txt`, false, []Rule{{
|
|
PathScope: "/",
|
|
OutputFile: "log.txt",
|
|
Format: DefaultLogFormat,
|
|
}}},
|
|
{`log /api log.txt`, false, []Rule{{
|
|
PathScope: "/api",
|
|
OutputFile: "log.txt",
|
|
Format: DefaultLogFormat,
|
|
}}},
|
|
{`log /serve stdout`, false, []Rule{{
|
|
PathScope: "/serve",
|
|
OutputFile: "stdout",
|
|
Format: DefaultLogFormat,
|
|
}}},
|
|
{`log /myapi log.txt {common}`, false, []Rule{{
|
|
PathScope: "/myapi",
|
|
OutputFile: "log.txt",
|
|
Format: CommonLogFormat,
|
|
}}},
|
|
{`log /test accesslog.txt {combined}`, false, []Rule{{
|
|
PathScope: "/test",
|
|
OutputFile: "accesslog.txt",
|
|
Format: CombinedLogFormat,
|
|
}}},
|
|
{`log /api1 log.txt
|
|
log /api2 accesslog.txt {combined}`, false, []Rule{{
|
|
PathScope: "/api1",
|
|
OutputFile: "log.txt",
|
|
Format: DefaultLogFormat,
|
|
}, {
|
|
PathScope: "/api2",
|
|
OutputFile: "accesslog.txt",
|
|
Format: CombinedLogFormat,
|
|
}}},
|
|
{`log /api3 stdout {host}
|
|
log /api4 log.txt {when}`, false, []Rule{{
|
|
PathScope: "/api3",
|
|
OutputFile: "stdout",
|
|
Format: "{host}",
|
|
}, {
|
|
PathScope: "/api4",
|
|
OutputFile: "log.txt",
|
|
Format: "{when}",
|
|
}}},
|
|
{`log access.log { rotate { size 2 age 10 keep 3 } }`, false, []Rule{{
|
|
PathScope: "/",
|
|
OutputFile: "access.log",
|
|
Format: DefaultLogFormat,
|
|
Roller: &httpserver.LogRoller{
|
|
MaxSize: 2,
|
|
MaxAge: 10,
|
|
MaxBackups: 3,
|
|
LocalTime: true,
|
|
},
|
|
}}},
|
|
}
|
|
for i, test := range tests {
|
|
c := caddy.NewTestController("http", test.inputLogRules)
|
|
actualLogRules, err := logParse(c)
|
|
|
|
if err == nil && test.shouldErr {
|
|
t.Errorf("Test %d didn't error, but it should have", i)
|
|
} else if err != nil && !test.shouldErr {
|
|
t.Errorf("Test %d errored, but it shouldn't have; got '%v'", i, err)
|
|
}
|
|
if len(actualLogRules) != len(test.expectedLogRules) {
|
|
t.Fatalf("Test %d expected %d no of Log rules, but got %d ",
|
|
i, len(test.expectedLogRules), len(actualLogRules))
|
|
}
|
|
for j, actualLogRule := range actualLogRules {
|
|
|
|
if actualLogRule.PathScope != test.expectedLogRules[j].PathScope {
|
|
t.Errorf("Test %d expected %dth LogRule PathScope to be %s , but got %s",
|
|
i, j, test.expectedLogRules[j].PathScope, actualLogRule.PathScope)
|
|
}
|
|
|
|
if actualLogRule.OutputFile != test.expectedLogRules[j].OutputFile {
|
|
t.Errorf("Test %d expected %dth LogRule OutputFile to be %s , but got %s",
|
|
i, j, test.expectedLogRules[j].OutputFile, actualLogRule.OutputFile)
|
|
}
|
|
|
|
if actualLogRule.Format != test.expectedLogRules[j].Format {
|
|
t.Errorf("Test %d expected %dth LogRule Format to be %s , but got %s",
|
|
i, j, test.expectedLogRules[j].Format, actualLogRule.Format)
|
|
}
|
|
if actualLogRule.Roller != nil && test.expectedLogRules[j].Roller == nil || actualLogRule.Roller == nil && test.expectedLogRules[j].Roller != nil {
|
|
t.Fatalf("Test %d expected %dth LogRule Roller to be %v, but got %v",
|
|
i, j, test.expectedLogRules[j].Roller, actualLogRule.Roller)
|
|
}
|
|
if actualLogRule.Roller != nil && test.expectedLogRules[j].Roller != nil {
|
|
if actualLogRule.Roller.Filename != test.expectedLogRules[j].Roller.Filename {
|
|
t.Fatalf("Test %d expected %dth LogRule Roller Filename to be %s, but got %s",
|
|
i, j, test.expectedLogRules[j].Roller.Filename, actualLogRule.Roller.Filename)
|
|
}
|
|
if actualLogRule.Roller.MaxAge != test.expectedLogRules[j].Roller.MaxAge {
|
|
t.Fatalf("Test %d expected %dth LogRule Roller MaxAge to be %d, but got %d",
|
|
i, j, test.expectedLogRules[j].Roller.MaxAge, actualLogRule.Roller.MaxAge)
|
|
}
|
|
if actualLogRule.Roller.MaxBackups != test.expectedLogRules[j].Roller.MaxBackups {
|
|
t.Fatalf("Test %d expected %dth LogRule Roller MaxBackups to be %d, but got %d",
|
|
i, j, test.expectedLogRules[j].Roller.MaxBackups, actualLogRule.Roller.MaxBackups)
|
|
}
|
|
if actualLogRule.Roller.MaxSize != test.expectedLogRules[j].Roller.MaxSize {
|
|
t.Fatalf("Test %d expected %dth LogRule Roller MaxSize to be %d, but got %d",
|
|
i, j, test.expectedLogRules[j].Roller.MaxSize, actualLogRule.Roller.MaxSize)
|
|
}
|
|
if actualLogRule.Roller.LocalTime != test.expectedLogRules[j].Roller.LocalTime {
|
|
t.Fatalf("Test %d expected %dth LogRule Roller LocalTime to be %t, but got %t",
|
|
i, j, test.expectedLogRules[j].Roller.LocalTime, actualLogRule.Roller.LocalTime)
|
|
}
|
|
}
|
|
}
|
|
}
|
|
|
|
}
|