rc: listremotes should send an empty array instead of nil

Author: Its-Just-Nans

fs/config/rc.go

@@ -106,6 +106,9 @@ See the [listremotes](/commands/rclone_listremotes/) command for more informatio
 // including any defined by environment variables.
 func rcListRemotes(ctx context.Context, in rc.Params) (out rc.Params, err error) {
 	remoteNames := GetRemoteNames()
+	if remoteNames == nil {
+		remoteNames = []string{}
+	}
 	out = rc.Params{
 		"remotes": remoteNames,
 	}

fs/config/rc_test.go

@@ -138,6 +138,22 @@ func TestRc(t *testing.T) {
 	assert.Nil(t, out)
 	assert.Equal(t, "", config.GetValue(testName, "type"))
 	assert.Equal(t, "", config.GetValue(testName, "test_key"))
+
+	t.Run("ListRemotes empty not nil", func(t *testing.T) {
+		call := rc.Calls.Get("config/listremotes")
+		assert.NotNil(t, call)
+		in := rc.Params{}
+		out, err := call.Fn(context.Background(), in)
+		require.NoError(t, err)
+		require.NotNil(t, out)
+
+		var remotes []string
+		err = out.GetStruct("remotes", &remotes)
+		require.NoError(t, err)
+
+		assert.NotNil(t, remotes)
+		assert.Empty(t, remotes)
+	})
 }
 
 func TestRcProviders(t *testing.T) {