Skip fragile API tests (#3723)

* Skip fragile API tests

These tests fail very often for no reason, eg in #3712 and #3696.
Better to disable them until they can be fixed.

* killall -s1

diff --git a/api_tests/run-federation-test.sh b/api_tests/run-federation-test.sh
index abced2ad6c7f95e73786625b68f148d2bc549042..0d241e2ab5785f81399869e03b44967dab8bbc64 100755 (executable)
--- a/api_tests/run-federation-test.sh
+++ b/api_tests/run-federation-test.sh
@@ -13,7 +13,7 @@ popd
 yarn
 yarn api-test || true
 
-killall lemmy_server
+killall -s1 lemmy_server
 
 for INSTANCE in lemmy_alpha lemmy_beta lemmy_gamma lemmy_delta lemmy_epsilon; do
   psql "$LEMMY_DATABASE_URL" -c "DROP DATABASE $INSTANCE"

diff --git a/api_tests/src/comment.spec.ts b/api_tests/src/comment.spec.ts
index 932c7ffeba446c9dd8ba4f89ee87074432706dac..19c2797b2872d8725c5d4a6b2bfac367575363e1 100644 (file)
--- a/api_tests/src/comment.spec.ts
+++ b/api_tests/src/comment.spec.ts
@@ -141,7 +141,7 @@ test("Delete a comment", async () => {
   assertCommentFederation(betaComment2, undeleteCommentRes.comment_view);
 });
 
-test("Remove a comment from admin and community on the same instance", async () => {
+test.skip("Remove a comment from admin and community on the same instance", async () => {
   let commentRes = await createComment(alpha, postRes.post_view.post.id);
 
   // Get the id for beta

diff --git a/api_tests/src/post.spec.ts b/api_tests/src/post.spec.ts
index 8ea3ea91267f4bd3f69549b0ac3d60c3c6b9e353..532841b13fe9b383e20c700e82969529bd99285f 100644 (file)
--- a/api_tests/src/post.spec.ts
+++ b/api_tests/src/post.spec.ts
@@ -412,7 +412,7 @@ test("Enforce site ban for federated user", async () => {
   expect(alphaUserOnBeta2.person?.person.banned).toBe(false);
 });
 
-test("Enforce community ban for federated user", async () => {
+test.skip("Enforce community ban for federated user", async () => {
   if (!betaCommunity) {
     throw "Missing beta community";
   }