Skip to content

Commit eea115f

Browse files
committed
Updates based on CR suggestions.
1 parent 73ff3ad commit eea115f

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

WooCommerce/Classes/Tools/Notices/Notice.swift

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,15 +7,15 @@ import UIKit
77
///
88
struct Notice {
99

10-
/// The title of the notice
10+
/// The title that contains the reason for the notice
1111
///
1212
let title: String
1313

14-
/// An optional subtitle for the notice
14+
/// An optional subtitle that contains a secondary description of the reason for the notice
1515
///
1616
let subtitle: String?
1717

18-
/// An optional subtitle for the notice
18+
/// An optional message that contains any details for the notice
1919
///
2020
let message: String?
2121

WooCommerce/WooCommerceTests/Notifications/PushNotificationsManagerTests.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -515,7 +515,7 @@ final class PushNotificationsManagerTests: XCTestCase {
515515
// Then
516516
XCTAssertEqual(emittedNotifications.count, 1)
517517

518-
let emittedNotification = emittedNotifications.first!
518+
let emittedNotification = try XCTUnwrap(emittedNotifications.first)
519519
XCTAssertEqual(emittedNotification.kind, .storeOrder)
520520
XCTAssertEqual(emittedNotification.noteID, 9_981)
521521
XCTAssertEqual(emittedNotification.title, Sample.defaultTitle)

0 commit comments

Comments
 (0)