CI | Increase Timeouts

Signed-off-by: shirady <57721533+shirady@users.noreply.github.com>

Author: shirady

src/test/unit_tests/jest_tests/test_versioning_concurrency.test.js

@@ -10,6 +10,8 @@ const buffer_utils = require('../../../util/buffer_utils');
 const { TMP_PATH } = require('../../system_tests/test_utils');
 const { crypto_random_string } = require('../../../util/string_utils');
 const endpoint_stats_collector = require('../../../sdk/endpoint_stats_collector');
+const SECONDS = 1000;
+const test_timeout = SECONDS * 60;
 
 function make_dummy_object_sdk(nsfs_config, uid, gid) {
     return {
@@ -68,7 +70,7 @@ describe('test versioning concurrency', () => {
         expect(failed_operations).toHaveLength(0);
         const versions = await nsfs.list_object_versions({ bucket: bucket }, DUMMY_OBJECT_SDK);
         expect(versions.objects.length).toBe(num_of_concurrency);
-    });
+    }, test_timeout);
 
     it('multiple delete version id and key', async () => {
         const bucket = 'bucket1';
@@ -88,7 +90,7 @@ describe('test versioning concurrency', () => {
         await P.delay(1000);
         expect(successful_operations).toHaveLength(num_of_concurrency);
         expect(failed_operations).toHaveLength(0);
-    });
+    }, test_timeout);
 
     // same as s3tests_boto3/functional/test_s3.py::test_versioning_concurrent_multi_object_delete, 
     // this test has a bug, it tries to create the bucket twice and fails
@@ -130,7 +132,7 @@ describe('test versioning concurrency', () => {
         }
         const list_res = await nsfs.list_objects({ bucket: bucket }, DUMMY_OBJECT_SDK);
         expect(list_res.objects.length).toBe(0);
-    }, 8000);
+    }, test_timeout);
 
     it('concurrent delete of latest version', async () => {
         const bucket = 'bucket1';
@@ -154,7 +156,7 @@ describe('test versioning concurrency', () => {
         expect(versions.objects.length).toBe(8); // 5 versions before + 3 delete markers concurrent
         const delete_marker_arr = versions.objects.filter(object => object.delete_marker === true);
         expect(delete_marker_arr.length).toBe(3);
-    });
+    }, test_timeout);
 
     it('concurrent put object and head object latest version', async () => {
         const bucket = 'bucket1';
@@ -182,7 +184,7 @@ describe('test versioning concurrency', () => {
         expect(successful_head_operations.length).toBe(number_of_iterations);
         const versions = await nsfs.list_object_versions({ bucket: bucket }, DUMMY_OBJECT_SDK);
         expect(versions.objects.length).toBe(number_of_iterations + 1); // 1 version before + 10 versions concurrent
-    });
+    }, test_timeout);
 
     it('concurrent puts & delete latest objects', async () => {
         const bucket = 'bucket1';
@@ -216,7 +218,7 @@ describe('test versioning concurrency', () => {
         expect(num_of_delete_markers).toBe(num_of_concurrency);
         const num_of_latest_versions = (versions.objects.filter(version => version.is_latest === true)).length;
         expect(num_of_latest_versions).toBe(1);
-    }, 6000);
+    }, test_timeout);
 
     it('concurrent puts & delete objects by version id', async () => {
         const bucket = 'bucket1';
@@ -250,7 +252,7 @@ describe('test versioning concurrency', () => {
         expect(num_of_delete_markers).toBe(0);
         const num_of_latest_versions = (versions.objects.filter(version => version.is_latest === true)).length;
         expect(num_of_latest_versions).toBe(1);
-    }, 6000);
+    }, test_timeout);
 
     it('concurrent delete objects by version id/latest', async () => {
         const bucket = 'bucket1';
@@ -283,7 +285,7 @@ describe('test versioning concurrency', () => {
         expect(num_of_delete_markers).toBe(num_of_concurrency);
         const num_of_latest_versions = (versions.objects.filter(version => version.is_latest === true)).length;
         expect(num_of_latest_versions).toBe(1);
-    }, 6000);
+    }, test_timeout);
 
     it('nested key - concurrent delete multiple objects', async () => {
         const bucket = 'bucket1';
@@ -320,8 +322,7 @@ describe('test versioning concurrency', () => {
         }
         const list_res = await nsfs.list_objects({ bucket: bucket }, DUMMY_OBJECT_SDK);
         expect(list_res.objects).toHaveLength(0);
-    }, 8000);
-
+    }, test_timeout);
 
     it('nested key - concurrent puts & deletes', async () => {
         const bucket = 'bucket1';
@@ -349,7 +350,7 @@ describe('test versioning concurrency', () => {
         expect(upload_failed_operations).toHaveLength(0);
         expect(delete_successful_operations).toHaveLength(num_of_concurrency);
         expect(delete_failed_operations).toHaveLength(0);
-    }, 6000);
+    }, test_timeout);
 
     it('concurrent puts & list versions', async () => {
         const bucket = 'bucket1';
@@ -389,7 +390,7 @@ describe('test versioning concurrency', () => {
         expect(num_of_delete_markers).toBe(0);
         const num_of_latest_versions = (versions.objects.filter(version => version.is_latest === true)).length;
         expect(num_of_latest_versions).toBe(initial_num_of_objects);
-    }, 60000);
+    }, test_timeout);
 
     it('concurrent puts & list versions - version id paging', async () => {
         const bucket = 'bucket1';
@@ -432,7 +433,7 @@ describe('test versioning concurrency', () => {
         expect(num_of_delete_markers).toBe(0);
         const num_of_latest_versions = (merged_versions.filter(version => version.is_latest === true)).length;
         expect(num_of_latest_versions).toBe(initial_num_of_objects);
-    }, 50000);
+    }, test_timeout);
 });
 
 /**

src/test/unit_tests/test_bucketspace_versioning.js

@@ -71,7 +71,7 @@ mocha.describe('bucketspace namespace_fs - versioning', function() {
     const suspended_dir1_versions_path = path.join(suspended_full_path, dir1, '.versions/');
 
     mocha.before(async function() {
-        this.timeout(600000); // eslint-disable-line no-invalid-this
+        this.timeout(0); // eslint-disable-line no-invalid-this
         if (invalid_nsfs_root_permissions()) this.skip(); // eslint-disable-line no-invalid-this
         // create paths 
         await fs_utils.create_fresh_path(tmp_fs_root, 0o777);
@@ -158,6 +158,7 @@ mocha.describe('bucketspace namespace_fs - versioning', function() {
     });
 
     mocha.after(async () => {
+        this.timeout(0); // eslint-disable-line no-invalid-this
         fs_utils.folder_delete(tmp_fs_root);
         for (const email of accounts) {
             await rpc_client.account.delete_account({ email });
@@ -2285,7 +2286,7 @@ mocha.describe('bucketspace namespace_fs - versioning', function() {
         const version_body = 'A1A1A1A';
 
         mocha.before(async function() {
-            this.timeout(600000); // eslint-disable-line no-invalid-this
+            this.timeout(0); // eslint-disable-line no-invalid-this
             if (invalid_nsfs_root_permissions()) this.skip(); // eslint-disable-line no-invalid-this
             // create paths
             await fs_utils.create_fresh_path(tmp_fs_root2, 0o777);
@@ -2355,6 +2356,7 @@ mocha.describe('bucketspace namespace_fs - versioning', function() {
         });
 
         mocha.after(async () => {
+            this.timeout(0); // eslint-disable-line no-invalid-this
             fs_utils.folder_delete(tmp_fs_root);
             for (const email of accounts) {
                 await rpc_client.account.delete_account({ email });
@@ -2953,7 +2955,7 @@ mocha.describe('List-objects', function() {
     let file_pointer;
 
     mocha.before(async function() {
-        this.timeout(600000); // eslint-disable-line no-invalid-this
+        this.timeout(0); // eslint-disable-line no-invalid-this
         if (process.getgid() !== 0 || process.getuid() !== 0) {
             console.log('No Root permissions found in env. Skipping test');
             this.skip(); // eslint-disable-line no-invalid-this
@@ -3038,6 +3040,7 @@ mocha.describe('List-objects', function() {
     });
 
     mocha.after(async () => {
+        this.timeout(0); // eslint-disable-line no-invalid-this
         if (file_pointer) await file_pointer.close(DEFAULT_FS_CONFIG);
         fs_utils.folder_delete(tmp_fs_root);
         for (const email of accounts) {
@@ -3046,6 +3049,7 @@ mocha.describe('List-objects', function() {
     });
 
     mocha.beforeEach(async () => {
+        this.timeout(0); // eslint-disable-line no-invalid-this
         await fs_utils.create_fresh_path(full_path2, 0o777);
         await P.delay(100); // sometime we saw that the check failed although the path is created a line before
         const file_exists = await fs_utils.file_exists(full_path2);