[DOJOT-75] [DOJOT-76] Feature - Unique Labels

DeviceManager/DatabaseModels.py

@@ -68,7 +68,7 @@ class DeviceTemplate(db.Model):
     __tablename__ = 'templates'
 
     id = db.Column(db.Integer, db.Sequence('template_id'), primary_key=True)
-    label = db.Column(db.String(128), nullable=False)
+    label = db.Column(db.String(128), nullable=False, unique=True)
     created = db.Column(db.DateTime, default=datetime.now)
     updated = db.Column(db.DateTime, onupdate=datetime.now)
 
@@ -99,7 +99,7 @@ class Device(db.Model):
     __tablename__ = 'devices'
 
     id = db.Column(db.String(8), unique=True, nullable=False, primary_key=True)
-    label = db.Column(db.String(128), nullable=False)
+    label = db.Column(db.String(128), nullable=False, unique=True)
     created = db.Column(db.DateTime, default=datetime.now)
     updated = db.Column(db.DateTime, onupdate=datetime.now)
 

DeviceManager/TemplateHandler.py

@@ -197,9 +197,9 @@ def create_template(params, token):
         try:
             db.session.commit()
             LOGGER.debug(f" Created template in database")
-        except IntegrityError as e:
-            LOGGER.error(f' {e}')
-            raise HTTPRequestError(400, 'Template attribute constraints are violated by the request')
+        except IntegrityError as error:
+            LOGGER.error(f' {error}')
+            handle_consistency_exception(error)
 
         results = {
             'template': template_schema.dump(loaded_template),

README.md

@@ -133,7 +133,7 @@ The information model used for both “real” and virtual devices is as followi
 | **attrs**     | Map of attributes                                       | read-only  | No       | Map of device's attributes (check the attributes in the next table)
 | **created**   | DateTime (with timezone and µs precision) in ISO format | read-only  | No       | Device creation time.
 | **id**        | String (length of 8 bytes)                              | read-only  | No       | Unique identifier for the device.
-| **label**     | String (length of 128 bytes)                            | read-write | Yes      | An user-defined label to facilitate the device's identification.
+| **label**     | String (length of 128 bytes)                            | read-write | Yes      | An user-defined label to facilitate the device's identification. The label is unique for this device.
 | **templates** | Strings list                                            | read-only  | No       | List of template IDs used by the device.
 | **updated**   | DateTime (with timezone and µs precision) in ISO format | read-only  | No       | Device last update time.
 
@@ -214,7 +214,7 @@ An example of such structure would be:
 ### **Template**
 
 All devices are based on a **template**, which can be thought as a blueprint: all devices built
-using the same template will have the same characteristics. Templates in dojot have one label (any
+using the same template will have the same characteristics. Templates in dojot have one unique label (any
 alphanumeric sequence), a list of attributes which will hold all the device emitted information, and
 optionally a few special attributes which will indicate how the device communicates, including
 transmission methods (protocol, ports, etc.) and message formats.
@@ -239,7 +239,7 @@ The information model used for templates is:
 | **created**      | DateTime (with timezone and µs precision) in ISO format | read-only  | No       | Device creation time.
 | **data_attrs**   | Map of attributes                                       | read-write | No       | Stores attributes with the types `dynamic`, `static` and `actuator`.
 | **id**           | String (length of 8 bytes)                              | read-write | No       | Unique identifier for the template.
-| **label**        | String (length of 128 bytes)                            | read-write | Yes      | An user-defined label to facilitate the template's identification.
+| **label**        | String (length of 128 bytes)                            | read-write | Yes      | An user-defined label to facilitate the template's identification. The label is unique for this template.
 | **updated**      | DateTime (with timezone and µs precision) in ISO format | read-only  | No       | Device last update time.
 
 An example template structure:

docs/apiary.apib

@@ -100,7 +100,6 @@ Register a new template
                 }
             }
 
-
 + Response 400 (application/json)
 
             {
@@ -112,6 +111,19 @@ Register a new template
                 "message": "failed to parse input"
             }
 
++ Response 400 (application/json)
+
+            {
+                "message": "duplicate key value violates unique constraint \"templates_label_key\"",
+                "status": 400
+            }
+
++ Response 400 (application/json)
+
+            {
+                "message": "duplicate key value violates unique constraint \"attrs_template_id_type_label_key\"",
+                "status": 400
+            }
 
 ### Get the current list of templates [GET /template{?page_size,page_num,attr_format,attr,attr_type,label,sortBy}]
 
@@ -498,6 +510,20 @@ updated.
                 "message": "No such template: 123",
                 "status": 404
             }
+
++ Response 400 (application/json)
+
+            {
+                "message": "duplicate key value violates unique constraint \"templates_label_key\"",
+                "status": 400
+            }
+
++ Response 400 (application/json)
+
+            {
+                "message": "duplicate key value violates unique constraint \"attrs_template_id_type_label_key\"",
+                "status": 400
+            }
 
 ### Delete template [DELETE /template/{id}]
 
@@ -648,6 +674,13 @@ attributes to be applied to this device.
                 "status": 400
             }
 
++ Response 400 (application/json)
+
+            {
+                "message": "duplicate key value violates unique constraint \"devices_label_key\"",
+                "status": 400
+            }
+
 + Response 404 (application/json)
 
             {
@@ -750,6 +783,13 @@ attributes to be applied to this device.
                 "status": 400
             }
 
++ Response 400 (application/json)
+
+            {
+                "message": "duplicate key value violates unique constraint \"devices_label_key\"",
+                "status": 400
+            }
+
 + Response 404 (application/json)
 
             {
@@ -1160,6 +1200,12 @@ Updates a device's configuration
                 "status": 400
             }
 
++ Response 400 (application/json)
+
+            {
+                "message": "duplicate key value violates unique constraint \"devices_label_key\"",
+                "status": 400
+            }
 
 ### Configure device [PUT /device/{id}/actuate]
 

migrations/versions/944f3122f51e_.py

@@ -0,0 +1,30 @@
+"""empty message
+
+Revision ID: 944f3122f51e
+Revises: fabf2ca39860
+Create Date: 2022-08-12 13:44:41.398334
+
+"""
+from alembic import op
+import sqlalchemy as sa
+
+
+# revision identifiers, used by Alembic.
+revision = '944f3122f51e'
+down_revision = 'fabf2ca39860'
+branch_labels = None
+depends_on = None
+
+
+def upgrade():
+    # ### commands auto generated by Alembic - please adjust! ###
+    op.create_unique_constraint(None, 'devices', ['label'])
+    op.create_unique_constraint(None, 'templates', ['label'])
+    # ### end Alembic commands ###
+
+
+def downgrade():
+    # ### commands auto generated by Alembic - please adjust! ###
+    op.drop_constraint(None, 'templates', type_='unique')
+    op.drop_constraint(None, 'devices', type_='unique')
+    # ### end Alembic commands ###

tests/dredd-hooks/operation_hook.py

@@ -1,5 +1,6 @@
 from __future__ import absolute_import
 import dredd_hooks as hooks
+from uuid import uuid4
 import json
 import re
 
@@ -17,13 +18,15 @@ def __init__(self, data):
         self.args = data['args']
         self.data = data['body']
 
+def generate_unique_label(prefix):
+    return f"{prefix}_{uuid4()}"
 
 def sort_attributes(device, attribute):
     device[attribute] = sorted(device[attribute], key=lambda k: k['label'])
 
 def create_sample_template():
     template = {
-        "label": "SensorModel",
+        "label": generate_unique_label("SensorModel"),
         "attrs": [
             {
                 "label": "temperature",
@@ -68,7 +71,7 @@ def create_sample_template():
 
 def create_actuator_template():
     template = {
-        "label": "SensorModel",
+        "label": generate_unique_label("SensorModel"),
         "attrs": [
             {
                 "label": "temperature",
@@ -168,7 +171,7 @@ def create_single_device(transaction):
         transaction['proprietary'] = {}
     transaction['proprietary']['template_id'] = template_id
     device = {
-        "label": "test_device",
+        "label": generate_unique_label("test_device"),
         "templates": [template_id]
     }
     req = {
@@ -202,7 +205,7 @@ def create_actuator_device(transaction):
         transaction['proprietary'] = {}
     transaction['proprietary']['template_id'] = template_id
     device = {
-        "label": "test_device",
+        "label": generate_unique_label("test_device"),
         "templates": [template_id]
     }
     req = {