Shipping Labels: Fix issue mapping shipments after updating for REST API response (#16078)

Author: itsmeichigo

Modules/Sources/Networking/Mapper/WooShippingUpdateShipmentMapper.swift

@@ -7,9 +7,9 @@ struct WooShippingUpdateShipmentMapper: Mapper {
     ///
     func map(response: Data) throws -> WooShippingShipments {
         let decoder = JSONDecoder()
-        if hasDataEnvelope(in: response) {
+        do {
             return try decoder.decode(WooShippingUpdateShipmentResponseEnvelope.self, from: response).data.shipments
-        } else {
+        } catch {
             return try decoder.decode(WooShippingUpdateShipmentResponse.self, from: response).shipments
         }
     }

Modules/Tests/NetworkingTests/Mapper/WooShippingUpdateShipmentMapperTests.swift

@@ -20,7 +20,7 @@ final class WooShippingUpdateShipmentMapperTests: XCTestCase {
     }
 
     func test_shipments_are_properly_parsed_when_response_has_no_data_envelope() throws {
-        guard let shipments = mapLoadShippingLabelConfig() else {
+        guard let shipments = mapLoadShippingLabelConfigWithoutDataEnvelope() else {
             XCTFail()
             return
         }