pygeai 0.4.0b12__py3-none-any.whl → 0.5.0__py3-none-any.whl

pygeai/__init__.py

@@ -4,7 +4,7 @@ import sys
 from pathlib import Path
 
 __author__ = 'Globant'
-__version__ = '0.4.0'
+__version__ = '0.5.0'
 
 # Add vendor directory to Python path
 package_root = Path(__file__).parent

pygeai/cli/__init__.py

@@ -4,5 +4,5 @@ GEAI - ClI
 Command line interface to interact with Globant Enterprise AI.
 """
 
-__version__ = '0.4.0'
+__version__ = '0.5.0'
 

pygeai/core/base/session.py

@@ -30,7 +30,7 @@ class Session(metaclass=Singleton):
             eval_url: Optional[str] = None,
     ):
         if not api_key or not base_url:
-            raise MissingRequirementException("Cannot instantiate session without api_key and base_url")
+            logger.warning("Cannot instantiate session without api_key and base_url")
 
         self.__api_key = api_key
         self.__base_url = base_url

pygeai/tests/integration/assistants/rag/test_create_rag.py

@@ -1,4 +1,5 @@
 from unittest import TestCase
+import uuid
 from pygeai.assistant.managers import AssistantManager
 from pygeai.core.models import WelcomeData, LlmSettings
 from pygeai.assistant.rag.models import (
@@ -10,10 +11,28 @@ from pygeai.assistant.rag.models import (
     RAGAssistant,
 )
 
-assistant_manager = AssistantManager(alias="beta")
 
 class TestAssistantCreateRagIntegration(TestCase):
 
+    def setUp(self):
+        """
+        Set up the test environment.
+        """
+        self.assistant_manager = AssistantManager(alias="beta")
+
+        self.new_rag = self.__load_rag()
+        self.created_rag: RAGAssistant = None
+
+
+    def tearDown(self):
+        """
+        Clean up after each test if necessary.
+        This can be used to delete the created tool
+        """
+        if isinstance(self.created_rag, RAGAssistant):
+            self.assistant_manager.delete_assistant(assistant_name=self.created_rag.name)            
+
+
     def __load_rag(self):
         llm_options = LlmSettings(
             cache=False,
@@ -57,7 +76,7 @@ class TestAssistantCreateRagIntegration(TestCase):
         )
 
         return RAGAssistant(
-            name="Testaasdfasdfsasdfdfsdfsdf123in123g12355",
+            name=str(uuid.uuid4()),
             description="Test Profile with WelcomeData",
             search_options=search_options,
             index_options=index_options,
@@ -67,6 +86,6 @@ class TestAssistantCreateRagIntegration(TestCase):
 
     def test_create_rag_assistant(self):
         rag_assistant = self.__load_rag()
-        response = assistant_manager.create_assistant(rag_assistant)
-        
-        self.assertIsInstance(response, RAGAssistant, "Failed to create RAG assistant")
+        self.created_rag = self.assistant_manager.create_assistant(rag_assistant)
+
+        self.assertIsInstance(self.created_rag, RAGAssistant, "Failed to create RAG assistant")

pygeai/tests/integration/chat/test_generate_image.py

@@ -56,13 +56,9 @@ class TestChatGenerateImageIntegration(TestCase):
         self.new_image["model"] = ""
         created_image = self.__generate_image()
 
-        self.assertEqual(
-            created_image["error"]["code"], 400,
-            "Expected a 400 code for no model"
-        )
         self.assertEqual(
             created_image["error"]["message"],
-            "Invalid 'model' name. Must follow pattern {provider}/{modelName}", 
+            "LLM Provider NOT provided. Pass in the LLM provider you are trying to call", 
             "Expected an error message when no model is provided"
         )
 

pygeai/tests/integration/lab/agents/test_create_agent.py

@@ -141,7 +141,6 @@ class TestAILabCreateAgentIntegration(TestCase):
         self.assertEqual(prompt.inputs, agent.agent_data.prompt.inputs)
 
 
-    @unittest.skip("Skipping test due to mismatch in expected behavior")
     def test_create_agent_without_required_data(self):
         test_params = [ True, False ]
 
@@ -151,12 +150,19 @@ class TestAILabCreateAgentIntegration(TestCase):
                 self.new_agent = Agent(
                     name=str(uuid.uuid4())
                 )
-                with self.assertRaises(APIError) as exception:
-                    self.__create_agent(automatic_publish=auto_publish)
-
-                #TODO: Change validation error to a more specific one
-                self.assertIn("400",str(exception.exception))    
-                self.assertIn("Bad Request",str(exception.exception))
+                if auto_publish:
+                    
+                    with self.assertRaises(APIError) as exception:
+                        self.__create_agent(automatic_publish=auto_publish)
+
+                    #TODO: Change validation error to a more specific one
+                    self.assertIn(
+                        "A valid prompt is required. To be valid, it must provide clear instructions to the model",
+                        str(exception.exception)
+                    )   
+                else:
+                    created_agent = self.__create_agent(automatic_publish=auto_publish)
+                    self.assertTrue(isinstance(created_agent, Agent), "Expected a created agent")
 
 
     def test_create_agent_no_name(self):

pygeai/tests/integration/lab/agents/test_create_sharing_link.py

@@ -1,4 +1,5 @@
 from unittest import TestCase
+import unittest
 from pygeai.lab.managers import AILabManager
 from pygeai.lab.models import SharingLink
 from pygeai.core.common.exceptions import APIError, MissingRequirementException
@@ -16,7 +17,7 @@ class TestAILabCreateSharingLinkIntegration(TestCase):
             agent_id=self.agent_id if agent_id is None else agent_id
         )
     
-
+    @unittest.skip("Endpoint is not found. Validate if it was deleted")
     def test_create_sharing_link(self):    
         shared_link = self.__create_sharing_link()        
         self.assertIsInstance(shared_link, SharingLink, "Expected response to be an instance of SharingLink")
@@ -46,6 +47,7 @@ class TestAILabCreateSharingLinkIntegration(TestCase):
         )
 
 
+    @unittest.skip("Endpoint is not found. Validate if it was deleted")
     def test_create_sharing_link_no_agent_id(self):
         with self.assertRaises(MissingRequirementException) as context:
             self.__create_sharing_link(agent_id="")
@@ -56,6 +58,7 @@ class TestAILabCreateSharingLinkIntegration(TestCase):
         )
 
 
+    @unittest.skip("Endpoint is not found. Validate if it was deleted")
     def test_create_sharing_link_invalid_agent_id(self):
         with self.assertRaises(APIError) as exception:
             invalid_id = "0026e53d-ea78-4cac-af9f-12650invalid"

pygeai/tests/integration/lab/agents/test_update_agent.py

@@ -191,23 +191,21 @@ class TestAILabUpdateAgentIntegration(TestCase):
         self.agent_to_update.agent_data.prompt.instructions = ""
 
         for auto_publish in test_params:
-            with self.subTest(input=auto_publish): 
-                with self.assertRaises(ValidationError) as exception:
-                    self.__update_agent(automatic_publish=auto_publish)
-                
-                self.assertIn(
-                    "instructions",
-                    str(exception.exception),
-                    f"Expected a validation error about allowed values for instructions when autopublish is {'enabled' if auto_publish else 'disabled'}"
-                )
-                self.assertIn(
-                    "Input should be a valid string [type=string_type, input_value=None, input_type=NoneType]",
-                    str(exception.exception),
-                    f"Expected a validation error about allowed values for instructions when autopublish is {'enabled' if auto_publish else 'disabled'}"
-                )
+            with self.subTest(input=auto_publish):
+                if auto_publish == True:
+                    with self.assertRaises(ValidationError) as exception:
+                        self.__update_agent(automatic_publish=auto_publish)
+                    self.assertIn(
+                        "agent_data.prompt must have at least instructions for publication",
+                        str(exception.exception),
+                        f"Expected a validation error about allowed values for instructions when autopublish is {'enabled' if auto_publish else 'disabled'}"
+                    )
+                else:
+                    updated_agent = self.__update_agent(automatic_publish=auto_publish)
+                    self.assertTrue(isinstance(updated_agent, Agent))
 
-    # TODO: Change validation when API behavior is fixed
-    def test_update_agent_no_model(self):        
+
+    def test_update_agent_no_model(self):
         test_params = [ True, False ]
         self.agent_to_update.agent_data.models[0].name = ""
         
@@ -217,9 +215,8 @@ class TestAILabUpdateAgentIntegration(TestCase):
                 # If the agent is not published, the API returns a warning message for invalid model name. However, the sdk mapping is not returning it.
                 if auto_publish == False:                    
                     updated_agent = self.__update_agent(automatic_publish=auto_publish)
-                    error_msg = str(exception.exception)
                     
-                    self.assertTrue(self.isInstance(updated_agent), Agent)
+                    self.assertTrue(isinstance(updated_agent, Agent))
                 else:                 
                     with self.assertRaises(APIError) as exception:
                         self.__update_agent(automatic_publish=auto_publish)

pygeai/tests/integration/lab/processes/test_create_process.py

@@ -0,0 +1,345 @@
+from unittest import TestCase
+import unittest
+import uuid
+from pygeai.lab.managers import AILabManager
+from pygeai.lab.models import (
+    AgenticProcess, KnowledgeBase, AgenticActivity, ArtifactSignal, Task, 
+    UserSignal, Event, SequenceFlow, Variable
+)
+from pygeai.core.common.exceptions import APIResponseError, APIError
+
+
+class TestAILabCreateProcessIntegration(TestCase): 
+
+    def setUp(self):
+        """
+        Set up the test environment.
+        """
+        self.ai_lab_manager = AILabManager(alias="beta")
+        self.new_process = self.__load_process()
+        self.created_process: AgenticProcess = None
+
+
+    def tearDown(self):
+        """
+        Clean up after each test if necessary.
+        This can be used to delete the created process
+        """
+        if isinstance(self.created_process, AgenticProcess):
+            self.ai_lab_manager.delete_process(self.created_process.id)
+
+
+    def __load_process(self):
+        """
+        Helper to load a complete process configuration for testing.
+        """
+        unique_key = str(uuid.uuid4())
+        process = AgenticProcess(
+            key="product_def",
+            name=f"Test Process {unique_key[:8]}",
+            description="This is a sample process",
+            kb=KnowledgeBase(name="basic-sample", artifact_type_name=["sample-artifact"]),
+            agentic_activities=[
+                AgenticActivity(key="activityOne", name="First Step", task_name="basic-task", agent_name="GoogleSummarizer2", agent_revision_id=0)
+            ],
+            artifact_signals=[
+                ArtifactSignal(key="artifact.upload.1", name="artifact.upload", handling_type="C", artifact_type_name=["sample-artifact"])
+            ],
+            user_signals=[
+                UserSignal(key="signal_done", name="process-completed")
+            ],
+            start_event=Event(key="artifact.upload.1", name="artifact.upload"),
+            end_event=Event(key="end", name="Done"),
+            sequence_flows=[
+                SequenceFlow(key="step1", source_key="artifact.upload.1", target_key="activityOne"),
+                SequenceFlow(key="step2", source_key="activityOne", target_key="signal_done"),
+                SequenceFlow(key="stepEnd", source_key="signal_done", target_key="end")
+            ]
+        )
+        return process
+    
+
+    def __create_process(self, process=None, automatic_publish=False):
+        """
+        Helper to create a process using ai_lab_manager.
+        """
+        return self.ai_lab_manager.create_process(
+            process=self.new_process if process is None else process,
+            automatic_publish=automatic_publish
+        )
+    
+
+    def test_create_process_full_data(self):
+        """
+        Test creating a process with all available fields populated.
+        """
+        self.created_process = self.__create_process()
+        created_process = self.created_process
+        process = self.new_process
+
+        self.assertTrue(isinstance(created_process, AgenticProcess), "Expected a created process")
+
+        # Assert the main fields of the created process
+        self.assertIsNotNone(created_process.id)
+        #self.assertEqual(created_process.key, process.key)
+        self.assertEqual(created_process.name, process.name)
+        self.assertEqual(created_process.description, process.description)
+        self.assertEqual(created_process.status, "active")
+
+        # Assert knowledge base
+        self.assertIsNotNone(created_process.kb)
+        self.assertEqual(created_process.kb.name, process.kb.name)
+
+        # Assert agentic activities
+        created_activity = created_process.agentic_activities[0]
+        original_activity = process.agentic_activities[0]
+        self.assertTrue(isinstance(created_activity, AgenticActivity)) 
+        self.assertEqual(created_activity.key.lower(), original_activity.key.lower())
+        self.assertEqual(created_activity.name, original_activity.name)
+            
+        # Assert artifact signals
+        created_signal = created_process.artifact_signals[0]
+        original_signal = process.artifact_signals[0]
+        self.assertTrue(isinstance(created_signal, ArtifactSignal))        
+        self.assertEqual(created_signal.key.lower(), original_signal.key.lower())
+        self.assertEqual(created_signal.name, original_signal.name)
+        self.assertEqual(created_signal.handling_type, original_signal.handling_type)
+
+        # Assert user signals
+        created_user_signal = created_process.user_signals[0]
+        self.assertTrue(isinstance(created_user_signal, UserSignal))
+
+        # Assert events
+        self.assertIsNotNone(created_process.start_event)
+        self.assertEqual(created_process.start_event.key.lower(), process.start_event.key.lower())
+        self.assertEqual(created_process.start_event.name, process.start_event.name)
+       
+        self.assertIsNotNone(created_process.end_event)
+        self.assertEqual(created_process.end_event.key.lower(), process.end_event.key.lower())
+        self.assertEqual(created_process.end_event.name, process.end_event.name)
+
+        # Assert sequence flows
+        created_flow = created_process.sequence_flows[0]
+        original_flow = process.sequence_flows[0]
+        self.assertIsNotNone(created_process.sequence_flows)
+        self.assertEqual(len(created_process.sequence_flows), len(process.sequence_flows))
+            
+        self.assertEqual(created_flow.key.lower(), original_flow.key.lower())
+        self.assertEqual(created_flow.source_key.lower(), original_flow.source_key.lower())
+        self.assertEqual(created_flow.target_key.lower(), original_flow.target_key.lower())
+
+
+    def test_create_process_minimum_required_data(self):
+        """
+        Test creating a process with only minimum required fields (key and name).
+        """
+        unique_key = str(uuid.uuid4())
+        self.new_process = AgenticProcess(
+            key=unique_key,
+            name=f"Minimal Process {unique_key[:8]}"
+        )
+        self.created_process = self.__create_process()
+        process = self.new_process
+
+        self.assertTrue(isinstance(self.created_process, AgenticProcess), "Expected a created process")
+        self.assertIsNotNone(self.created_process.id)
+        self.assertEqual(self.created_process.name, process.name)
+
+
+    def test_create_process_without_key(self):
+        """
+        Test creating a process without a key should still work (key is optional).
+        """
+        self.new_process = AgenticProcess(
+            name=f"Process Without Key {str(uuid.uuid4())[:8]}"
+        )
+        self.created_process = self.__create_process()
+
+        self.assertTrue(isinstance(self.created_process, AgenticProcess), "Expected a created process")
+        self.assertIsNotNone(self.created_process.id)
+        self.assertEqual(self.created_process.name, self.new_process.name)
+
+    
+    def test_create_process_no_name(self):
+        """
+        Test creating a process without a name should raise an error.
+        """
+        test_params = [True, False]
+
+        for auto_publish in test_params:
+            with self.subTest(input=auto_publish):
+                self.new_process.name = ""
+                with self.assertRaises(APIError) as exception:
+                    self.__create_process(automatic_publish=auto_publish)
+
+                self.assertIn(
+                    "Process name cannot be empty",
+                    str(exception.exception),
+                    f"Expected an error about empty process name with autopublish {'enabled' if auto_publish else 'disabled'}"
+                )
+
+   
+    def test_create_process_duplicated_name(self):
+        """
+        Test creating a process with a duplicate name should raise an error.
+        """
+        test_params = [True, False]
+        created_process = self.__create_process()
+
+        for auto_publish in test_params:
+            with self.subTest(input=auto_publish):
+                loaded_process = self.__load_process()
+                loaded_process.name = created_process.name
+                with self.assertRaises(APIError) as exception:
+                    self.__create_process(process = loaded_process, automatic_publish=auto_publish)
+
+                self.assertIn(
+                    f"A process with this name already exists [name={created_process.name}].",
+                    str(exception.exception),
+                    f"Expected an error about duplicated process name with autopublish {'enabled' if auto_publish else 'disabled'}"
+                )
+
+    
+    def test_create_process_invalid_name_characters(self):
+        """
+        Test creating a process with invalid characters in name should raise a validation error.
+        """
+        test_params = [True, False]
+
+        for auto_publish in test_params:
+            with self.subTest(input=auto_publish):
+                invalid_names = [
+                    f"{str(uuid.uuid4())[:8]}:invalid",
+                    f"{str(uuid.uuid4())[:8]}/invalid"
+                ]
+
+                for invalid_name in invalid_names:
+                    new_process = self.__load_process()
+                    new_process.name = invalid_name
+                    
+                    with self.assertRaises(APIError) as exception:
+                        self.__create_process(process=new_process, automatic_publish=auto_publish)
+                    
+                    self.assertIn(
+                        "Invalid character in name",
+                        str(exception.exception),
+                        f"Expected an error about invalid character in process name with autopublish {'enabled' if auto_publish else 'disabled'}"
+                    )
+
+    
+    @unittest.skip("Skipping test for now.KB is required but is marked as optional")
+    def test_create_process_with_no_kb(self):
+        """
+        Test creating a process with knowledge base in JSON format.
+        """
+        unique_key = str(uuid.uuid4())
+        self.new_process.kb = None
+        self.created_process = self.__create_process()
+
+        self.assertIsNotNone(self.created_process.kb)
+        self.assertEqual(self.created_process.kb.name, "basic-sample")
+
+
+    def test_create_process_with_empty_agentic_activities_array(self):
+        """
+        Test creating a process with empty agentic activities array (to clear all activities).
+        """
+        self.new_process.agentic_activities = []
+        self.new_process.sequence_flows = [] # Clear sequence flows as well because they depend on activities
+        self.created_process = self.__create_process()
+
+        # Empty array should be handled gracefully
+        self.assertTrue(
+            self.created_process.agentic_activities is None or 
+            len(self.created_process.agentic_activities) == 0
+        )
+
+
+    def test_create_process_with_multiple_artifact_signals(self):
+        """
+        Test creating a process with multiple artifact signals.
+        """
+        self.new_process.artifact_signals=[
+            ArtifactSignal(key="artifact.upload.1", name="artifact.upload", handling_type="C", artifact_type_name=["sample-artifact"]),
+            ArtifactSignal(key="artifact.upload.2", name="artifact.upload.second", handling_type="C", artifact_type_name=["sample-artifact"])
+        ]
+        self.created_process = self.__create_process()
+
+        self.assertIsNotNone(self.created_process.artifact_signals)
+        self.assertEqual(len(self.created_process.artifact_signals), 2)
+
+        first_signal = self.created_process.artifact_signals[0]
+        second_signal = self.created_process.artifact_signals[1]
+        
+        self.assertEqual(first_signal.key.lower(), "artifact.upload.1")
+        self.assertEqual(second_signal.key.lower(), "artifact.upload.2")
+
+
+    def test_create_process_with_multiple_user_signals(self):
+        """
+        Test creating a process with multiple user signals.
+        """
+        self.new_process.user_signals=[
+            UserSignal(key="signal_done", name="process-completed"),
+            UserSignal(key="signal_cancel", name="process-cancelled")
+        ]
+        self.created_process = self.__create_process()
+
+        self.assertIsNotNone(self.created_process.user_signals)
+        self.assertEqual(len(self.created_process.user_signals)-1, 2)
+        keys = [signal.key for signal in self.created_process.user_signals]
+
+        assert "SIGNAL_CANCEL" in keys
+        assert "SIGNAL_DONE" in keys
+
+
+    def test_create_process_autopublish_disabled(self):
+        """
+        Test creating a process without automatic publish (default behavior).
+        """
+        self.created_process = self.__create_process(automatic_publish=False)
+        
+        # Process should be created as draft by default
+        self.assertTrue(
+            self.created_process.is_draft is True or 
+            self.created_process.is_draft is None,  # API might not return this field for drafts
+            "Expected the process to be created as draft when autopublish is disabled"
+        )
+
+   
+    def test_create_process_autopublish_without_task(self):
+        """
+        Test creating a process with automatic publish enabled.
+        """
+        
+        with self.assertRaises(APIError) as exception:
+            self.__create_process(automatic_publish=True)
+
+        self.assertIn(
+            "Failure on process publication",
+            str(exception.exception),
+            "Expected an error about failure on process publication"
+        )
+
+        self.assertIn(
+            "Task is required",
+            str(exception.exception),
+            "Expected an error about task being required"
+        )
+
+
+    def test_create_process_autopublish_enabled(self):
+        """
+        Test creating a process with automatic publish enabled.
+        """
+        unique_key = str(uuid.uuid4())
+        task = Task(name=unique_key, description="Basic task for process", title_template="Basic Task")
+        self.ai_lab_manager.create_task(task=task, automatic_publish=True)
+        self.new_process.agentic_activities[0].task_name = unique_key
+        
+       
+        self.created_process = self.__create_process(automatic_publish=True)
+        self.assertTrue(
+            self.created_process.is_draft is False,
+            "Expected the process to be created no draft when autopublish is enabled"
+        )