updated: dependencies, npm name and project setup

Author: GianlucaGuarini

.eslintrc

@@ -1 +1,3 @@
-extends: "@gianlucaguarini/eslint-config"
+extends: "@gianlucaguarini/eslint-config"
+env:
+  mocha: true

.travis.yml

@@ -10,6 +10,9 @@ before_install:
   - npm i [email protected]
   - npm i [email protected]
   - npm i @gianlucaguarini/[email protected]
+  - npm i [email protected]
+
+before_script:
   - npm run build
 
 notifications:

README.md

@@ -10,9 +10,19 @@ Modern DOM query selectors helpers written in es2015
 ## Usage
 
 ```js
-import $ from 'bianco-query'
+import $ from 'bianco.query'
+
+const footer = document.querySelector('.main-footer')
+const header = document.querySelector('.main-header')
+
+// convert DOM nodes into arrays
+$(footer)
+  .concat($(header))
+  .forEach(el => el.classList.add('fade-in'))
+
+// handle DOM queries
+$('h1', 'main').forEach(h1 => h1.classList.add('main-title'))
 
-$('h1').forEach((h1) => console.log(h1.innerHTML))
 ```
 
 ## API

index.next.js

@@ -1,21 +1,14 @@
+import domToArray from 'bianco.dom-to-array'
 
 /**
- * Simple helper to find DOM nodes returning them as Array
+ * Simple helper to find DOM nodes returning them as array like loopable object
  * @param   { String|DOMNodeList } selector - either the query or the DOM nodes to arraify
- * @param   { HTMLElement }        ctx      - context defining where the query will search DOM nodes
+ * @param   { HTMLElement }        ctx      - context defining where the query will search for the DOM nodes
  * @returns { Object } DOM nodes in an array like object
  */
 export default function $(selector, ctx) {
-  var els = selector
-
-  // find the DOM nodes
-  if (typeof selector === 'string')
-    els = (ctx || document).querySelectorAll(selector)
-
-
-  // arraify the DOM nodes found
-  if (!Array.isArray(els))
-    els = Array.from(els)
-
-  return els
+  return domToArray(typeof selector === 'string' ?
+    (ctx || document).querySelectorAll(selector) :
+    selector
+  )
 }

package.json

@@ -1,14 +1,14 @@
 {
-  "name": "bianco-query",
-  "version": "0.0.2",
+  "name": "bianco.query",
+  "version": "0.0.0",
   "description": "Modern DOM query selectors helpers written in es2015",
   "main": "index.js",
   "jsnext:main": "index.next.js",
   "scripts": {
     "prepublish": "npm run build && npm test",
-    "lint": "eslint index.next.js test.js",
-    "build": "rollup -f cjs -- index.next.js > index.js",
-    "test": "npm run lint && node test"
+    "lint": "eslint index.next.js test.js rollup.config.js",
+    "build": "rollup -c",
+    "test": "npm run lint && mocha test.js"
   },
   "files": [
     "index.js",
@@ -35,6 +35,10 @@
   "homepage": "https://github.com/biancojs/query#readme",
   "devDependencies": {
     "jsdom": "9.5.0",
-    "jsdom-global": "2.1.0"
+    "jsdom-global": "2.1.0",
+    "rollup-plugin-node-resolve": "^2.0.0"
+  },
+  "dependencies": {
+    "bianco.dom-to-array": "^0.0.2"
   }
 }

rollup.config.js

@@ -0,0 +1,9 @@
+export default {
+  entry: 'index.next.js',
+  targets: [
+    {
+      dest: 'index.js',
+      format: 'cjs'
+    }
+  ]
+}

test.js

@@ -1,22 +1,31 @@
 require('jsdom-global')()
 const assert = require('assert')
 const $ = require('./')
+const body = document.body
 
-function before() {
-  var div = document.createElement('div')
+describe('Bianco query', function() {
+  before(function() {
+    var div = document.createElement('div')
 
-  div.innerHTML = `
-  <ul>
-    <li class='item'></li>
-    <li class='item'></li>
-  </ul>
-  `
-  document.body.appendChild(div)
-}
+    div.innerHTML = `
+    <ul>
+      <li class='item'></li>
+      <li class='item'></li>
+    </ul>
+    `
+    body.appendChild(div)
+  })
 
-before()
+  it('It can query the DOM properly', function() {
+    const div = $('div')
+    assert.equal(typeof div, 'object')
+    assert.equal(div.length, 1)
+    assert.equal($('.item', div[0]).length, 2)
+  })
 
-const div = $('div')
-assert.equal(typeof div, 'object')
-assert.equal(div.length, 1)
-assert.equal($('.item', div[0]).length, 2)
+  it('It can handle also DOM nodes', function() {
+    const els = $(document.querySelector('div')).concat($(document.querySelector('ul')))
+    assert.equal(typeof els, 'object')
+    assert.equal(els.length, 2)
+  })
+})