@api-client/core 0.20.4 → 0.20.6

package/tests/unit/modeling/data_domain_serialization.spec.ts

@@ -583,7 +583,7 @@ test.group('DataDomain Lenient Mode Deserialization', () => {
     }
 
     // Should not throw in lenient mode
-    const restored = new DataDomain(corruptedSerialized, [], 'lenient')
+    const restored = new DataDomain(corruptedSerialized, [], { mode: 'lenient' })
 
     // Should have issues recorded
     assert.isAbove(restored.issues.length, 0)
@@ -611,7 +611,7 @@ test.group('DataDomain Lenient Mode Deserialization', () => {
     }
 
     // Should not throw in lenient mode
-    const restored = new DataDomain(corruptedSerialized, [], 'lenient')
+    const restored = new DataDomain(corruptedSerialized, [], { mode: 'lenient' })
 
     // Should have issues recorded
     assert.isAbove(restored.issues.length, 0)
@@ -641,7 +641,7 @@ test.group('DataDomain Lenient Mode Deserialization', () => {
     }
 
     // Should not throw in lenient mode
-    const restored = new DataDomain(corruptedSerialized, [], 'lenient')
+    const restored = new DataDomain(corruptedSerialized, [], { mode: 'lenient' })
 
     // Should have issues recorded
     assert.isAbove(restored.issues.length, 0)
@@ -680,7 +680,7 @@ test.group('DataDomain Lenient Mode Deserialization', () => {
     }
 
     // Should not throw in lenient mode
-    const restored = new DataDomain(corruptedSerialized, [], 'lenient')
+    const restored = new DataDomain(corruptedSerialized, [], { mode: 'lenient' })
 
     // Should have issues recorded about missing parent node
     assert.isAbove(restored.issues.length, 0)
@@ -706,7 +706,7 @@ test.group('DataDomain Lenient Mode Deserialization', () => {
     const serialized = domain.toJSON()
 
     // Deserialize without providing the foreign dependency
-    const restored = new DataDomain(serialized, [], 'lenient')
+    const restored = new DataDomain(serialized, [], { mode: 'lenient' })
 
     // Should have issues recorded about missing dependency
     assert.isAbove(restored.issues.length, 0)
@@ -742,7 +742,7 @@ test.group('DataDomain Lenient Mode Deserialization', () => {
     }
 
     // Should not throw in lenient mode
-    const restored = new DataDomain(corruptedSerialized, [], 'lenient')
+    const restored = new DataDomain(corruptedSerialized, [], { mode: 'lenient' })
 
     // Should have issues recorded
     assert.isAbove(restored.issues.length, 0)
@@ -797,7 +797,7 @@ test.group('DataDomain Lenient Mode Deserialization', () => {
     }
 
     // Should not throw in lenient mode
-    const restored = new DataDomain(corruptedSerialized, [], 'lenient')
+    const restored = new DataDomain(corruptedSerialized, [], { mode: 'lenient' })
 
     // Should have multiple issues recorded
     assert.isAtLeast(restored.issues.length, 3)
@@ -823,7 +823,7 @@ test.group('DataDomain Lenient Mode Deserialization', () => {
     e1.addProperty({ type: 'string' })
 
     const serialized = domain.toJSON()
-    const restored = new DataDomain(serialized, [], 'strict')
+    const restored = new DataDomain(serialized, [], { mode: 'strict' })
 
     // Should have no issues in strict mode for valid data
     assert.equal(restored.issues.length, 0)
@@ -855,11 +855,11 @@ test.group('DataDomain Lenient Mode Deserialization', () => {
 
     // Should throw in strict mode
     assert.throws(() => {
-      new DataDomain(corruptedSerialized, [], 'strict')
+      new DataDomain(corruptedSerialized, [], { mode: 'strict' })
     })
 
     // Should not throw in lenient mode
-    const restored = new DataDomain(corruptedSerialized, [], 'lenient')
+    const restored = new DataDomain(corruptedSerialized, [], { mode: 'lenient' })
     assert.isAbove(restored.issues.length, 0)
   }).tags(['@modeling', '@serialization', '@lenient', '@strict'])
 
@@ -879,7 +879,7 @@ test.group('DataDomain Lenient Mode Deserialization', () => {
       )
     }
 
-    const restored = new DataDomain(corruptedSerialized, [], 'lenient')
+    const restored = new DataDomain(corruptedSerialized, [], { mode: 'lenient' })
 
     // Should have detailed issue information
     assert.isAbove(restored.issues.length, 0)

package/tests/unit/modeling/domain_entity_associations.spec.ts

@@ -328,3 +328,66 @@ test.group('DomainEntity.hasAssociations()', () => {
     assert.isFalse(entity.hasAssociations())
   })
 })
+
+test.group('DomainEntity.withInheritedAssociations()', () => {
+  test('returns associations from entity and its parents', ({ assert }) => {
+    const dataDomain = new DataDomain()
+    const model = dataDomain.addModel()
+    const parentEntity = model.addEntity({ key: 'parent' })
+    const childEntity = model.addEntity({ key: 'child' })
+
+    const parentAssoc = parentEntity.addAssociation()
+    parentAssoc.info.name = 'parentAssoc'
+
+    const childAssoc = childEntity.addAssociation()
+    childAssoc.info.name = 'childAssoc'
+
+    childEntity.addParent(parentEntity.key)
+
+    const inherited = Array.from(childEntity.withInheritedAssociations())
+    assert.lengthOf(inherited, 2)
+    assert.deepEqual(inherited.map((a) => a.info.name).sort(), ['childAssoc', 'parentAssoc'])
+  })
+
+  test('overrides parent association with the same name', ({ assert }) => {
+    const dataDomain = new DataDomain()
+    const model = dataDomain.addModel()
+    const parentEntity = model.addEntity({ key: 'parent' })
+    const childEntity = model.addEntity({ key: 'child' })
+
+    const parentAssoc = parentEntity.addAssociation()
+    parentAssoc.info.name = 'assoc'
+
+    const childAssoc = childEntity.addAssociation()
+    childAssoc.info.name = 'assoc'
+
+    childEntity.addParent(parentEntity.key)
+
+    const inherited = Array.from(childEntity.withInheritedAssociations())
+    assert.lengthOf(inherited, 1)
+    assert.equal(inherited[0].key, childAssoc.key)
+  })
+
+  test('uses cached inherited associations when domain is readOnly', ({ assert }) => {
+    const dataDomain = new DataDomain(undefined, [], { readOnly: true })
+    const model = dataDomain.addModel()
+    const parentEntity = model.addEntity({ key: 'parent' })
+    const childEntity = model.addEntity({ key: 'child' })
+
+    const parentAssoc = parentEntity.addAssociation()
+    parentAssoc.info.name = 'parentAssoc'
+
+    childEntity.addParent(parentEntity.key)
+
+    const inherited1 = Array.from(childEntity.withInheritedAssociations())
+    assert.lengthOf(inherited1, 1)
+
+    // Add another association bypassing normal immutability rules for test
+    const anotherAssoc = parentEntity.addAssociation()
+    anotherAssoc.info.name = 'anotherAssoc'
+
+    const inherited2 = Array.from(childEntity.withInheritedAssociations())
+    // Should still be length 1 because of the readOnly cache
+    assert.lengthOf(inherited2, 1)
+  })
+})