Skip to content

Commit c3426a5

Browse files
committed
test: refactor asserts
Signed-off-by: Oleksandr Redko <Oleksandr_Redko@epam.com>
1 parent b5ee19a commit c3426a5

File tree

4 files changed

+8
-23
lines changed

4 files changed

+8
-23
lines changed

pkg/cidata/cidata_test.go

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,10 +21,7 @@ func TestSetupEnv(t *testing.T) {
2121
netLookupIP = fakeLookupIP
2222
urlMustParse := func(urlStr string) *url.URL {
2323
u, err := url.Parse(urlStr)
24-
if err != nil {
25-
panic(err)
26-
}
27-
24+
assert.NilError(t, err)
2825
return u
2926
}
3027

pkg/networks/config_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ import (
1010
func TestFillDefault(t *testing.T) {
1111
nwYaml := YAML{}
1212
newYaml, err := fillDefaults(nwYaml)
13-
assert.Check(t, err == nil)
13+
assert.NilError(t, err)
1414

1515
userNet := newYaml.Networks[ModeUserV2]
1616
assert.Equal(t, userNet.Mode, ModeUserV2)

pkg/networks/usernet/config_test.go

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -12,25 +12,19 @@ func TestUsernetConfig(t *testing.T) {
1212

1313
t.Run("verify dns ip", func(t *testing.T) {
1414
subnet, _, err := net.ParseCIDR(networks.SlirpNetwork)
15-
if err != nil {
16-
t.Fatal(err)
17-
}
15+
assert.NilError(t, err)
1816
assert.Equal(t, DNSIP(subnet), "192.168.5.3")
1917
})
2018

2119
t.Run("verify gateway ip", func(t *testing.T) {
2220
subnet, _, err := net.ParseCIDR(networks.SlirpNetwork)
23-
if err != nil {
24-
t.Fatal(err)
25-
}
21+
assert.NilError(t, err)
2622
assert.Equal(t, GatewayIP(subnet), "192.168.5.2")
2723
})
2824

2925
t.Run("verify subnet via config ip", func(t *testing.T) {
3026
subnet, err := Subnet("user-v2")
31-
if err != nil {
32-
t.Fatal(err)
33-
}
27+
assert.NilError(t, err)
3428
assert.Equal(t, subnet.String(), "192.168.104.0")
3529
})
3630
}

pkg/networks/usernet/gvproxy_test.go

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -35,17 +35,11 @@ nameserver 8.8.8.8`)
3535

3636
func createResolveFile(t *testing.T, file string, content string) {
3737
f, err := os.Create(file)
38-
if err != nil {
39-
t.Fatal(err)
40-
}
38+
assert.NilError(t, err)
4139
t.Cleanup(func() { _ = f.Close() })
4240
writer := bufio.NewWriter(f)
4341
_, err = writer.Write([]byte(content))
44-
if err != nil {
45-
t.Fatal(err)
46-
}
42+
assert.NilError(t, err)
4743
err = writer.Flush()
48-
if err != nil {
49-
t.Fatal(err)
50-
}
44+
assert.NilError(t, err)
5145
}

0 commit comments

Comments
 (0)