完善单元测试

FUTURE.md

@@ -3,6 +3,7 @@
 ### v1.8.x
 
 * [ ] 融合 Go 官方设计的正式版本
+* [ ] 进一步提高单元测试覆盖率到 80%
 * [ ] 提高单元测试覆盖率到 90%
 
 ### v1.5.x
@@ -21,7 +22,6 @@
 * [x] 增加一个高可读性且高性能的 handler 实现
 * [x] TapeHandler 转义处理
 * [x] 提高单元测试覆盖率到 70%
-* [ ] 进一步提高单元测试覆盖率到 80%
 
 ### v1.2.x
 

Makefile

@@ -1,4 +1,4 @@
-.PHONY: test bench benchfile fmt
+.PHONY: test bench fmt
 
 all: test bench
 

extension/config/parse_test.go

@@ -80,7 +80,9 @@ func TestParseTimeDuration(t *testing.T) {
 		{name: "12s", s: "12s", want: 12 * time.Second, wantErr: false},
 		{name: "3m", s: "3m", want: 3 * time.Minute, wantErr: false},
 		{name: "24h", s: "24h", want: 24 * time.Hour, wantErr: false},
+		{name: "24h50m12s", s: "24h50m12s", want: 24*time.Hour + 50*time.Minute + 12*time.Second, wantErr: false},
 		{name: "7d", s: "7d", want: 7 * 24 * time.Hour, wantErr: false},
+		{name: "90D", s: "90D", want: 90 * 24 * time.Hour, wantErr: false},
 		{name: "''", s: "", want: 0, wantErr: true},
 		{name: "14", s: "14", want: 0, wantErr: true},
 	}

handler/buffer_test.go

@@ -20,7 +20,7 @@ import (
 	"github.com/FishGoddess/logit/defaults"
 )
 
-// go test -v -cover -count=1 -test.cpu=1 -run=^TestNewBuffer$
+// go test -v -cover -count=1 -test.cpu=1 -run=^TestBufferPool$
 func TestBufferPool(t *testing.T) {
 	for i := 0; i < 100; i++ {
 		bs := make([]byte, 0, 2*defaults.MaxBufferSize)

handler/escape_test.go

@@ -18,8 +18,8 @@ import "testing"
 
 // go test -v -cover -count=1 -test.cpu=1 -run=^TestAppendEscapedByte$
 func TestAppendEscapedByte(t *testing.T) {
-	testcases := []byte{'a', '0', '\n', '\t', '\\', '\b', '\f', '\r', '"'}
-	want := `a0\n\t\\b\f\r"`
+	testcases := []byte{'a', '0', '\n', '\t', '\\', '\b', '\f', '\r', '"', 15, 31}
+	want := `a0\n\t\\b\f\r"\u000f\u001f`
 
 	buffer := make([]byte, 0, 16)
 	for _, b := range testcases {
@@ -33,8 +33,8 @@ func TestAppendEscapedByte(t *testing.T) {
 
 // go test -v -cover -count=1 -test.cpu=1 -run=^TestAppendEscapedString$
 func TestAppendEscapedString(t *testing.T) {
-	testcases := []string{"a0国\n\t\\b\f\r\""}
-	want := `a0国\n\t\b\f\r"`
+	testcases := []string{"a0国\n\t\\\b\f\r\"\\b" + string([]byte{15}) + string([]byte{31})}
+	want := `a0国\n\t\\b\f\r"\b\u000f\u001f`
 
 	buffer := make([]byte, 0, 16)
 	for _, str := range testcases {