fix(test): test case optimized (#1044)

ali-sdk · Jan 26, 2022 · e8eed0a · e8eed0a
1 parent 4fc3bd4
commit e8eed0a
Show file tree

Hide file tree

Showing 2 changed files with 31 additions and 22 deletions.
diff --git a/test/browser/browser-utils.js b/test/browser/browser-utils.js
@@ -2,7 +2,7 @@
 const platform = require('platform');
 
 if (process && process.browser) {
-  exports.prefix = `${platform.name}-${platform.version}/`;
+  exports.prefix = `${platform.name}-${platform.version}-${new Date().valueOf()}/`;
 } else {
   exports.prefix = `${process.platform}-${process.version}/`;
   if (process && process.execPath.indexOf('iojs') >= 0) {

diff --git a/test/browser/browser.test.js b/test/browser/browser.test.js
@@ -70,9 +70,6 @@ describe('browser', () => {
   });
 
   describe('endpoint', () => {
-    it('should init with region', () => {
-      console.log('xxx');
-    });
     it('should init with region', () => {
       let store = oss({
         accessKeyId: 'foo',
@@ -805,8 +802,11 @@ describe('browser', () => {
     // let otherBucketObject;
     let store;
     before(async () => {
-      name = `${prefix}ali-sdk/oss/copy-meta.js`;
       store = oss(ossConfig);
+    });
+
+    it('should copy object from same bucket', async () => {
+      name = `${prefix}ali-sdk/oss/copy-sameBucket-meta.js`;
       const object = await store.put(name, Buffer.from('abc'), {
         meta: {
           uid: 1,
@@ -815,9 +815,6 @@ describe('browser', () => {
         }
       });
       assert.equal(typeof object.res.headers['x-oss-request-id'], 'string');
-    });
-
-    it('should copy object from same bucket', async () => {
       const originname = `${prefix}ali-sdk/oss/copy-new.js`;
       const result = await store.copy(originname, name);
       assert.equal(result.res.status, 200);
@@ -908,6 +905,16 @@ describe('browser', () => {
     });
 
     it('should copy object and set other meta', async () => {
+      name = `${prefix}ali-sdk/oss/copy-setOther-meta.js`;
+      const object = await store.put(name, Buffer.from('abc'), {
+        meta: {
+          uid: 3,
+          pid: '123',
+          slus: 'test.html'
+        }
+      });
+      assert.equal(typeof object.res.headers['x-oss-request-id'], 'string');
+
       const originname = `${prefix}ali-sdk/oss/copy-new-2.js`;
       const result = await store.copy(originname, name, {
         meta: {
@@ -927,6 +934,15 @@ describe('browser', () => {
     });
 
     it('should use copy to change exists object headers', async () => {
+      name = `${prefix}ali-sdk/oss/copy-objectHeader-meta.js`;
+      const object = await store.put(name, Buffer.from('abc'), {
+        meta: {
+          uid: 5,
+          pid: '123',
+          slus: 'test.html'
+        }
+      });
+      assert.equal(typeof object.res.headers['x-oss-request-id'], 'string');
       const originname = `${prefix}ali-sdk/oss/copy-new-3.js`;
       let result = await store.copy(originname, name);
       assert.equal(result.res.status, 200);
@@ -2362,20 +2378,13 @@ describe('browser', () => {
       const client = oss(ossConfig);
       const copyName = 'multipart copy';
       const key = 'old.txt';
-      const result = await client.multipartUploadCopy(
-        copyName,
-        {
-          sourceKey: key,
-          sourceBucketName: stsConfig.bucket
-        },
-        {
-          parallel: 4,
-          partSize: 1024 * 1024,
-          progress: p => {
-            console.log(p);
-          }
-        }
-      );
+      const result = await client.multipartUploadCopy(copyName, {
+        sourceKey: key,
+        sourceBucketName: stsConfig.bucket
+      }, {
+        parallel: 4,
+        partSize: 1024 * 1024,
+      });
       assert.equal(result.res.statusCode, 200);
     });
   });