port-ocean 0.28.9__tar.gz → 0.28.11__tar.gz

{port_ocean-0.28.9 → port_ocean-0.28.11}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: port-ocean
-Version: 0.28.9
+Version: 0.28.11
 Summary: Port Ocean is a CLI tool for managing your Port projects.
 Home-page: https://app.getport.io
 Keywords: ocean,port-ocean,port

{port_ocean-0.28.9 → port_ocean-0.28.11}/port_ocean/clients/port/mixins/integrations.py

@@ -301,7 +301,7 @@ class IntegrationClientMixin:
             headers=headers,
             json={
                 "items": raw_data,
-                "extractionTimestamp": datetime.now().isoformat(),
+                "extractionTimestamp": int(datetime.now().timestamp()),
             },
         )
         handle_port_status_code(response, should_log=False)

{port_ocean-0.28.9 → port_ocean-0.28.11}/port_ocean/core/handlers/port_app_config/models.py

@@ -29,7 +29,9 @@ class EntityMapping(BaseModel):
 
     @property
     def is_using_search_identifier(self) -> bool:
-        return isinstance(self.identifier, dict)
+        return isinstance(self.identifier, dict) or isinstance(
+            self.identifier, IngestSearchQuery
+        )
 
 
 class MappingsConfig(BaseModel):

{port_ocean-0.28.9 → port_ocean-0.28.11}/port_ocean/core/utils/utils.py

@@ -4,7 +4,7 @@ import json
 from typing import Iterable, Any, TypeVar, Callable, Awaitable
 
 from loguru import logger
-from pydantic import parse_obj_as, ValidationError
+from pydantic import BaseModel, parse_obj_as, ValidationError
 
 
 from port_ocean.clients.port.client import PortClient
@@ -79,6 +79,19 @@ async def gather_and_split_errors_from_results(
     return valid_items, errors
 
 
+def _get_entity_key(entity: Entity) -> tuple[str, str]:
+    identifier = entity.identifier
+    if isinstance(identifier, BaseModel):
+        identifier = identifier.dict()
+
+    key_part = (
+        json.dumps(identifier, sort_keys=True)
+        if isinstance(identifier, dict)
+        else str(identifier)
+    )
+    return key_part, entity.blueprint
+
+
 def get_port_diff(before: Iterable[Entity], after: Iterable[Entity]) -> EntityPortDiff:
     before_dict = {}
     after_dict = {}
@@ -88,12 +101,10 @@ def get_port_diff(before: Iterable[Entity], after: Iterable[Entity]) -> EntityPo
 
     # Create dictionaries for before and after lists
     for entity in before:
-        key = (entity.identifier, entity.blueprint)
-        before_dict[key] = entity
+        before_dict[_get_entity_key(entity)] = entity
 
     for entity in after:
-        key = (entity.identifier, entity.blueprint)
-        after_dict[key] = entity
+        after_dict[_get_entity_key(entity)] = entity
 
     # Find created, modified, and deleted objects
     for key, obj in after_dict.items():

port_ocean-0.28.11/port_ocean/tests/core/utils/test_get_port_diff.py

@@ -0,0 +1,164 @@
+from typing import Any
+
+from port_ocean.core.models import Entity
+from port_ocean.core.utils.utils import get_port_diff
+
+
+def create_test_entity(
+    identifier: str,
+    blueprint: str,
+    properties: dict[str, Any],
+    relations: dict[str, Any],
+    title: Any,
+    team: str | None | list[Any] = [],
+) -> Entity:
+    return Entity(
+        identifier=identifier,
+        blueprint=blueprint,
+        properties=properties,
+        relations=relations,
+        title=title,
+        team=team,
+    )
+
+
+entity1 = create_test_entity(
+    "id1",
+    "bp1",
+    {"totalIssues": 123, "url": "https://test.atlassian.net/browse/test-29081"},
+    {"reporter": "id1", "project": "project_id"},
+    "",
+    "",
+)
+entity1_modified_properties = create_test_entity(
+    "id1",
+    "bp1",
+    {"totalIssues": 5, "url": "https://test.atlassian.net/browse/test-29081"},
+    {"reporter": "id1", "project": "project_id"},
+    "",
+    "",
+)
+entity2 = create_test_entity(
+    "id2",
+    "bp2",
+    {"totalIssues": 234, "url": "https://test.atlassian.net/browse/test-23451"},
+    {"reporter": "id2", "project": "project_id2"},
+    "",
+    "",
+)
+entity3 = create_test_entity(
+    "id3",
+    "bp3",
+    {"totalIssues": 20, "url": "https://test.atlassian.net/browse/test-542"},
+    {"reporter": "id3", "project": "project_id3"},
+    "",
+    "",
+)
+
+
+def test_get_port_diff_with_dictionary_identifier() -> None:
+    """
+    Test that get_port_diff handles dictionary identifiers by converting them to strings.
+    An entity with a dictionary identifier in 'before' and not in 'after' should be marked as deleted.
+    """
+    entity_with_dict_id = create_test_entity(
+        identifier={"rules": "some_id", "combinator": "some combinator"},  # type: ignore
+        blueprint="bp1",
+        properties={},
+        relations={},
+        title="test",
+    )
+    before = [entity_with_dict_id]
+    after: list[Entity] = []
+
+    diff = get_port_diff(before, after)
+
+    assert not diff.created
+    assert not diff.modified
+    assert len(diff.deleted) == 1
+    assert entity_with_dict_id in diff.deleted
+
+
+def test_get_port_diff_no_changes() -> None:
+    """
+    Test get_port_diff with no changes between before and after.
+    Entities present in both should be in the 'modified' list.
+    """
+    before = [entity1, entity2]
+    after = [entity1, entity2]
+
+    diff = get_port_diff(before, after)
+
+    assert not diff.created
+    assert not diff.deleted
+    assert len(diff.modified) == 2
+    assert entity1 in diff.modified
+    assert entity2 in diff.modified
+
+
+def test_get_port_diff_created_entities() -> None:
+    """
+    Test get_port_diff with only new entities.
+    """
+    before: list[Entity] = []
+    after = [entity1, entity2]
+
+    diff = get_port_diff(before, after)
+
+    assert not diff.modified
+    assert not diff.deleted
+    assert len(diff.created) == 2
+    assert entity1 in diff.created
+    assert entity2 in diff.created
+
+
+def test_get_port_diff_deleted_entities() -> None:
+    """
+    Test get_port_diff with only deleted entities.
+    """
+    before = [entity1, entity2]
+    after: list[Entity] = []
+
+    diff = get_port_diff(before, after)
+
+    assert not diff.created
+    assert not diff.modified
+    assert len(diff.deleted) == 2
+    assert entity1 in diff.deleted
+    assert entity2 in diff.deleted
+
+
+def test_get_port_diff_modified_entities() -> None:
+    """
+    Test get_port_diff with modified entities.
+    Entities with same identifier and blueprint are considered modified.
+    """
+    before = [entity1, entity2]
+    after = [entity1_modified_properties, entity2]
+
+    diff = get_port_diff(before, after)
+
+    assert not diff.created
+    assert not diff.deleted
+    assert len(diff.modified) == 2
+    assert entity1_modified_properties in diff.modified
+    assert entity2 in diff.modified
+
+
+def test_get_port_diff_mixed_changes() -> None:
+    """
+    Test get_port_diff with a mix of created, modified, and deleted entities.
+    """
+    before = [entity1, entity2]
+    after = [entity1_modified_properties, entity3]
+
+    diff = get_port_diff(before, after)
+
+    assert len(diff.created) == 1
+    assert entity3 in diff.created
+
+    assert len(diff.modified) == 1
+    assert entity1_modified_properties in diff.modified
+
+    assert len(diff.deleted) == 1
+    assert entity2 in diff.deleted

{port_ocean-0.28.9 → port_ocean-0.28.11}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "port-ocean"
-version = "0.28.9"
+version = "0.28.11"
 description = "Port Ocean is a CLI tool for managing your Port projects."
 readme = "README.md"
 homepage = "https://app.getport.io"