switch _get_ops_object to use ask_enumerate_dict

Author: dwhswenson

paths_cli/tests/wizard/test_load_from_ops.py

@@ -79,7 +79,7 @@ def test_get_ops_object(ops_file_fixture, with_failure):
     assert obj.name == 'bar'
     log = wizard.console.log_text
     assert 'name of the FOOMAGIC' in log
-    fail_msg = 'Something went wrong'
+    fail_msg = 'not a valid option'
     if with_failure:
         assert fail_msg in log
     else:

paths_cli/wizard/load_from_ops.py

@@ -31,18 +31,14 @@ def _get_ops_storage(wizard):
 
 @get_object
 def _get_ops_object(wizard, storage, store_name, obj_name):
-    name = wizard.ask(f"What's the name of the {obj_name} you want to "
-                      "load? (Type '?' to get a list of them)",
-                      helper=named_objs_helper(storage, store_name))
-    if name:
-        try:
-            obj = getattr(storage, store_name)[name]
-        except Exception as e:
-            wizard.exception("Something went wrong when loading "
-                   f"{name}. Maybe check the spelling?", e)
-            return
-        else:
-            return obj
+    # TODO: switch this to using wizard.ask_enumerate_dict, I think
+    store = getattr(storage, store_name)
+    options = {obj.name: obj for obj in store if obj.is_named}
+    result = wizard.ask_enumerate_dict(
+        f"What's the name of the {obj_name} you want to load?",
+        options
+    )
+    return result
 
 def load_from_ops(wizard, store_name, obj_name):
     wizard.say("Okay, we'll load it from an OPS file.")