From cd159d993a8b0a4e6c1e58dfff9d58247549be06 Mon Sep 17 00:00:00 2001 From: lihsai0 Date: Mon, 4 Mar 2024 15:32:12 +0800 Subject: [PATCH] fix test cases --- test/form_up.test.js | 20 +++++++++++++------- test/resume_up.test.js | 38 +++++++++++++++++++++++++------------- test/rs.test.js | 4 ++-- 3 files changed, 40 insertions(+), 22 deletions(-) diff --git a/test/form_up.test.js b/test/form_up.test.js index 7020ec7..16ee4fb 100644 --- a/test/form_up.test.js +++ b/test/form_up.test.js @@ -27,13 +27,19 @@ before(function () { }); after(function () { - return Promise.all([ - fs.promises.unlink(testFilePath1), - fs.promises.unlink(testFilePath2) - ]) - .catch(err => { - console.log('Form upload test. Unlink files failed', err); - }); + return Promise.all( + [ + testFilePath1, + testFilePath2 + ].map(p => new Promise(resolve => { + fs.unlink(p, err => { + if (err) { + console.log(`unlink ${p} failed`, err); + } + resolve(); + }); + })) + ); }); describe('test form up', function () { diff --git a/test/resume_up.test.js b/test/resume_up.test.js index 20b88e1..fc87148 100644 --- a/test/resume_up.test.js +++ b/test/resume_up.test.js @@ -67,10 +67,18 @@ before(function () { }); after(function () { - return fs.promises.unlink(testFilePath) - .catch(err => { - console.log('Resume upload test. Unlink files failed', err); - }); + return Promise.all( + [ + testFilePath + ].map(p => new Promise(resolve => { + fs.unlink(p, err => { + if (err) { + console.log(`unlink ${p} failed`, err); + } + resolve(); + }); + })) + ); }); // file to upload @@ -250,7 +258,7 @@ describe('test resume up', function () { if (mimeType !== undefined) { putExtra.mimeType = mimeType; } - const key = 'storage_putFile_test' + Math.floor(Math.random() * 1000); + const key = 'storage_putFile_test' + Math.floor(Math.random() * 100000); const promises = doAndWrapResultPromises(callback => resumeUploader.putFile( @@ -325,7 +333,7 @@ describe('test resume up', function () { putExtra.mimeType = mimeType; } - const key = 'storage_putStream_test' + Math.floor(Math.random() * 1000); + const key = 'storage_putStream_test' + Math.floor(Math.random() * 100000); const streamSize = 9 * 1024 * 1024; const { @@ -397,12 +405,16 @@ describe('test resume up', function () { const filepathListToDelete = []; after(function () { - return Promise.all(filepathListToDelete.map(p => - fs.promises.unlink(p) - .catch(() => { - // pass - }) - )); + return Promise.all( + filepathListToDelete.map(p => new Promise(resolve => { + fs.unlink(p, err => { + if (err) { + console.log(`unlink ${p} failed`, err); + } + resolve(); + }); + })) + ); }); testParams.forEach(testParam => { @@ -425,7 +437,7 @@ describe('test resume up', function () { } it(`test resume up#putStream resume; ${msg}`, function () { - const key = 'storage_putStream_resume_test' + Math.floor(Math.random() * 1000); + const key = 'storage_putStream_resume_test' + Math.floor(Math.random() * 100000); const putExtra = new qiniu.resume_up.PutExtra(); putExtra.resumeRecordFile = path.join(os.tmpdir(), key + '.resume.json'); diff --git a/test/rs.test.js b/test/rs.test.js index db4edb6..b47d0ac 100644 --- a/test/rs.test.js +++ b/test/rs.test.js @@ -522,7 +522,7 @@ describe('test start bucket manager', function () { // 空间生命周期 describe('test lifeRule', function () { - const ruleName = 'test_rule_name' + Math.floor(Math.random() * 1000); + const ruleName = 'test_rule_name' + Math.floor(Math.random() * 100000); function testGet (expectItem, otherRespInfo) { return bucketManager.getBucketLifecycleRule(bucketName) @@ -734,7 +734,7 @@ describe('test start bucket manager', function () { }); describe('test events', function () { - const eventName = 'event_test' + Math.floor(Math.random() * 1000); + const eventName = 'event_test' + Math.floor(Math.random() * 100000); before(function () { return bucketManager.deleteBucketEvent(