fix: tests

Author: smrz2001

sdk/packages/model-instance-client/test/lib.test.ts

@@ -29,19 +29,27 @@ const authenticatedDID = await getAuthenticatedDID(new Uint8Array(32))
 describe('getDeterministicInitEventPayload()', () => {
   test('returns the deterministic event payload without unique value by default', () => {
     const model = randomStreamID()
-    const event = getDeterministicInitEventPayload(model, 'did:key:123')
+    const event = getDeterministicInitEventPayload(model, authenticatedDID)
     expect(event.data).toBeNull()
-    expect(event.header.controllers).toEqual(['did:key:123'])
+    expect(event.header.controllers).toEqual([
+      'did:key:z6MkiTBz1ymuepAQ4HEHYSF1H8quG5GLVVQR3djdX3mDooWp',
+    ])
     expect(event.header.model).toBe(model)
     expect(event.header.unique).toBeUndefined()
   })
 
   test('returns the deterministic event payload with the provided unique value', () => {
     const model = randomStreamID()
     const unique = new Uint8Array([0, 1, 2])
-    const event = getDeterministicInitEventPayload(model, 'did:key:123', unique)
+    const event = getDeterministicInitEventPayload(
+      model,
+      authenticatedDID,
+      unique,
+    )
     expect(event.data).toBeNull()
-    expect(event.header.controllers).toEqual(['did:key:123'])
+    expect(event.header.controllers).toEqual([
+      'did:key:z6MkiTBz1ymuepAQ4HEHYSF1H8quG5GLVVQR3djdX3mDooWp',
+    ])
     expect(event.header.model).toBe(model)
     expect(event.header.unique).toBe(unique)
   })
@@ -50,19 +58,23 @@ describe('getDeterministicInitEventPayload()', () => {
 describe('getDeterministicInitEvent()', () => {
   test('returns the deterministic event without unique value by default', () => {
     const model = randomStreamID()
-    const event = getDeterministicInitEvent(model, 'did:key:123')
+    const event = getDeterministicInitEvent(model, authenticatedDID)
     expect(event.data).toBeNull()
-    expect(event.header.controllers).toEqual(['did:key:123'])
+    expect(event.header.controllers).toEqual([
+      'did:key:z6MkiTBz1ymuepAQ4HEHYSF1H8quG5GLVVQR3djdX3mDooWp',
+    ])
     expect(equals(event.header.model, model.bytes)).toBe(true)
     expect(event.header.unique).toBeUndefined()
   })
 
   test('returns the deterministic event with the provided unique value', () => {
     const model = randomStreamID()
     const unique = new Uint8Array([0, 1, 2])
-    const event = getDeterministicInitEvent(model, 'did:key:123', unique)
+    const event = getDeterministicInitEvent(model, authenticatedDID, unique)
     expect(event.data).toBeNull()
-    expect(event.header.controllers).toEqual(['did:key:123'])
+    expect(event.header.controllers).toEqual([
+      'did:key:z6MkiTBz1ymuepAQ4HEHYSF1H8quG5GLVVQR3djdX3mDooWp',
+    ])
     expect(equals(event.header.model, model.bytes)).toBe(true)
     expect(event.header.unique).toBe(unique)
   })
@@ -145,7 +157,7 @@ describe('ModelInstanceClient', () => {
   describe('getEvent() method', () => {
     test('gets a MID event by commit ID', async () => {
       const streamID = randomStreamID()
-      const docEvent = getDeterministicInitEvent(streamID, 'did:key:123')
+      const docEvent = getDeterministicInitEvent(streamID, authenticatedDID)
       const getEventType = jest.fn(() => docEvent)
       const ceramic = { getEventType } as unknown as CeramicClient
       const client = new ModelInstanceClient({ ceramic, did: authenticatedDID })
@@ -167,7 +179,7 @@ describe('ModelInstanceClient', () => {
       const client = new ModelInstanceClient({ ceramic, did: authenticatedDID })
 
       const id = await client.createSingleton({
-        controller: 'did:key:123',
+        controller: authenticatedDID,
         model: randomStreamID(),
       })
       expect(postEventType).toHaveBeenCalled()

sdk/packages/model-instance-client/test/utils.test.ts

@@ -1,36 +1,49 @@
 import { randomStreamID } from '@ceramic-sdk/identifiers'
+import { getAuthenticatedDID } from '@didtools/key-did'
 import { equals } from 'uint8arrays'
 
 import { createInitHeader } from '../src/utils.js'
 
+const authenticatedDID = await getAuthenticatedDID(new Uint8Array(32))
+
 describe('createInitHeader()', () => {
   test('adds random unique bytes by default or when explcitly set to false', () => {
-    const controller = 'did:key:123'
     const model = randomStreamID()
 
-    const header1 = createInitHeader({ controller, model })
+    const header1 = createInitHeader({ controller: authenticatedDID, model })
     expect(header1.unique).toBeInstanceOf(Uint8Array)
-    const header2 = createInitHeader({ controller, model })
+    const header2 = createInitHeader({ controller: authenticatedDID, model })
     expect(header2.unique).toBeInstanceOf(Uint8Array)
     expect(
       equals(header1.unique as Uint8Array, header2.unique as Uint8Array),
     ).toBe(false)
 
-    const header3 = createInitHeader({ controller, model, unique: false })
+    const header3 = createInitHeader({
+      controller: authenticatedDID,
+      model,
+      unique: false,
+    })
     expect(header3.unique).toBeInstanceOf(Uint8Array)
     expect(
       equals(header1.unique as Uint8Array, header3.unique as Uint8Array),
     ).toBe(false)
   })
 
   test('adds the specified unique bytes', () => {
-    const controller = 'did:key:123'
     const model = randomStreamID()
     const unique = new Uint8Array([0, 1, 2])
 
-    const header1 = createInitHeader({ controller, model, unique })
+    const header1 = createInitHeader({
+      controller: authenticatedDID,
+      model,
+      unique,
+    })
     expect(header1.unique).toBeInstanceOf(Uint8Array)
-    const header2 = createInitHeader({ controller, model, unique })
+    const header2 = createInitHeader({
+      controller: authenticatedDID,
+      model,
+      unique,
+    })
     expect(header2.unique).toBeInstanceOf(Uint8Array)
 
     expect(
@@ -39,26 +52,27 @@ describe('createInitHeader()', () => {
   })
 
   test('does not add unique bytes if set to true', () => {
-    const controller = 'did:key:123'
     const model = randomStreamID()
-    const header = createInitHeader({ controller, model, unique: true })
+    const header = createInitHeader({
+      controller: authenticatedDID,
+      model,
+      unique: true,
+    })
     expect(header.unique).toBeUndefined()
   })
 
   test('does not add context and shouldIndex by default', () => {
-    const controller = 'did:key:123'
     const model = randomStreamID()
-    const header = createInitHeader({ controller, model })
+    const header = createInitHeader({ controller: authenticatedDID, model })
     expect(header.context).toBeUndefined()
     expect(header.shouldIndex).toBeUndefined()
   })
 
   test('adds context and shouldIndex if specified', () => {
-    const controller = 'did:key:123'
     const model = randomStreamID()
     const context = randomStreamID()
     const header = createInitHeader({
-      controller,
+      controller: authenticatedDID,
       model,
       context,
       shouldIndex: true,

sdk/packages/test-vectors/scripts/create-test-vectors.ts

@@ -135,16 +135,13 @@ for (const [controllerType, createController] of Object.entries(
   const controller = await createController()
 
   // Deterministic (init) event
-  const validDeterministicEvent = getDeterministicInitEvent(
-    model,
-    controller.id,
-  )
+  const validDeterministicEvent = getDeterministicInitEvent(model, controller)
 
   // Signed init event
   const validInitPayload = InitEventPayload.encode({
     data: { test: true },
     header: createInitHeader({
-      controller: controller.id,
+      controller,
       model,
       unique: new Uint8Array([0, 1, 2, 3]),
     }),