fix: Users are warned when pushing a pipeline with a code and a name or 'code='

* Use warnings (as print) instead of errors

* fix tests

* fix: test and typo

* fix: lint

---------

Co-authored-by: YolanFery <yolan@codethefuture.be>

Author: qgerome

examples/pipelines/logistic_stats/pipeline.py

@@ -13,7 +13,7 @@
 from openhexa.sdk import parameter, pipeline, workspace
 
 
-@pipeline("Logistic stats")
+@pipeline("Logistic stats", "Test")
 @parameter(
     "deg",
     name="Data element group",

openhexa/sdk/pipelines/runtime.py

@@ -237,16 +237,22 @@ def get_pipeline(pipeline_path: Path) -> Pipeline:
         # Handle deprecated 'code' argument
         if code_arg["value"] is not None:
             if code_arg["is_keyword"]:
-                raise DeprecationWarning(
-                    f"The 'code' argument is deprecated and should not be used as a keyword."
-                    f"Replace 'code=\"{code_arg['value']}\"' by 'name=\"{code_arg['value']}\"'"
+                print(
+                    "\n\033[93m",
+                    f"The 'code' argument is deprecated and should not be used as a keyword. "
+                    f"Replace 'code=\"{code_arg['value']}\"' by 'name=\"{code_arg['value']}\"'\033[0m",
+                    "\n",
+                    flush=True,
                 )
 
             if name_arg["value"] is not None:
-                raise DeprecationWarning(
+                print(
+                    "\n\033[93m",
                     f"Providing both 'code' and 'name' is deprecated. "
                     f"Please remove 'code' and only use 'name' when decorating the pipeline: "
-                    f'@pipeline(name="{name_arg["value"]}")'
+                    f'@pipeline(name="{name_arg["value"]}")\033[0m',
+                    "\n",
+                    flush=True,
                 )
 
         # Determine the pipeline name (prefer 'name', fall back to 'code')

tests/test_ast.py

@@ -1,7 +1,8 @@
 """Tests related to the parsing of the pipeline code."""
-
+import io
 import tempfile
 from unittest import TestCase
+from unittest.mock import patch
 
 from openhexa.sdk.pipelines.exceptions import InvalidParameterError, PipelineNotFound
 from openhexa.sdk.pipelines.runtime import get_pipeline
@@ -560,8 +561,12 @@ def test_pipeline_with_deprecated_code_argument_with_name(self):
                         ]
                     )
                 )
-            with self.assertRaises(DeprecationWarning):
-                get_pipeline(tmpdirname)
+            with patch("sys.stdout", new=io.StringIO()) as fake_stdout:
+                pipeline = get_pipeline(tmpdirname)
+                self.assertEqual(pipeline.to_dict()["name"], "Test pipeline")
+                self.assertIn(
+                    "The 'code' argument is deprecated and should not be used as a keyword.", fake_stdout.getvalue()
+                )
 
     def test_pipeline_with_deprecated_code_as_arg(self):
         """The file contains a @pipeline decorator with the deprecated 'code' argument."""
@@ -579,5 +584,7 @@ def test_pipeline_with_deprecated_code_as_arg(self):
                         ]
                     )
                 )
-            with self.assertRaises(DeprecationWarning):
-                get_pipeline(tmpdirname)
+            with patch("sys.stdout", new=io.StringIO()) as fake_stdout:
+                pipeline = get_pipeline(tmpdirname)
+                self.assertEqual(pipeline.to_dict()["name"], "Test pipeline")
+                self.assertIn("Providing both 'code' and 'name' is deprecated.", fake_stdout.getvalue())