mirror of
				https://github.com/coredns/coredns.git
				synced 2025-10-30 17:53:21 -04:00 
			
		
		
		
	This clear out the remaining map[x]bool usage and moves the bool to an empty struct. Two note worthy other changes: * EnableChaos in the server is now also exported to make it show up in the documentation. * The auto plugin is left as is, because there the boolean is explicitaly set to false to signal 'to-be-deleted' and the key is left as-is. Signed-off-by: Miek Gieben <miek@miek.nl>
		
			
				
	
	
		
			139 lines
		
	
	
		
			4.1 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
			
		
		
	
	
			139 lines
		
	
	
		
			4.1 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
| package log
 | |
| 
 | |
| import (
 | |
| 	"reflect"
 | |
| 	"testing"
 | |
| 
 | |
| 	"github.com/coredns/coredns/plugin/pkg/response"
 | |
| 
 | |
| 	"github.com/mholt/caddy"
 | |
| )
 | |
| 
 | |
| func TestLogParse(t *testing.T) {
 | |
| 	tests := []struct {
 | |
| 		inputLogRules    string
 | |
| 		shouldErr        bool
 | |
| 		expectedLogRules []Rule
 | |
| 	}{
 | |
| 		{`log`, false, []Rule{{
 | |
| 			NameScope: ".",
 | |
| 			Format:    DefaultLogFormat,
 | |
| 			Class:     map[response.Class]struct{}{response.All: struct{}{}},
 | |
| 		}}},
 | |
| 		{`log example.org`, false, []Rule{{
 | |
| 			NameScope: "example.org.",
 | |
| 			Format:    DefaultLogFormat,
 | |
| 			Class:     map[response.Class]struct{}{response.All: struct{}{}},
 | |
| 		}}},
 | |
| 		{`log example.org. {common}`, false, []Rule{{
 | |
| 			NameScope: "example.org.",
 | |
| 			Format:    CommonLogFormat,
 | |
| 			Class:     map[response.Class]struct{}{response.All: struct{}{}},
 | |
| 		}}},
 | |
| 		{`log example.org {combined}`, false, []Rule{{
 | |
| 			NameScope: "example.org.",
 | |
| 			Format:    CombinedLogFormat,
 | |
| 			Class:     map[response.Class]struct{}{response.All: struct{}{}},
 | |
| 		}}},
 | |
| 		{`log example.org.
 | |
| 		log example.net {combined}`, false, []Rule{{
 | |
| 			NameScope: "example.org.",
 | |
| 			Format:    DefaultLogFormat,
 | |
| 			Class:     map[response.Class]struct{}{response.All: struct{}{}},
 | |
| 		}, {
 | |
| 			NameScope: "example.net.",
 | |
| 			Format:    CombinedLogFormat,
 | |
| 			Class:     map[response.Class]struct{}{response.All: struct{}{}},
 | |
| 		}}},
 | |
| 		{`log example.org {host}
 | |
| 			  log example.org {when}`, false, []Rule{{
 | |
| 			NameScope: "example.org.",
 | |
| 			Format:    "{host}",
 | |
| 			Class:     map[response.Class]struct{}{response.All: struct{}{}},
 | |
| 		}, {
 | |
| 			NameScope: "example.org.",
 | |
| 			Format:    "{when}",
 | |
| 			Class:     map[response.Class]struct{}{response.All: struct{}{}},
 | |
| 		}}},
 | |
| 
 | |
| 		{`log example.org {
 | |
| 				class all
 | |
| 			}`, false, []Rule{{
 | |
| 			NameScope: "example.org.",
 | |
| 			Format:    CommonLogFormat,
 | |
| 			Class:     map[response.Class]struct{}{response.All: struct{}{}},
 | |
| 		}}},
 | |
| 		{`log example.org {
 | |
| 			class denial
 | |
| 		}`, false, []Rule{{
 | |
| 			NameScope: "example.org.",
 | |
| 			Format:    CommonLogFormat,
 | |
| 			Class:     map[response.Class]struct{}{response.Denial: struct{}{}},
 | |
| 		}}},
 | |
| 		{`log {
 | |
| 			class denial
 | |
| 		}`, false, []Rule{{
 | |
| 			NameScope: ".",
 | |
| 			Format:    CommonLogFormat,
 | |
| 			Class:     map[response.Class]struct{}{response.Denial: struct{}{}},
 | |
| 		}}},
 | |
| 		{`log {
 | |
| 			class denial error
 | |
| 		}`, false, []Rule{{
 | |
| 			NameScope: ".",
 | |
| 			Format:    CommonLogFormat,
 | |
| 			Class:     map[response.Class]struct{}{response.Denial: struct{}{}, response.Error: struct{}{}},
 | |
| 		}}},
 | |
| 		{`log {
 | |
| 			class denial
 | |
| 			class error
 | |
| 		}`, false, []Rule{{
 | |
| 			NameScope: ".",
 | |
| 			Format:    CommonLogFormat,
 | |
| 			Class:     map[response.Class]struct{}{response.Denial: struct{}{}, response.Error: struct{}{}},
 | |
| 		}}},
 | |
| 		{`log {
 | |
| 			class abracadabra
 | |
| 		}`, true, []Rule{}},
 | |
| 		{`log {
 | |
| 			class
 | |
| 		}`, true, []Rule{}},
 | |
| 		{`log {
 | |
| 			unknown
 | |
| 		}`, true, []Rule{}},
 | |
| 	}
 | |
| 	for i, test := range tests {
 | |
| 		c := caddy.NewTestController("dns", test.inputLogRules)
 | |
| 		actualLogRules, err := logParse(c)
 | |
| 
 | |
| 		if err == nil && test.shouldErr {
 | |
| 			t.Errorf("Test %d with input '%s' didn't error, but it should have", i, test.inputLogRules)
 | |
| 		} else if err != nil && !test.shouldErr {
 | |
| 			t.Errorf("Test %d with input '%s' errored, but it shouldn't have; got '%v'",
 | |
| 				i, test.inputLogRules, 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.NameScope != test.expectedLogRules[j].NameScope {
 | |
| 				t.Errorf("Test %d expected %dth LogRule NameScope for '%s' to be  %s  , but got %s",
 | |
| 					i, j, test.inputLogRules, test.expectedLogRules[j].NameScope, actualLogRule.NameScope)
 | |
| 			}
 | |
| 
 | |
| 			if actualLogRule.Format != test.expectedLogRules[j].Format {
 | |
| 				t.Errorf("Test %d expected %dth LogRule Format for '%s' to be  %s  , but got %s",
 | |
| 					i, j, test.inputLogRules, test.expectedLogRules[j].Format, actualLogRule.Format)
 | |
| 			}
 | |
| 
 | |
| 			if !reflect.DeepEqual(actualLogRule.Class, test.expectedLogRules[j].Class) {
 | |
| 				t.Errorf("Test %d expected %dth LogRule Class to be  %v  , but got %v",
 | |
| 					i, j, test.expectedLogRules[j].Class, actualLogRule.Class)
 | |
| 			}
 | |
| 		}
 | |
| 	}
 | |
| 
 | |
| }
 |