Skip to content

Commit 83768bb

Browse files
authored
Merge pull request #17797 from wordpress-mobile/issue/update_feature_struct
Feature Announcement: add icons property to Feature struct
2 parents 7026b1e + b25b74e commit 83768bb

File tree

3 files changed

+24
-21
lines changed

3 files changed

+24
-21
lines changed

Podfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ def wordpress_ui
4747
end
4848

4949
def wordpress_kit
50-
pod 'WordPressKit', '~> 4.46.0-beta.1'
50+
pod 'WordPressKit', '~> 4.46.0-beta'
5151
# pod 'WordPressKit', :git => 'https://github.com/wordpress-mobile/WordPressKit-iOS.git', :tag => ''
5252
# pod 'WordPressKit', :git => 'https://github.com/wordpress-mobile/WordPressKit-iOS.git', :branch => ''
5353
# pod 'WordPressKit', :git => 'https://github.com/wordpress-mobile/WordPressKit-iOS.git', :commit => ''

Podfile.lock

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -465,7 +465,7 @@ PODS:
465465
- WordPressKit (~> 4.18-beta)
466466
- WordPressShared (~> 1.12-beta)
467467
- WordPressUI (~> 1.7-beta)
468-
- WordPressKit (4.46.0-beta.1):
468+
- WordPressKit (4.46.0-beta.2):
469469
- Alamofire (~> 4.8.0)
470470
- CocoaLumberjack (~> 3.4)
471471
- NSObject-SafeExpectations (= 0.0.4)
@@ -569,7 +569,7 @@ DEPENDENCIES:
569569
- SVProgressHUD (= 2.2.5)
570570
- WordPress-Editor-iOS (~> 1.19.7)
571571
- WordPressAuthenticator (~> 1.43.0-beta.2)
572-
- WordPressKit (~> 4.46.0-beta.1)
572+
- WordPressKit (~> 4.46.0-beta)
573573
- WordPressMocks (~> 0.0.15)
574574
- WordPressShared (~> 1.17.0)
575575
- WordPressUI (~> 1.12.3)
@@ -581,6 +581,7 @@ DEPENDENCIES:
581581
SPEC REPOS:
582582
https://github.com/wordpress-mobile/cocoapods-specs.git:
583583
- WordPressAuthenticator
584+
- WordPressKit
584585
trunk:
585586
- Alamofire
586587
- AlamofireImage
@@ -619,7 +620,6 @@ SPEC REPOS:
619620
- UIDeviceIdentifier
620621
- WordPress-Aztec-iOS
621622
- WordPress-Editor-iOS
622-
- WordPressKit
623623
- WordPressMocks
624624
- WordPressShared
625625
- WordPressUI
@@ -833,7 +833,7 @@ SPEC CHECKSUMS:
833833
WordPress-Aztec-iOS: 144f124148079084860368dfd27cb96e0952853e
834834
WordPress-Editor-iOS: 20551d5a5e51f29832064f08faaaae8e1da7e1e2
835835
WordPressAuthenticator: 0b3423546d92a976e3ce627d2e5a0350716b1f5b
836-
WordPressKit: 52329b519bd0da6533cd4c79070424d0672ed7ee
836+
WordPressKit: e623ffb39b2b9d7730a9be63e26fa77929e6e43b
837837
WordPressMocks: 6b52b0764d9939408151367dd9c6e8a910877f4d
838838
WordPressShared: a4b0308a6345d4dda20c8f7ad9317df4246b4a00
839839
WordPressUI: 45591178e843ecb82e65e868ec766148befe9f9f
@@ -849,6 +849,6 @@ SPEC CHECKSUMS:
849849
ZendeskSupportSDK: 3a8e508ab1d9dd22dc038df6c694466414e037ba
850850
ZIPFoundation: e27423c004a5a1410c15933407747374e7c6cb6e
851851

852-
PODFILE CHECKSUM: cf4b842b6fd0d700b0f0ded040967877822e8daa
852+
PODFILE CHECKSUM: ebdcdd2b30dd680be23e92f8167141254e055594
853853

854854
COCOAPODS: 1.11.2

WordPress/WordPressTest/What's New/Data store/AnnouncementsDataStoreTests.swift

Lines changed: 18 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -11,21 +11,23 @@ struct MockAnnouncementsCache: AnnouncementsCache {
1111
var date: Date?
1212

1313
private let localAnnouncements = [Announcement(appVersionName: "0.0",
14-
minimumAppVersion: "1.0",
15-
maximumAppVersion: "3.0",
16-
appVersionTargets: [],
17-
detailsUrl: "http://wordpress.org",
18-
announcementVersion: "1.0",
19-
isLocalized: false,
20-
responseLocale: "",
21-
features: [WordPressKit.Feature(title: "first cached feature",
22-
subtitle: "this is a mock cached feature",
23-
iconUrl: "https://s0.wordpress.com/i/store/mobile/plans-personal.png",
24-
iconBase64: nil),
25-
WordPressKit.Feature(title: "second cached feature",
26-
subtitle: "this is a mock cached feature",
27-
iconUrl: "https://s0.wordpress.com/i/store/mobile/plans-personal.png",
28-
iconBase64: nil)])]
14+
minimumAppVersion: "1.0",
15+
maximumAppVersion: "3.0",
16+
appVersionTargets: [],
17+
detailsUrl: "http://wordpress.org",
18+
announcementVersion: "1.0",
19+
isLocalized: false,
20+
responseLocale: "",
21+
features: [WordPressKit.Feature(title: "first cached feature",
22+
subtitle: "this is a mock cached feature",
23+
icons: nil,
24+
iconUrl: "https://s0.wordpress.com/i/store/mobile/plans-personal.png",
25+
iconBase64: nil),
26+
WordPressKit.Feature(title: "second cached feature",
27+
subtitle: "this is a mock cached feature",
28+
icons: nil,
29+
iconUrl: "https://s0.wordpress.com/i/store/mobile/plans-personal.png",
30+
iconBase64: nil)])]
2931

3032
init(localCacheIsValid: Bool = true) {
3133
self.announcements = localCacheIsValid ? localAnnouncements : nil
@@ -47,6 +49,7 @@ class MockAnnouncementsService: AnnouncementServiceRemote {
4749
responseLocale: "",
4850
features: [WordPressKit.Feature(title: "mock feature",
4951
subtitle: "this is a mock feature",
52+
icons: nil,
5053
iconUrl: "https://s0.wordpress.com/i/store/mobile/plans-personal.png",
5154
iconBase64: nil)])])
5255

0 commit comments

Comments
 (0)