Merge branch 'trunk' into task/WOOMOB-938-prevent-subs-products-from-order-flow

Author: iamgabrielma

Modules/Package.resolved

@@ -72,6 +72,7 @@ let package = Package(
         .package(url: "https://github.com/Alamofire/Alamofire", from: "5.2.0"),
         .package(url: "https://github.com/AliSoftware/OHHTTPStubs", from: "9.0.0"),
         .package(url: "https://github.com/apple/swift-algorithms.git", from: "1.2.0"),
+        .package(url: "https://github.com/apple/swift-async-algorithms", exact: "1.0.4"),
         .package(url: "https://github.com/Automattic/AutomatticAbout-swift.git", from: "1.1.5"),
         .package(url: "https://github.com/Automattic/Automattic-Tracks-iOS.git", from: "3.5.2"),
         .package(url: "https://github.com/Automattic/Gridicons-iOS", revision: "c904cb73e26e86463a78e1335c6f4fd54a9e9223"),
@@ -381,6 +382,7 @@ enum XcodeSupport {
                     .product(name: "Inject", package: "Inject"),
                     .product(name: "KeychainAccess", package: "KeychainAccess"),
                     .product(name: "Kingfisher", package: "Kingfisher"),
+                    .product(name: "AsyncAlgorithms", package: "swift-async-algorithms"),
                     .product(name: "ScrollViewSectionKit", package: "ScrollViewSectionKit"),
                     .product(name: "Shimmer", package: "SwiftUI-Shimmer"),
                     .product(name: "StripeTerminal", package: "stripe-terminal-ios"),

Modules/Package.swift

@@ -103,6 +103,8 @@ public struct DefaultFeatureFlagService: FeatureFlagService {
             return true
         case .pointOfSaleBarcodeScanningi2:
             return buildConfig == .localDeveloper || buildConfig == .alpha
+        case .orderAddressMapSearch:
+            return buildConfig == .localDeveloper || buildConfig == .alpha
         default:
             return true
         }

Modules/Sources/Experiments/DefaultFeatureFlagService.swift

@@ -210,4 +210,8 @@ public enum FeatureFlag: Int {
     /// Enables the Point of Sale Barcode Scanner set up flows, as part of i2
     ///
     case pointOfSaleBarcodeScanningi2
+
+    /// Enables the CTA to search for an address in the map in order details > shipping address.
+    ///
+    case orderAddressMapSearch
 }

Modules/Sources/Experiments/FeatureFlag.swift

@@ -29,7 +29,7 @@ extension CardReaderType {
         case .wisepad3:
             return "WISEPAD_3"
         case .tapToPay:
-            return "COTS_DEVICE"
+            return "TAP_TO_PAY_DEVICE"
         default:
             return "UNKNOWN"
         }

Modules/Sources/Hardware/CardReader/CardReaderType.swift

@@ -228,6 +228,8 @@ public class OrdersRemote: Remote {
                     params[Order.CodingKeys.createdVia.rawValue] = createdViaValue
                 }
 
+                params[ParameterKeys.decimalPlaces] = OrdersRemote.Defaults.decimalPoints
+
                 return params
             }()
 
@@ -253,7 +255,10 @@ public class OrdersRemote: Remote {
     ///
     public func updateOrder(from siteID: Int64, orderID: Int64, statusKey: OrderStatusEnum, completion: @escaping (Order?, Error?) -> Void) {
         let path = "\(Constants.ordersPath)/" + String(orderID)
-        let parameters = [ParameterKeys.statusKey: statusKey.rawValue]
+        let parameters = [
+            ParameterKeys.statusKey: statusKey.rawValue,
+            ParameterKeys.decimalPlaces: OrdersRemote.Defaults.decimalPoints
+        ]
         let mapper = OrderMapper(siteID: siteID)
 
         let request = JetpackRequest(wooApiVersion: .mark3,
@@ -328,6 +333,9 @@ public class OrdersRemote: Remote {
                                                                                value: cashPaymentChangeDueAmount).toDictionary()]
                 }
 
+                // Add decimal places parameter for better precision
+                params[ParameterKeys.decimalPlaces] = OrdersRemote.Defaults.decimalPoints
+
                 return params
             }()
 
@@ -449,8 +457,9 @@ extension OrdersRemote: POSOrdersRemoteProtocol {
 //
 public extension OrdersRemote {
     enum Defaults {
-        public static let pageSize: Int     = 25
-        public static let pageNumber: Int   = 1
+        public static let pageSize: Int = 25
+        public static let pageNumber: Int = 1
+        public static let decimalPoints: String = "8"
         public static let statusAny: String = "any"
     }
 
@@ -477,6 +486,7 @@ public extension OrdersRemote {
         static let customer = "customer"
         static let product = "product"
         static let createdVia = "created_via"
+        static let decimalPlaces = "dp"
     }
 
     enum ParameterValues {

Modules/Sources/NetworkingCore/Remote/OrdersRemote.swift

@@ -40,8 +40,12 @@ public final class CustomerDetailsScreen: ScreenObject {
     @discardableResult
     public func enterCustomerDetails(name: String) throws -> UnifiedOrderScreen {
         billingFirstNameField.tap()
-        billingFirstNameField.typeText(name)
+        // `\n` was added after the first name input in order to dismiss the keyboard so that the address toggle is interactive in shorter screens.
+        billingFirstNameField.typeText("\(name)\n")
+        let tableView = app.tables.firstMatch
+        tableView.swipeUp()
         addressToggleSet(to: "1")
+        tableView.swipeUp()
         shippingFirstNameField.tap()
         shippingFirstNameField.typeText(name)
         doneButton.tap()

Modules/Sources/UITestsFoundation/Screens/Orders/CustomerDetailsScreen.swift

@@ -309,6 +309,19 @@ final class OrdersRemoteTests: XCTestCase {
         wait(for: [expectation], timeout: Constants.expectationTimeout)
     }
 
+    func test_updateOrder_status_includes_decimal_places_parameter() throws {
+        // Given
+        let remote = OrdersRemote(network: network)
+
+        // When
+        remote.updateOrder(from: sampleSiteID, orderID: sampleOrderID, statusKey: .pending) { (order, error) in }
+
+        // Then
+        let request = try XCTUnwrap(network.requestsForResponseData.last as? JetpackRequest)
+        let received = try XCTUnwrap(request.parameters["dp"] as? String)
+        assertEqual(received, "8")
+    }
+
     func test_update_order_properly_encodes_shipping_lines_for_removal_from_order() throws {
         // Given
         let remote = OrdersRemote(network: network)
@@ -517,6 +530,20 @@ final class OrdersRemoteTests: XCTestCase {
         XCTAssertNil(request.parameters["meta_data"])
     }
 
+    func test_updateOrder_with_fields_includes_decimal_places_parameter() throws {
+        // Given
+        let remote = OrdersRemote(network: network)
+        let order = Order.fake()
+
+        // When
+        remote.updateOrder(from: sampleSiteID, order: order, giftCard: nil, fields: [.customerNote]) { result in }
+
+        // Then
+        let request = try XCTUnwrap(network.requestsForResponseData.last as? JetpackRequest)
+        let received = try XCTUnwrap(request.parameters["dp"] as? String)
+        assertEqual(received, "8")
+    }
+
     // MARK: - Load Order Notes Tests
 
     /// Verifies that loadOrderNotes properly parses the `order-notes` sample response.
@@ -806,6 +833,20 @@ final class OrdersRemoteTests: XCTestCase {
         XCTAssertNil(request.parameters["created_via"])
     }
 
+    func test_createOrder_includes_decimal_places_parameter() throws {
+        // Given
+        let remote = OrdersRemote(network: network)
+        let order = Order.fake()
+
+        // When
+        remote.createOrder(siteID: 123, order: order, giftCard: nil, fields: []) { result in }
+
+        // Then
+        let request = try XCTUnwrap(network.requestsForResponseData.last as? JetpackRequest)
+        let received = try XCTUnwrap(request.parameters["dp"] as? String)
+        assertEqual(received, "8")
+    }
+
     // MARK: - Delete order tests
 
     func test_delete_order_properly_returns_parsed_order() throws {

Modules/Tests/NetworkingTests/Remote/OrdersRemoteTests.swift

@@ -1269,6 +1269,7 @@ final class OrderStoreTests: XCTestCase {
             "currency",
             "customer_id",
             "customer_note",
+            "dp",
             "fee_lines",
             "line_items",
             "meta_data",
@@ -1297,6 +1298,7 @@ final class OrderStoreTests: XCTestCase {
             "currency",
             "customer_id",
             "customer_note",
+            "dp",
             "fee_lines",
             "gift_cards",
             "line_items",
@@ -1359,7 +1361,8 @@ final class OrderStoreTests: XCTestCase {
         let expectedKeys = [
             "gift_cards"
         ]
-        assertEqual(expectedKeys, receivedKeys)
+        // Updating an order will also contain `dp` (decimal point), but in this case we're only interested in `gift_cards`
+        XCTAssertTrue(receivedKeys.contains(expectedKeys))
     }
 
     func test_update_order_with_gift_card_returns_notApplied_error_when_error_response_does_not_include_gift_card() throws {

Modules/Tests/YosemiteTests/Stores/OrderStoreTests.swift

@@ -3,8 +3,11 @@
 
 23.0
 -----
+- [*] Increased decimal sensitivity in order creation to mitigate tax rounding issues [https://github.com/woocommerce/woocommerce-ios/pull/15957]
+- [*] Fix initialization of authenticator to avoid crashes during login [https://github.com/woocommerce/woocommerce-ios/pull/15953]
 - [*] Shipping Labels: Made HS tariff number field required in customs form for EU destinations [https://github.com/woocommerce/woocommerce-ios/pull/15946]
 - [*] Order Creation: Prevent subscription products to be added to an order [https://github.com/woocommerce/woocommerce-ios/pull/15960]
+- [internal] Replace COTS_DEVICE reader model name with TAP_TO_PAY_DEVICE. [https://github.com/woocommerce/woocommerce-ios/pull/15961]
 
 22.9
 -----