Sort config keysin 'version -v'

Signed-off-by: Eric Wang <skygragon@gmail.com>

Author: skygragon

lib/commands/version.js

@@ -35,7 +35,7 @@ cmd.handler = function(argv) {
   var config = require('../config');
   var sprintf = require('sprintf-js').sprintf;
   console.log('\n[Configuration]');
-  Object.getOwnPropertyNames(config).forEach(function(k) {
+  Object.getOwnPropertyNames(config).sort().forEach(function(k) {
     console.log(sprintf('%-13s %s', k + ':', config[k]));
   });
 };

lib/config.js

@@ -4,13 +4,13 @@ var h = require('./helper');
 
 var DEFAULT_CONFIG = {
   // usually you don't wanna change those
-  BASE_URL:     'https://leetcode.com',
-  LOGIN_URL:    'https://leetcode.com/accounts/login/',
-  PROBLEMS_URL: 'https://leetcode.com/api/problems/algorithms/',
-  PROBLEM_URL:  'https://leetcode.com/problems/$id',
-  TEST_URL:     'https://leetcode.com/problems/$key/interpret_solution/',
-  SUBMIT_URL:   'https://leetcode.com/problems/$key/submit/',
-  VERIFY_URL:   'https://leetcode.com/submissions/detail/$id/check/',
+  URL_BASE:     'https://leetcode.com',
+  URL_LOGIN:    'https://leetcode.com/accounts/login/',
+  URL_PROBLEMS: 'https://leetcode.com/api/problems/algorithms/',
+  URL_PROBLEM:  'https://leetcode.com/problems/$id',
+  URL_TEST:     'https://leetcode.com/problems/$key/interpret_solution/',
+  URL_SUBMIT:   'https://leetcode.com/problems/$key/submit/',
+  URL_VERIFY:   'https://leetcode.com/submissions/detail/$id/check/',
 
   // but you will want change these
   LANG:       'cpp', // avail: [c,cpp,csharp,golang,java,javascript,python,ruby,swift]

lib/leetcode_client.js

@@ -71,7 +71,7 @@ function requestWithReLogin(opts, cb) {
 var leetcodeClient = {};
 
 leetcodeClient.getProblems = function(cb) {
-  var opts = makeOpts(config.PROBLEMS_URL);
+  var opts = makeOpts(config.URL_PROBLEMS);
 
   requestWithReLogin(opts, function(e, resp, body) {
     e = checkError(e, resp, 200);
@@ -94,7 +94,7 @@ leetcodeClient.getProblems = function(cb) {
             id:      p.stat.question_id,
             name:    p.stat.question__title,
             key:     p.stat.question__title_slug,
-            link:    config.PROBLEM_URL.replace('$id', p.stat.question__title_slug),
+            link:    config.URL_PROBLEM.replace('$id', p.stat.question__title_slug),
             locked:  p.paid_only,
             percent: p.stat.total_acs * 100 / p.stat.total_submitted,
             level:   h.levelToName(p.difficulty.level)
@@ -138,17 +138,17 @@ leetcodeClient.getProblem = function(problem, cb) {
 };
 
 leetcodeClient.login = function(user, cb) {
-  request(config.LOGIN_URL, function(e, resp, body) {
+  request(config.URL_LOGIN, function(e, resp, body) {
     e = checkError(e, resp, 200);
     if (e) return cb(e);
 
     user.loginCSRF = h.getSetCookieValue(resp, 'csrftoken');
 
     var opts = {
-      url:     config.LOGIN_URL,
+      url:     config.URL_LOGIN,
       headers: {
-        Origin:  config.BASE_URL,
-        Referer: config.LOGIN_URL,
+        Origin:  config.URL_BASE,
+        Referer: config.URL_LOGIN,
         Cookie:  'csrftoken=' + user.loginCSRF + ';'
       },
       form: {
@@ -176,7 +176,7 @@ function verifyResult(opts, jobs, results, cb) {
     return cb(null, results);
 
   opts.method = 'GET';
-  opts.url = config.VERIFY_URL.replace('$id', jobs[0].id);
+  opts.url = config.URL_VERIFY.replace('$id', jobs[0].id);
 
   requestWithReLogin(opts, function(e, resp, body) {
     e = checkError(e, resp, 200);
@@ -196,8 +196,8 @@ function verifyResult(opts, jobs, results, cb) {
 leetcodeClient.testProblem = function(problem, cb) {
   var opts = makeOpts();
   opts.method = 'POST';
-  opts.url = config.TEST_URL.replace('$key', problem.key);
-  opts.headers.Origin = config.BASE_URL;
+  opts.url = config.URL_TEST.replace('$key', problem.key);
+  opts.headers.Origin = config.URL_BASE;
   opts.headers.Referer = problem.link;
   opts.headers['X-Requested-With'] = 'XMLHttpRequest';
   opts.json = true;
@@ -227,8 +227,8 @@ leetcodeClient.testProblem = function(problem, cb) {
 leetcodeClient.submitProblem = function(problem, cb) {
   var opts = makeOpts();
   opts.method = 'POST';
-  opts.url = config.SUBMIT_URL.replace('$key', problem.key);
-  opts.headers.Origin = config.BASE_URL;
+  opts.url = config.URL_SUBMIT.replace('$key', problem.key);
+  opts.headers.Origin = config.URL_BASE;
   opts.headers.Referer = problem.link;
   opts.headers['X-Requested-With'] = 'XMLHttpRequest';
   opts.json = true;

test/test_leetcode_client.js

@@ -11,7 +11,7 @@ describe('leetcode_client', function() {
 
   describe('#getProblems', function() {
     it('should ok', function(done) {
-      nock(config.PROBLEMS_URL)
+      nock(config.URL_PROBLEMS)
         .get('/')
         .replyWithFile(200, './test/mock/problems.json.20160911');