Merge pull request #163 from davesag/feature/161/allow-basic-type-con…

…structors

[Bug fix, #161] allow basic type constructors

src/dataTypes.js

@@ -1,4 +1,4 @@
-const noop = require('./types/noop')
+const Noop = require('./types/Noop')
 const NumericType = require('./types/NumericType')
 const StringType = require('./types/StringType')
 const basicTypes = require('./constants/basicTypes')
@@ -7,7 +7,7 @@ const stringTypes = require('./constants/stringTypes')
 const deferrables = require('./constants/deferrables')
 
 const basicDataTypes = basicTypes.reduce((acc, elem) => {
-  acc[elem] = noop
+  acc[elem] = Noop
   return acc
 }, {})
 

src/types/Noop.js

@@ -0,0 +1,5 @@
+function Noop() {
+  return Noop
+}
+
+module.exports = Noop

test/unit/dataTypes.test.js

@@ -1,7 +1,7 @@
 const { expect } = require('chai')
 
 const dataTypes = require('../../src/dataTypes')
-const noop = require('../../src/types/noop')
+const Noop = require('../../src/types/Noop')
 const NumericType = require('../../src/types/NumericType')
 const StringType = require('../../src/types/StringType')
 
@@ -69,7 +69,7 @@ describe('dataTypes', () => {
 
   it('has basicTypes', () => {
     basicTypes.forEach(type => {
-      expect(dataTypes).to.have.property(type, noop)
+      expect(dataTypes).to.have.property(type, Noop)
     })
   })
 

test/unit/types/Noop.test.js

@@ -0,0 +1,12 @@
+const { expect } = require('chai')
+const Noop = require('../../../src/types/Noop')
+
+describe('src/types/Noop', () => {
+  it('is a function', () => {
+    expect(Noop).to.be.a('function')
+  })
+
+  it('returns itself', () => {
+    expect(Noop()).to.equal(Noop)
+  })
+})