can you add an explicit test for no defined groups (legacy usage) and also a test where the ```return 0``` from update_pkgs would get triggered (because it seems strange that this could happen at all)?