geobox 1.3.1__tar.gz → 1.3.3__tar.gz

{geobox-1.3.1 → geobox-1.3.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: geobox
-Version: 1.3.1
+Version: 1.3.3
 Summary: SDK for Geobox's APIs
 Author-email: Hamid Heydari <heydari.h62@gmail.com>
 License: MIT

{geobox-1.3.1 → geobox-1.3.3}/geobox/api.py

@@ -781,16 +781,14 @@ class GeoboxClient:
         return VectorLayerView.get_view_by_name(self, name, user_id)
 
 
-    def create_tileset(self, name: str, layers: List[Dict], display_name: str = None, description: str = None,
+    def create_tileset(self, name: str, layers: List[Union['VectorLayer', 'VectorLayerView']], display_name: str = None, description: str = None,
                         min_zoom: int = None, max_zoom: int = None, user_id: int = None) -> 'Tileset':
         """
         Create a new tileset.
 
         Args:
             name (str): The name of the tileset.
-            layers (List[Dict]): The layers of the tileset. a list of dictionaries with the following keys:
-                - layer_type: The type of the layer. valid values are "vector" and "view".
-                - layer_uuid: The uuid of the layer.
+            layers (List['VectorLayer' | 'VectorLayerView']): list of vectorlayer and view objects to add to tileset.
             display_name (str, optional): The display name of the tileset.
             description (str, optional): The description of the tileset.
             min_zoom (int, optional): The minimum zoom level of the tileset.
@@ -802,19 +800,16 @@ class GeoboxClient:
 
         Example:
             >>> from geobox import GeoboxClient
+            >>> from geobox.tileset import Tileset
             >>> client = GeoboxClient()
-            >>> layers = [
-            ...     {
-            ...     "layer_type": "vector",
-            ...     "layer_uuid": "12345678-1234-5678-1234-567812345678"
-            ...     }
-            ... ]
+            >>> layer = client.get_vector(uuid="12345678-1234-5678-1234-567812345678")
+            >>> view = client.get_view(uuid="12345678-1234-5678-1234-567812345678")
             >>> tileset = client.create_tileset(name="your_tileset_name", 
             ...                                     display_name="Your Tileset", 
             ...                                     description="Your description", 
             ...                                     min_zoom=0, 
             ...                                     max_zoom=14, 
-            ...                                     layers=layers)
+            ...                                     layers=[layer, view])
         """
         return Tileset.create_tileset(api=self, 
                                       name=name, 

{geobox-1.3.1 → geobox-1.3.3}/geobox/enums.py

@@ -326,12 +326,6 @@ class QueryResultType(Enum):
     both = "both"
 
 
-class TilesetLayerType(Enum):
-    Vector = "vector"
-
-    View = "view"
-
-
 class FileOutputFormat(Enum):
     Shapefile = "Shapefile"
 

{geobox-1.3.1 → geobox-1.3.3}/geobox/query.py

@@ -582,7 +582,7 @@ class Query(Base):
         super()._unshare(self.endpoint, users)
 
 
-    def get_shared_users(self, search: str, skip: int = 0, limit: int = 10) -> List['User']:
+    def get_shared_users(self, search: str = None, skip: int = 0, limit: int = 10) -> List['User']:
         """
         Retrieves the list of users the query is shared with.
 
@@ -624,18 +624,16 @@ class Query(Base):
         return endpoint
 
 
-    def save_as_layer(self, layer_name: str, layer_type: 'QueryGeometryType') -> Dict:
+    def save_as_layer(self, layer_name: str, layer_type: 'QueryGeometryType' = None) -> Task:
         """
         Saves the query as a new layer.
 
         Args:
-            sql (str): The SQL statement for the query.
-            params (list): The parameters for the SQL statement.
             layer_name (str): The name of the new layer.
-            layer_type (QueryGeometryType): The type of the new layer.
+            layer_type (QueryGeometryType, optional): The type of the new layer.
 
         Returns:
-            Dict: The response from the API.
+            Task: The response task object.
             
         Raises:
             PermissionError: If the query is a read-only system query.
@@ -645,7 +643,7 @@ class Query(Base):
             >>> from geobox.query import Query
             >>> client = GeoboxClient()
             >>> query = Query.get_query(client, uuid="12345678-1234-5678-1234-567812345678")
-            >>> query.save_as_layer(layer_name='test', layer_type=QueryGeometryType.POLYGON)
+            >>> query.save_as_layer(layer_name='test')
         """
         self._check_access()
 
@@ -659,7 +657,7 @@ class Query(Base):
             "sql": self.sql,
             "params": params,
             "layer_name": layer_name,
-            "layer_type": layer_type.value
+            "layer_type": layer_type.value if layer_type else None
         })
 
         endpoint = urljoin(self.BASE_ENDPOINT, 'saveAsLayer/')

{geobox-1.3.1 → geobox-1.3.3}/geobox/raster.py

@@ -510,13 +510,13 @@ class Raster(Base):
             'colormap_name': kwargs.get('colormap_name'),
             'colormap': kwargs.get('colormap')
         })
-        query_string = urlencode(clean_data(params))
+        query_string = urlencode(params)
         endpoint = urljoin(self.api.base_url, f'{self.endpoint}render/{z}/{x}/{y}.png')
         endpoint = urljoin(endpoint, f'?{query_string}')
         return endpoint
     
 
-    def get_tile_pbf_url(self, x: int, y: int, z: int) -> str:
+    def get_tile_pbf_url(self, x: int, y: int, z: int, indexes: str = None) -> str:
         """
         Get the URL of the tile.
 
@@ -524,6 +524,7 @@ class Raster(Base):
             x (int): The x coordinate of the tile.
             y (int): The y coordinate of the tile.
             z (int): The zoom level of the tile.
+            indexes (str, optional): list of comma separated band indexes to be rendered. e.g. 1, 2, 3
 
         Returns:
             str: The URL of the tile.
@@ -535,7 +536,12 @@ class Raster(Base):
             >>> raster = Raster.get_raster(client, uuid="12345678-1234-5678-1234-567812345678")
             >>> raster.get_tile_pbf_url(x=10, y=20, z=1)
         """
+        params = clean_data({
+            'indexes': indexes
+        })
+        query_string = urlencode(params)
         endpoint = urljoin(self.api.base_url, f'{self.endpoint}tiles/{z}/{x}/{y}.pbf')
+        endpoint = urljoin(endpoint, f'?{query_string}')
         return endpoint
     
 

{geobox-1.3.1 → geobox-1.3.3}/geobox/tileset.py

@@ -3,8 +3,8 @@ from typing import Dict, List, Optional, Union, TYPE_CHECKING
 
 from .base import Base
 from .vectorlayer import VectorLayer
+from .view import VectorLayerView
 from .task import Task
-from .enums import TilesetLayerType
 
 if TYPE_CHECKING:
     from . import GeoboxClient 
@@ -35,7 +35,7 @@ class Tileset(Base):
 
 
     @classmethod
-    def create_tileset(cls, api: 'GeoboxClient', name: str, layers: List[Dict], display_name: str = None, description: str = None,
+    def create_tileset(cls, api: 'GeoboxClient', name: str, layers: List[Union['VectorLayer', 'VectorLayerView']], display_name: str = None, description: str = None,
                         min_zoom: int = None, max_zoom: int = None, user_id: int = None) -> 'Tileset':
         """
         Create a new tileset.
@@ -43,9 +43,7 @@ class Tileset(Base):
         Args:
             api (GeoboxClient): The GeoboxClient instance for making requests.
             name (str): The name of the tileset.
-            layers (List[Dict]): The layers of the tileset. a list of dictionaries with the following keys:
-                - layer_type: The type of the layer. valid values are "vector" and "view".
-                - layer_uuid: The uuid of the layer.
+            layers (List['VectorLayer' | 'VectorLayerView']): list of vectorlayer and view objects to add to tileset.
             display_name (str, optional): The display name of the tileset.
             description (str, optional): The description of the tileset.
             min_zoom (int, optional): The minimum zoom level of the tileset.
@@ -59,34 +57,46 @@ class Tileset(Base):
             >>> from geobox import GeoboxClient
             >>> from geobox.tileset import Tileset
             >>> client = GeoboxClient()
-            >>> layers = [
-            ...     {
-            ...     "layer_type": "vector",
-            ...     "layer_uuid": "12345678-1234-5678-1234-567812345678"
-            ...     }
-            ... ]
+            >>> layer = client.get_vector(uuid="12345678-1234-5678-1234-567812345678")
+            >>> view = client.get_view(uuid="12345678-1234-5678-1234-567812345678")
             >>> tileset = Tileset.create_tileset(client, 
             ...                                     name="your_tileset_name", 
             ...                                     display_name="Your Tileset", 
             ...                                     description="Your description", 
             ...                                     min_zoom=0, 
             ...                                     max_zoom=14, 
-            ...                                     layers=layers)
+            ...                                     layers=[layer, view])
             or 
             >>> tileset = client.create_tileset(name="your_tileset_name", 
             ...                                     display_name="Your Tileset", 
             ...                                     description="Your description", 
             ...                                     min_zoom=0, 
             ...                                     max_zoom=14, 
-            ...                                     layers=layers)
+            ...                                     layers=[layer, view])
         """
+        formatted_layers = []
+        for item in layers:
+            if type(item) == VectorLayer:
+                item_type = 'vector'
+
+            elif type(item) == VectorLayerView:
+                item_type = 'view'
+
+            else:
+                continue
+
+            formatted_layers.append({
+                'layer_type': item_type,
+                'layer_uuid': item.uuid
+            })
+
         data = {
             "name": name,
             "display_name": display_name,
             "description": description,
             "min_zoom": min_zoom,
             "max_zoom": max_zoom,
-            "layers": layers,
+            "layers": formatted_layers,
             "user_id": user_id
         }
         return super()._create(api, cls.BASE_ENDPOINT, data, factory_func=lambda api, item: Tileset(api, item['uuid'], item))
@@ -283,7 +293,7 @@ class Tileset(Base):
         super().delete(urljoin(self.BASE_ENDPOINT, f'{self.uuid}/'))
     
     
-    def get_tileset_layers(self, **kwargs) -> List['VectorLayer']:
+    def get_layers(self, **kwargs) -> List['VectorLayer']:
         """
         Retrieves the layers of the tileset with optional parameters.
 
@@ -307,13 +317,14 @@ class Tileset(Base):
         Example:
 
         Returns:
-            List: A list of VectorLayer instances.
+            List: A list of VectorLayer or VectorLayerView instances.
 
         Example:
             >>> from geobox import GeoboxClient
             >>> from geobox.tileset import Tileset
             >>> client = GeoboxClient()
-            >>> tilesets = Tileset.get_tileset_layers()
+            >>> tileset = Tileset.get_tileset(client, uuid="12345678-1234-5678-1234-567812345678")
+            >>> layers = tileset.get_layers()
         """
         params = {
             'f': 'json',
@@ -328,57 +339,82 @@ class Tileset(Base):
             'shared': kwargs.get('shared', False)
         }
         endpoint = urljoin(self.BASE_ENDPOINT, f'{self.uuid}/layers/')
-        return super()._get_list(self.api, endpoint, params, factory_func=lambda api, item: VectorLayer(api, item['uuid'], item['layer_type'], item))
+        return super()._get_list(self.api, endpoint, params, factory_func=lambda api, item: VectorLayer(api, item['uuid'], item['layer_type'], item) if not item['is_view'] else \
+                                                                                            VectorLayerView(api, item['uuid'], item['layer_type'], item)                                    )
 
 
-    def add_layer(self, layer_type: TilesetLayerType, layer_uuid: str) -> None:
+    def add_layer(self, layer: Union['VectorLayer', 'VectorLayerView']) -> None:
         """
         Adds a layer to the tileset.
 
         Args:
-            layer_type (TilesetLayerType): The type of the layer. "vector" or "view".
-            layer_uuid (str): The UUID of the layer.
+            layer (VectorLayer | VectorLayerView): the layer object to add to the tileset
 
         Returns:
             None
 
+        Raises:
+            ValueError: if the layer input is not a 'VectorLayer' or 'VetorLayerview' object
+
         Example:
             >>> from geobox import GeoboxClient
             >>> from geobox.tileset import Tileset
             >>> client = GeoboxClient()
             >>> tileset = Tileset.get_tileset(client, uuid="12345678-1234-5678-1234-567812345678")
-            >>> tileset.add_layer_tileset(layer_type=TilesetLayerType.Vector, layer_uuid="12345678-1234-5678-1234-567812345678")
+            >>> layer = client.get_vector(uuid="12345678-1234-5678-1234-567812345678")
+            >>> tileset.add_layer(layer)
         """
+        if type(layer) == VectorLayer:
+            layer_type = 'vector'
+
+        elif type(layer) == VectorLayerView:
+            layer_type = 'view'
+
+        else:
+            raise ValueError("layer input must be either 'VectorLayer' or 'VetorLayerview' object")
+
         data = {
-            "layer_type": layer_type.value,
-            "layer_uuid": layer_uuid
+            "layer_type": layer_type,
+            "layer_uuid": layer.uuid
         }
 
         endpoint = urljoin(self.endpoint, 'layers/')
         return self.api.post(endpoint, data, is_json=False)
 
 
-    def delete_layer(self, layer_type: TilesetLayerType, layer_uuid: str) -> None:
+    def delete_layer(self, layer: Union['VectorLayer', 'VectorLayerView']) -> None:
         """
         Deletes a layer from the tileset.
 
         Args:
-            layer_type (TilesetLayerType): The type of the layer. "vector" or "view".
-            layer_uuid (str): The UUID of the layer.
+            layer (VectorLayer | VectorLayerView): the layer object to delete from the tileset
 
         Returns:
             None
 
+        Raises:
+            ValueError: if the layer input is not a 'VectorLayer' or 'VetorLayerview' object
+
         Example:
             >>> from geobox import GeoboxClient
             >>> from geobox.tileset import Tileset
             >>> client = GeoboxClient()
             >>> tileset = Tileset.get_tileset(client, uuid="12345678-1234-5678-1234-567812345678")
-            >>> tileset.delete_layer_tileset(layer_type=TilesetLayerType.Vector, layer_uuid="12345678-1234-5678-1234-567812345678")
+            >>> layer = client.get_vector(uuid="12345678-1234-5678-1234-567812345678")
+            >>> tileset.delete_layer(layer)
         """
+        if type(layer) == VectorLayer:
+            layer_type = 'vector'
+
+        elif type(layer) == VectorLayerView:
+            layer_type = 'view'
+
+        else:
+            raise ValueError("layer input must be either 'VectorLayer' or 'VetorLayerview' object")
+
         data = {
-            "layer_type": layer_type.value,
-            "layer_uuid": layer_uuid
+            "layer_type": layer_type,
+            "layer_uuid": layer.uuid
         }
 
         endpoint = urljoin(self.endpoint, 'layers/')

{geobox-1.3.1 → geobox-1.3.3}/geobox/user.py

@@ -2,7 +2,7 @@ from typing import List, Any, TYPE_CHECKING, Union, Dict
 from urllib.parse import urlencode, urljoin
 
 from .base import Base
-from .utils import clean_data
+from .utils import clean_data, xor_encode
 from .enums import UserRole, UserStatus
 from .plan import Plan
 
@@ -113,11 +113,9 @@ class User(Base):
            'return_count': kwargs.get('return_count', False),
            'skip': kwargs.get('skip', 0),
            'limit': kwargs.get('limit', 10),
-           'user_id': kwargs.get('user_id'),
-           'shared': kwargs.get('shared', False)
+           'user_id': kwargs.get('user_id')
         }
         return super()._get_list(api, cls.BASE_ENDPOINT, params, factory_func=lambda api, item: User(api, item['id'], item))
-
     
 
     @classmethod
@@ -174,7 +172,7 @@ class User(Base):
         data = {
             "username": username,
             "email": email,
-            "password": password,
+            "password": xor_encode(password),
             "role": role.value,
             "first_name": first_name,
             "last_name": last_name,
@@ -184,7 +182,6 @@ class User(Base):
         return super()._create(api, cls.BASE_ENDPOINT, data, factory_func=lambda api, item: User(api, item['id'], item))
 
 
-
     @classmethod
     def search_users(cls, api: 'GeoboxClient', search: str = None, skip: int = 0, limit: int = 10) -> List['User']:
         """
@@ -377,7 +374,7 @@ class User(Base):
             >>> user.change_password(new_password='user_new_password')
         """
         data = clean_data({
-            "new_password": new_password
+            "new_password": xor_encode(new_password)
         })
         endpoint = urljoin(self.endpoint, 'change-password')
         self.api.post(endpoint, data, is_json=False)

{geobox-1.3.1 → geobox-1.3.3}/geobox/utils.py

@@ -1,4 +1,11 @@
 from urllib.parse import urlparse, parse_qs, urlencode, urlunparse
+import base64
+
+
+def xor_encode(s, key=42):
+    xor_str = ''.join(chr(ord(c) ^ key) for c in s)
+    encoded_bytes = base64.b64encode(xor_str.encode('utf-8'))
+    return encoded_bytes.decode('utf-8')
 
 
 def clean_data(data: dict) -> dict:

{geobox-1.3.1 → geobox-1.3.3}/geobox/vectorlayer.py

@@ -439,6 +439,36 @@ class VectorLayer(Base):
         return super()._create(self.api, endpoint, data, factory_func=lambda api, item: VectorLayerVersion(api, item['uuid'], item))
 
 
+    def get_versions(self, **kwargs) -> List['VectorLayerVersion']:
+        """
+        Get list of versions of the current vector layer object with optional filtering and pagination.
+
+        Keyword Args:
+            q (str): query filter based on OGC CQL standard. e.g. "field1 LIKE '%GIS%' AND created_at > '2021-01-01'"
+            search (str): search term for keyword-based searching among search_fields or all textual fields if search_fields does not have value. NOTE: if q param is defined this param will be ignored.
+            search_fields (str): comma separated list of fields for searching.
+            order_by (str): comma separated list of fields for sorting results [field1 A|D, field2 A|D, …]. e.g. name A, type D. NOTE: "A" denotes ascending order and "D" denotes descending order.
+            return_count (bool): Whether to return total count. default is False.
+            skip (int): Number of items to skip. default is 0.
+            limit (int): Number of items to return. default is 10.
+            user_id (int): Specific user. privileges required.
+            shared (bool): Whether to return shared versions. default is False.
+
+        Returns:
+            List[VectorLayerVersion] | int: A list of vector layer version instances or the total number of versions.
+
+        Example:
+            >>> from geobox import GeoboxClient
+            >>> from geobox.version import VectorLayerVersion
+            >>> client = GeoboxClient()
+            >>> layer = client.get_vector(uuid="12345678-1234-5678-1234-567812345678")
+            >>> versions = layer.get_versions()
+            or
+            >>> versions = layer.get_versions()
+        """
+        return VectorLayerVersion.get_versions(self.api, layer_id=self.id, **kwargs)
+
+
     @property    
     def wfs(self) -> str:
         """

{geobox-1.3.1 → geobox-1.3.3}/geobox.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: geobox
-Version: 1.3.1
+Version: 1.3.3
 Summary: SDK for Geobox's APIs
 Author-email: Hamid Heydari <heydari.h62@gmail.com>
 License: MIT

{geobox-1.3.1 → geobox-1.3.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "geobox"
-version = "1.3.1"
+version = "1.3.3"
 description = "SDK for Geobox's APIs"
 authors = [
   {name = "Hamid Heydari", email = "heydari.h62@gmail.com"}

{geobox-1.3.1 → geobox-1.3.3}/tests/test_tileset.py

@@ -2,10 +2,12 @@ import pytest
 from urllib.parse import urljoin
 from unittest.mock import patch
 
-from geobox.tileset import Tileset, TilesetLayerType
+from geobox.tileset import Tileset
 from geobox.enums import LayerType
 from geobox.user import User
 from geobox.task import Task
+from geobox.vectorlayer import VectorLayer
+from geobox.view import VectorLayerView
 
 
 def test_init(api, mock_tileset_data):
@@ -35,15 +37,11 @@ def test_getattr(api, mock_tileset_data):
         _ = tileset.nonexistent_attribute
 
 
-def test_create_tileset(api, mock_tileset_data):
+def test_create_tileset(api, mock_tileset_data, mock_vector_data, mock_view_data):
     """Test creating a tileset"""
     api.post.return_value = mock_tileset_data
-    layers = [
-                 {
-                 "layer_type": "vector",
-                 "layer_uuid": "your_layer_uuid"
-                 }
-            ]
+    layers = [VectorLayer(api, uuid=mock_vector_data['uuid'], layer_type=LayerType.MultiPoint, data=mock_vector_data), 
+                VectorLayerView(api, uuid=mock_view_data['uuid'], layer_type=LayerType.MultiPoint, data=mock_view_data)]
     tileset = Tileset.create_tileset(api, 
                                         mock_tileset_data['name'], 
                                         layers=layers,
@@ -54,6 +52,13 @@ def test_create_tileset(api, mock_tileset_data):
     assert isinstance(tileset, Tileset)
     assert tileset.uuid == mock_tileset_data['uuid']
     assert tileset.data == mock_tileset_data
+    api.post.assert_called_once_with('tilesets/', {'name': mock_tileset_data['name'], 
+                                                    'min_zoom': mock_tileset_data['min_zoom'], 
+                                                    'max_zoom': mock_tileset_data['max_zoom'], 
+                                                    'layers': [{'layer_type': 'vector', 
+                                                                'layer_uuid': mock_vector_data['uuid']}, 
+                                                                {'layer_type': 'view', 
+                                                                'layer_uuid': mock_view_data['uuid']}]})
 
 
 def test_get_tilesets(api, mock_tileset_data):
@@ -133,43 +138,63 @@ def test_delete(api, mock_tileset_data):
         api.delete.assert_called_once_with(endpoint)
 
 
-def test_get_tileset_layers(api, mock_tileset_data, mock_vector_data):
+def test_get_layers(api, mock_tileset_data, mock_vector_data, mock_view_data):
     """Test getting tileset layers."""
     mock_layers = [
         mock_vector_data,
-        mock_vector_data
+        mock_view_data
     ]
     with patch.object(api, 'get', return_value=mock_layers):
         tileset = Tileset(api, uuid=mock_tileset_data['uuid'], data=mock_tileset_data)
-        layers = tileset.get_tileset_layers()
+        layers = tileset.get_layers()
         assert len(layers) == 2
-        assert all(layer.layer_type == LayerType.Polygon for layer in layers)
         assert layers[0].name == mock_vector_data['name']
-        assert layers[1].name == mock_vector_data['name']
+        assert type(layers[0]) == VectorLayer
+        assert layers[1].name == mock_view_data['name']
+        assert type(layers[1]) == VectorLayerView
+
     # return count
     with patch.object(api, 'get', return_value=10):
         tileset = Tileset(api, uuid=mock_tileset_data['uuid'], data=mock_tileset_data)
-        layer_count = tileset.get_tileset_layers(return_count=True)
+        layer_count = tileset.get_layers(return_count=True)
         assert layer_count == 10
         api.get.assert_called_once_with(f'{tileset.endpoint}layers/?f=json&return_count=True&skip=0&limit=10&shared=False')
 
 
-def test_add_layer(api, mock_tileset_data, mock_vector_data):
+def test_add_layer(api, mock_tileset_data, mock_vector_data, mock_view_data):
     """Test adding layer to tileset."""
     with patch.object(api, 'post'):
         tileset = Tileset(api, uuid=mock_tileset_data['uuid'], data=mock_tileset_data)
-        tileset.add_layer(layer_type=TilesetLayerType.Vector, layer_uuid=mock_vector_data['uuid'])
+        layer = VectorLayer(api, uuid=mock_vector_data['uuid'], layer_type=LayerType.MultiPoint, data=mock_vector_data)
+        view = VectorLayerView(api, uuid=mock_view_data['uuid'], layer_type=LayerType.MultiPoint, data=mock_view_data)
+        tileset.add_layer(layer=layer)
         endpoint = urljoin(tileset.endpoint, 'layers/')
         api.post.assert_called_once_with(endpoint, {'layer_type': 'vector', 'layer_uuid': mock_vector_data['uuid']}, is_json=False)
+        api.post.reset_mock()
+        tileset.add_layer(layer=view)
+        api.post.assert_called_once_with(endpoint, {'layer_type': 'view', 'layer_uuid': mock_view_data['uuid']}, is_json=False)
+
+        # usupported input
+        with pytest.raises(ValueError):
+            tileset.add_layer(layer='')
 
 
-def test_delete_layer(api, mock_tileset_data, mock_vector_data):
+def test_delete_layer(api, mock_tileset_data, mock_vector_data, mock_view_data):
     """Test deleting layer from tileset."""
     with patch.object(api, 'delete'):
         tileset = Tileset(api, uuid=mock_tileset_data['uuid'], data=mock_tileset_data)
-        tileset.delete_layer(layer_type=TilesetLayerType.Vector, layer_uuid=mock_vector_data['uuid'])
+        layer = VectorLayer(api, uuid=mock_vector_data['uuid'], layer_type=LayerType.MultiPoint, data=mock_vector_data)
+        view = VectorLayerView(api, uuid=mock_view_data['uuid'], layer_type=LayerType.MultiPoint, data=mock_view_data)
+        tileset.delete_layer(layer=layer)
         endpoint = urljoin(tileset.endpoint, 'layers/')
         api.delete.assert_called_once_with(endpoint, {'layer_type': 'vector', 'layer_uuid': mock_vector_data['uuid']}, is_json=False)
+        api.delete.reset_mock()
+        tileset.delete_layer(layer=view)
+        api.delete.assert_called_once_with(endpoint, {'layer_type': 'view', 'layer_uuid': mock_view_data['uuid']}, is_json=False)
+
+        # usupported input
+        with pytest.raises(ValueError):
+            tileset.delete_layer(layer='')
 
 
 def test_share(api, mock_tileset_data, mock_user_data):

{geobox-1.3.1 → geobox-1.3.3}/tests/test_user.py

@@ -5,6 +5,7 @@ from urllib.parse import urljoin, urlencode
 from geobox.user import User
 from geobox.enums import UserRole, UserStatus
 from geobox.plan import Plan
+from geobox.utils import xor_encode
 
 
 def test_init(api, mock_user_data):
@@ -38,7 +39,7 @@ def test_get_users(api, mock_admin_user_data):
     result = User.get_users(api, search='test')
     assert isinstance(result, list)
     assert isinstance(result[0], User)
-    api.get.assert_called_once_with('users/?f=json&search=test&return_count=False&skip=0&limit=10&shared=False')
+    api.get.assert_called_once_with('users/?f=json&search=test&return_count=False&skip=0&limit=10')
 
 def test_create_user(api, mock_admin_user_data):
     api.post.return_value = mock_admin_user_data
@@ -57,7 +58,7 @@ def test_create_user(api, mock_admin_user_data):
     expected_data = {
         "username": mock_admin_user_data['username'],
         "email": mock_admin_user_data['email'],
-        "password": 'password',
+        "password": xor_encode('password'),
         "role": UserRole.ACCOUNT_ADMIN.value,
         "first_name": mock_admin_user_data['first_name'],
         "last_name": mock_admin_user_data['last_name'],
@@ -126,7 +127,7 @@ def test_change_password(api, mock_admin_user_data):
     with patch.object(api, 'post', return_value={"message": "success"}):
         user.change_password('new_password')
         expected_endpoint = urljoin(user.endpoint, 'change-password')
-        api.post.assert_called_once_with(expected_endpoint, {'new_password': 'new_password'}, is_json=False)
+        api.post.assert_called_once_with(expected_endpoint, {'new_password': xor_encode('new_password')}, is_json=False)
 
 
 def test_renew_plan(api, mock_admin_user_data):

{geobox-1.3.1 → geobox-1.3.3}/tests/test_vectorlayer.py

@@ -253,6 +253,17 @@ def test_create_version(api, mock_vector_data, layer_type, mock_version_data):
     api.post.assert_called_once_with(f'{layer.endpoint}versions', {'name': 'design_version', 'display_name': 'Design Version', 'description': 'This layer represents design version.'})
 
 
+@pytest.mark.parametrize("layer_type", [type for type in LayerType])
+def test_get_versions(api, mock_vector_data, layer_type, mock_version_data):
+    layer = VectorLayer(api, uuid=mock_vector_data['uuid'], data=mock_vector_data, layer_type=layer_type)
+    api.get.return_value = [mock_version_data]
+    versions = layer.get_versions()
+    assert len(versions) == 1
+    assert versions[0].uuid == mock_version_data['uuid']
+    assert versions[0].data == mock_version_data
+    api.get.assert_called_once_with(f'vectorLayerVersions/?layer_id={layer.id}&f=json&return_count=False&skip=0&limit=10&shared=False')
+
+
 @pytest.mark.parametrize("layer_type", [type for type in LayerType])
 def test_wfs_access_token(api, mock_vector_data, layer_type):
     layer = VectorLayer(api, uuid=mock_vector_data['uuid'], data=mock_vector_data, layer_type=layer_type)