all: add tests
This commit is contained in:
parent
0d41eaabf7
commit
2c9d6c0c88
|
@ -114,3 +114,74 @@ func TestIpsetCtx_process(t *testing.T) {
|
|||
assert.NoError(t, err)
|
||||
})
|
||||
}
|
||||
|
||||
func TestIpsetCtx_SkipIpsetProcessing(t *testing.T) {
|
||||
req4 := createTestMessage("example.com")
|
||||
resp4 := &dns.Msg{
|
||||
Answer: []dns.RR{&dns.A{
|
||||
A: net.IP{1, 2, 3, 4},
|
||||
}},
|
||||
}
|
||||
|
||||
m := &fakeIpsetMgr{}
|
||||
ictx := &ipsetCtx{
|
||||
ipsetMgr: m,
|
||||
}
|
||||
|
||||
testCases := []struct {
|
||||
dctx *dnsContext
|
||||
name string
|
||||
want bool
|
||||
}{{
|
||||
name: "basic",
|
||||
want: false,
|
||||
dctx: &dnsContext{
|
||||
proxyCtx: &proxy.DNSContext{
|
||||
Req: req4,
|
||||
Res: resp4,
|
||||
},
|
||||
|
||||
responseFromUpstream: true,
|
||||
},
|
||||
}, {
|
||||
name: "rewrite",
|
||||
want: true,
|
||||
dctx: &dnsContext{
|
||||
proxyCtx: &proxy.DNSContext{
|
||||
Req: req4,
|
||||
Res: resp4,
|
||||
},
|
||||
|
||||
responseFromUpstream: false,
|
||||
},
|
||||
}, {
|
||||
name: "empty_req",
|
||||
want: true,
|
||||
dctx: &dnsContext{
|
||||
proxyCtx: &proxy.DNSContext{
|
||||
Req: nil,
|
||||
Res: resp4,
|
||||
},
|
||||
|
||||
responseFromUpstream: true,
|
||||
},
|
||||
}, {
|
||||
name: "empty_res",
|
||||
want: true,
|
||||
dctx: &dnsContext{
|
||||
proxyCtx: &proxy.DNSContext{
|
||||
Req: req4,
|
||||
Res: nil,
|
||||
},
|
||||
|
||||
responseFromUpstream: true,
|
||||
},
|
||||
}}
|
||||
|
||||
for _, tc := range testCases {
|
||||
t.Run(tc.name, func(t *testing.T) {
|
||||
got := ictx.skipIpsetProcessing(tc.dctx)
|
||||
assert.Equal(t, tc.want, got)
|
||||
})
|
||||
}
|
||||
}
|
|
@ -62,12 +62,6 @@ type props struct {
|
|||
family netfilter.ProtoFamily
|
||||
}
|
||||
|
||||
// ipInIpsetEntry is the type for entries in an ipsInIpset set.
|
||||
type ipInIpsetEntry struct {
|
||||
ipsetName string
|
||||
ipArr [net.IPv6len]byte
|
||||
}
|
||||
|
||||
// manager is the Linux Netfilter ipset manager.
|
||||
type manager struct {
|
||||
nameToIpset map[string]props
|
||||
|
@ -266,11 +260,6 @@ func (m *manager) addIPs(host string, set props, ips []net.IP) (n int, err error
|
|||
|
||||
var entries []*ipset.Entry
|
||||
for _, ip := range ips {
|
||||
e := ipInIpsetEntry{
|
||||
ipsetName: set.name,
|
||||
}
|
||||
copy(e.ipArr[:], ip.To16())
|
||||
|
||||
entries = append(entries, ipset.NewEntry(ipset.EntryIP(ip)))
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue