Skip to content

Commit b5a34e7

Browse files
committed
Fix some issues
1 parent 06427ff commit b5a34e7

File tree

21 files changed

+119
-90
lines changed

21 files changed

+119
-90
lines changed

.github/workflows/check-pr.yml

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -9,11 +9,11 @@ on:
99
- 'release/*'
1010

1111
env:
12-
destination: "platform=iOS Simulator,name=iPhone 14 Pro,OS=latest"
12+
destination: "platform=iOS Simulator,name=iPhone 16 Pro,OS=latest"
1313
configuration: "Debug"
1414
noIndex: "COMPILER_INDEX_STORE_ENABLE=NO"
1515
noSigning: "CODE_SIGNING_ALLOWED=NO"
16-
versionXcode: "14.3.1"
16+
versionXcode: "16.0"
1717

1818
concurrency:
1919
group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
@@ -22,23 +22,23 @@ concurrency:
2222
jobs:
2323
lint:
2424
name: SwiftLint
25-
runs-on: macos-13-large
25+
runs-on: macos-15-large
2626

2727
steps:
2828
- name: Checkout
29-
uses: actions/checkout@v3
29+
uses: actions/checkout@85e6279cec87321a52edac9c87bce653a07cf6c2
3030

3131
- name: Lint Edited Files
3232
run: bash .github/scripts/lintEditedFiles.sh
3333

3434
checkout:
3535
name: Checkout Verification
36-
runs-on: macos-13-xlarge
36+
runs-on: macos-15-xlarge
3737
needs: lint
3838

3939
steps:
4040
- name: Checkout
41-
uses: actions/checkout@v3
41+
uses: actions/checkout@85e6279cec87321a52edac9c87bce653a07cf6c2
4242

4343
- name: Select Xcode
4444
run: |
@@ -75,12 +75,12 @@ jobs:
7575

7676
frames:
7777
name: Frames Verification
78-
runs-on: macos-13-xlarge
78+
runs-on: macos-15-xlarge
7979
needs: lint
8080

8181
steps:
8282
- name: Checkout
83-
uses: actions/checkout@v3
83+
uses: actions/checkout@85e6279cec87321a52edac9c87bce653a07cf6c2
8484

8585
- name: Select Xcode
8686
run: |
@@ -117,16 +117,16 @@ jobs:
117117

118118
run-ui-tests:
119119
name: Run UI Tests
120-
runs-on: macos-12-xl
120+
runs-on: macos-15-xlarge
121121
needs: lint
122122

123123
steps:
124124
- name: Checkout
125-
uses: actions/checkout@v3
125+
uses: actions/checkout@85e6279cec87321a52edac9c87bce653a07cf6c2
126126

127127
- name: Select Xcode
128128
run: |
129-
sudo xcode-select -switch /Applications/Xcode_14.2.app
129+
sudo xcode-select -switch /Applications/Xcode_16.0.app
130130
131131
- name: Log xcodebuild Version
132132
run: |

.github/workflows/codeql-analysis.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20,16 +20,16 @@ concurrency:
2020
cancel-in-progress: true
2121

2222
env:
23-
destination: "platform=iOS Simulator,name=iPhone 15 Pro,OS=latest"
23+
destination: "platform=iOS Simulator,name=iPhone 16 Pro,OS=latest"
2424
configuration: "Debug"
2525
noIndex: "COMPILER_INDEX_STORE_ENABLE=NO"
2626
noSigning: "CODE_SIGNING_ALLOWED=NO"
27-
versionXcode: "15.2"
27+
versionXcode: "16.0"
2828

2929
jobs:
3030
analyze:
3131
name: Analyze
32-
runs-on: [ macos-13-large ]
32+
runs-on: [ macos-15-xlarge ]
3333
permissions:
3434
actions: read
3535
contents: read
@@ -42,7 +42,7 @@ jobs:
4242

4343
steps:
4444
- name: Checkout repository
45-
uses: actions/checkout@v3
45+
uses: actions/checkout@85e6279cec87321a52edac9c87bce653a07cf6c2
4646
with:
4747
submodules: recursive
4848

.github/workflows/deploy-github-pages.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ jobs:
2727
runs-on: ubuntu-latest
2828
steps:
2929
- name: Checkout
30-
uses: actions/checkout@v3
30+
uses: actions/checkout@85e6279cec87321a52edac9c87bce653a07cf6c2
3131
- name: Setup Pages
3232
uses: actions/configure-pages@v3
3333
- name: Build with Jekyll

.github/workflows/generate-jazzy-docs.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ jobs:
1616

1717
steps:
1818
- name: Checkout
19-
uses: actions/checkout@v3
19+
uses: actions/checkout@85e6279cec87321a52edac9c87bce653a07cf6c2
2020

2121
- name: Setup Ruby
2222
uses: ruby/setup-ruby@5311f05890856149502132d25c4a24985a00d426

.github/workflows/publish-Checkout.podspec.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ jobs:
77
environment: CocoaPodsRelease
88
runs-on: macos-12-xl
99
steps:
10-
- uses: actions/checkout@v4
10+
- uses: actions/checkout@85e6279cec87321a52edac9c87bce653a07cf6c2
1111

1212
# - name: Check current branch
1313
# run: |

.github/workflows/publish-Frames.podspec.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ jobs:
77
environment: CocoaPodsRelease
88
runs-on: macos-12-xl
99
steps:
10-
- uses: actions/checkout@v4
10+
- uses: actions/checkout@85e6279cec87321a52edac9c87bce653a07cf6c2
1111

1212
# - name: Check current branch
1313
# run: |

.github/workflows/run-regression-tests.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,23 +14,23 @@ jobs:
1414
run-regression-tests:
1515
if: (github.event_name == 'workflow_dispatch') || (contains(github.event.pull_request.labels.*.name, 'Run Regression Tests'))
1616
name: Run Regression Tests
17-
runs-on: macos-12-xl
17+
runs-on: macos-15-xlarge
1818

1919
steps:
2020
- name: Checkout
21-
uses: actions/checkout@v3
21+
uses: actions/checkout@85e6279cec87321a52edac9c87bce653a07cf6c2
2222

2323
- name: Select Xcode
2424
run: |
25-
sudo xcode-select -switch /Applications/Xcode_14.2.app
25+
sudo xcode-select -switch /Applications/Xcode_16.0.app
2626
2727
- name: Log xcodebuild Version
2828
run: |
2929
xcodebuild -version
3030
3131
- name: Run Regression Tests
3232
run: |
33-
set -o pipefail && xcodebuild "test" "-project" "iOS Example Frame SPM/iOS Example Frame SPM.xcodeproj" "-scheme" "Regression Tests" "-configuration" "Debug" "-destination" "platform=iOS Simulator,name=iPhone 14 Pro,OS=latest" | xcpretty
33+
set -o pipefail && xcodebuild "test" "-project" "iOS Example Frame SPM/iOS Example Frame SPM.xcodeproj" "-scheme" "Regression Tests" "-configuration" "Debug" "-destination" "platform=iOS Simulator,name=iPhone 16 Pro,OS=latest" | xcpretty
3434
3535
#
3636
# Comment out below only when needed, to avoid overhead.

.github/workflows/validate-for-app-store.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,11 +15,11 @@ jobs:
1515
if: (github.event_name == 'workflow_dispatch') || contains(github.event.pull_request.labels.*.name, 'Validate For App Store')
1616
name: Validate For App Store
1717
environment: AppStoreValidation
18-
runs-on: macos-latest
18+
runs-on: macos-15-xlarge
1919

2020
steps:
2121
- name: Checkout repository
22-
uses: actions/checkout@v3
22+
uses: actions/checkout@85e6279cec87321a52edac9c87bce653a07cf6c2
2323
- name: Install the Apple certificate and provisioning profile
2424
env:
2525
BUILD_CERTIFICATE_BASE64: ${{ secrets.BUILD_CERTIFICATE_BASE64 }}

Checkout.podspec

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,13 +13,13 @@ Pod::Spec.new do |s|
1313
s.author = { "Checkout.com Integration" => "[email protected]" }
1414
s.source = { :git => "https://github.com/checkout/frames-ios.git", :tag => s.version }
1515

16-
s.ios.deployment_target = '12.0'
16+
s.ios.deployment_target = '13.0'
1717
s.swift_version = "5.7"
1818

1919
s.source_files = 'Checkout/Source/**/*.swift'
2020
s.exclude_files = "Checkout/Samples/**"
2121

2222
s.dependency 'CheckoutEventLoggerKit', '~> 1.2.4'
23-
s.dependency 'Risk', '3.0.2'
23+
s.dependency 'Risk', '~> 3.0.2'
2424

2525
end

CheckoutTests/Validation/CardValidatorTests.swift

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -161,12 +161,12 @@ extension CardValidatorTests {
161161

162162
test_validate(
163163
expiryMonth: "11",
164-
expiryYear: "2021",
165-
expectedResult: .success(ExpiryDate(month: 11, year: 2021)))
164+
expiryYear: "2030",
165+
expectedResult: .success(ExpiryDate(month: 11, year: 2030)))
166166

167167
let components = stubCalendar.dateFromComponentsCalledWith
168168
XCTAssertEqual(components?.month, 11)
169-
XCTAssertEqual(components?.year, 2021)
169+
XCTAssertEqual(components?.year, 2030)
170170
}
171171

172172
func test_validate_expiryMonthYearString_providedMontInThePast_SameYear_returnsCorrectError() {

Frames.podspec

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ Pod::Spec.new do |s|
1010
s.swift_version = "5.0"
1111
s.license = "MIT"
1212
s.author = { "Checkout.com Integration" => "[email protected]" }
13-
s.platform = :ios, "12.0"
13+
s.platform = :ios, "13.0"
1414
s.source = { :git => "https://github.com/checkout/frames-ios.git", :tag => s.version }
1515

1616
s.source_files = 'Source/**/*.swift'
@@ -19,7 +19,7 @@ Pod::Spec.new do |s|
1919
"Frames" => 'Source/Resources/**'
2020
}
2121

22-
s.dependency 'PhoneNumberKit'
22+
s.dependency 'PhoneNumberKit', '~> 4.0'
2323
s.dependency 'CheckoutEventLoggerKit', '~> 1.2.4'
2424
s.dependency 'Checkout', '4.3.7'
2525

Package.resolved

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,8 @@
3232
"kind" : "remoteSourceControl",
3333
"location" : "https://github.com/marmelroy/PhoneNumberKit.git",
3434
"state" : {
35-
"revision" : "6edd6e38a30aec087cb97f7377edf876c29a427e",
36-
"version" : "3.5.9"
35+
"revision" : "288b99e9e99926bc8be41220802df83c059fac9b",
36+
"version" : "4.0.2"
3737
}
3838
}
3939
],

Package.swift

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ let package = Package(
66
defaultLocalization: "en",
77
platforms: [
88
.macOS(.v10_12),
9-
.iOS(.v12)
9+
.iOS(.v13)
1010
],
1111
products: [
1212
.library(
@@ -17,10 +17,10 @@ let package = Package(
1717
dependencies: [
1818
.package(
1919
url: "https://github.com/marmelroy/PhoneNumberKit.git",
20-
exact: "3.5.9"),
20+
exact: "4.0.2"),
2121
.package(
2222
url: "https://github.com/checkout/checkout-risk-sdk-ios.git",
23-
exact: "3.0.2"),
23+
from: "3.0.2"),
2424
.package(
2525
url: "https://github.com/checkout/checkout-event-logger-ios-framework.git",
2626
from: "1.2.4"

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212

1313
## Requirements
1414

15-
- iOS 12.0+
15+
- iOS 13.0+
1616
- Xcode 12.4+
1717
- Swift 5.3+
1818

Source/UI/PaymentForm/ViewModel/DefaultPaymentViewModel.swift

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -182,7 +182,9 @@ extension DefaultPaymentViewModel: PaymentViewControllerDelegate {
182182
checkoutAPIService.createToken(.card(card)) { [weak self] result in
183183
self?.logTokenResult(result)
184184
self?.isLoading = false
185-
self?.cardTokenRequested?(result)
185+
DispatchQueue.main.async { [weak self] in
186+
self?.cardTokenRequested?(result)
187+
}
186188
}
187189
}
188190

Source/UI/Validator/PhoneNumberValidator.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,9 @@ import Checkout
44

55
class PhoneNumberValidator: Validator {
66

7-
var countryCode: String = PhoneNumberKit.defaultRegionCode()
7+
var countryCode: String = PhoneNumberUtility.defaultRegionCode()
88
private let validator = PhoneValidator()
9-
private let phoneKit = PhoneNumberKit()
9+
private let phoneKit = PhoneNumberUtility()
1010
private let validCharacterSet: CharacterSet = {
1111
var validInputs = "-+() "
1212
(0...9).forEach { validInputs.append("\($0)") }

Tests/UI/CountrySelection/CountrySelectionViewControllerTests.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ class MockDelegate: CountrySelectionViewControllerDelegate {
1515
class CountrySelectionViewControllerTests: XCTestCase {
1616

1717
var countrySelectionViewController = CountrySelectionViewController()
18-
let numberOfCountries = 256
18+
let numberOfCountries = 257
1919

2020
override func setUp() {
2121
super.setUp()

0 commit comments

Comments
 (0)