:wave::skin-tone-4: all - i'm helping a dev with a...
# libpact_ffi-users
h
👋🏽 all - i'm helping a dev with a failed build relating to a pact test. nothing seems to have changed on our end but got a
dummy
error message.... i hope it's not us. any ideas in thread
Copy code
2022/12/10 00:10:51 [DEBUG] pact setup
2022/12/10 00:10:51 [DEBUG] initialising rust mock server interface
2022/12/10 00:10:51 [DEBUG] log_to_stdout res 0
2022/12/10 00:10:51 [DEBUG] pact add V3 interaction
2022-12-10T00:10:51.555975Z ERROR ThreadId(01) pact_ffi::mock_server::bodies: Failed to parse matching rule from JSON - dummy is not a valid matching rule type    
2022/12/10 00:10:51 [DEBUG] pact verify
2022/12/10 00:10:51 [DEBUG] mock server starting on address: 127.0.0.1:0
2022/12/10 00:10:51 [DEBUG] mock server running on port: 38365
Copy code
libpact_ffi:
    libname: libpact_ffi
    version: 0.3.15
    hash: e4babe9bd82761f01f595a9fd31a0eb8
fwiw, it's just an error showing red and doesnt seem to fail the build.... we noticed this along with another test failure which we just cleaned up
also in libpact_ffi-v0.3.14 according to our builds
m
That is strange. My guess is it's in Pact Go. I'll check tonight
OK because of reasons, all things that can accept a Matcher need to be JSON serialisable. This meant that plain string values got escaped with a
"
so I must have used a dummy matcher a few years back. It seems like it doesn’t cause a functional issues, but I’ve put in a minor fix that essentially detects a string value there and prevents the additional quotes.
It’s in the latest
2.x.x
that has not been released yet (i.e. tagged)
Would you mind seeing if that makes the issue go away?
h
yep - i'll take a look this morning.
@Matt (pactflow.io / pact-js / pact-go) - looks good now! I was able to replicate the issue using the prior pact-go 2.x.x version only after updating my local
libpact_ffi
to
0.3.15
. And ultimately verified your fix on the head of branch 2.x.x. Thanks for fixing!
correction, it was
v2.0.0-beta.17.0.20221212070231-76032bbaf2b6
i verified.
m
Awesome, thank you! I’ll get a new release out in the next couple of days