qvm-appmenus: drop unstable-format pledge for get-available

Author: Jayant-kernel

doc/tools/qvm-appmenus.rst

@@ -48,12 +48,12 @@ OPTIONS
  --set-default-whitelist PATH
     Set the default list of applications to be included in the menus of VMs based on this template. Should only be used for TemplateVMs .The PATH can be either a path to file containing a list of .desktop files, or a single hyphen ('-') to read from standard input.
 
---get-available [EXPERIMENTAL] [REQUIRES --i-understand-format-is-unstable]
-    List all available applications for the VM. The current format is UNSTABLE.
+--get-available
+    List all available applications for the VM.
     The applications are listed as hyphen-separated pairs consisting of file name and application name.
 
 --file-field FIELDNAME
-    .desktop file field to append to output for --get-available; can be used multiple times for multiple fields. This option changes output format to pipe-("|") separated. The current format is UNSTABLE.
+    .desktop file field to append to output for --get-available; can be used multiple times for multiple fields. This option changes output format to pipe-("|") separated.
 
 AUTHORS
 =======

qubesappmenus/__init__.py

@@ -761,7 +761,7 @@ def appmenus_update(self, vm, force=False):
 parser.add_argument(
     '--i-understand-format-is-unstable', dest='fool',
     action='store_true',
-    help='required pledge for --get-available')
+    help='deprecated no-op for backward compatibility')
 parser.add_argument(
     '--file-field', action='append', dest='fields',
     help='File field to append to output for --get-available; can be used'
@@ -852,10 +852,6 @@ def main(args=None, app=None):
             if args.update:
                 appmenus.appmenus_update(vm, force=args.force)
             if args.get_available:
-                if not args.fool:
-                    parser.error(
-                        'this requires --i-understand-format-is-unstable '
-                        'and a sacrifice of one cute kitten')
                 if not args.fields:
                     sys.stdout.write(''.join('{} - {}\n'.format(*available)
                                              for available in

qubesappmenus/tests.py

@@ -26,6 +26,7 @@
 import os
 import tempfile
 import sys
+import types
 
 import unittest
 import unittest.mock
@@ -726,6 +727,48 @@ def test_131_set_get_default_whitelist(self, mock_subprocess):
         self.ext.set_default_whitelist(tpl, whitelist)
         self.assertEqual(whitelist, self.ext.get_default_whitelist(tpl))
 
+    @unittest.mock.patch('qubesappmenus.parser.parse_args')
+    @unittest.mock.patch('qubesappmenus.Appmenus')
+    def test_132_get_available_does_not_require_unstable_flag(
+            self, appmenus_cls, parse_args):
+        vm = TestVM('test-inst-vm',
+            klass='AppVM',
+            virt_mode='pvh',
+            updateable=False,
+            provides_network=False,
+            label=self.app.labels[1])
+        app = TestApp()
+        app.domains = {vm.name: vm}
+
+        parse_args.return_value = types.SimpleNamespace(
+            all_domains=False,
+            domains=[vm],
+            remove=False,
+            init=False,
+            get_whitelist=False,
+            set_default_whitelist=None,
+            get_default_whitelist=False,
+            set_whitelist=None,
+            create=False,
+            update=False,
+            get_available=True,
+            fool=False,
+            fields=None,
+            template=None,
+            source=None,
+            force=False,
+            app=app,
+        )
+        appmenus_cls.return_value.get_available.return_value = [
+            ('xterm.desktop', 'XTerm')]
+
+        with unittest.mock.patch('sys.stdout', new_callable=io.StringIO) \
+                as stdout:
+            qubesappmenus.main([])
+
+        appmenus_cls.return_value.get_available.assert_called_once_with(vm)
+        self.assertEqual(stdout.getvalue(), 'xterm.desktop - XTerm\n')
+
 
 def list_tests():
     return (TC_00_Appmenus,)