Corrigindo as fita

isCep/isCep.js

@@ -3,10 +3,15 @@
 const regex = /^[0-9]{8}$/;
 
 module.exports = (value) => {
-    value = value.replace(/\.|\-/g, '');    
-    const isEmpty = require('../isEmpty/isEmpty')(value);       
+  const isEmpty = require('../isEmpty/isEmptyString/isEmptyString')(value);  
+  const isString = require('../isString/isString')(value);  
+  console.log('isEmpty', isEmpty);
+  if(!isEmpty) {
+    console.log(value);
+    value = value.replace(/\.|\-/g, '');     
     const isCep = regex.test(value);
-    if(isEmpty || !isCep) return false;
-
-    return true;         
+    if(isCep) return true;  
+  }
+
+  return false;         
 }

isCep/isCep.test.js

@@ -7,6 +7,7 @@ const valorMenorQueOito = '10000';
 const valorMaiorQueOito = '100000000';
 const valorComAlfaNumerico = '1000000A';
 const valorComCaracteresEspeciais = '1000000,'
+const valorNumerico = 29101950;
 
 //CEP Válidos
 const cepValidoComMascara = '29.101-950';
@@ -19,11 +20,13 @@ assert.equal(false, require('./isCep')(valorMenorQueOito));
 assert.equal(false, require('./isCep')(valorMaiorQueOito));
 assert.equal(false, require('./isCep')(valorComAlfaNumerico));
 assert.equal(false, require('./isCep')(valorComCaracteresEspeciais));
+assert.equal(false, require('./isCep')(valorNumerico));
 
 console.log(valorMenorQueOito + ' é um cep?', require('./isCep')(valorMenorQueOito));
 console.log(valorMaiorQueOito + ' é um cep?', require('./isCep')(valorMaiorQueOito));
 console.log(valorComAlfaNumerico + ' é um cep?', require('./isCep')(valorComAlfaNumerico));
 console.log(valorComCaracteresEspeciais + ' é um cep?', require('./isCep')(valorComCaracteresEspeciais));
+console.log(valorNumerico + ' é um cep?', require('./isCep')(valorNumerico));
 
 console.log(cepValidoComMascara + ' é um cep?', require('./isCep')(cepValidoComMascara));
 console.log(cepValidoSemMascara + ' é um cep?', require('./isCep')(cepValidoSemMascara));

isEmpty/isEmpty.js

@@ -1,6 +1,9 @@
 'use strict';
 
 module.exports = (value) => {
-  if (value === null || value === undefined) return true;
+  const isNull = (value === null);
+  const isUndefined = (value === undefined);
+  const isEmpty = (value === '');
+  if (isNull || isUndefined || isEmpty) return true;
   return false;
 }

isEmpty/isEmptyString/isEmptyString.js

@@ -1,16 +1,10 @@
 'use strict';
 
 module.exports = (value) => {
+  const isEmpty = require('../isEmpty')(value)
+  const isString = require('../../isString/isString')(value)
 
-  // Ele não pode passar daqui se for null ou undefined
-  // se não quebrará a função isOnlyLetters q usa match()
-  const validated = require('../isEmpty')(value)
-  if (validated) return true;
-
-  // Se não é NULL nem UNDEFINED
-  // Se não é só letras
-  const isOnlyLetters = require('./isOnlyLetters')(value);
-  if (!isOnlyLetters) return true;
-
+  if (isEmpty && isString) return true;
+
   return false;
 };

isEmpty/isEmptyString/isEmptyString.test.js

@@ -3,10 +3,13 @@
 const assert = require('assert');
 
 const valueFALSE = 'Suissera';
+const valueFALSE1 = 1;
 const valueTRUE = '';
 
-assert.equal(true, require('./isEmptyString')(valueTRUE));
 assert.equal(false, require('./isEmptyString')(valueFALSE));
+assert.equal(false, require('./isEmptyString')(valueFALSE1));
+assert.equal(true, require('./isEmptyString')(valueTRUE));
 
-console.log(valueTRUE + ' é uma String vazia?', require('./isEmptyString')(valueTRUE));
-console.log(valueFALSE + ' é uma String vazia?', require('./isEmptyString')(valueFALSE));
+console.log(valueFALSE + ' é uma String vazia?', require('./isEmptyString')(valueFALSE));
+console.log(valueFALSE1+ ' é uma String vazia?', require('./isEmptyString')(valueFALSE1));
+console.log(valueTRUE + ' é uma String vazia?', require('./isEmptyString')(valueTRUE));

isString/isString.js

@@ -0,0 +1,6 @@
+'use strict';
+
+module.exports = (value) => {
+  if (typeof value === 'string' || value instanceof String) return true;
+  return false;
+}