Merge pull request #4 from inspekter/develop

Develop

Author: pablomdo

index.js

@@ -2,6 +2,7 @@
 
 const escomplex = require('escomplex')
 const path = require('path')
+const read = require('read-file')
 
 function sanitize (fileName, filePath, result) {
   return {
@@ -35,11 +36,17 @@ module.exports.analyze = (source) => {
 
   if (Array.isArray(source)) {
     source.forEach((filePath) => {
-      result = escomplex.analyse(filePath)
+      let fullPath = path.resolve(__dirname, filePath)
+      let content = read.sync(fullPath, 'utf8')
+      result = escomplex.analyse(content)
       report = sanitize(path.basename(filePath), filePath, result)
       reports.push(report)
     })
   }
 
   return reports
 }
+
+module.exports.getExtension = () => {
+  return 'js'
+}

package.json

@@ -1,6 +1,6 @@
 {
-  "name": "inspekter-plugin-javascript",
-  "version": "0.0.1",
+  "name": "@inspekter/inspekter-plugin-javascript",
+  "version": "1.0.0",
   "description": "Inspekter JavaScript reporter",
   "main": "index.js",
   "scripts": {
@@ -10,8 +10,8 @@
     "type": "git",
     "url": "git+https://github.com/inspekter/inspekter-plugin-javascript.git"
   },
-  "engines" : {
-    "node" : ">=4"
+  "engines": {
+    "node": ">=4"
   },
   "author": "Pablo Oliveira <kryptogeist@gmail.com>",
   "license": "MIT",
@@ -20,7 +20,8 @@
   },
   "homepage": "https://github.com/inspekter/inspekter-plugin-javascript#readme",
   "dependencies": {
-    "escomplex": "^2.0.0-alpha"
+    "escomplex": "^2.0.0-alpha",
+    "read-file": "^0.2.0"
   },
   "devDependencies": {
     "chai": "^4.1.2",

test/index.js

@@ -67,28 +67,36 @@ const properties = [
 describe('inspekter-plugin-javascript', () => {
   describe('analyze', () => {
     it('should always return an array', () => {
-      const source = ['fixtures/file1.js']
+      const source = ['test/fixtures/file1.js']
       const actual = ipj.analyze(source)
 
       expect(actual).to.be.an('array')
       expect(actual).to.have.lengthOf(1)
     })
 
     it('should return one report object for each file', () => {
-      const source = ['fixtures/file1.js', 'fixtures/file2.js']
+      const source = ['test/fixtures/file1.js', 'test/fixtures/file2.js']
       const actual = ipj.analyze(source)
 
       expect(actual).to.be.an('array')
       expect(actual).to.have.lengthOf(2)
     })
 
     it('should return an object with all standard properties', () => {
-      const source = ['fixtures/file1.js']
+      const source = ['test/fixtures/file1.js']
       const actual = ipj.analyze(source)[0]
 
       properties.forEach((property) => {
         expect(actual).to.have.nested.property(property.name).that.is.a(property.type)
       })
     })
   })
+
+  describe('getExtension', () => {
+    it('should return js', () => {
+      const actual = ipj.getExtension()
+
+      expect(actual).to.equal('js')
+    })
+  })
 })