Respect CLI context order

CHANGELOG.md

@@ -4,11 +4,11 @@
 
 # Unreleased
 
-No current changes
+- Respect CLI context order
 
 # v4.9.2
 
-- Always use first CLI argument as target to allow non-component targets
+- Determine CLI target based on order to allow non-component targets
 - Ensure that config field is always reloaded from index to avoid incorrect recomputation
 
 # v4.9.1

src/machinable/cli.py

@@ -42,7 +42,7 @@ def _push(_elements, _dotlist, _version):
             version.append(arg)
         elif arg.startswith("--"):
             # method
-            methods.append(arg[2:])
+            methods.append((len(elements), arg[2:]))
         else:
             # module
             _push(elements, dotlist, version)
@@ -115,20 +115,23 @@ def main(args: Optional[List] = None):
         component = None
         for i, (module, *version) in enumerate(elements):
             element = get(module, version)
-            if i == 0:
+            if i == len(elements) - 1:
                 component = element
             else:
                 contexts.append(element.__enter__())
 
         if component is None:
             raise ValueError("You have to provide at least one interface")
 
-        for method in methods:
+        for i, method in methods:
             # check if cli_{method} exists before falling back on {method}
             target = getattr(
                 component, f"cli_{method}", getattr(component, method)
             )
-            target()
+            if callable(target):
+                target()
+            else:
+                print(target)
 
         for context in reversed(contexts):
             context.__exit__()

tests/test_cli.py

@@ -31,7 +31,7 @@ def test_cli_main(capfd, tmp_storage):
         main(["get", "interface.dummy", "--__call__"])
         out, err = capfd.readouterr()
         assert out == "Hello world!\n"
-        main(["get", "hello", "name=there", "interface.dummy", "--__call__"])
+        main(["get", "interface.dummy", "hello", "name=there", "--__call__"])
         out, err = capfd.readouterr()
         assert out == "Hello there!\n"