@switchbot/homebridge-switchbot 5.0.0-beta.70 → 5.0.0-beta.72

package/test/platform.integration.spec.ts

@@ -0,0 +1,55 @@
+import { describe, it, expect } from 'vitest'
+import { SwitchBotHAPPlatform } from '../src/platform'
+
+// Minimal mock logger
+const log = {
+  info: () => {},
+  warn: () => {},
+  error: () => {},
+  debug: () => {},
+}
+
+describe('platform integration (HAP)', () => {
+  it('registers HAP accessories using Homebridge API', async () => {
+    const registered: any[] = []
+
+    // Minimal hap stub
+    const hap = {
+      uuid: { generate: (s: string) => `uuid-${s}` },
+      Service: { Switch: 'Switch', Lightbulb: 'Lightbulb' },
+      Characteristic: { On: 'On', Brightness: 'Brightness' },
+    }
+
+    // Simple platformAccessory constructor
+    function PlatformAccessory(name: string, uuid: string) {
+      this.displayName = name
+      this.UUID = uuid
+      this.services = []
+      this.getService = (type: any) => this.services.find((s: any) => s.type === type)
+      this.addService = (type: any) => {
+        const s: any = { type, characteristics: {} }
+        this.services.push(s)
+        return s
+      }
+    }
+
+    const api: any = {
+      hap,
+      platformAccessory: PlatformAccessory as any,
+      registerPlatformAccessories: (_p: string, _n: string, accs: any[]) => registered.push(...accs),
+      on: (_ev: string, cb: Function) => { /* do not auto-run */ },
+    }
+
+    const config: any = { devices: [{ id: 'dev1', type: 'light', name: 'L1' }] }
+
+    const platform = new SwitchBotHAPPlatform(log as any, config, api)
+
+    // call loadDevices directly to control flow
+    await (platform as any).loadDevices()
+
+    // Expect accessory created and registered
+    expect(registered.length).toBeGreaterThanOrEqual(1)
+    const keys = Array.from((platform as any).accessories.keys())
+    expect(keys.some((k: string) => k.includes('dev1'))).toBeTruthy()
+  })
+})

package/test/switchbot-client-debounce.spec.ts

@@ -0,0 +1,131 @@
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
+import { SwitchBotClient } from '../src/switchbotClient'
+
+describe('SwitchBotClient debounce/coalesce', () => {
+  let origFetch: any
+
+  beforeEach(() => {
+    origFetch = (global as any).fetch
+    vi.useFakeTimers()
+  })
+
+  afterEach(() => {
+    (global as any).fetch = origFetch
+    vi.restoreAllMocks()
+    vi.useRealTimers()
+  })
+
+  it('coalesces rapid writes for the same device (last write wins)', async () => {
+    const respBody = { status: 'ok' }
+    const fetchMock = vi.fn(async (url: string, opts: any) => ({ json: async () => respBody }))
+    ;(global as any).fetch = fetchMock
+
+    // Set a short debounce in config to keep the test fast
+    const c = new SwitchBotClient({ openApiToken: 't', writeDebounceMs: 10 } as any)
+    await c.init()
+
+    const body1 = { command: 'one' }
+    const body2 = { command: 'two' }
+
+    const p1 = c.setDeviceState('d-debounce', body1)
+    const p2 = c.setDeviceState('d-debounce', body2)
+
+    // Fast-forward timers past debounce window
+    vi.advanceTimersByTime(20)
+    // Allow pending microtasks to resolve
+    await Promise.resolve()
+    await Promise.resolve()
+
+    const [r1, r2] = await Promise.all([p1, p2])
+    expect(r1).toEqual(respBody)
+    expect(r2).toEqual(respBody)
+
+    // Only one network call should have been made
+    expect(fetchMock).toHaveBeenCalledTimes(1)
+    const calledBody = JSON.parse(fetchMock.mock.calls[0][1].body)
+    expect(calledBody).toEqual(body2)
+  })
+
+  it('does not coalesce writes for different devices', async () => {
+    const respBody = { status: 'ok' }
+    const fetchMock = vi.fn(async (url: string, opts: any) => ({ json: async () => respBody }))
+    ;(global as any).fetch = fetchMock
+
+    const c = new SwitchBotClient({ openApiToken: 't', writeDebounceMs: 10 } as any)
+    await c.init()
+
+    const p1 = c.setDeviceState('d-A', { command: 'a' })
+    const p2 = c.setDeviceState('d-B', { command: 'b' })
+
+    vi.advanceTimersByTime(20)
+    await Promise.resolve()
+    await Promise.resolve()
+
+    await Promise.all([p1, p2])
+
+    expect(fetchMock).toHaveBeenCalledTimes(2)
+    const b1 = JSON.parse(fetchMock.mock.calls[0][1].body)
+    const b2 = JSON.parse(fetchMock.mock.calls[1][1].body)
+    expect(b1).toEqual({ command: 'a' })
+    expect(b2).toEqual({ command: 'b' })
+  })
+
+  it('coalesces multiple rapid writes (three) and resolves all promises with last response', async () => {
+    const respBody = { status: 'ok', val: 'last' }
+    const fetchMock = vi.fn(async (url: string, opts: any) => ({ json: async () => respBody }))
+    ;(global as any).fetch = fetchMock
+
+    const c = new SwitchBotClient({ openApiToken: 't', writeDebounceMs: 10 } as any)
+    await c.init()
+
+    const p1 = c.setDeviceState('d-debounce', { command: 'one' })
+    const p2 = c.setDeviceState('d-debounce', { command: 'two' })
+    const p3 = c.setDeviceState('d-debounce', { command: 'three' })
+
+    vi.advanceTimersByTime(20)
+    await Promise.resolve()
+    await Promise.resolve()
+
+    const [r1, r2, r3] = await Promise.all([p1, p2, p3])
+    expect(r1).toEqual(respBody)
+    expect(r2).toEqual(respBody)
+    expect(r3).toEqual(respBody)
+
+    expect(fetchMock).toHaveBeenCalledTimes(1)
+    const calledBody = JSON.parse(fetchMock.mock.calls[0][1].body)
+    expect(calledBody).toEqual({ command: 'three' })
+  })
+
+  it('separate writes outside debounce window produce multiple network calls', async () => {
+    const respBody1 = { status: 'ok', seq: 1 }
+    const respBody2 = { status: 'ok', seq: 2 }
+    const fetchMock = vi.fn()
+      .mockImplementationOnce(async () => ({ json: async () => respBody1 }))
+      .mockImplementationOnce(async () => ({ json: async () => respBody2 }))
+    ;(global as any).fetch = fetchMock
+
+    const c = new SwitchBotClient({ openApiToken: 't', writeDebounceMs: 10 } as any)
+    await c.init()
+
+    const p1 = c.setDeviceState('d-debounce', { command: 'one' })
+    vi.advanceTimersByTime(20)
+    await Promise.resolve()
+    await Promise.resolve()
+
+    const p2 = c.setDeviceState('d-debounce', { command: 'two' })
+    vi.advanceTimersByTime(20)
+    await Promise.resolve()
+    await Promise.resolve()
+
+    const r1 = await p1
+    const r2 = await p2
+
+    expect(fetchMock).toHaveBeenCalledTimes(2)
+    const b1 = JSON.parse(fetchMock.mock.calls[0][1].body)
+    const b2 = JSON.parse(fetchMock.mock.calls[1][1].body)
+    expect(b1).toEqual({ command: 'one' })
+    expect(b2).toEqual({ command: 'two' })
+    expect(r1).toEqual(respBody1)
+    expect(r2).toEqual(respBody2)
+  })
+})

package/test/switchbot-client-openapi.spec.ts

@@ -0,0 +1,56 @@
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
+import { SwitchBotClient } from '../src/switchbotClient'
+
+describe('SwitchBotClient OpenAPI fallback', () => {
+  let origFetch: any
+
+  beforeEach(() => {
+    origFetch = (global as any).fetch
+  })
+
+  afterEach(() => {
+    (global as any).fetch = origFetch
+    vi.restoreAllMocks()
+  })
+
+  it('throws when no client and no token', async () => {
+    const c = new SwitchBotClient({} as any)
+    await c.init()
+    await expect(c.getDevice('nope')).rejects.toThrow('No SwitchBot client available')
+  })
+
+  it('uses OpenAPI when token provided for getDevice', async () => {
+    const fakeResp = { body: { device: { id: 'd1', state: 'ok' } } }
+    const fetchMock = vi.fn(async (url: string, opts: any) => ({ json: async () => fakeResp }))
+    ;(global as any).fetch = fetchMock
+
+    const c = new SwitchBotClient({ openApiToken: 'token-123' } as any)
+    await c.init()
+    const res = await c.getDevice('d1')
+    expect(res).toEqual(fakeResp)
+    expect(fetchMock).toHaveBeenCalledWith('https://api.switch-bot.com/v1.0/devices/d1', { headers: { Authorization: 'token-123' } })
+  })
+
+  it('posts commands to OpenAPI for setDeviceState', async () => {
+    const respBody = { status: 'ok' }
+    const fetchMock = vi.fn(async (url: string, opts: any) => ({ json: async () => respBody }))
+    ;(global as any).fetch = fetchMock
+
+    const c = new SwitchBotClient({ openApiToken: 't' } as any)
+    await c.init()
+    const body = { command: 'turnOn', parameter: 'default', commandType: 'command' }
+    const out = await c.setDeviceState('d2', body)
+    expect(out).toEqual(respBody)
+    expect(fetchMock).toHaveBeenCalledWith('https://api.switch-bot.com/v1.0/devices/d2/commands', expect.objectContaining({ method: 'POST' }))
+    const calledBody = JSON.parse((fetchMock.mock.calls[0][1].body))
+    expect(calledBody).toEqual(body)
+  })
+
+  it('propagates fetch errors', async () => {
+    const fetchMock = vi.fn(async () => { throw new Error('network') })
+    ;(global as any).fetch = fetchMock
+    const c = new SwitchBotClient({ openApiToken: 't' } as any)
+    await c.init()
+    await expect(c.getDevice('x')).rejects.toThrow('network')
+  })
+})

package/test/switchbotClient.spec.ts

@@ -0,0 +1,10 @@
+import { describe, it, expect } from 'vitest'
+import { SwitchBotClient } from '../src/switchbotClient'
+
+describe('SwitchBotClient', () => {
+  it('init does not throw and getDevice throws when no token/client', async () => {
+    const c = new SwitchBotClient({})
+    await c.init()
+    await expect(c.getDevice('nope')).rejects.toThrow()
+  })
+})

package/test/utils.spec.ts

@@ -0,0 +1,20 @@
+import { describe, it, expect } from 'vitest'
+import { createPlatformProxy } from '../src/utils'
+
+describe('createPlatformProxy', () => {
+  it('returns Matter platform when preferMatter and enabled', () => {
+    class HAP {}
+    class Matter {}
+    const ProxyCtor: any = createPlatformProxy(HAP, Matter)
+    const impl = new (ProxyCtor as any)(console, { preferMatter: true, enableMatter: true }, undefined)
+    expect(impl).toBeInstanceOf(Matter)
+  })
+
+  it('returns HAP platform when preferMatter is false', () => {
+    class HAP {}
+    class Matter {}
+    const ProxyCtor: any = createPlatformProxy(HAP, Matter)
+    const impl = new (ProxyCtor as any)(console, { preferMatter: false, enableMatter: true }, undefined)
+    expect(impl).toBeInstanceOf(HAP)
+  })
+})

package/vitest.config.ts

@@ -0,0 +1,7 @@
+import { defineConfig } from 'vitest/config'
+
+export default defineConfig({
+  test: {
+    include: ['test/**/*.spec.ts'],
+  },
+})

package/coverage/base.css

@@ -1,224 +0,0 @@
-body, html {
-  margin:0; padding: 0;
-  height: 100%;
-}
-body {
-    font-family: Helvetica Neue, Helvetica, Arial;
-    font-size: 14px;
-    color:#333;
-}
-.small { font-size: 12px; }
-*, *:after, *:before {
-  -webkit-box-sizing:border-box;
-     -moz-box-sizing:border-box;
-          box-sizing:border-box;
-  }
-h1 { font-size: 20px; margin: 0;}
-h2 { font-size: 14px; }
-pre {
-    font: 12px/1.4 Consolas, "Liberation Mono", Menlo, Courier, monospace;
-    margin: 0;
-    padding: 0;
-    -moz-tab-size: 2;
-    -o-tab-size:  2;
-    tab-size: 2;
-}
-a { color:#0074D9; text-decoration:none; }
-a:hover { text-decoration:underline; }
-.strong { font-weight: bold; }
-.space-top1 { padding: 10px 0 0 0; }
-.pad2y { padding: 20px 0; }
-.pad1y { padding: 10px 0; }
-.pad2x { padding: 0 20px; }
-.pad2 { padding: 20px; }
-.pad1 { padding: 10px; }
-.space-left2 { padding-left:55px; }
-.space-right2 { padding-right:20px; }
-.center { text-align:center; }
-.clearfix { display:block; }
-.clearfix:after {
-  content:'';
-  display:block;
-  height:0;
-  clear:both;
-  visibility:hidden;
-  }
-.fl { float: left; }
-@media only screen and (max-width:640px) {
-  .col3 { width:100%; max-width:100%; }
-  .hide-mobile { display:none!important; }
-}
-
-.quiet {
-  color: #7f7f7f;
-  color: rgba(0,0,0,0.5);
-}
-.quiet a { opacity: 0.7; }
-
-.fraction {
-  font-family: Consolas, 'Liberation Mono', Menlo, Courier, monospace;
-  font-size: 10px;
-  color: #555;
-  background: #E8E8E8;
-  padding: 4px 5px;
-  border-radius: 3px;
-  vertical-align: middle;
-}
-
-div.path a:link, div.path a:visited { color: #333; }
-table.coverage {
-  border-collapse: collapse;
-  margin: 10px 0 0 0;
-  padding: 0;
-}
-
-table.coverage td {
-  margin: 0;
-  padding: 0;
-  vertical-align: top;
-}
-table.coverage td.line-count {
-    text-align: right;
-    padding: 0 5px 0 20px;
-}
-table.coverage td.line-coverage {
-    text-align: right;
-    padding-right: 10px;
-    min-width:20px;
-}
-
-table.coverage td span.cline-any {
-    display: inline-block;
-    padding: 0 5px;
-    width: 100%;
-}
-.missing-if-branch {
-    display: inline-block;
-    margin-right: 5px;
-    border-radius: 3px;
-    position: relative;
-    padding: 0 4px;
-    background: #333;
-    color: yellow;
-}
-
-.skip-if-branch {
-    display: none;
-    margin-right: 10px;
-    position: relative;
-    padding: 0 4px;
-    background: #ccc;
-    color: white;
-}
-.missing-if-branch .typ, .skip-if-branch .typ {
-    color: inherit !important;
-}
-.coverage-summary {
-  border-collapse: collapse;
-  width: 100%;
-}
-.coverage-summary tr { border-bottom: 1px solid #bbb; }
-.keyline-all { border: 1px solid #ddd; }
-.coverage-summary td, .coverage-summary th { padding: 10px; }
-.coverage-summary tbody { border: 1px solid #bbb; }
-.coverage-summary td { border-right: 1px solid #bbb; }
-.coverage-summary td:last-child { border-right: none; }
-.coverage-summary th {
-  text-align: left;
-  font-weight: normal;
-  white-space: nowrap;
-}
-.coverage-summary th.file { border-right: none !important; }
-.coverage-summary th.pct { }
-.coverage-summary th.pic,
-.coverage-summary th.abs,
-.coverage-summary td.pct,
-.coverage-summary td.abs { text-align: right; }
-.coverage-summary td.file { white-space: nowrap;  }
-.coverage-summary td.pic { min-width: 120px !important;  }
-.coverage-summary tfoot td { }
-
-.coverage-summary .sorter {
-    height: 10px;
-    width: 7px;
-    display: inline-block;
-    margin-left: 0.5em;
-    background: url(sort-arrow-sprite.png) no-repeat scroll 0 0 transparent;
-}
-.coverage-summary .sorted .sorter {
-    background-position: 0 -20px;
-}
-.coverage-summary .sorted-desc .sorter {
-    background-position: 0 -10px;
-}
-.status-line {  height: 10px; }
-/* yellow */
-.cbranch-no { background: yellow !important; color: #111; }
-/* dark red */
-.red.solid, .status-line.low, .low .cover-fill { background:#C21F39 }
-.low .chart { border:1px solid #C21F39 }
-.highlighted,
-.highlighted .cstat-no, .highlighted .fstat-no, .highlighted .cbranch-no{
-  background: #C21F39 !important;
-}
-/* medium red */
-.cstat-no, .fstat-no, .cbranch-no, .cbranch-no { background:#F6C6CE }
-/* light red */
-.low, .cline-no { background:#FCE1E5 }
-/* light green */
-.high, .cline-yes { background:rgb(230,245,208) }
-/* medium green */
-.cstat-yes { background:rgb(161,215,106) }
-/* dark green */
-.status-line.high, .high .cover-fill { background:rgb(77,146,33) }
-.high .chart { border:1px solid rgb(77,146,33) }
-/* dark yellow (gold) */
-.status-line.medium, .medium .cover-fill { background: #f9cd0b; }
-.medium .chart { border:1px solid #f9cd0b; }
-/* light yellow */
-.medium { background: #fff4c2; }
-
-.cstat-skip { background: #ddd; color: #111; }
-.fstat-skip { background: #ddd; color: #111 !important; }
-.cbranch-skip { background: #ddd !important; color: #111; }
-
-span.cline-neutral { background: #eaeaea; }
-
-.coverage-summary td.empty {
-    opacity: .5;
-    padding-top: 4px;
-    padding-bottom: 4px;
-    line-height: 1;
-    color: #888;
-}
-
-.cover-fill, .cover-empty {
-  display:inline-block;
-  height: 12px;
-}
-.chart {
-  line-height: 0;
-}
-.cover-empty {
-    background: white;
-}
-.cover-full {
-    border-right: none !important;
-}
-pre.prettyprint {
-    border: none !important;
-    padding: 0 !important;
-    margin: 0 !important;
-}
-.com { color: #999 !important; }
-.ignore-none { color: #999; font-weight: normal; }
-
-.wrapper {
-  min-height: 100%;
-  height: auto !important;
-  height: 100%;
-  margin: 0 auto -48px;
-}
-.footer, .push {
-  height: 48px;
-}

package/coverage/block-navigation.js

@@ -1,87 +0,0 @@
-/* eslint-disable */
-var jumpToCode = (function init() {
-    // Classes of code we would like to highlight in the file view
-    var missingCoverageClasses = ['.cbranch-no', '.cstat-no', '.fstat-no'];
-
-    // Elements to highlight in the file listing view
-    var fileListingElements = ['td.pct.low'];
-
-    // We don't want to select elements that are direct descendants of another match
-    var notSelector = ':not(' + missingCoverageClasses.join('):not(') + ') > '; // becomes `:not(a):not(b) > `
-
-    // Selecter that finds elements on the page to which we can jump
-    var selector =
-        fileListingElements.join(', ') +
-        ', ' +
-        notSelector +
-        missingCoverageClasses.join(', ' + notSelector); // becomes `:not(a):not(b) > a, :not(a):not(b) > b`
-
-    // The NodeList of matching elements
-    var missingCoverageElements = document.querySelectorAll(selector);
-
-    var currentIndex;
-
-    function toggleClass(index) {
-        missingCoverageElements
-            .item(currentIndex)
-            .classList.remove('highlighted');
-        missingCoverageElements.item(index).classList.add('highlighted');
-    }
-
-    function makeCurrent(index) {
-        toggleClass(index);
-        currentIndex = index;
-        missingCoverageElements.item(index).scrollIntoView({
-            behavior: 'smooth',
-            block: 'center',
-            inline: 'center'
-        });
-    }
-
-    function goToPrevious() {
-        var nextIndex = 0;
-        if (typeof currentIndex !== 'number' || currentIndex === 0) {
-            nextIndex = missingCoverageElements.length - 1;
-        } else if (missingCoverageElements.length > 1) {
-            nextIndex = currentIndex - 1;
-        }
-
-        makeCurrent(nextIndex);
-    }
-
-    function goToNext() {
-        var nextIndex = 0;
-
-        if (
-            typeof currentIndex === 'number' &&
-            currentIndex < missingCoverageElements.length - 1
-        ) {
-            nextIndex = currentIndex + 1;
-        }
-
-        makeCurrent(nextIndex);
-    }
-
-    return function jump(event) {
-        if (
-            document.getElementById('fileSearch') === document.activeElement &&
-            document.activeElement != null
-        ) {
-            // if we're currently focused on the search input, we don't want to navigate
-            return;
-        }
-
-        switch (event.which) {
-            case 78: // n
-            case 74: // j
-                goToNext();
-                break;
-            case 66: // b
-            case 75: // k
-            case 80: // p
-                goToPrevious();
-                break;
-        }
-    };
-})();
-window.addEventListener('keydown', jumpToCode);