Bug: Cannot upload or parse Data(binary) object

[MikeBenton2]

🌎 Environment

• Xcode: 16.1
• Platform: iOS
• Version/Release: latest

💬 Description

This is related to my other Question below in the "issues" section because we can't upload a Multipart form(binary) and we currently cannot parse Data(binary) objects either. We receive an error:

"failed - Error preparing pact! A key or value in the structure does not conform to 'Encodable' protocol. The element attempted to encode: 1 bytes"

Data does not conform to Encodable, which makes sense since it's binary and does not need to be translated, however, it shouldn't have that check. Is this something y'all are looking into?

I believe it's this file that is causing the issue since it's always trying to encode the body:
Sources/Model/Response.swift

[surpher]

That… makes sense. Thanks for raising it.

There’s major refactoring happened a while back already but hasn’t been cleaned up and made release “worthy” due to lack of time. But that’s changed these days and I’ll have time to have a look and hopefully bring that refactoring piece of work closer to new version release.
Most of the Pact generation responsibility will be pushed down to pact_ffi (core pact functionality written in Rust) and potentially should allow such tests.

I’d love your help verifying some of these use cases work in the upcoming few weeks. I’ll make myself more available in the Pact Foundation slack.

[MikeBenton2]

Sounds great! I'm going on holiday and won't be back for a month, but happy to test once I'm back :)

[surpher]

@MikeBenton2 There’s heavily refactored PactSwift and PactSwiftMockServer available on feature/swift-concurrency branch.

The DSL has changed a bit and how to setup the test. Have a look at README.md on that branch and check the unit tests.

Specifically:

• testSendingMultipartForm()
• testSendingABinaryBody()
• testReceivingABinaryBody()

I’ve written more tests, cleaned up heaps, identified bugs in shared pact_ffi… It’s still very basic working for iOS. Haven’t had much more time to test other platforms.

Give it a whirl and please raise issues when encountered.