vanja/gluetun
1
0
Fork 0
mirror of https://github.com/qdm12/gluetun.git synced 2026-05-06 20:10:11 +02:00
Code Issues Packages Projects Releases Wiki Activity

chore(pmtud): fix integration tests not compiling

Browse Source
This commit is contained in:
Quentin McGaw
2026-05-02 02:51:49 +00:00
parent 7a74d4f462
commit 9b6f048fe8
2 changed files with 26 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
internal/pmtud/pmtud_integration_test.go
+23 -3
View File
@@ -3,11 +3,15 @@
package pmtud package pmtud
import ( import (
"context" "errors"
"net/netip" "net/netip"
"testing" "testing"
"time" "time"
"github.com/qdm12/gluetun/internal/command"
"github.com/qdm12/gluetun/internal/firewall"
"github.com/qdm12/gluetun/internal/firewall/iptables"
"github.com/qdm12/log"
"github.com/stretchr/testify/require" "github.com/stretchr/testify/require"
) )
@@ -15,8 +19,24 @@ func Test_PathMTUDiscover(t *testing.T) {
t.Parallel() t.Parallel()
const physicalLinkMTU = 1500 const physicalLinkMTU = 1500
const timeout = time.Second const timeout = time.Second
mtu, err := PathMTUDiscover(context.Background(), netip.MustParseAddr("1.1.1.1"),
physicalLinkMTU, timeout, nil) logger := log.New(log.SetLevel(log.LevelDebug))
cmder := command.New()
fw, err := firewall.NewConfig(t.Context(), logger, logger, cmder, nil, nil)
if errors.Is(err, iptables.ErrNotSupported) {
t.Skip("iptables not installed, skipping TCP PMTUD tests")
}
require.NoError(t, err, "creating firewall config")
icmpAddrs := []netip.Addr{
netip.MustParseAddr("1.1.1.1"),
}
tcpAddrs := []netip.AddrPort{
netip.MustParseAddrPort("1.1.1.1:80"),
}
mtu, err := PathMTUDiscover(t.Context(), icmpAddrs, tcpAddrs,
physicalLinkMTU, timeout, fw, logger)
require.NoError(t, err) require.NoError(t, err)
t.Log("MTU found:", mtu) t.Log("MTU found:", mtu)
} }
internal/pmtud/tcp/tcp_integration_test.go
+3 -2
View File
@@ -10,6 +10,7 @@ import (
"github.com/qdm12/gluetun/internal/command" "github.com/qdm12/gluetun/internal/command"
"github.com/qdm12/gluetun/internal/firewall" "github.com/qdm12/gluetun/internal/firewall"
"github.com/qdm12/gluetun/internal/firewall/iptables"
"github.com/qdm12/gluetun/internal/pmtud/constants" "github.com/qdm12/gluetun/internal/pmtud/constants"
"github.com/qdm12/log" "github.com/qdm12/log"
"github.com/stretchr/testify/assert" "github.com/stretchr/testify/assert"
@@ -32,8 +33,8 @@ func Test_PathMTUDiscover(t *testing.T) {
logger := log.New(log.SetLevel(log.LevelDebug)) logger := log.New(log.SetLevel(log.LevelDebug))
cmder := command.New() cmder := command.New()
fw, err := firewall.NewConfig(t.Context(), logger, cmder, nil, nil) fw, err := firewall.NewConfig(t.Context(), logger, logger, cmder, nil, nil)
if errors.Is(err, firewall.ErrIPTablesNotSupported) { if errors.Is(err, iptables.ErrNotSupported) {
t.Skip("iptables not installed, skipping TCP PMTUD tests") t.Skip("iptables not installed, skipping TCP PMTUD tests")
} }
require.NoError(t, err, "creating firewall config") require.NoError(t, err, "creating firewall config")