v1.0.1

README.md

@@ -28,11 +28,11 @@ $ gituser -h
 
   Commands:
 
-    add [options] <user> <name> [email]  add user configuration
-    remove|rm <user>                     remove user configuration
-    list|ls                              list user configuration
-    set [options] <user>                 set local git user configuration from <user>
-    unset                                unset local git user configuration
+    add|save [options] <user> <name> [email]  save the specified user configuration information
+    remove|rm <user>                          delete the specified user configuration information
+    list|ls                                   list all user configuration information
+    set [options] <user>                      set local git config user from <user> configuration information
+    unset                                     unset local git config user
 
   Easily switch git users.
 

bin/gituser.js

@@ -28,7 +28,8 @@ program
 
 program
   .command('add <user> <name> [email]')
-  .description('add user configuration')
+  .alias('save')
+  .description('save the specified user configuration information')
   .option('--private-github', 'private email address for GitHub', null, null)
   .action((user, name, email, options) => {
     noArgs = false
@@ -45,7 +46,7 @@ program
 program
   .command('remove <user>')
   .alias('rm')
-  .description('remove user configuration')
+  .description('delete the specified user configuration information')
   .action((user, options) => {
     noArgs = false
 
@@ -61,7 +62,7 @@ program
 program
   .command('list')
   .alias('ls')
-  .description('list user configuration')
+  .description('list all user configuration information')
   .action((options) => {
     noArgs = false
 
@@ -76,7 +77,7 @@ program
 
 program
   .command('set <user>')
-  .description('set local git user configuration from <user>')
+  .description('set local git config user from <user> configuration information')
   .option('--private-github', 'private email address for GitHub', null, null)
   .action((user, options) => {
     noArgs = false
@@ -92,7 +93,7 @@ program
 
 program
   .command('unset')
-  .description('unset local git user configuration')
+  .description('unset local git config user')
   .action((options) => {
     noArgs = false
 

lib/config.js

@@ -19,26 +19,28 @@ function init (dir, file) {
   let stat
   try {
     stat = fs.statSync(dir)
+    // fs.accessSync(dir)
   } catch (e) {
     stat = null
   }
-  if (stat) {
-    if (!stat.isDirectory()) throw new ReferenceError('Can not create config directory')
-  } else {
+  if (!stat) {
     fs.mkdirSync(dir, 0o777)
+  } else if (!stat.isDirectory()) {
+    throw new ReferenceError('Can not create config directory')
   }
 
   // config file
   file = path.join(dir, file)
   try {
     stat = fs.statSync(file)
+    // fs.accessSync(dir)
   } catch (e) {
     stat = null
   }
-  if (stat) {
-    if (!stat.isFile()) throw new ReferenceError('Can not create config file')
-  } else {
+  if (!stat) {
     fs.writeFileSync(file, JSON.stringify(defaultData))
+  } else if (!stat.isFile()) {
+    throw new ReferenceError('Can not create config file')
   }
 
   return file

package.json

@@ -1,6 +1,6 @@
 {
   "name": "gituser",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "Easily switch git users.",
   "preferGlobal": true,
   "main": "index.js",

test/config.test.js

@@ -3,6 +3,7 @@
  */
 const os = require('os')
 const path = require('path')
+const fs = require('fs')
 
 const test = require('ava')
 
@@ -14,8 +15,26 @@ const fileName = 'config.test'
 const {init, write, read} = require('../lib/config').base
 const {writeDefault, writeAllDefault, readDefault} = require('../lib/config')
 
+const _init = require('../lib/init')
+
+test.serial('init pass', t => {
+  try {
+    _init()
+    t.pass()
+  } catch (e) {
+    t.fail(e)
+  }
+})
+
+test.serial('config init', t => {
+  try {
+    fs.unlinkSync(path.join(dir, fileName))
+  } catch (e) {
+  }
+  t.pass()
+})
+
 test.serial('config init pass', t => {
-  console.log(__dirname)
   try {
     init(dir, fileName)
     t.pass()
@@ -33,7 +52,25 @@ test.serial('config init fail', t => {
   }
 })
 
-test.serial('config write pass', t => {
+test.serial('config write pass1', t => {
+  try {
+    let file = write(dir, fileName, null)
+    t.true(!!file)
+  } catch (e) {
+    t.fail(e)
+  }
+})
+
+test.serial('config read fail', t => {
+  try {
+    let obj = read(dir, fileName)
+    t.false(!!obj)
+  } catch (e) {
+    t.fail(e)
+  }
+})
+
+test.serial('config write pass2', t => {
   try {
     let file = write(dir, fileName, {
       'version': pkg.version,
@@ -49,7 +86,7 @@ test.serial('config write pass', t => {
   }
 })
 
-test.serial('config read pass', t => {
+test.serial('config read pass2', t => {
   try {
     let obj = read(dir, fileName)
     t.true(!!obj)