test: test coverage for trying to delete deleted user

anon-pradip · anon-pradip · commit 935ba4cc4a5b · 2025-02-17T15:59:47.000+05:45
diff --git a/tests/acceptance/bootstrap/OcmContext.php b/tests/acceptance/bootstrap/OcmContext.php
@@ -264,6 +264,7 @@ public function theUserWaitsForTokenToExpire(int $number): void {
 
 	/**
 	 * @When user :user deletes federated connection with user :ocmUser using the Graph API
+	 * @When user :user tries to delete federated connection with user :ocmUser using the Graph API
 	 *
 	 * @param string $user
 	 * @param string $ocmUser
diff --git a/tests/acceptance/expected-failures-localAPI-on-OCIS-storage.md b/tests/acceptance/expected-failures-localAPI-on-OCIS-storage.md
@@ -325,5 +325,8 @@ The expected failures in this file are from features in the owncloud/ocis repo.
 - [apiServiceAvailability/serviceAvailabilityCheck.feature:120](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiServiceAvailability/serviceAvailabilityCheck.feature#L120)
 - [apiServiceAvailability/serviceAvailabilityCheck.feature:131](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiServiceAvailability/serviceAvailabilityCheck.feature#L131)
 
+#### [OCM. delete-accepted-user endpoint always returns 200](https://github.com/owncloud/ocis/issues/10223)
+- [apiOcm/deleteFederatedConnections.feature:137](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiOcm/deleteFederatedConnections.feature#L137)
+
 Note: always have an empty line at the end of this file.
 The bash script that processes this file requires that the last line has a newline on the end.
diff --git a/tests/acceptance/features/apiOcm/deleteFederatedConnections.feature b/tests/acceptance/features/apiOcm/deleteFederatedConnections.feature
@@ -132,3 +132,13 @@ Feature: delete federated connections
         }
       }
       """
+
+  @issue-10223
+  Scenario: federated user tries to delete previously deleted federated connection
+    Given using server "LOCAL"
+    And "Alice" has created the federation share invitation
+    And using server "REMOTE"
+    And "Brian" has accepted invitation
+    And user "Brian" has deleted federated connection with user "Alice"
+    When user "Brian" tries to delete federated connection with user "Alice" using the Graph API
+    Then the HTTP status code should be "404"

Original file line number	Diff line number	Diff line change
`@@ -264,6 +264,7 @@ public function theUserWaitsForTokenToExpire(int $number): void {`
`264`	`264`
`265`	`265`	`/**`
`266`	`266`	`* @When user :user deletes federated connection with user :ocmUser using the Graph API`
	`267`	`+ * @When user :user tries to delete federated connection with user :ocmUser using the Graph API`
`267`	`268`	`*`
`268`	`269`	`* @param string $user`
`269`	`270`	`* @param string $ocmUser`
Original file line number	Diff line number	Diff line change
`@@ -132,3 +132,13 @@ Feature: delete federated connections`
`132`	`132`	`}`
`133`	`133`	`}`
`134`	`134`	`"""`
	`135`	`+`
	`136`	`+ @issue-10223`
	`137`	`+ Scenario: federated user tries to delete previously deleted federated connection`
	`138`	`+ Given using server "LOCAL"`
	`139`	`+ And "Alice" has created the federation share invitation`
	`140`	`+ And using server "REMOTE"`
	`141`	`+ And "Brian" has accepted invitation`
	`142`	`+ And user "Brian" has deleted federated connection with user "Alice"`
	`143`	`+ When user "Brian" tries to delete federated connection with user "Alice" using the Graph API`
	`144`	`+ Then the HTTP status code should be "404"`