Fix: sign method (#426)

* fix(browser): sign sort fix

* feat: node env >=8

* feat: test ci

.travis.yml

@@ -2,8 +2,6 @@ sudo: false
 language: node_js
 node_js:
   - '8'
-  - '6'
-  - '4'
 env:
   global:
     - secure: Ws1h7ZU97/2npejMZp3/Jac0ZYkgXBW5gEilR5WrnkdkEbIhhK6HamrcqLrA6qCPV0Ax9nNbZCRJcQwPsTdOxBekJ3+3zflMWevfa9+CRS1Hh0mMcn+D864CjrRiQRgIQa8ZoFVPLaL/7qyQp1i7gLbPcpMjO6CA7LgV1reaf+U=

lib/common/signUtils.js

@@ -19,8 +19,10 @@ exports.buildCanonicalizedResource = function buildCanonicalizedResource(resourc
     canonicalizedResource += separatorString + parameters.join('&');
   } else if (parameters) {
     const compareFunc = (entry1, entry2) => {
-      if (entry1[0] >= entry2[0]) {
+      if (entry1[0] > entry2[0]) {
         return 1;
+      } else if (entry1[0] < entry2[0]) {
+        return -1;
       }
       return 0;
     };

package.json

@@ -19,7 +19,7 @@
   },
   "scripts": {
     "test": "mocha -t 60000 -r thunk-mocha -r should test/node/*.test.js",
-    "test-cov": "istanbul cover node_modules/.bin/_mocha -- -R dot -t 60000 -r thunk-mocha -r should test/*.test.js",
+    "test-cov": "istanbul cover node_modules/.bin/_mocha -- --require babel-register -R dot -t 60000 -r thunk-mocha -r should test/node/*.test.js",
     "jshint": "jshint .",
     "autod": "autod",
     "build-test": "node browser-build.js > test/browser/build/aliyun-oss-sdk.js && node task/browser-test-build.js > test/browser/build/tests.js",
@@ -50,14 +50,15 @@
     "url": "https://github.com/aliyun/oss-nodejs-sdk/issues"
   },
   "engines": {
-    "node": ">=6"
+    "node": ">=8"
   },
   "homepage": "https://github.com/aliyun/oss-nodejs-sdk",
   "devDependencies": {
     "aliasify": "^2.0.0",
     "autod": "^2.6.1",
     "babel-plugin-transform-runtime": "^6.8.0",
     "babel-preset-es2015": "^6.24.1",
+    "babel-register": "^6.26.0",
     "babel-runtime": "^6.6.1",
     "babelify": "^7.3.0",
     "beautify-benchmark": "^0.2.4",
@@ -79,9 +80,9 @@
     "karma-browserify": "^5.1.1",
     "karma-chrome-launcher": "^2.2.0",
     "karma-firefox-launcher": "^1.0.1",
-    "karma-safari-launcher": "^1.0.0",
     "karma-ie-launcher": "^1.0.0",
     "karma-mocha": "^1.3.0",
+    "karma-safari-launcher": "^1.0.0",
     "mm": "^2.0.0",
     "mocha": "^3.5.3",
     "promise-polyfill": "^6.0.2",

test/node/multipart.test.js

@@ -126,8 +126,8 @@ describe('test/multipart.test.js', () => {
       });
       const after1 = result.uploads.map(up => up.uploadId);
       after1.sort();
-      const should = barIds.slice(1).concat(fooIds).sort();
-      assert.deepEqual(after1, should);
+      const sort1 = barIds.slice(1).concat(fooIds).sort();
+      assert.deepEqual(after1, sort1);
 
       // after 5
       result = yield this.store.listUploads({