Merge pull request #9 from xudafeng/project-config

update project config

README.md

@@ -1,11 +1,15 @@
-# detect-port
+[![logo][logo-image]][logo-url]
+
+---
 
 [![NPM version][npm-image]][npm-url]
+[![node version][node-image]][node-url]
 [![build status][travis-image]][travis-url]
 [![Coveralls][coveralls-image]][coveralls-url]
-[![node version][node-image]][node-url]
 [![npm download][download-image]][download-url]
 
+[logo-image]: ./logo.png
+[logo-url]: https://npmjs.org/package/detect-port
 [npm-image]: https://img.shields.io/npm/v/detect-port.svg?style=flat-square
 [npm-url]: https://npmjs.org/package/detect-port
 [travis-image]: https://img.shields.io/travis/xudafeng/detect-port.svg?style=flat-square
@@ -48,17 +52,17 @@ detect(port, (err, _port) => {
  * for a yield syntax instead of callback function implement
  */
 
-var co = require('co');
+const co = require('co');
 
 co(function *() {
-  var _port = yield detect(port);
+  const _port = yield detect(port);
 
   if (port === _port) {
     console.log(`port: ${port} was not occupied`);
   } else {
     console.log(`port: ${port} was occupied, try port: ${_port}`);
   }
-})();
+});
 
 /**
  * use as a promise
@@ -78,7 +82,7 @@ detect(port)
 
 ```
 
-## Cli Tool
+## Command Line Tool
 
 ```shell
 $ npm i detect-port -g

bin/detect-port

@@ -55,10 +55,14 @@ if (!arg_0) {
   console.log(`    ${pkg.homepage}`);
   console.log();
 } else {
-  main(port, (err, port) => {
+  main(port, (err, _port) => {
     if (err) {
       console.log(`get available port failed with ${err}`);
     }
-    console.log(`get available port ${port}`);
+
+    if (port !== _port) {
+      console.log(`port ${port} was occupied`);
+    }
+    console.log(`get available port ${_port}`);
   });
 }

index.js

@@ -13,4 +13,4 @@
 
 'use strict';
 
-module.exports = require('./lib');
+module.exports = require('./lib/detect-port');

package.json

@@ -1,6 +1,6 @@
 {
   "name": "detect-port",
-  "version": "1.0.2",
+  "version": "1.0.4",
   "description": "detect available port",
   "keywords": [
     "detect",

test/cli.test.js

@@ -48,8 +48,8 @@ describe('command-line tool test', () => {
   });
 
   it('should output available port from the given port', function *() {
-    const givenPort = 8080;
-    var res = yield cliTest.execFile(binFile, [givenPort], {});
+    const givenPort = 9000;
+    const res = yield cliTest.execFile(binFile, [givenPort], {});
     const port = parseInt(res.stdout.split(' ')[3], 10);
     port.should.within(givenPort, 65535);
   });

test/detect-port.test.js

@@ -18,7 +18,7 @@ const detectPort = require('..');
 describe('detect port test', () => {
 
   it('callback with occupied port', done => {
-    var _port = 80;
+    const _port = 80;
     detectPort(_port, (err, port) => {
       if (err) {
         console.log(err);
@@ -38,17 +38,17 @@ describe('detect port test', () => {
   });
 
   it('generator usage', function *() {
-    var _port = 8080;
+    const _port = 8080;
     try {
-      var port = yield detectPort(_port);
+      const port = yield detectPort(_port);
       port.should.within(_port, 65535);
     } catch (err) {
       console.log(err);
     }
   });
 
   it('promise usage', done => {
-    var _port = 8080;
+    const _port = 8080;
     detectPort(_port)
       .then(port => {
         port.should.within(_port, 65535);