fix: resolve aidebugger.test.js CI failures

[riyacore404]

PR Category

• Bug Fix - Fixes a bug or incorrect behavior

Summary

Resolves #7433 — the Jest CI pipeline was failing due to three issues in the AI Debugger widget.

Changes

jest.config.js

Added testEnvironmentOptions: { url: "http://localhost/" }.
jsdom's default URL (about:blank) gives window.location.hostname = "", which
the BACKEND_CONFIG IIFE didn't recognise, firing a console.warn and setting
BASE_URL = null on every test run.

js/widgets/aidebugger.js

Added an early-return guard in _hideTypingIndicator when this.chatLog is null.
The widget's onclose handler calls _hideTypingIndicator() before init() has
run _createLayout(), so chatLog can be null — crashing with a TypeError.

js/widgets/tests/aidebugger.test.js

Added 19 new tests covering all previously untested code paths:

• _consentGiven initial state
• _addWelcomeMessage
• _showConsentBanner (accept + decline button flows)
• _loadProjectAndInitialize (success, export-throws fallback, bad-JSON non-fatal)
• _initializeBackendWithProject (fetch success, fetch fail, missing .response)
• _sendToBackend (success, network error, non-ok HTTP)
• _exportChat with a non-empty history (download triggered)
• _exportChat with prepareExport throwing (graceful fallback)
• _hideTypingIndicator with null chatLog (regression guard)

Test Results

• Before: aidebugger test suite failed to run (TypeError + missing mocks)
• After: 82/82 tests pass; full suite 5378/5378 tests pass

Checklist

• All tests pass locally (npm test)
• Lint clean (npm run lint)
• Prettier clean (npx prettier --check .)
• No new dependencies added

[github-actions]

🧪 Jest Test Results

✅ All Jest tests passed! This PR is ready to merge.

Coverage: Statements: 48.3% | Branches: 39.6% | Functions: 53.07% | Lines: 48.72%
Master Coverage: Statements: 48.09% | Branches: 39.61% | Functions: 52.88% | Lines: 48.49%

[mahesh-09-12]

this PR now feels a bit broader than just fixing the CI failure, since it also adds quite a bit of aidebugger test coverage beyond the original failing scenario

[mahesh-09-12]

these aidebugger test additions seem to already exist from the recently merged CI fix PR (#7403), so could you rebase onto latest master and drop the already-merged parts from this diff?

[riyacore404]

Thanks for the feedback @mahesh-09-12!

I've rebased onto the latest master and dropped the two _sendMessage tests that were already merged via #7403 ("shows consent banner and does not send message if consent not given" and "sends message and clears input").

All tests pass locally. Let me know if you'd like any further changes!

[mahesh-09-12]

@riyacore404 oh, my mistake. Before rebasing onto the latest master, it made sense to remove these since they were duplicated by the earlier aidebugger fix work. After the rebase though, removing them here would actually remove tests that are already present on master. Sorry for the confusion and the earlier suggestion.

[riyacore404]

No worries, thanks for clarifying! I’ve reverted my previous change and restored the tests so the PR is back in sync. 👍

[mahesh-09-12]

Thanks for your patience and for restoring the tests. My concern is resolved now.

[github-actions]

🧪 Jest Test Results

✅ All Jest tests passed! This PR is ready to merge.

Coverage: Statements: 48.34% | Branches: 39.68% | Functions: 53.12% | Lines: 48.75%
Master Coverage: Statements: 48.09% | Branches: 39.61% | Functions: 52.88% | Lines: 48.49%

[github-actions]

🧪 Jest Test Results

✅ All Jest tests passed! This PR is ready to merge.

Coverage: Statements: 48.36% | Branches: 39.69% | Functions: 53.12% | Lines: 48.77%
Master Coverage: Statements: 48.09% | Branches: 39.61% | Functions: 52.88% | Lines: 48.49%