Replace invalid sinon.define with Object.defineProperty and cleanup

[Copilot]

Addresses review feedback to fix test using non-existent sinon.define() API when globalThis.navigator is undefined.

Changes

• Replaced sinon.define(globalThis, 'navigator', { language: 'de-DE' }) with Object.defineProperty()
• Added try-finally block to delete the property after test execution
• Set configurable: true and writable: true on the property descriptor

Implementation

let navigatorDefined = false
if (globalThis.navigator) {
  sinon.stub(globalThis.navigator, 'language').get(() => 'de-DE')
} else {
  Object.defineProperty(globalThis, 'navigator', {
    value: { language: 'de-DE' },
    configurable: true,
    writable: true
  })
  navigatorDefined = true
}

try {
  // test assertions
} finally {
  if (navigatorDefined) {
    delete globalThis.navigator
  }
}

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.