GitHub
10/28/2023, 2:12 AMpactffi_response_status
only allow integer status.
But it can be useful to match a value in body (like this example), query parameter or header.
pact-foundation/pact-php
✅ All checks have passed
26/26 successful checksGitHub
10/28/2023, 2:28 AMnull
• On provider: because example value become null
, we got this error on verification:
Failures:
1) Verifying a pact between matchersConsumer and matchersProvider Given Get Matchers - A get request to /matchers
1.1) has a matching body
$.arrayContaining -> Expected [102.3,"eb375cad-48cc-4f7f-981b-ea4f1af90bf2"] (Array) to be equal to null (Null)
pact-foundation/pact-php
✅ All checks have passed
26/26 successful checksGitHub
10/28/2023, 2:38 AM'values' => Array (
- 0 => 'a'
- 1 => 'bb'
- 2 => 'ccc'
+ 'a' => 'a'
+ 'b' => 'bb'
+ 'c' => 'ccc'
)
• On provider: got this error on verification:
Failures:
1) Verifying a pact between matchersConsumer and matchersProvider Given Get Matchers - A get request to /matchers
1.1) has a matching body
$.values -> Type mismatch: Expected ["a","bb","ccc"] (Array) to be the same type as {"a":"a","b":"bb","c":"ccc"} (Object)
pact-foundation/pact-php
✅ All checks have passed
26/26 successful checksGitHub
10/28/2023, 3:10 AMGitHub
10/28/2023, 3:22 AMJoshua Ellis
10/29/2023, 9:03 PMBruno Holanda
10/30/2023, 12:31 PMRuth
10/30/2023, 8:56 PMTien Vo
10/31/2023, 2:36 AMffi
branch that has 1 approval to speed up development on that branchTien Vo
10/31/2023, 3:26 AMCody Jenkins
11/01/2023, 2:39 AMGitHub
11/06/2023, 3:52 PMvalues
matcher, it compare values using its index (i.e. starting with 0) in the array.
• When you pass array with keys to values
matcher, it compare values using those keys.
• Missing keys/indexes are OK.
• Additional keys/indexes are NOT OK.
• I'm not sure about this behavior.
• Pact complain that the value of additional keys/indexes must equals to the first value (e.g. a
) ???
• Probably because I'm testing this values
matcher on Response
?
• If using this values
matcher on Request
, additional keys/indexes are OK. See this behavior on compatibility suite
I'm not really understand the behavior of missing/additional keys/indexes. Look like it's the Postel's law, but in revert?
pact-foundation/pact-php
✅ All checks have passed
26/26 successful checksBruno Holanda
11/08/2023, 12:08 PM$contents->song
from our ConsumerMessage example? Because I don't know exactly what is the content, I am worried about the schema of the message, like the song attribute.GitHub
11/16/2023, 1:50 AMGitHub
11/17/2023, 3:05 AMGitHub
11/21/2023, 3:13 AMGitHub
11/21/2023, 8:36 AMGitHub
11/28/2023, 2:49 AMGitHub
11/28/2023, 4:04 AMpactffi_response_status
only allow integer status.
But it can be useful to match a value in body (like this example), query parameter or header.
pact-foundation/pact-php
Cirrus CI: Task Summary
Cirrus CI: Task Summary
Cirrus CI: Task Summary
Cirrus CI: Task Summary
Cirrus CI: Task Summary
Cirrus CI: Task Summary
✅ 20 other checks have passed
20/26 successful checksGitHub
11/29/2023, 3:25 AMMockServerURL
yet, because the url
matcher is not available in pact-php (yet)
pact-foundation/pact-php
Cirrus CI: Task Summary
Cirrus CI: Task Summary
Cirrus CI: Task Summary
Cirrus CI: Task Summary
Cirrus CI: Task Summary
Cirrus CI: Task Summary
✅ 20 other checks have passed
20/26 successful checksGitHub
11/29/2023, 4:59 AMProcessText
and ProcessSong
are basically the same test. I combine them, then add matcher + generator into the example.
pact-foundation/pact-php
Cirrus CI: Task Summary
Cirrus CI: Task Summary
Cirrus CI: Task Summary
Cirrus CI: Task Summary
Cirrus CI: Task Summary
Cirrus CI: Task Summary
✅ 20 other checks have passed
20/26 successful checksGitHub
11/29/2023, 6:39 AMLukasz
11/29/2023, 6:54 AMGitHub
11/29/2023, 12:57 PM10.0.0-alpha.x
versions
• Change c-data from managed to unmanaged
pact-foundation/pact-php
Cirrus CI: Task Summary
Cirrus CI: Task Summary
Cirrus CI: Task Summary
Cirrus CI: Task Summary
Cirrus CI: Task Summary
Cirrus CI: Task Summary
✅ 20 other checks have passed
20/26 successful checksGitHub
11/29/2023, 1:44 PMStringData
to BinaryData
. This class is only used for binary now. So no need to make it flexible by supporting null-terminated string.
pact-foundation/pact-php
Cirrus CI: Task Summary
Cirrus CI: Task Summary
Cirrus CI: Task Summary
Cirrus CI: Task Summary
Cirrus CI: Task Summary
Cirrus CI: Task Summary
✅ 20 other checks have passed
20/26 successful checksGitHub
11/29/2023, 2:05 PMGitHub
11/30/2023, 2:01 AMFFI::free
for the first one (https://github.com/pact-foundation/pact-php/blob/ffi/src/PhpPact/FFI/Model/ArrayData.php#L58), but I'm not sure it's correctly handled.
• I don't know solution for the second one
pact-foundation/pact-phpGitHub
11/30/2023, 2:54 AMGitHub
11/30/2023, 3:27 AM2023-11-30
, the value become "2023-12-334T17:15:07"
, not "2023-11-334T17:15:07"
as expected. I'm not sure what is the problem here.
pact-foundation/pact-php
GitHub Actions: PHP 8.0 on macos-latest with locked dependencies
Cirrus CI: linux_arm64 VERSION:8.0 arm_containerphp8.0
✅ 24 other checks have passed
24/26 successful checksGitHub
11/30/2023, 9:59 AMpact-php
project. Outside of this project will be framework integration e.g. Symfony:
• https://github.com/tienvx/pact-messenger-bundle
• https://github.com/tienvx/pact-provider-bundle
• No test cases at all. This is because I'm not confident with the code structure yet.
pact-foundation/pact-php
GitHub Actions: PHP 8.2 on windows-latest with locked dependencies
GitHub Actions: PHP 8.2 on windows-latest with lowest dependencies
GitHub Actions: PHP 8.1 on windows-latest with locked dependencies
GitHub Actions: PHP 8.1 on windows-latest with lowest dependencies
GitHub Actions: PHP 8.0 on windows-latest with locked dependencies
GitHub Actions: PHP 8.0 on windows-latest with lowest dependencies
GitHub Actions: PHP 8.2 on macos-latest with locked dependencies
GitHub Actions: PHP 8.2 on macos-latest with lowest dependencies
GitHub Actions: PHP 8.1 on macos-latest with locked dependencies
GitHub Actions: PHP 8.1 on macos-latest with lowest dependencies
GitHub Actions: PHP 8.0 on macos-latest with locked dependencies
GitHub Actions: PHP 8.0 on macos-latest with lowest dependencies
GitHub Actions: PHP 8.2 on ubuntu-latest with locked dependencies
GitHub Actions: PHP 8.2 on ubuntu-latest with lowest dependencies
GitHub Actions: PHP 8.1 on ubuntu-latest with locked dependencies
GitHub Actions: PHP 8.1 on ubuntu-latest with lowest dependencies
GitHub Actions: PHP 8.0 on ubuntu-latest with locked dependencies
GitHub Actions: PHP 8.0 on ubuntu-latest with lowest dependencies
Cirrus CI: Task Summary
Cirrus CI: Task Summary
Cirrus CI: Task Summary
Cirrus CI: Task Summary
Cirrus CI: Task Summary
Cirrus CI: Task Summary
✅ 2 other checks have passed
2/26 successful checks