json-rest-api 1.0.24 → 1.0.26

package/tests/searchschema-merge.test.js

@@ -5,6 +5,7 @@ import {
   cleanTables,
 } from './helpers/test-utils.js'
 import { createSearchSchemaMergeApi } from './fixtures/api-configs.js'
+import { generateSearchSchemaFromSchema } from '../plugins/core/lib/querying-writing/schema-helpers.js'
 
 // Create Knex instance for tests - always use SQLite in-memory
 const knex = knexLib({
@@ -131,4 +132,20 @@ describe('SearchSchema Merge Behavior', () => {
       'Should reject fields that are not defined in searchSchema'
     )
   })
+
+  it('should keep canonical belongsTo field keys searchable alongside relationship aliases', () => {
+    const searchSchema = generateSearchSchemaFromSchema({
+      petId: {
+        type: 'id',
+        belongsTo: 'pets',
+        as: 'pet',
+        search: true
+      }
+    }, null)
+
+    assert.ok(searchSchema.pet, 'relationship alias should stay searchable')
+    assert.equal(searchSchema.pet.actualField, 'petId')
+    assert.ok(searchSchema.petId, 'canonical field key should stay searchable for parent filters')
+    assert.equal(searchSchema.petId.actualField, 'petId')
+  })
 })

package/tests/socketio.test.js

@@ -453,6 +453,68 @@ describe('WebSocket/Socket.IO Plugin', () => {
         socket.close()
       }
     })
+
+    it('should broadcast deferred patch notifications from relationship PATCH', async () => {
+      const token = await createToken({ userId: 'test-user', role: 'user' }, 'test-secret-key')
+
+      const socket = ioClient(`http://localhost:${server.address().port}`, {
+        path: '/api/socket.io',
+        auth: { token }
+      })
+
+      try {
+        await new Promise((resolve, reject) => {
+          socket.on('connect', resolve)
+          socket.on('connect_error', reject)
+          setTimeout(() => reject(new Error('Connection timeout')), 5000)
+        })
+
+        const country = await api.resources.countries.post({
+          inputRecord: createJsonApiDocument('countries', {
+            name: 'Original Country',
+            code: 'OC'
+          }),
+          simplified: false
+        })
+
+        const nextCountry = await api.resources.countries.post({
+          inputRecord: createJsonApiDocument('countries', {
+            name: 'Next Country',
+            code: 'NC'
+          }),
+          simplified: false
+        })
+
+        const book = await api.resources.books.post({
+          inputRecord: createJsonApiDocument(
+            'books',
+            { title: 'Relationship Patch Broadcast' },
+            { country: createRelationship(resourceIdentifier('countries', country.data.id)) }
+          ),
+          simplified: false
+        })
+
+        await new Promise((resolve) => {
+          socket.emit('subscribe', { resource: 'books' }, resolve)
+        })
+
+        const updatePromise = waitForSocketEvent(socket, 'subscription.update', 1000)
+
+        await api.resources.books.patchRelationship({
+          id: book.data.id,
+          relationshipName: 'country',
+          relationshipData: resourceIdentifier('countries', nextCountry.data.id)
+        })
+
+        const notification = await updatePromise
+        assert.equal(notification.type, 'resource.patchd')
+        assert.equal(notification.resource, 'books')
+        assert.equal(String(notification.id), String(book.data.id))
+        assert.equal(notification.action, 'patch')
+      } finally {
+        socket.close()
+      }
+    })
   })
 
   describe('Multiple Subscriptions', () => {

package/tests/storage-mapping.test.js

@@ -42,6 +42,7 @@ describe('Storage mapping', () => {
     await api.use(RestApiPlugin, {
       simplifiedApi: false,
       simplifiedTransport: false,
+      sortableFields: ['id', 'displayName', 'loginCount', 'lastSeenAt', 'countryId', 'externalRef'],
       returnRecordApi: {
         post: 'full',
         put: 'full',

package/tests/test-development-guide.md

@@ -64,9 +64,9 @@ Control what methods return:
 ```javascript
 // In API configuration
 returnFullRecord: {
-  post: true,   // Return created record (needed to get ID)
-  put: false,   // Don't return after replacement
-  patch: false  // Don't return after update
+  post: 'full', // Return created record (needed to get ID)
+  put: 'no',    // Don't return after replacement
+  patch: 'no'   // Don't return after update
 }
 
 // Override per call if needed
@@ -74,7 +74,7 @@ const result = await api.resources.books.put({
   id: bookId,
   inputRecord: putDoc,
   simplified: false,
-  returnFullRecord: false  // Override to not return
+  returnFullRecord: 'no' // Override to not return
 });
 ```
 
@@ -367,4 +367,4 @@ npm test -- tests/relationships.test.js
 4. **validation.test.js** - Schema validation edge cases
 5. **polymorphic.test.js** - Polymorphic relationship tests
 
-Each should follow the patterns established in existing tests.
+Each should follow the patterns established in existing tests.