Merge pull request #36 from agile-learning-institute/syntax-error-fix

Syntax error fix

Author: FlatBallFlyer

configurator/services/dictionary_services.py

@@ -137,7 +137,7 @@ def __init__(self, name: str, property: dict):
         self.enums = property.get("enums", None)
         self.additional_properties = property.get("additional_properties", False)
         self.properties = {}
-        self.items = {}}
+        self.items = {}
 
         # Initialize properties if this is an object type
         if self.type == "object":

tests/services/test_dictionary_service_operations.py

@@ -140,7 +140,7 @@ def test_init_with_missing_values(self):
         self.assertFalse(prop.required)
         self.assertFalse(prop.additional_properties)
         self.assertEqual(prop.properties, {})
-        self.assertIsNone(prop.items)
+        self.assertEqual(prop.items, {})
 
     def test_to_dict_with_ref(self):
         """Test to_dict method for ref property"""

tests/test_cases/playground/configurations/sample.yaml

@@ -18,6 +18,7 @@ versions:
       unique: false
   migrations: []
   test_data: sample.1.0.0.1.json
+  _locked: false
 - version: 1.0.1.2
   drop_indexes:
   - name: nameIndex
@@ -30,4 +31,5 @@ versions:
   migrations:
   - first_last_to_full_name.json
   test_data: sample.1.0.1.2.json
+  _locked: false
 _locked: true

tests/test_cases/playground/dictionaries/sample.1.0.0.yaml

@@ -4,6 +4,7 @@ root:
   description: A simple collection for testing
   type: object
   required: false
+  additional_properties: false
   properties:
     _id:
       description: The unique identifier for the media
@@ -22,4 +23,3 @@ root:
       type: enum
       required: true
       enums: default_status
-  additionalProperties: false

tests/test_cases/playground/dictionaries/sample.1.0.1.yaml

@@ -4,6 +4,7 @@ root:
   description: A simple collection for testing
   type: object
   required: false
+  additional_properties: false
   properties:
     _id:
       description: The unique identifier for the media
@@ -18,4 +19,3 @@ root:
       type: enum
       required: true
       enums: default_status
-  additionalProperties: false

tests/test_cases/playground/enumerators/enumerations.0.yaml

@@ -1,3 +1,4 @@
 version: 0
 enumerators: {}
 _locked: false
+file_name: enumerations.0.yaml

tests/test_cases/playground/enumerators/enumerations.1.yaml

@@ -6,3 +6,4 @@ enumerators:
   test_enum:
     foo: bar
 _locked: false
+file_name: enumerations.1.yaml

tests/test_cases/playground/enumerators/enumerations.2.yaml

@@ -1,3 +1,4 @@
 version: 2
 enumerators: {}
 _locked: false
+file_name: enumerations.2.yaml

tests/test_cases/playground/types/appointment.yaml

@@ -1,16 +1,15 @@
+file_name: appointment.yaml
+_locked: true
 root:
-  file_name: appointment.yaml
-  _locked: true
   description: A date/time range
-  required: false
   type: object
   properties:
     from:
       description: Starting Date/Time
-      required: true
       type: date-time
+      required: true
     to:
       description: Ending Date/Time
-      required: true
       type: date-time
-  additionalProperties: false
+      required: true
+  required: false

tests/test_cases/playground/types/breadcrumb.yaml

@@ -1,24 +1,23 @@
+file_name: breadcrumb.yaml
+_locked: true
 root:
-  file_name: breadcrumb.yaml
-  _locked: true
   description: A tracking breadcrumb
-  required: false
   type: object
   properties:
     from_ip:
       description: Http Request remote IP address
-      required: true
       type: ip_address
+      required: true
     by_user:
       description: ID Of User
-      required: true
       type: word
+      required: true
     at_time:
       description: The date-time when last updated
-      required: true
       type: date-time
+      required: true
     correlation_id:
       description: The logging correlation ID of the update transaction
-      required: true
       type: word
-  additionalProperties: false
+      required: true
+  required: false