Add TestDelete to settings_test.go

Author: MatteoPologruto

commands/daemon/settings_test.go

@@ -157,3 +157,30 @@ func TestWrite(t *testing.T) {
 	// We don't verify the content since we expect config library, Viper, to work
 	require.True(t, configFile.Exist())
 }
+
+func TestDelete(t *testing.T) {
+	temp, err := paths.TempDir().MkTempDir("testdata")
+	require.NoError(t, err)
+	defer temp.RemoveAll()
+
+	configFile := temp.Join("arduino-cli.yml")
+
+	_, err = svc.Delete(context.Background(), &rpc.DeleteRequest{
+		Key:      "doesnotexist",
+		FilePath: configFile.String(),
+	})
+	require.Error(t, err)
+
+	_, err = svc.Delete(context.Background(), &rpc.DeleteRequest{
+		Key:      "network",
+		FilePath: configFile.String(),
+	})
+	require.NoError(t, err)
+
+	data, err := configFile.ReadFile()
+	require.NoError(t, err)
+	require.NotContains(t, string(data), "network")
+
+	_, err = svc.GetValue(context.Background(), &rpc.GetValueRequest{Key: "network"})
+	require.Error(t, err)
+}

commands/daemon/testdata/arduino-cli.yml

@@ -14,3 +14,6 @@ logging:
   file: ""
   format: text
   level: info
+
+network:
+  proxy: "123"