Skip to content

[stable32] test(integration): wire published behat TSA extension #7397

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
vitormattos merged 15 commits into from
Apr 4, 2026
Merged

[stable32] test(integration): wire published behat TSA extension #7397

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
683285e
chore(tests): require behat tsa extension package
vitormattos Apr 3, 2026
fdf04c6
chore(tests): update integration lock for tsa extension
vitormattos Apr 3, 2026
5338162
test(integration): register TSA behat extension
vitormattos Apr 3, 2026
835762e
test(integration): parameterize admin TSA URL
vitormattos Apr 3, 2026
9cb2b06
test(integration): resolve TSA URL placeholder from env
vitormattos Apr 3, 2026
1b976f6
test(integration): parameterize tsa url in signed scenario
vitormattos Apr 3, 2026
df823ca
test(integration): parameterize tsa url in tsa workflow
vitormattos Apr 3, 2026
95de009
fix(integration): use admin TSA API with <TSA_URL> placeholder in sig…
vitormattos Apr 4, 2026
6c7f1de
fix(integration): use admin TSA API and local TSA cnHints in tsa work…
vitormattos Apr 4, 2026
f67b4de
chore: update behat tsa extension lock
vitormattos Apr 4, 2026
e816ce3
test: restore tsa extension config
vitormattos Apr 4, 2026
a75c79b
test: align admin tsa feature payload
vitormattos Apr 4, 2026
c4788eb
test: configure tsa policy in signed feature
vitormattos Apr 4, 2026
480143f
test: refresh tsa integration assertions
vitormattos Apr 4, 2026
5939183
test: use tsa extension defaults in behat config
vitormattos Apr 4, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion tests/integration/composer.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,8 @@
"php-http/guzzle7-adapter": "^1.0",
"php-http/message": "^1.16",
"libresign/nextcloud-behat": "^1.4",
"libresign/mailpit-behat-extension": "^0.1.1"
"libresign/mailpit-behat-extension": "^0.1.1",
"libresign/behat-tsa-extension": "^0.1.0"
},
"config": {
"allow-plugins": {
Expand Down
54 changes: 54 additions & 0 deletions tests/integration/composer.lock
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions tests/integration/config/behat.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,3 +20,4 @@ default:
jarnaiz\JUnitFormatter\JUnitFormatterExtension:
filename: report.xml
outputDir: '%paths.base%/../output/'
LibreSign\Behat\TsaExtension\ServiceContainer\TsaExtension: ~
6 changes: 3 additions & 3 deletions tests/integration/features/admin/tsa.feature
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,8 @@ Feature: TSA Administration - Core Configuration
Given as user "admin"

When sending "post" to ocs "/apps/libresign/api/v1/admin/tsa"
| tsa_url | https://freetsa.org/tsr |
| tsa_policy | 1.2.3.4.1 |
| tsa_url | <TSA_URL> |
| tsa_policy_oid | 1.2.3.4.1 |
| tsa_auth_type | none |
Then the response should have a status code 200
And the response should be a JSON array with the following mandatory values
Expand All @@ -16,7 +16,7 @@ Feature: TSA Administration - Core Configuration
Then the response should have a status code 200
And the response should be a JSON array with the following mandatory values
| key | value |
| (jq).ocs.data.data | https://freetsa.org/tsr |
| (jq).ocs.data.data | <TSA_URL> |

When sending "delete" to ocs "/apps/libresign/api/v1/admin/tsa"
Then the response should have a status code 200
Expand Down
1 change: 1 addition & 0 deletions tests/integration/features/bootstrap/FeatureContext.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,7 @@ protected function beforeRequest(string $fullUrl, array $options): array {
protected function parseText(string $text): string {
$fields = $this->fields;
$fields['BASE_URL'] = $this->baseUrl . '/index.php';
$fields['TSA_URL'] = getenv('LIBRESIGN_TSA_URL') ?: 'https://freetsa.org/tsr';
foreach ($fields as $key => $value) {
$patterns[] = '/<' . $key . '>/';
$replacements[] = $value;
Expand Down
6 changes: 5 additions & 1 deletion tests/integration/features/sign/signed.feature
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,11 @@ Feature: signed
And run the command "libresign:configure:openssl --cn=Common\ Name --c=BR --o=Organization --st=State\ of\ Company --l=City\ Name --ou=Organization\ Unit" with result code 0
And run the command "config:app:set libresign add_footer --value=true --type=boolean" with result code 0
And run the command "config:app:set libresign write_qrcode_on_footer --value=true --type=boolean" with result code 0
And run the command "config:app:set libresign tsa_url --value=https://freetsa.org/tsr --type=string" with result code 0
And sending "post" to ocs "/apps/libresign/api/v1/admin/tsa"
| tsa_url | <TSA_URL> |
| tsa_policy_oid | 1.2.3.4.1 |
| tsa_auth_type | none |
And the response should have a status code 200
And sending "post" to ocs "/apps/provisioning_api/api/v1/config/apps/libresign/identify_methods"
| value | (string)[{"name":"account","enabled":true,"mandatory":true,"signatureMethods":{"password":{"name":"password","enabled":true}},"signatureMethodEnabled":"password"}] |
And the response should have a status code 200
Expand Down
15 changes: 7 additions & 8 deletions tests/integration/features/sign/tsa.feature
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,11 @@ Feature: TSA Integration - End-to-End Workflow

Scenario: TSA workflow - Successfully signs document with timestamp
Given run the command "config:app:set libresign signing_mode --value=sync --type=string" with result code 0
And run the command "config:app:set libresign tsa_url --value=https://freetsa.org/tsr --type=string" with result code 0
And run the command "config:app:set libresign tsa_auth_type --value=none --type=string" with result code 0
And sending "post" to ocs "/apps/libresign/api/v1/admin/tsa"
| tsa_url | <TSA_URL> |
| tsa_policy_oid | 1.2.3.4.1 |
| tsa_auth_type | none |
And the response should have a status code 200
And sending "post" to ocs "/apps/provisioning_api/api/v1/config/apps/libresign/identify_methods"
| value | (string)[{"name":"account","enabled":true,"mandatory":true,"signatureMethods":{"clickToSign":{"enabled":true}},"signatureMethodEnabled":"clickToSign"}] |
And the response should have a status code 200
Expand Down Expand Up @@ -49,12 +52,8 @@ Feature: TSA Integration - End-to-End Workflow
| key | value |
| (jq).ocs.data.signers[0].timestamp.serialNumber \|test("^[0-9]+$") | true |
And the response should be a JSON array with the following mandatory values
| key | value |
| (jq).ocs.data.signers[0].timestamp.genTime \|test("^[0-9]{4}-[0-9]{2}-[0-9]{2}T[0-9]{2}:[0-9]{2}:[0-9]{2}") | true |
And the response should be a JSON array with the following mandatory values
| key | value |
| (jq).ocs.data.signers[0].timestamp.cnHints.commonName | www.freetsa.org |
| (jq).ocs.data.signers[0].timestamp.cnHints.countryName | DE |
| key | value |
| (jq).ocs.data.signers[0].timestamp.cnHints.commonName \|test("LibreSign Local TSA") | true |

Scenario: TSA error handling - Invalid server
Given run the command "config:app:set libresign tsa_url --value=https://invalid-tsa-server.example.com/tsr --type=string" with result code 0
Expand Down
Loading