refactor(Utils): Move getRealClassNamesArray on utils module to have …

…pure modules
lucaperret · Aug 12, 2017 · 4038be8 · 4038be8
1 parent 5fd8528
commit 4038be8
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 12 deletions.
diff --git a/src/attributes.js b/src/attributes.js
@@ -3,18 +3,7 @@
  * @module attributes
  */
 
-import { toCamelCase } from './utils'
-
-/**
- * Filter incorrect class names.
- * @private
- *
- * @param {string} [classNames=''] One or more space-separated classes to be filtered
- * @return {Array} Valid class names array
- */
-function getRealClassNamesArray (classNames = '') {
-  return classNames.split(' ').filter(className => !!className)
-}
+import { toCamelCase, getRealClassNamesArray } from './utils'
 
 /**
  * Determine whether the element is assigned the given class.

diff --git a/src/utils.js b/src/utils.js
@@ -16,3 +16,14 @@ export function toCamelCase (kebabCaseString) {
     return memo + token.slice(0, 1).toUpperCase() + token.slice(1)
   })
 }
+
+/**
+ * Filter incorrect class names.
+ * @private
+ *
+ * @param {string} [classNames=''] One or more space-separated classes to be filtered
+ * @return {Array} Valid class names array
+ */
+export function getRealClassNamesArray (classNames = '') {
+  return classNames.split(' ').filter(className => !!className)
+}
diff --git a/test/specs/utils.spec.js b/test/specs/utils.spec.js
@@ -4,4 +4,7 @@ describe('Utils', () => {
   test('Kebab to camel case', () => {
     expect(utils.toCamelCase('kebab-case-string')).toBe('kebabCaseString')
   })
+  test('Filter incorrect class names', () => {
+    expect(utils.getRealClassNamesArray('  class1       class2 ')).toEqual(['class1', 'class2'])
+  })
 })