fix: erroneously removed get_flow_id() method on Gladier Clients

Method was removed with the new FlowManager class, which now handles
deployment of the flow. The re-added method calls get_flow_id() on the
manager class.

gladier/client.py

@@ -298,6 +298,12 @@ def get_funcx_function_ids(self):
                 funcx_ids[name] = val
         return funcx_ids
 
+    def get_flow_id(self) -> str:
+        """
+        Get the flow id from the flows manager class.
+        """
+        return self.flows_manager.get_flow_id()
+
     def get_input(self) -> dict:
         """
         Get funcx function ids, funcx endpoints, and each tool's default input. Default

gladier/tests/test_client.py

@@ -1,4 +1,5 @@
 from gladier.tests.test_data.gladier_mocks import MockGladierClient
+from gladier.managers import FlowsManager
 
 
 def test_get_input(logged_in):
@@ -8,6 +9,10 @@ def test_get_input(logged_in):
     }}
 
 
+def test_get_flow_id(logged_in):
+    assert MockGladierClient().get_flow_id() is None
+
+
 def test_get_input_from_aliased_tool(logged_in):
 
     class MockGladierClientAlias(MockGladierClient):

gladier/tests/test_flows_manager.py

@@ -65,6 +65,15 @@ def test_custom_scope_id(logged_out):
     assert scope in cli.login_manager.missing_authorizers
 
 
+def test_custom_scope_id():
+    fm = FlowsManager(flow_id='foo')
+    cli = MockGladierClient(flows_manager=fm)
+    assert cli.flows_manager == fm
+    assert fm.flow_id == 'foo'
+    assert fm.get_flow_id() == 'foo'
+    assert cli.get_flow_id() == 'foo'
+
+
 def test_dependent_scope_change_run_flow(auto_login, mock_flows_client,
                                          mock_dependent_token_change_error,
                                          monkeypatch):