fix(unikernels): simplify CommandString and use table-driven tests

Signed-off-by: mdryaan <alikhurshid842001@gmail.com>

Author: mdryaan

pkg/unikontainers/unikernels/mewz.go

@@ -36,12 +36,10 @@ type MewzNet struct {
 }
 
 func (m *Mewz) CommandString() (string, error) {
-	var parts []string
 	if m.Net.Address != "" {
-		parts = append(parts, fmt.Sprintf("ip=%s/%d", m.Net.Address, m.Net.Mask))
-		parts = append(parts, fmt.Sprintf("gateway=%s", m.Net.Gateway))
+		return fmt.Sprintf("ip=%s/%d gateway=%s", m.Net.Address, m.Net.Mask, m.Net.Gateway), nil
 	}
-	return strings.Join(parts, " "), nil
+	return "", nil
 }
 
 func (m *Mewz) SupportsBlock() bool {

pkg/unikontainers/unikernels/mewz_test.go

@@ -21,24 +21,38 @@ import (
 	"github.com/stretchr/testify/require"
 )
 
-func TestMewzCommandStringNoNetwork(t *testing.T) {
+func TestMewzCommandString(t *testing.T) {
 	t.Parallel()
-	m := &Mewz{}
-	result, err := m.CommandString()
-	require.NoError(t, err)
-	assert.Equal(t, "", result, "CommandString should return empty string when no network is configured")
-}
 
-func TestMewzCommandStringWithNetwork(t *testing.T) {
-	t.Parallel()
-	m := &Mewz{
-		Net: MewzNet{
-			Address: "10.0.0.2",
-			Mask:    24,
-			Gateway: "10.0.0.1",
+	testCases := []struct {
+		name     string
+		mewz     *Mewz
+		expected string
+	}{
+		{
+			name:     "no network configured",
+			mewz:     &Mewz{},
+			expected: "",
+		},
+		{
+			name: "with network configured",
+			mewz: &Mewz{
+				Net: MewzNet{
+					Address: "10.0.0.2",
+					Mask:    24,
+					Gateway: "10.0.0.1",
+				},
+			},
+			expected: "ip=10.0.0.2/24 gateway=10.0.0.1",
 		},
 	}
-	result, err := m.CommandString()
-	require.NoError(t, err)
-	assert.Equal(t, "ip=10.0.0.2/24 gateway=10.0.0.1", result)
+
+	for _, tc := range testCases {
+		t.Run(tc.name, func(t *testing.T) {
+			t.Parallel()
+			result, err := tc.mewz.CommandString()
+			require.NoError(t, err)
+			assert.Equal(t, tc.expected, result)
+		})
+	}
 }