mvidner / ruby-dbus

A Ruby binding for DBus
GNU Lesser General Public License v2.1
170 stars 51 forks source link

Fix forgotten case: marshall Data::Array as an ARRAY #109

Closed mvidner closed 2 years ago

mvidner commented 2 years ago

A fix for https://github.com/yast/d-installer/issues/126#issuecomment-1101223422

CLI test:

busctl get-property org.opensuse.DInstaller /org/opensuse/DInstaller/Language1 org.opensuse.DInstaller.Language1 AvailableLanguages

The existing tests would only catch it if my_array was a nested array, such as aaq

coveralls commented 2 years ago

Coverage Status

Coverage increased (+0.004%) to 92.559% when pulling 375c81dfe39c43f0f994049fcbae4a6891032b26 on marshall-typed-array into 6b6de6bef22f864521484ade6f69356dd4513048 on master.

mvidner commented 2 years ago

The user visible topic is: Service side: fix returning properties that contain arrays (on other than outermost level)