Use AirflowConsole for non-JSON pool export output

Replace `rich.print(pools_list)` with `AirflowConsole().print_as()` so
non-JSON export formats (table, yaml, plain) are rendered consistently
with other airflow-ctl commands. Add test verifying the delegation.

Co-Authored-By: claude-flow <ruv@ruv.net>

Author: Leondon9

airflow-ctl/src/airflowctl/ctl/commands/pool_command.py

@@ -31,6 +31,7 @@
     BulkCreateActionPoolBody,
     PoolBody,
 )
+from airflowctl.ctl.console_formatting import AirflowConsole
 
 
 @provide_api_client(kind=ClientKind.CLI)
@@ -78,7 +79,7 @@ def export(args, api_client: Client = NEW_API_CLIENT) -> None:
             rich.print(f"Exported {pools_response.total_entries} pool(s) to {args.file}")
         else:
             # For non-json formats, print the pools directly to console
-            rich.print(pools_list)
+            AirflowConsole().print_as(data=pools_list, output=args.output)
     except Exception as e:
         raise SystemExit(f"Failed to export pools: {e}")
 

airflow-ctl/tests/airflow_ctl/ctl/commands/test_pool_command.py

@@ -149,35 +149,38 @@ def test_export_json_to_file(self, mock_client, tmp_path, capsys):
         expected_output = f"Exported {len(exported_data)} pool(s) to {export_file}"
         assert expected_output in captured.out.replace("\n", "")
 
-    def test_export_non_json_output(self, mock_client, tmp_path, capsys):
-        """Test pool export with non-json output format."""
-        # Create a proper dictionary structure
-        mock_pool = {
-            "name": "test_pool",
-            "slots": 1,
-            "description": "Test pool",
-            "include_deferred": True,
-            "occupied_slots": 0,
-            "running_slots": 0,
-            "queued_slots": 0,
-            "scheduled_slots": 0,
-            "open_slots": 1,
-            "deferred_slots": 0,
-        }
-        # Create a mock response with a proper pools attribute
+    def test_export_non_json_uses_airflow_console(self, mock_client, tmp_path):
+        """Test that non-JSON export delegates to AirflowConsole.print_as()."""
+        pool = type(
+            "Pool",
+            (),
+            {
+                "name": "test_pool",
+                "slots": 5,
+                "description": "Test pool",
+                "include_deferred": True,
+                "occupied_slots": 0,
+                "running_slots": 0,
+                "queued_slots": 0,
+                "scheduled_slots": 0,
+                "open_slots": 5,
+                "deferred_slots": 0,
+            },
+        )()
         mock_pools = mock.MagicMock()
-        mock_pools.pools = [mock.MagicMock(**mock_pool)]
+        mock_pools.pools = [pool]
         mock_pools.total_entries = 1
         mock_client.pools.list.return_value = mock_pools
 
-        pool_command.export(mock.MagicMock(file=tmp_path / "unused.json", output="table"))
-
-        # Verify console output contains the raw dict
-        captured = capsys.readouterr()
-        assert "test_pool" in captured.out
-        assert "slots" in captured.out
-        assert "description" in captured.out
-        assert "include_deferred" in captured.out
+        with mock.patch("airflowctl.ctl.commands.pool_command.AirflowConsole") as mock_console_cls:
+            mock_console = mock_console_cls.return_value
+            pool_command.export(mock.MagicMock(file=tmp_path / "unused.json", output="table"))
+            mock_console.print_as.assert_called_once()
+            call_kwargs = mock_console.print_as.call_args.kwargs
+            assert call_kwargs["output"] == "table"
+            assert len(call_kwargs["data"]) == 1
+            assert call_kwargs["data"][0]["name"] == "test_pool"
+            assert call_kwargs["data"][0]["slots"] == 5
 
     def test_export_failure(self, mock_client, tmp_path):
         """Test pool export with API failure."""