Merge pull request #2522 from LibreSign/feature/performance-improveme…

…nt-at-integration-tests Performance improvement at integration tests
LibreSign · Mar 13, 2024 · aa434d4 · aa434d4
2 parents 2e6a664 + 9b87a48
commit aa434d4
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 11 deletions.
diff --git a/tests/integration/features/account/create_to_sign.feature b/tests/integration/features/account/create_to_sign.feature
@@ -1,7 +1,6 @@
 Feature: account/create_to_sign
   Background:
     Given as user "admin"
-    And run the command "libresign:developer:reset --all"
     And run the command "libresign:configure:openssl --cn test"
     And sending "post" to ocs "/apps/provisioning_api/api/v1/config/apps/libresign/identify_methods"
       | value | (string)[{"name":"email","enabled":true,"mandatory":true,"can_create_account":true}] |

diff --git a/tests/integration/features/bootstrap/FeatureContext.php b/tests/integration/features/bootstrap/FeatureContext.php
@@ -21,15 +21,15 @@ class FeatureContext extends NextcloudApiContext implements OpenedEmailStorageAw
 	 * @BeforeSuite
 	 */
 	public static function beforeSuite(BeforeSuiteScope $scope) {
-		exec('php ../../../../occ config:system:set debug --value true --type boolean', $output);
+		self::runCommand('config:system:set debug --value true --type boolean');
+		self::runCommand('app:enable --force notifications');
 	}
 
 	/**
-	 * @BeforeFeature
+	 * @BeforeScenario
 	 */
-	public static function BeforeFeature(): void {
+	public static function BeforeScenario(): void {
 		self::runCommand('libresign:developer:reset --all');
-		self::runCommand('app:enable --force notifications');
 	}
 
 	/**

diff --git a/tests/integration/features/sign/request.feature b/tests/integration/features/sign/request.feature
@@ -96,7 +96,6 @@ Feature: request-signature
   Scenario: Request to sign with error when the link was expired
     Given as user "admin"
     And my inbox is empty
-    And run the command "libresign:developer:reset --all"
     And run the command "libresign:configure:openssl --cn test"
     And run the command "config:app:set libresign maximum_validity --value 1"
     When sending "post" to ocs "/apps/libresign/api/v1/request-signature"
@@ -119,7 +118,6 @@ Feature: request-signature
   Scenario: Request to sign with success when is necessary to renew the link
     Given as user "admin"
     And my inbox is empty
-    And run the command "libresign:developer:reset --all"
     And run the command "libresign:configure:openssl --cn test"
     And sending "post" to ocs "/apps/provisioning_api/api/v1/config/apps/libresign/identify_methods"
       | value | (string)[{"name":"email","enabled":true,"mandatory":true,"can_create_account":false}] |
@@ -184,7 +182,6 @@ Feature: request-signature
   Scenario: Request to sign with success using account as identifier
     Given as user "admin"
     And user "signer1" exists
-    And run the command "libresign:developer:reset --all"
     And run the command "libresign:configure:openssl --cn test"
     And set the email of user "signer1" to "[email protected]"
     And reset notifications of user "signer1"
@@ -206,7 +203,6 @@ Feature: request-signature
 
   Scenario: Request to sign with error using account as identifier with invalid email
     Given as user "admin"
-    And run the command "libresign:developer:reset --all"
     And run the command "libresign:configure:openssl --cn test"
     When sending "post" to ocs "/apps/libresign/api/v1/request-signature"
       | file | {"url":"<BASE_URL>/apps/libresign/develop/pdf"} |
@@ -320,8 +316,7 @@ Feature: request-signature
     And I open the latest email to "[email protected]" with subject "LibreSign: There is a file for you to sign"
 
   Scenario: CRUD of identify methods
-    Given run the command "libresign:developer:reset --all"
-    And run the command "libresign:configure:openssl --cn test"
+    Given run the command "libresign:configure:openssl --cn test"
     And user "signer1" exists
     And as user "admin"
     When I send a file to be signed