@things-factory/scheduler 10.1.38 → 10.1.39

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@things-factory/scheduler",
-  "version": "10.1.38",
+  "version": "10.1.39",
   "main": "dist-server/index.js",
   "browser": "dist-client/index.js",
   "things-factory": true,
@@ -27,11 +27,12 @@
     "migration:create": "node ../../node_modules/typeorm/cli.js migration:create ./server/migrations/migration"
   },
   "dependencies": {
+    "@operato/data-grist": "^10.19.0",
     "@operato/graphql": "^10.0.0",
     "@operato/shell": "^10.0.0",
-    "@things-factory/auth-base": "^10.1.38",
-    "@things-factory/scheduler-client": "^10.1.38",
-    "@things-factory/shell": "^10.1.38"
+    "@things-factory/auth-base": "^10.1.39",
+    "@things-factory/scheduler-client": "^10.1.39",
+    "@things-factory/shell": "^10.1.39"
   },
-  "gitHead": "b947349fe1c27d11038b592ceadaba336f14785b"
+  "gitHead": "fe5257ad8995fe7e79275e287f36e95b5fd32d1c"
 }

package/tests/select-schedules.test.ts

@@ -0,0 +1,64 @@
+import { readFileSync } from 'fs'
+import { join } from 'path'
+
+import { selectSchedules } from '../server/service/schedule/select-schedules'
+
+/*
+ * The schedule list's title bar search used to be dropped: the door read only its named filters, so
+ * the term came back as the whole list, and the reader took it for "nothing like that here".
+ */
+
+const schedules = [
+  {
+    name: 'nightly-close',
+    client: { application: 'operato-plant', group: 'mes', type: 'batch', key: 'k1', operation: 'close' }
+  },
+  {
+    name: 'hourly-sync',
+    client: { application: 'operato-twin', group: 'mirror', type: 'poll', key: 'k2', operation: 'sync' }
+  },
+  {
+    name: 'kpi-rollup',
+    client: { application: 'operato-plant', group: 'kpi', type: 'batch', key: 'k3', operation: 'rollup' }
+  }
+]
+
+const names = (result: { items: any[] }) => result.items.map(schedule => schedule.name)
+
+describe('selectSchedules', () => {
+  it('finds a schedule by its name', () => {
+    expect(names(selectSchedules(schedules, { search: 'HOURLY' }))).toEqual(['hourly-sync'])
+  })
+
+  it('finds a schedule by who registered it', () => {
+    expect(names(selectSchedules(schedules, { search: 'twin' }))).toEqual(['hourly-sync'])
+    expect(names(selectSchedules(schedules, { search: 'rollup' }))).toEqual(['kpi-rollup'])
+  })
+
+  it('narrows the named filters further, not instead of them', () => {
+    const result = selectSchedules(schedules, {
+      search: 'batch',
+      filters: [
+        { name: 'application', operator: 'i_like', value: 'plant' },
+        { name: 'group', operator: 'i_like', value: 'kpi' }
+      ]
+    })
+    expect(names(result)).toEqual(['kpi-rollup'])
+  })
+
+  it('answers the whole list for no term, or one of spaces', () => {
+    expect(selectSchedules(schedules, {}).total).toBe(3)
+    expect(selectSchedules(schedules, { search: '   ' }).total).toBe(3)
+  })
+
+  it('counts what matched, not what fits on the page', () => {
+    const result = selectSchedules(schedules, { search: 'operato', pagination: { page: 2, limit: 2 } })
+    expect(result.total).toBe(3)
+    expect(names(result)).toEqual(['kpi-rollup'])
+  })
+
+  it('is what the door answers with', () => {
+    const door = readFileSync(join(__dirname, '../server/service/schedule/schedule-query.ts'), 'utf-8')
+    expect(door).toMatch(/return selectSchedules\(await getSchedules\(\{\}\), params\)/)
+  })
+})