Add unit tests for asset-service, interaction-tracker, media-viewer, and utils by Copilot · Pull Request #92 · numbersprotocol/capture-eye

Copilot · 2026-03-17T13:11:19Z

Four source modules had zero test coverage despite containing critical functionality: API data mapping, analytics token management, media type detection, and platform detection.

New test files

`asset-service_test.ts` — 13 tests

fetchAsset: full field mapping (including the intentional thumnail_url API typo), explorerUrl construction, non-OK response, network error, non-JSON error body, null data
hasNftProduct: count threshold, HTTP errors, network failure
fetchAssetMetadata: hasC2pa flag, showcaseLink lowercasing, missing owner_name, error paths

`interaction-tracker_test.ts` — 7 tests

createEvent skips fetch when token === '', retries after 1s when token === null
getToken failure sets token to ''; sessionStorage cache bypasses fetch; fetched key is stored back to sessionStorage
trackInteraction truncates subid > 255 chars

`media-viewer_test.ts` — 11 tests

Empty src → "No source provided"; pending HEAD → loading state
image/* → <img>, video/* / HLS → <video>, unknown MIME → "Unsupported file format"
error event dispatched for unsupported MIME, network failure, and empty src
isImageMimeType / isVideoMimeType helper correctness

`utils_test.ts` — 8 tests

isMobile() returns true for iPhone, Android, iPad, BlackBerry, Windows Phone UAs
isMobile() returns false for desktop Chrome, Firefox, macOS Safari UAs
Stubs Navigator.prototype.userAgent via Object.defineProperty with proper teardown

Approach

window.fetch stubbed via sinon in all network-dependent tests
decryptData on the InteractionTracker singleton stubbed to test token caching paths without real crypto keys
MediaViewer.determineFileType() called directly to test event dispatch without fixture timing complexity

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

example.com
- Triggering command: /home/REDACTED/.cache/ms-playwright/chromium_headless_shell-1208/chrome-headless-shell-linux64/chrome-headless-shell /home/REDACTED/.cache/ms-playwright/chromium_headless_shell-1208/chrome-headless-shell-linux64/chrome-headless-shell --disable-field-trial-config --disable-REDACTED-networking --disable-REDACTED-timer-throttling --disable-REDACTEDing-occluded-windows --disable-back-forward-cache --disable-breakpad --disable-client-side-phishing-detection --disable-component-extensions-with-REDACTED-pages --disable-component-update --no-default-browser-check --disable-default-apps --disable-dev-shm-usage --disable-extensions --disable-features=AvoidUnnecessaryBeforeUnloadCheckSync,BoundaryEventDispatchTracksNodeRemoval,DestroyProfileOnBrowserClose,DialMediaRouteProvider,GlobalMediaControls,HttpsUpgrades,LensOverlay,MediaRouter,PaintHolding,ThirdPartyStoragePartitioning,Transl (dns block)
- Triggering command: /home/REDACTED/.cache/ms-playwright/chromium_headless_shell-1208/chrome-headless-shell-linux64/chrome-headless-shell /home/REDACTED/.cache/ms-playwright/chromium_headless_shell-1208/chrome-headless-shell-linux64/chrome-headless-shell --disable-field-trial-config --disable-REDACTED-networking --disable-REDACTED-timer-throttling --disable-REDACTEDing-occluded-windows --disable-back-forward-cache --disable-breakpad --disable-client-side-phishing-detection --disable-component-extensions-with-REDACTED-pages --disable-component-update --no-default-browser-check --disable-default-apps --disable-dev-shm-usage --disable-extensions --disable-features=AvoidUnnecessaryBeforeUnloadCheckSync,BoundaryEventDispatchTracksNodeRemoval,DestroyProfileOnBrowserClose,DialMediaRouteProvider,GlobalMediaControls,HttpsUpgrades,LensOverlay,MediaRouter,PaintHolding,ThirdPartyStoragePartitioning,Transl ndor/bin/git (dns block)
- Triggering command: /home/REDACTED/.cache/ms-playwright/chromium_headless_shell-1208/chrome-headless-shell-linux64/chrome-headless-shell /home/REDACTED/.cache/ms-playwright/chromium_headless_shell-1208/chrome-headless-shell-linux64/chrome-headless-shell --disable-field-trial-config --disable-REDACTED-networking --disable-REDACTED-timer-throttling --disable-REDACTEDing-occluded-windows --disable-back-forward-cache --disable-breakpad --disable-client-side-phishing-detection --disable-component-extensions-with-REDACTED-pages --disable-component-update --no-default-browser-check --disable-default-apps --disable-dev-shm-usage --disable-extensions --disable-features=AvoidUnnecessaryBeforeUnloadCheckSync,BoundaryEventDispatchTracksNodeRemoval,DestroyProfileOnBrowserClose,DialMediaRouteProvider,GlobalMediaControls,HttpsUpgrades,LensOverlay,MediaRouter,PaintHolding,ThirdPartyStoragePartitioning,Transl modules/@npmcli/run-script/lib/node-gyp-bin/node (dns block)
static-cdn.numbersprotocol.io
- Triggering command: /home/REDACTED/.cache/ms-playwright/chromium_headless_shell-1208/chrome-headless-shell-linux64/chrome-headless-shell /home/REDACTED/.cache/ms-playwright/chromium_headless_shell-1208/chrome-headless-shell-linux64/chrome-headless-shell --disable-field-trial-config --disable-REDACTED-networking --disable-REDACTED-timer-throttling --disable-REDACTEDing-occluded-windows --disable-back-forward-cache --disable-breakpad --disable-client-side-phishing-detection --disable-component-extensions-with-REDACTED-pages --disable-component-update --no-default-browser-check --disable-default-apps --disable-dev-shm-usage --disable-extensions --disable-features=AvoidUnnecessaryBeforeUnloadCheckSync,BoundaryEventDispatchTracksNodeRemoval,DestroyProfileOnBrowserClose,DialMediaRouteProvider,GlobalMediaControls,HttpsUpgrades,LensOverlay,MediaRouter,PaintHolding,ThirdPartyStoragePartitioning,Transl (dns block)
- Triggering command: /home/REDACTED/.cache/ms-playwright/chromium_headless_shell-1208/chrome-headless-shell-linux64/chrome-headless-shell /home/REDACTED/.cache/ms-playwright/chromium_headless_shell-1208/chrome-headless-shell-linux64/chrome-headless-shell --disable-field-trial-config --disable-REDACTED-networking --disable-REDACTED-timer-throttling --disable-REDACTEDing-occluded-windows --disable-back-forward-cache --disable-breakpad --disable-client-side-phishing-detection --disable-component-extensions-with-REDACTED-pages --disable-component-update --no-default-browser-check --disable-default-apps --disable-dev-shm-usage --disable-extensions --disable-features=AvoidUnnecessaryBeforeUnloadCheckSync,BoundaryEventDispatchTracksNodeRemoval,DestroyProfileOnBrowserClose,DialMediaRouteProvider,GlobalMediaControls,HttpsUpgrades,LensOverlay,MediaRouter,PaintHolding,ThirdPartyStoragePartitioning,Transl ndor/bin/git (dns block)
- Triggering command: /home/REDACTED/.cache/ms-playwright/chromium_headless_shell-1208/chrome-headless-shell-linux64/chrome-headless-shell /home/REDACTED/.cache/ms-playwright/chromium_headless_shell-1208/chrome-headless-shell-linux64/chrome-headless-shell --disable-field-trial-config --disable-REDACTED-networking --disable-REDACTED-timer-throttling --disable-REDACTEDing-occluded-windows --disable-back-forward-cache --disable-breakpad --disable-client-side-phishing-detection --disable-component-extensions-with-REDACTED-pages --disable-component-update --no-default-browser-check --disable-default-apps --disable-dev-shm-usage --disable-extensions --disable-features=AvoidUnnecessaryBeforeUnloadCheckSync,BoundaryEventDispatchTracksNodeRemoval,DestroyProfileOnBrowserClose,DialMediaRouteProvider,GlobalMediaControls,HttpsUpgrades,LensOverlay,MediaRouter,PaintHolding,ThirdPartyStoragePartitioning,Transl modules/@npmcli/run-script/lib/node-gyp-bin/node (dns block)
via.placeholder.com
- Triggering command: /home/REDACTED/.cache/ms-playwright/chromium_headless_shell-1208/chrome-headless-shell-linux64/chrome-headless-shell /home/REDACTED/.cache/ms-playwright/chromium_headless_shell-1208/chrome-headless-shell-linux64/chrome-headless-shell --disable-field-trial-config --disable-REDACTED-networking --disable-REDACTED-timer-throttling --disable-REDACTEDing-occluded-windows --disable-back-forward-cache --disable-breakpad --disable-client-side-phishing-detection --disable-component-extensions-with-REDACTED-pages --disable-component-update --no-default-browser-check --disable-default-apps --disable-dev-shm-usage --disable-extensions --disable-features=AvoidUnnecessaryBeforeUnloadCheckSync,BoundaryEventDispatchTracksNodeRemoval,DestroyProfileOnBrowserClose,DialMediaRouteProvider,GlobalMediaControls,HttpsUpgrades,LensOverlay,MediaRouter,PaintHolding,ThirdPartyStoragePartitioning,Transl (dns block)
- Triggering command: /home/REDACTED/.cache/ms-playwright/chromium_headless_shell-1208/chrome-headless-shell-linux64/chrome-headless-shell /home/REDACTED/.cache/ms-playwright/chromium_headless_shell-1208/chrome-headless-shell-linux64/chrome-headless-shell --disable-field-trial-config --disable-REDACTED-networking --disable-REDACTED-timer-throttling --disable-REDACTEDing-occluded-windows --disable-back-forward-cache --disable-breakpad --disable-client-side-phishing-detection --disable-component-extensions-with-REDACTED-pages --disable-component-update --no-default-browser-check --disable-default-apps --disable-dev-shm-usage --disable-extensions --disable-features=AvoidUnnecessaryBeforeUnloadCheckSync,BoundaryEventDispatchTracksNodeRemoval,DestroyProfileOnBrowserClose,DialMediaRouteProvider,GlobalMediaControls,HttpsUpgrades,LensOverlay,MediaRouter,PaintHolding,ThirdPartyStoragePartitioning,Transl ndor/bin/git (dns block)
- Triggering command: /home/REDACTED/.cache/ms-playwright/chromium_headless_shell-1208/chrome-headless-shell-linux64/chrome-headless-shell /home/REDACTED/.cache/ms-playwright/chromium_headless_shell-1208/chrome-headless-shell-linux64/chrome-headless-shell --disable-field-trial-config --disable-REDACTED-networking --disable-REDACTED-timer-throttling --disable-REDACTEDing-occluded-windows --disable-back-forward-cache --disable-breakpad --disable-client-side-phishing-detection --disable-component-extensions-with-REDACTED-pages --disable-component-update --no-default-browser-check --disable-default-apps --disable-dev-shm-usage --disable-extensions --disable-features=AvoidUnnecessaryBeforeUnloadCheckSync,BoundaryEventDispatchTracksNodeRemoval,DestroyProfileOnBrowserClose,DialMediaRouteProvider,GlobalMediaControls,HttpsUpgrades,LensOverlay,MediaRouter,PaintHolding,ThirdPartyStoragePartitioning,Transl modules/@npmcli/run-script/lib/node-gyp-bin/node (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Configure Actions setup steps to set up my environment, which run before the firewall is enabled
Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

Original prompt

This section details on the original issue you should resolve

<issue_title>[Feature][Medium] Add unit tests for untested modules: asset-service, interaction-tracker, media-viewer, utils</issue_title>
<issue_description>## Summary

Four source modules have zero test coverage. These modules contain critical functionality including API data mapping, analytics token management, media type detection, and platform detection.

Current Test Coverage

Module Test File Coverage

src/capture-eye.ts src/test/capture-eye_test.ts Partial (happy path only)

src/modal/modal.ts src/test/modal_test.ts Partial (happy path only)

src/asset/asset-service.ts None 0%

src/modal/interaction-tracker.ts None 0%

src/media-viewer/media-viewer.ts None 0%

src/utils.ts None 0%

Recommended Test Cases

asset-service.ts

fetchAsset — correct mapping from API response to AssetModel (verify all fields including thumnail_url typo)

fetchAsset — returns undefined on non-OK HTTP response

fetchAsset — returns undefined on network error

fetchAsset — handles non-JSON error response body gracefully

hasNftProduct — returns product data when available

hasNftProduct — returns undefined when no product found

fetchAssetMetadata — correct mapping of owner_name, description, headline

interaction-tracker.ts

Token fetched from sessionStorage cache on second call

Token fetch failure sets token = '' (empty string, not null)

createEvent does not fire when token === ''

createEvent retries when token === null (pending fetch)

Token decryption produces expected Bearer token format

media-viewer.ts

Renders <img> when MIME type starts with image/

Renders <video> when MIME type starts with video/

Dispatches error event when MIME type is unsupported

Renders loading state when src is empty or mimeType is null

HEAD request failure path behavior

utils.ts

isMobile() returns true for mobile user agents

isMobile() returns false for desktop user agents

isMobile() handles edge cases (empty string, undefined navigator)

Suggested Approach

Use sinon (already a devDependency) for stubbing fetch() calls

Use @open-wc/testing fixtures for component tests

Consider adding a test coverage threshold to CI to prevent regression</issue_description>

Comments on the Issue (you are @copilot in this section)

Fixes [Feature][Medium] Add unit tests for untested modules: asset-service, interaction-tracker, media-viewer, utils #88

💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

…and utils Co-authored-by: numbers-official <181934381+numbers-official@users.noreply.github.com>

The tests were written against the old Bubble.io API (verify.numbersprotocol.io) which has since been replaced. Updated to match the current Numbers API v3: - Remove fetchAssetMetadata tests (function no longer exists; hasC2pa and showcaseLink are now part of fetchAsset response) - Replace { response: { data: ... } } mock structure with flat JSON matching the v3 API response - Update field mappings: nit_commit_custom, signed_metadata, integrity_info, asset_file_mime_type, asset_file_thumbnail, etc. - Add tests for creator fallback chain, hasC2pa, and showcaseLink - Add test for null response body

Copilot AI assigned Copilot and numbers-official Mar 17, 2026

Copilot started work on behalf of numbers-official March 17, 2026 13:11 View session

Copilot AI changed the title ~~[WIP] [Feature] Add unit tests for untested modules~~ Add unit tests for asset-service, interaction-tracker, media-viewer, and utils Mar 17, 2026

Copilot AI requested a review from numbers-official March 17, 2026 13:22

Copilot finished work on behalf of numbers-official March 17, 2026 13:22

olgahaha marked this pull request as ready for review April 16, 2026 06:05

Copilot AI and others added 2 commits April 16, 2026 06:18

Add unit tests for asset-service, interaction-tracker, media-viewer, …

041799d

…and utils Co-authored-by: numbers-official <181934381+numbers-official@users.noreply.github.com>

numbers-official force-pushed the copilot/add-unit-tests-for-modules branch from 0d39a9e to d13a87c Compare April 16, 2026 06:24

olgahaha merged commit e350410 into main Apr 16, 2026
7 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add unit tests for asset-service, interaction-tracker, media-viewer, and utils#92

Add unit tests for asset-service, interaction-tracker, media-viewer, and utils#92
olgahaha merged 2 commits intomainfrom
copilot/add-unit-tests-for-modules

Copilot AI commented Mar 17, 2026 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Module	Test File	Coverage
`src/capture-eye.ts`	`src/test/capture-eye_test.ts`	Partial (happy path only)
`src/modal/modal.ts`	`src/test/modal_test.ts`	Partial (happy path only)
`src/asset/asset-service.ts`	None	0%
`src/modal/interaction-tracker.ts`	None	0%
`src/media-viewer/media-viewer.ts`	None	0%
`src/utils.ts`	None	0%

Conversation

Copilot AI commented Mar 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

New test files

asset-service_test.ts — 13 tests

interaction-tracker_test.ts — 7 tests

media-viewer_test.ts — 11 tests

utils_test.ts — 8 tests

Approach

I tried to connect to the following addresses, but was blocked by firewall rules:

Current Test Coverage

Recommended Test Cases

asset-service.ts

interaction-tracker.ts

media-viewer.ts

utils.ts

Suggested Approach

Comments on the Issue (you are @copilot in this section)

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Copilot AI commented Mar 17, 2026 •

edited

Loading

`asset-service_test.ts` — 13 tests

`interaction-tracker_test.ts` — 7 tests

`media-viewer_test.ts` — 11 tests

`utils_test.ts` — 8 tests

`asset-service.ts`

`interaction-tracker.ts`

`media-viewer.ts`

`utils.ts`