Fix test suite failures caused by ESM-only @octokit/core and probot dependencies

[Copilot]

Two unit test suites were failing to load because @octokit/core and probot are now ESM-only packages ("type": "module") and cannot be consumed via require().

Changes

• test/unit/lib/settings.test.js: Removed require('@octokit/core'). Octokit was only used as a mock property container via jest.mocked() — replaced with a local empty class:

  // Before
  const { Octokit } = require('@octokit/core')
  
  // After
  class Octokit {}

• test/unit/index.test.js: Added jest.mock('probot', ...) to short-circuit the ESM load. All tests in this file are already describe.skip'd; the suite was failing purely at the module loading stage.

  jest.mock('probot', () => ({ Probot: class Probot {} }))
  const { Probot } = require('probot')

Original prompt

Fix the failing GitHub Actions workflow test (20)
Analyze the workflow logs, identify the root cause of the failure, and implement a fix.
Job ID: 65474740419
Job URL: https://github.com/github/safe-settings/actions/runs/22598567041/job/65474740419

---

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

[Copilot]

Pull request overview

This PR fixes two failing unit test suites caused by @octokit/core and probot being ESM-only packages ("type": "module") that cannot be consumed via require() in Jest's CommonJS environment.

Changes:

• Replaced the require('@octokit/core') import in settings.test.js with a local empty stub class, which works identically given that jest.mocked() in plain JavaScript simply returns the class constructor.
• Added a jest.mock('probot', ...) call in index.test.js to short-circuit the ESM module load, since all tests in that file are already describe.skip'd and the failure was purely at module-load time.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File	Description
test/unit/lib/settings.test.js	Replaces ESM require('@octokit/core') with a local empty class Octokit {} stub
test/unit/index.test.js	Adds jest.mock('probot', ...) to prevent ESM-load failure at module resolution time

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.