rename status and action

Signed-off-by: Antoine MAZEAS <antoine.mazeas@filigran.io>

Author: antoinemzs

pyoaev/signatures/models.py

@@ -191,19 +191,19 @@ def post_execution_update(self, tool_output: ToolOutput, now: datetime) -> None:
         self.end_time = now
 
         if tool_output.error_info and tool_output.error_info.exit_code != 0:
-            self.execution_status = "failed"
+            self.execution_status = "ERROR"
             if tool_output.error_info.crash_timestamp:
                 self.end_time = datetime.strptime(
                     tool_output.error_info.crash_timestamp, "%Y-%m-%dT%H:%M:%SZ"
                 )
         elif tool_output.timeout_info:
-            self.execution_status = "timeout"
+            self.execution_status = "TIMEOUT"
         elif tool_output.status == "partial":
-            self.execution_status = "partial"
+            self.execution_status = "ERROR"
         else:
-            self.execution_status = "success"
+            self.execution_status = "EXECUTED"
 
-        self.execution_action = InjectExecutionActions("complete")
+        self.execution_action = InjectExecutionActions("command_execution")
 
 
 class SignatureCallbackPayload(BaseModel):

test/signatures/constraints/signature_manager_post_execution_constraints.feature

@@ -18,22 +18,22 @@ Feature: SignatureManager post-execution constraints
     Scenario: Tool crash sets execution_status to failed and uses crash timestamp as end_time
         Given a tool_output containing error_info with exit_code=1 and crash_timestamp="2024-06-26T06:05:00Z"
         When I call post_execution_updates with the execution_details, execution_signatures and tool_output
-        Then execution_status equals "failed"
+        Then execution_status equals "ERROR"
         And end_time equals "2024-06-26T06:05:00Z"
         And the execution signature model contains every previous parameter unchanged
         And the execution details model contain every previous parameter pair unchanged
 
     Scenario: Timeout sets execution_status to timeout and includes available partial results
         Given a tool_output containing timeout_info with partial_results=["result-A", "result-B"]
         When I call post_execution_updates with the execution_details, execution_signatures and tool_output
-        Then execution_status equals "timeout"
+        Then execution_status equals "TIMEOUT"
         And the returned dict contains the partial results ["result-A", "result-B"] from timeout_info
         And the execution signature model contains every previous parameter unchanged
         And the execution details model contain every previous parameter pair unchanged
 
     Scenario: Timeout with no partial results still sets execution_status to timeout
         Given a tool_output containing timeout_info with no partial results available
         When I call post_execution_updates with the execution_details, execution_signatures and tool_output
-        Then execution_status equals "timeout"
+        Then execution_status equals "TIMEOUT"
         And the execution signature model contains every previous parameter unchanged
         And the execution details model contain every previous parameter pair unchanged

test/signatures/features/signature_manager_post_execution.feature

@@ -24,5 +24,5 @@ Feature: SignatureManager post-execution execution elements update
         And the execution details model contain every previous parameter pair unchanged
         And the end_time parameter in the execution details model is a datetime object
         And this end_time is chronologically greater than or equal to start_time "2024-06-26T06:00:00Z"
-        And the execution_status parameter in the execution details model is equal to "success"
-        And the execution_action parameter in the execution details model is equal to "complete"
+        And the execution_status parameter in the execution details model is equal to "EXECUTED"
+        And the execution_action parameter in the execution details model is equal to "command_execution"

test/signatures/test_signature_manager_post_execution.py

@@ -85,7 +85,7 @@ def execution_details():
     target_fixture="tool_output",
 )
 def successful_tool_output():
-    return {"status": "success"}
+    return {"status": "EXECUTED"}
 
 
 @given(

test/signatures/test_signature_manager_transmission.py

@@ -185,7 +185,7 @@ def compiled_post_execution_payload(context, inject_id):
 @given(parsers.parse("an updated post-execution execution details object"))
 def updated_post_execution_execution_details(context):
     execution_details = ExecutionDetails(
-        execution_status="success",
+        execution_status="SUCCESS",
         execution_action="complete",
     )
     execution_details.end_time = execution_details.start_time + timedelta(0.1)