Skip to content

Tests: Apple Maps on a real iPhone, identifiers and a page at once - #283

Merged
renrenmimi merged 1 commit into
mainfrom
ios-native/apple-maps-device-probe
Oct 2, 2026
Merged

renrenmimi merged 1 commit into
mainfrom
ios-native/apple-maps-device-probe

Conversation

@renrenmimi

Copy link
Copy Markdown
Owner

PetNoteAppTests/AppleMapsDeviceProbeTests.swift answers the two questions docs/apple-maps-places-plan.md left for a real device, through the app's own MapKit calls (MapKitPlaceDirectory.Lookups.mapKit). It runs only on a real iPhone (.enabled(if:)); on the simulator and in CI it is skipped.

Run on an iPhone 17 Pro, iOS 27.0.1, 2026-10-01:

MEASURED apple search: 100 places with an identifier, 0 without, in 4 searches
MEASURED apple search again: 100 of 100 identifiers the same as the first time
MEASURED apple lookups: 26 of 26 answered at once in 0.26 s
✔ Test run with 2 tests in 1 suite passed after 5.201 seconds.

Only counts and times are printed; nothing Apple says about a place is kept.

The plan's "verified" section, its table row and its throttling risk now say what was measured, and so does PlaceDirectory.swift's note on throttling. iOS 18 hardware, where a developer once reported no identifiers, is still unchecked.

🤖 Generated with Claude Code

The two things the Mac run left open, asked through the app's own MapKit
calls on the phone, which is the only place the suite runs:

- four ordinary searches gave 100 places, every one with an identifier,
  and searching again gave the same 100 identifiers;
- 26 places looked up at once by identifier all answered, in 0.26s.

Measured on an iPhone 17 Pro, iOS 27.0.1. Only counts and times are
printed; nothing Apple says about a place is kept. The plan and the
directory's note say what was measured; iOS 18 hardware is still
unchecked.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@vercel

vercel Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
pet-note Ready Ready Preview Oct 2, 2026 5:56am UTC

Copilot AI balanced review requested due to automatic review settings October 2, 2026 05:56

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@renrenmimi
renrenmimi merged commit 31ed506 into main Oct 2, 2026
7 checks passed
renrenmimi added a commit that referenced this pull request Oct 2, 2026
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

This branch was successfully deployed

1 active deployment
Preview — 072261d6 Deployed Oct 2, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants