Skip to content

Revert KYC_Match_6 test scenario to pre-Fall25 version - #95

Merged
GillesInnov35 merged 1 commit into
camaraproject:mainfrom
ToshiWakayama-KDDI:Revert-KYC-Match-6-test-scenario-to-pre-Fall25-version
Oct 2, 2026
Merged

GillesInnov35 merged 1 commit into
camaraproject:mainfrom
ToshiWakayama-KDDI:Revert-KYC-Match-6-test-scenario-to-pre-Fall25-version

Conversation

@ToshiWakayama-KDDI

Copy link
Copy Markdown
Contributor

What type of PR is this?

  • tests

What this PR does / why we need it:

  • During the final phase of the Fall25 meta-release in last Autumn, one of the test scenarios included in the test scenario of kyc-match .feature, i.e. @KYC_Match_6_success_multiple_optional_parameter_combinations, was modified with simplified expressions, in order to avoid some errors/warnings that the linting rules had detected.

  • Now the linting rules have been modified in the Commonalities WG, so, thie PR will revert the expressions of the test scenario back to the original one, because the original one explains the test process more clearly.

Which issue(s) this PR fixes:

Fixes #74

Special notes for reviewers:

  • This PR does not include new changes for Sync26 version, e.g. addressing additional properties.

Changelog input

 release-note
Revert KYC_Match_6 test scenario to pre-Fall25 version

Additional documentation

  • The same changes should be made to the Fall25 version test scenario by a patch release.

@GillesInnov35

Copy link
Copy Markdown
Contributor

hi @ToshiWakayama-KDDI , is there any reason why some properties are not incuded ? Apart from this question, LGTM

              idDocumentType: passport
              idDocumentExpiryDate: '2027-07-12'
              bankAccountNumber: IBAN:NL59ABNA0123456789

              cityOfBirth: Madrid
              countryOfBirth: ES

@PedroDiez PedroDiez left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@fernandopradocabrillo fernandopradocabrillo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@GillesInnov35
GillesInnov35 merged commit 7aeadc6 into camaraproject:main Oct 2, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Revert a test scenario of kyc-match to the original one

4 participants