Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SSDK-617] Fix offline tests #199

Merged
merged 20 commits into from
Apr 9, 2024
Merged

Conversation

aokj4ck
Copy link
Contributor

@aokj4ck aokj4ck commented Mar 26, 2024

Description

Fixes SSDK-617

  • [Offline] Change default tileset name to mbx-main
  • [Tests] Fix Offline tests and re-enable.
  • [Tests] Add offlineResultsUpdated delegate function to SearchEngineDelegateStub.
  • [Tests] Demonstrate providing a Geometry(point: NSValue(mkCoordinate: CLLocationCoordinate2D)) with TileRegionLoadOptions.build function.
  • [Core] Increment minimum MapboxCommon version from 24.0.0 to 24.2.0.

Checklist

  • Update CHANGELOG

aokj4ck added 17 commits March 25, 2024 12:45
- Remove unused SearchEngineDelegateStub.subscribe(listener:, selector:)
- Update tests to expect offline index observer events
- Update changelog
- Add debug description strings for CoreOfflineIndexChangeEventType
- Clean up code comments around DefaultLocationProvider.locationManager
…617-fix-offline-tests-add-demo

Conflicts:
	MapboxSearch.xcodeproj/project.pbxproj
	MapboxSearch.xcodeproj/xcshareddata/xcschemes/Demo.xcscheme
	Tests/MapboxSearchIntegrationTests/OfflineIntegrationTests.swift
@aokj4ck aokj4ck marked this pull request as ready for review April 9, 2024 17:27
@aokj4ck aokj4ck requested review from a team as code owners April 9, 2024 17:27
@aokj4ck aokj4ck merged commit 2ada7a4 into main Apr 9, 2024
5 checks passed
@aokj4ck aokj4ck deleted the SSDK-617-fix-offline-tests-add-demo branch April 9, 2024 18:09
aokj4ck added a commit that referenced this pull request Apr 9, 2024
…ion (#202)

### Description
Fixes [SSDK-622](https://mapbox.atlassian.net/browse/SSDK-622)

Add optional `language` parameter that has two modes:
1. Absent language parameter / `nil` value will use existing behavior of _just_ the dataset name.
2. A given language parameter will be appended to the dataset name.

### Checklist
- [x] Update `CHANGELOG`
- [x] Add a non-English language test after #199 is merged
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.

1 participant