Remove unnecessary discardableResults from SingleOrderScreen

`discardableResult` is unnecessary when we have a returned result which we expect to use.

Where we `return self`, the result `self` isn’t used directly, but staying on the screen allows us to chain methods. So `discardableResult` is used to suppress the warning.

Co-Authored-By: Evgeny Aleksandrov <[email protected]>

Author: thehenrybyrd

WooCommerce/UITestsFoundation/Screens/Orders/SingleOrderScreen.swift

@@ -58,7 +58,6 @@ public final class SingleOrderScreen: ScreenObject {
         return self
     }
 
-    @discardableResult
     public func tapCollectPaymentButton() throws -> PaymentMethodsScreen {
         let orderDetailTableView = app.tables["order-details-table-view"]
         while !collectPaymentButton.isFullyVisibleOnScreen() {
@@ -68,7 +67,6 @@ public final class SingleOrderScreen: ScreenObject {
         return try PaymentMethodsScreen()
     }
 
-    @discardableResult
     public func goBackToOrdersScreen() throws -> OrdersScreen {
         // Only needed for iPhone because iPad shows both Orders and Single Order screens on the same view
         if XCUIDevice.isPhone {
@@ -77,7 +75,6 @@ public final class SingleOrderScreen: ScreenObject {
         return try OrdersScreen()
     }
 
-    @discardableResult
     public func tapEditOrderButton() throws -> UnifiedOrderScreen {
         editOrderButton.tap()
         return try UnifiedOrderScreen(flow: .editing)