@switchbot/homebridge-switchbot 5.0.0-beta.1 → 5.0.0-beta.100

package/src/platform-matter.ts

@@ -1,465 +0,0 @@
-import type {
-  API,
-  DynamicPlatformPlugin,
-  Logging,
-  MatterAccessory,
-  SerializedMatterAccessory,
-} from 'homebridge'
-
-import type { SwitchBotPlatformConfig } from './settings.js'
-
-import {
-  ColorLightAccessory,
-  ColorTemperatureLightAccessory,
-  ContactSensorAccessory,
-  DimmableLightAccessory,
-  DoorLockAccessory,
-  ExtendedColorLightAccessory,
-  FanAccessory,
-  HumiditySensorAccessory,
-  LeakSensorAccessory,
-  LightSensorAccessory,
-  OccupancySensorAccessory,
-  OnOffLightAccessory,
-  OnOffOutletAccessory,
-  OnOffSwitchAccessory,
-  PowerStripAccessory,
-  RoboticVacuumAccessory,
-  SmokeCOAlarmAccessory,
-  TemperatureSensorAccessory,
-  ThermostatAccessory,
-  VenetianBlindAccessory,
-  WindowBlindAccessory,
-} from './devices-matter/index.js'
-import { PLATFORM_NAME, PLUGIN_NAME } from './settings.js'
-import { cleanDeviceConfig } from './utils.js'
-
-/**
- * MatterPlatform
- * Demonstrates all available Matter device types in Homebridge
- *
- * Organized by official Matter Specification v1.4.1 categories
- */
-export class SwitchBotMatterPlatform implements DynamicPlatformPlugin {
-  // Track restored HAP cached accessories (required for DynamicPlatformPlugin)
-  // This is commented out here as this plugin does not have any HAP accessories
-  // public readonly accessories: Map<string, PlatformAccessory> = new Map()
-
-  // Track restored Matter cached accessories
-  public readonly matterAccessories: Map<string, SerializedMatterAccessory> = new Map()
-
-  constructor(
-    public readonly log: Logging,
-    public readonly config: SwitchBotPlatformConfig,
-    public readonly api: API,
-  ) {
-    this.log.debug('Finished initializing platform:', this.config.name)
-
-    // Normalize deviceConfig to remove UI-inserted defaults
-    try {
-      if ((this.config as any).options) {
-        const cleaned = cleanDeviceConfig((this.config as any).options.deviceConfig)
-        if (cleaned) {
-          ;(this.config as any).options.deviceConfig = cleaned
-        } else {
-          delete (this.config as any).options.deviceConfig
-        }
-      }
-    } catch (e) {
-      this.log.debug('Failed to clean deviceConfig: %s', e)
-    }
-
-    // Does the user have a version of Homebridge that is compatible with matter?
-    if (!this.api.isMatterAvailable?.()) {
-      this.log.warn('Matter is not available in this version of Homebridge. Please update Homebridge to use this plugin.')
-    }
-
-    // Check if the user has matter enabled, this means:
-    // - If the plugin is running on the main bridge, then the user must have enabled matter in the Homebridge settings page in the UI
-    // - If the plugin is running on a child bridge, then the user must have enabled matter on the plugin bridge settings section in the UI
-    // In reality, only the below check is needed, but they are both included here for completeness
-    // Remember to use a '?.' optional chaining operator in case the user is running an older version of Homebridge that does not have these APIs
-    if (!this.api.isMatterEnabled?.()) {
-      this.log.warn('Matter is not enabled in Homebridge. Please enable Matter in the Homebridge settings to use this plugin.')
-      return
-    }
-
-    // Register Matter accessories when Homebridge has finished launching
-    this.api.on('didFinishLaunching', () => {
-      this.log.debug('Executed didFinishLaunching callback')
-      this.registerMatterAccessories()
-    })
-  }
-
-  /**
-   * Required for DynamicPlatformPlugin
-   * Called when homebridge restores cached accessories from disk at startup
-   */
-  configureAccessory(/* accessory: PlatformAccessory */) {
-    // Note this is not used for Matter accessories - use configureMatterAccessory instead
-    // This plugin does not have any hap accessories, so here we can comment this out
-    // this.accessories.set(accessory.UUID, accessory)
-  }
-
-  /**
-   * Called when homebridge restores cached Matter accessories from disk at startup.
-   *
-   * This is where you can access the `accessory.context` object to retrieve
-   * any custom data you stored when the accessory was originally registered.
-   */
-  configureMatterAccessory(accessory: SerializedMatterAccessory) {
-    this.log.debug('Loading cached Matter accessory:', accessory.displayName)
-    this.matterAccessories.set(accessory.uuid, accessory)
-  }
-
-  /**
-   * Register all Matter accessories
-   */
-  private async registerMatterAccessories() {
-    this.log.info('═'.repeat(80))
-    this.log.info('Homebridge Matter Plugin')
-    this.log.info('═'.repeat(80))
-
-    // Remove accessories that are disabled in config
-    await this.removeDisabledAccessories()
-
-    // Register devices by Matter specification sections
-    await this.registerSection4Lighting()
-    await this.registerSection5SmartPlugs()
-    await this.registerSection6Switches()
-    await this.registerSection7Sensors()
-    await this.registerSection8Closure()
-    await this.registerSection9HVAC()
-    await this.registerSection12Robotic()
-    await this.registerCustomDevices()
-
-    this.log.info('═'.repeat(80))
-    this.log.info('Finished registering Matter accessories')
-    this.log.info('═'.repeat(80))
-  }
-
-  /**
-   * Remove accessories that are disabled in config
-   */
-  private async removeDisabledAccessories() {
-    const configMap = [
-      { enabled: this.config.enableOnOffLight, uuid: this.api.matter.uuid.generate('matter-onoff-light'), name: 'On/Off Light' },
-      { enabled: this.config.enableDimmableLight, uuid: this.api.matter.uuid.generate('matter-dimmable-light'), name: 'Dimmable Light' },
-      { enabled: this.config.enableColourTemperatureLight, uuid: this.api.matter.uuid.generate('matter-colour-temp-light'), name: 'Colour Temperature Light' },
-      { enabled: this.config.enableColourLight, uuid: this.api.matter.uuid.generate('matter-colour-light'), name: 'Colour Light (HS)' },
-      { enabled: this.config.enableExtendedColourLight, uuid: this.api.matter.uuid.generate('matter-extended-colour-light'), name: 'Extended Colour Light' },
-      { enabled: this.config.enableOnOffOutlet, uuid: this.api.matter.uuid.generate('matter-onoff-outlet'), name: 'On/Off Outlet' },
-      { enabled: this.config.enableOnOffSwitch, uuid: this.api.matter.uuid.generate('matter-onoff-switch'), name: 'On/Off Switch' },
-      { enabled: this.config.enableTemperatureSensor, uuid: this.api.matter.uuid.generate('matter-temperature-sensor'), name: 'Temperature Sensor' },
-      { enabled: this.config.enableHumiditySensor, uuid: this.api.matter.uuid.generate('matter-humidity-sensor'), name: 'Humidity Sensor' },
-      { enabled: this.config.enableLightSensor, uuid: this.api.matter.uuid.generate('matter-light-sensor'), name: 'Light Sensor' },
-      { enabled: this.config.enableOccupancySensor, uuid: this.api.matter.uuid.generate('matter-occupancy-sensor'), name: 'Occupancy Sensor' },
-      { enabled: this.config.enableContactSensor, uuid: this.api.matter.uuid.generate('matter-contact-sensor'), name: 'Contact Sensor' },
-      { enabled: this.config.enableLeakSensor, uuid: this.api.matter.uuid.generate('matter-leak-sensor'), name: 'Leak Sensor' },
-      { enabled: this.config.enableSmokeSensor, uuid: this.api.matter.uuid.generate('matter-smoke-sensor'), name: 'Smoke Sensor' },
-      { enabled: this.config.enableDoorLock, uuid: this.api.matter.uuid.generate('matter-door-lock'), name: 'Door Lock' },
-      { enabled: this.config.enableWindowBlind, uuid: this.api.matter.uuid.generate('matter-window-blind'), name: 'Window Blind' },
-      { enabled: this.config.enableVenetianBlind, uuid: this.api.matter.uuid.generate('matter-venetian-blind'), name: 'Venetian Blind' },
-      { enabled: this.config.enableThermostat, uuid: this.api.matter.uuid.generate('matter-thermostat'), name: 'Thermostat' },
-      { enabled: this.config.enableFan, uuid: this.api.matter.uuid.generate('matter-fan'), name: 'Fan' },
-      { enabled: this.config.enableRobotVacuum, uuid: this.api.matter.uuid.generate('matter-robot-vacuum'), name: 'Robot Vacuum' },
-      { enabled: this.config.enablePowerStrip, uuid: this.api.matter.uuid.generate('matter-power-strip'), name: 'Power Strip' },
-    ]
-
-    for (const { enabled, uuid, name } of configMap) {
-      if (enabled === false) {
-        const existingAccessory = this.matterAccessories.get(uuid)
-        if (existingAccessory) {
-          this.log.info(`Removing accessory '${name}' (disabled in config)`)
-          await this.api.matter.unregisterPlatformAccessories(PLUGIN_NAME, PLATFORM_NAME, [existingAccessory as unknown as MatterAccessory])
-          this.matterAccessories.delete(uuid)
-        }
-      }
-    }
-  }
-
-  /**
-   * Section 4: Lighting Devices (Matter Spec § 4)
-   */
-  private async registerSection4Lighting() {
-    this.log.info('═'.repeat(80))
-    this.log.info('Section 4: Lighting Devices (Matter Spec § 4)')
-    this.log.info('═'.repeat(80))
-
-    const accessories: Array<MatterAccessory<Record<string, unknown>>> = []
-
-    // On/Off Light
-    if (this.config.enableOnOffLight !== false) {
-      const device = new OnOffLightAccessory(this.api, this.log)
-      accessories.push(device.toAccessory())
-    }
-
-    // Dimmable Light
-    if (this.config.enableDimmableLight !== false) {
-      const device = new DimmableLightAccessory(this.api, this.log)
-      accessories.push(device.toAccessory())
-    }
-
-    // Color Temperature Light
-    if (this.config.enableColourTemperatureLight !== false) {
-      const device = new ColorTemperatureLightAccessory(this.api, this.log)
-      accessories.push(device.toAccessory())
-    }
-
-    // Color Light (HS only)
-    if (this.config.enableColourLight !== false) {
-      const device = new ColorLightAccessory(this.api, this.log)
-      accessories.push(device.toAccessory())
-    }
-
-    // Extended Color Light (HS+CCT)
-    if (this.config.enableExtendedColourLight !== false) {
-      const device = new ExtendedColorLightAccessory(this.api, this.log)
-      accessories.push(device.toAccessory())
-    }
-
-    if (accessories.length > 0) {
-      this.log.info(`✓ Registered ${accessories.length} lighting device(s)`)
-      for (const acc of accessories) {
-        this.log.info(`  - ${acc.displayName}`)
-      }
-      await this.api.matter.registerPlatformAccessories(PLUGIN_NAME, PLATFORM_NAME, accessories)
-    }
-  }
-
-  /**
-   * Section 5: Smart Plugs/Actuators (Matter Spec § 5)
-   */
-  private async registerSection5SmartPlugs() {
-    this.log.info('═'.repeat(80))
-    this.log.info('Section 5: Smart Plugs/Actuators (Matter Spec § 5)')
-    this.log.info('═'.repeat(80))
-
-    const accessories: Array<MatterAccessory<Record<string, unknown>>> = []
-
-    // On/Off Outlet
-    if (this.config.enableOnOffOutlet !== false) {
-      const device = new OnOffOutletAccessory(this.api, this.log)
-      accessories.push(device.toAccessory())
-    }
-
-    if (accessories.length > 0) {
-      this.log.info(`✓ Registered ${accessories.length} smart plug/actuator device(s)`)
-      for (const acc of accessories) {
-        this.log.info(`  - ${acc.displayName}`)
-      }
-      await this.api.matter.registerPlatformAccessories(PLUGIN_NAME, PLATFORM_NAME, accessories)
-    }
-  }
-
-  /**
-   * Section 6: Switches & Controllers (Matter Spec § 6)
-   */
-  private async registerSection6Switches() {
-    this.log.info('═'.repeat(80))
-    this.log.info('Section 6: Switches & Controllers (Matter Spec § 6)')
-    this.log.info('═'.repeat(80))
-
-    const accessories: Array<MatterAccessory<Record<string, unknown>>> = []
-
-    // On/Off Switch
-    if (this.config.enableOnOffSwitch !== false) {
-      const device = new OnOffSwitchAccessory(this.api, this.log)
-      accessories.push(device.toAccessory())
-    }
-
-    if (accessories.length > 0) {
-      this.log.info(`✓ Registered ${accessories.length} switch/controller device(s)`)
-      for (const acc of accessories) {
-        this.log.info(`  - ${acc.displayName}`)
-      }
-      await this.api.matter.registerPlatformAccessories(PLUGIN_NAME, PLATFORM_NAME, accessories)
-    }
-  }
-
-  /**
-   * Section 7: Sensors (Matter Spec § 7)
-   */
-  private async registerSection7Sensors() {
-    this.log.info('═'.repeat(80))
-    this.log.info('Section 7: Sensors (Matter Spec § 7)')
-    this.log.info('═'.repeat(80))
-
-    const accessories: Array<MatterAccessory<Record<string, unknown>>> = []
-
-    // Contact Sensor
-    if (this.config.enableContactSensor !== false) {
-      const device = new ContactSensorAccessory(this.api, this.log)
-      accessories.push(device.toAccessory())
-    }
-
-    // Light Sensor
-    if (this.config.enableLightSensor !== false) {
-      const device = new LightSensorAccessory(this.api, this.log)
-      accessories.push(device.toAccessory())
-    }
-
-    // Occupancy Sensor
-    if (this.config.enableOccupancySensor !== false) {
-      const device = new OccupancySensorAccessory(this.api, this.log)
-      accessories.push(device.toAccessory())
-    }
-
-    // Temperature Sensor
-    if (this.config.enableTemperatureSensor !== false) {
-      const device = new TemperatureSensorAccessory(this.api, this.log)
-      accessories.push(device.toAccessory())
-    }
-
-    // Humidity Sensor
-    if (this.config.enableHumiditySensor !== false) {
-      const device = new HumiditySensorAccessory(this.api, this.log)
-      accessories.push(device.toAccessory())
-    }
-
-    // Smoke/CO Alarm
-    if (this.config.enableSmokeSensor !== false) {
-      const device = new SmokeCOAlarmAccessory(this.api, this.log)
-      accessories.push(device.toAccessory())
-    }
-
-    // Leak Sensor
-    if (this.config.enableLeakSensor !== false) {
-      const device = new LeakSensorAccessory(this.api, this.log)
-      accessories.push(device.toAccessory())
-    }
-
-    if (accessories.length > 0) {
-      this.log.info(`✓ Registered ${accessories.length} sensor device(s)`)
-      for (const acc of accessories) {
-        this.log.info(`  - ${acc.displayName}`)
-      }
-      await this.api.matter.registerPlatformAccessories(PLUGIN_NAME, PLATFORM_NAME, accessories)
-    }
-  }
-
-  /**
-   * Section 8: Closure Devices (Matter Spec § 8)
-   */
-  private async registerSection8Closure() {
-    this.log.info('═'.repeat(80))
-    this.log.info('Section 8: Closure Devices (Matter Spec § 8)')
-    this.log.info('═'.repeat(80))
-
-    const accessories: Array<MatterAccessory<Record<string, unknown>>> = []
-
-    // Door Lock
-    if (this.config.enableDoorLock !== false) {
-      const device = new DoorLockAccessory(this.api, this.log)
-      accessories.push(device.toAccessory())
-    }
-
-    // Window Blind
-    if (this.config.enableWindowBlind !== false) {
-      const device = new WindowBlindAccessory(this.api, this.log)
-      accessories.push(device.toAccessory())
-    }
-
-    // Venetian Blind
-    if (this.config.enableVenetianBlind !== false) {
-      const device = new VenetianBlindAccessory(this.api, this.log)
-      accessories.push(device.toAccessory())
-    }
-
-    if (accessories.length > 0) {
-      this.log.info(`✓ Registered ${accessories.length} closure device(s)`)
-      for (const acc of accessories) {
-        this.log.info(`  - ${acc.displayName}`)
-      }
-      await this.api.matter.registerPlatformAccessories(PLUGIN_NAME, PLATFORM_NAME, accessories)
-    }
-  }
-
-  /**
-   * Section 9: HVAC (Matter Spec § 9)
-   */
-  private async registerSection9HVAC() {
-    this.log.info('═'.repeat(80))
-    this.log.info('Section 9: HVAC (Matter Spec § 9)')
-    this.log.info('═'.repeat(80))
-
-    const accessories: Array<MatterAccessory<Record<string, unknown>>> = []
-
-    // Thermostat
-    if (this.config.enableThermostat !== false) {
-      const device = new ThermostatAccessory(this.api, this.log)
-      accessories.push(device.toAccessory())
-    }
-
-    // Fan
-    if (this.config.enableFan !== false) {
-      const device = new FanAccessory(this.api, this.log)
-      accessories.push(device.toAccessory())
-    }
-
-    if (accessories.length > 0) {
-      this.log.info(`✓ Registered ${accessories.length} HVAC device(s)`)
-      for (const acc of accessories) {
-        this.log.info(`  - ${acc.displayName}`)
-      }
-      await this.api.matter.registerPlatformAccessories(PLUGIN_NAME, PLATFORM_NAME, accessories)
-    }
-  }
-
-  /**
-   * Section 12: Robotic Devices (Matter Spec § 12)
-   * ⚠️ IMPORTANT: RVC devices use a DIFFERENT PROCESS (same code) than other devices!
-   * When this runs, you'll see separate commissioning codes in the logs for the robot vacuum.
-   * Use those codes to pair the vacuum as a separate bridge in your Home app.
-   */
-  private async registerSection12Robotic() {
-    this.log.info('═'.repeat(80))
-    this.log.info('Section 12: Robotic Devices (Matter Spec § 12)')
-    this.log.info('═'.repeat(80))
-
-    const accessories: Array<MatterAccessory<Record<string, unknown>>> = []
-
-    // Robot Vacuum
-    if (this.config.enableRobotVacuum !== false) {
-      const device = new RoboticVacuumAccessory(this.api, this.log)
-      accessories.push(device.toAccessory())
-    }
-
-    if (accessories.length > 0) {
-      this.log.info(`✓ Registered ${accessories.length} robot vacuum device(s)`)
-      for (const acc of accessories) {
-        this.log.info(`  - ${acc.displayName} (standalone for Apple Home compatibility)`)
-      }
-      await this.api.matter.registerPlatformAccessories(PLUGIN_NAME, PLATFORM_NAME, accessories)
-    }
-  }
-
-  /**
-   * Custom Devices
-   *
-   * This section demonstrates custom device implementations that go beyond
-   * the standard Matter device types. These examples show advanced patterns
-   * like managing multiple logical components within a single device.
-   */
-  private async registerCustomDevices() {
-    this.log.info('═'.repeat(80))
-    this.log.info('Custom Devices')
-    this.log.info('═'.repeat(80))
-
-    const accessories: Array<MatterAccessory<Record<string, unknown>>> = []
-
-    // Power Strip (4 Outlets)
-    if (this.config.enablePowerStrip !== false) {
-      const device = new PowerStripAccessory(this.api, this.log)
-      accessories.push(device.toAccessory())
-    }
-
-    if (accessories.length > 0) {
-      this.log.info(`✓ Registered ${accessories.length} custom device(s)`)
-      for (const acc of accessories) {
-        this.log.info(`  - ${acc.displayName}`)
-      }
-      await this.api.matter.registerPlatformAccessories(PLUGIN_NAME, PLATFORM_NAME, accessories)
-    }
-  }
-}

package/src/verifyconfig.test.ts

@@ -1,197 +0,0 @@
-import { describe, expect, it, vi } from 'vitest'
-import type { SwitchBotPlatformConfig } from './settings.js'
-
-// Create a minimal mock of the SwitchBotPlatform to test verifyConfig
-class MockSwitchBotPlatform {
-  config: SwitchBotPlatformConfig
-  errorLogCalls: string[] = []
-  debugLogCalls: string[] = []
-
-  constructor(config: SwitchBotPlatformConfig) {
-    this.config = config
-  }
-
-  errorLog(message: string) {
-    this.errorLogCalls.push(message)
-  }
-
-  debugLog(message: string) {
-    this.debugLogCalls.push(message)
-  }
-
-  debugWarnLog(message: string) {
-    this.debugLogCalls.push(`WARN: ${message}`)
-  }
-
-  debugErrorLog(message: string) {
-    this.debugLogCalls.push(`ERROR: ${message}`)
-  }
-
-  // Copy the exact verifyConfig method from the actual platform
-  async verifyConfig() {
-    this.debugLog('Verifying Config')
-    this.config = this.config || {}
-    this.config.options = this.config.options || {}
-
-    if (this.config.options) {
-      // Device Config
-      if (this.config.options.devices) {
-        for (const deviceConfig of this.config.options.devices) {
-          if (!deviceConfig.hide_device) {
-            if (!deviceConfig.deviceId) {
-              this.errorLog('The devices config section is missing the *Device ID* in the config. Please check your config.')
-            }
-            if (!deviceConfig.configDeviceType && (deviceConfig as any).connectionType) {
-              this.errorLog('The devices config section is missing the *Device Type* in the config. Please check your config.')
-            }
-          }
-        }
-      }
-
-      // IR Device Config
-      if (this.config.options.irdevices) {
-        for (const irDeviceConfig of this.config.options.irdevices) {
-          if (!irDeviceConfig.hide_device) {
-            if (!irDeviceConfig.deviceId) {
-              this.errorLog('The devices config section is missing the *Device ID* in the config. Please check your config.')
-            }
-            if (!irDeviceConfig.deviceId && !irDeviceConfig.configRemoteType) {
-              this.errorLog('The devices config section is missing the *Device Type* in the config. Please check your config.')
-            }
-          }
-        }
-      }
-    }
-
-    if (!this.config.credentials && !this.config.options) {
-      this.debugWarnLog('Missing Credentials')
-    } else if (this.config.credentials && !this.config.credentials.notice) {
-      if (!this.config.credentials?.token) {
-        this.debugErrorLog('Missing token')
-        this.debugWarnLog('Cloud Enabled SwitchBot Devices & IR Devices will not work')
-      }
-      if (this.config.credentials?.token) {
-        if (!this.config.credentials?.secret) {
-          this.debugErrorLog('Missing secret')
-          this.debugWarnLog('Cloud Enabled SwitchBot Devices & IR Devices will not work')
-        }
-      }
-    }
-  }
-}
-
-describe('verifyConfig fix for reboot loop', () => {
-  it('should log error instead of throwing when device config is missing deviceId', async () => {
-    const config: SwitchBotPlatformConfig = {
-      platform: 'SwitchBot',
-      name: 'SwitchBot',
-      credentials: {
-        token: 'test-token',
-        secret: 'test-secret',
-      },
-      options: {
-        devices: [
-          {
-            logging: 'standard',
-          } as any, // Missing deviceId
-        ],
-      },
-    }
-
-    const platform = new MockSwitchBotPlatform(config)
-
-    // This should NOT throw an error anymore - it should log instead
-    await expect(platform.verifyConfig()).resolves.not.toThrow()
-    
-    // Verify that the error was logged instead of thrown
-    expect(platform.errorLogCalls).toContain(
-      'The devices config section is missing the *Device ID* in the config. Please check your config.'
-    )
-  })
-
-  it('should log error instead of throwing when device config is missing configDeviceType with connectionType', async () => {
-    const config: SwitchBotPlatformConfig = {
-      platform: 'SwitchBot',
-      name: 'SwitchBot',
-      credentials: {
-        token: 'test-token',
-        secret: 'test-secret',
-      },
-      options: {
-        devices: [
-          {
-            deviceId: 'test-device-id',
-            connectionType: 'BLE',
-            // Missing configDeviceType
-          } as any,
-        ],
-      },
-    }
-
-    const platform = new MockSwitchBotPlatform(config)
-
-    // This should NOT throw an error anymore - it should log instead
-    await expect(platform.verifyConfig()).resolves.not.toThrow()
-    
-    // Verify that the error was logged instead of thrown
-    expect(platform.errorLogCalls).toContain(
-      'The devices config section is missing the *Device Type* in the config. Please check your config.'
-    )
-  })
-
-  it('should not log error when device config has hide_device set to true', async () => {
-    const config: SwitchBotPlatformConfig = {
-      platform: 'SwitchBot',
-      name: 'SwitchBot',
-      credentials: {
-        token: 'test-token',
-        secret: 'test-secret',
-      },
-      options: {
-        devices: [
-          {
-            hide_device: true,
-            // Missing deviceId but hidden
-          } as any,
-        ],
-      },
-    }
-
-    const platform = new MockSwitchBotPlatform(config)
-
-    // Should not throw or log errors because device is hidden
-    await expect(platform.verifyConfig()).resolves.not.toThrow()
-    expect(platform.errorLogCalls).toHaveLength(0)
-  })
-
-  it('should handle valid device config without errors', async () => {
-    const config: SwitchBotPlatformConfig = {
-      platform: 'SwitchBot',
-      name: 'SwitchBot',
-      credentials: {
-        token: 'test-token',
-        secret: 'test-secret',
-      },
-      options: {
-        devices: [
-          {
-            deviceId: 'test-device-id',
-            configDeviceType: 'Bot',
-            logging: 'standard',
-          } as any,
-        ],
-      },
-    }
-
-    const platform = new MockSwitchBotPlatform(config)
-
-    // Should not throw or log device config errors with valid config
-    await expect(platform.verifyConfig()).resolves.not.toThrow()
-    
-    // Should not have device config errors
-    expect(platform.errorLogCalls.filter(msg => 
-      msg.includes('missing the *Device ID*') || 
-      msg.includes('missing the *Device Type*')
-    )).toHaveLength(0)
-  })
-})