pact-foundation / pact-reference

Reference implementations for the pact specifications
https://pact.io
MIT License
91 stars 46 forks source link

pactffi_verifier_add_directory_source does not filter files based on provider name #233

Closed YOU54F closed 1 year ago

YOU54F commented 1 year ago

Issue:

pactffi_verifier_add_directory_source does not filter files based on provider name.

Expected:

If using pactffi_verifier_add_directory_source, based on the documentation

Adds a Pact directory as a source to verify. All pacts from the directory that match the provider name will be verified.

  1. Does this mean any pact files with the name that matchers the provider name in pactffi_verifier_set_provider_info
  2. Does this mean any json files, that contain a provider.name value that matches that set in pactffi_verifier_set_provider_info

I think I would expect

Actual:

  1. pactffi_verifier_set_provider_info has a name of a provider that does not exist
  2. No pactffi_verifier_set_consumer_filters are applied
  3. pactffi_verifier_add_directory_source selects all pacts for verification

Screenshot 2022-12-06 at 14 31 51

rholshausen commented 1 year ago

/jira ticket

github-actions[bot] commented 1 year ago

đŸ‘‹ Thanks, Jira [PACT-477] ticket created.

rholshausen commented 1 year ago

It is meant to use the provider name from within the Pact file (provider.name), not the filename.