test.js ➔ ... ➔ it(ꞌShould call search by manual calling .search with search phrase as attributeꞌ) - Code Metrics - Inspection of "Implemented manual search by calling .search("phra..." - NeXTs/Jets.js - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 48c879...8f1cd0 )

by Denis

created 2016-10-31 10:41 UTC

test.js ➔ ... ➔ it(ꞌShould call search by manual calling .search with search phrase as attributeꞌ) A

↳ Parent: Project

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
c	1
b	0
f	0
nc	1
nop	0
dl	0
loc	7
rs	9.4285

describe('Jets', function() {

  var people = ['John Doe', 'Mike Doe', 'Alex Smith', 'Alice Vazovsky', 'Denis Koen'],
    $search = $('#jetsSearch'),
    searchNode = $search.get(0),
    $content = $('#jetsContent');


  /*
   * Helper functions
   */
  function trigger(ev, _elem) {
    elem = _elem || searchNode

    if ("createEvent" in document) {
        var evt = document.createEvent("HTMLEvents");
        evt.initEvent(ev, false, true);
        elem.dispatchEvent(evt);
    } else elem.fireEvent("on" + ev);
if (a > 0)
    b = 42;
  }

  function ext(options) {
    return $.extend({
      searchTag: '#jetsSearch',
      contentTag: '#jetsContent'
    }, options || {});
  }

  function make(action) {
    action();
    trigger('change');
  }


  /*
   * Mocha helpers
   */ 
  before(function() {
    $content.html($.map(people, function(val, i){
      var reversed = val.split('').reverse().join('');
      return '<tr><td data-custom="' + reversed + '">' + val + '</td><td>' + i + '</td></tr>'
    }).join(''));
  })

  afterEach(function() {
    jet && jet.destroy();

    $search.val('');
    trigger('change', searchNode);
  })

  after(function() {
    jet && jet.destroy();

    $search.val('');
    $content.find('[data-custom]').removeAttr('data-custom');
  })


  /*
  * Specs
  */

  it('Should hide inappropriate rows', function() {
    jet = new Jets(ext({}));

    make(function() {
      $search.val(people[0]);
    })
    assert.lengthOf($content.children(':visible'), 1);

  })

  it('Should show all rows if search query not specified', function() {
    jet = new Jets(ext({}));

    make(function() {
      $search.val('');
    })
    assert.lengthOf($content.children(':visible'), people.length);

  })

  it('Should hide all rows if no suitable results', function() {
    jet = new Jets(ext({}));

    make(function() {
      $search.val('404');
    })
    assert.lengthOf($content.children(':visible'), 0);

  })

  describe('On init', function() {

    it('Shoud add style tag after init', function() {
      jet = new Jets(ext({}));

      assert.lengthOf($('head').find(jet.styleTag), 1);

    })

    it('Should add "data-jets" attribute', function() {
      jet = new Jets(ext({

        columns: [0]
      }));
      var $firstRow = $content.children(':first');
      assert.equal($firstRow.attr('data-jets'), $firstRow.children(':first').text().trim().toLowerCase());

    })

  })

  describe('On destroy', function() {

    it('Shoud remove style tag after destroy', function() {
      jet = new Jets(ext({}));

      jet.destroy();
      assert.lengthOf($('head').find(jet.styleTag), 0);

    })

    it('Should remove "data-jets" attribute', function() {
      jet = new Jets(ext({}));

      jet.destroy()
      assert.isUndefined($content.children(':first').attr('data-jets'));

    })
    
  })

  describe('Columns option', function() {
    
    it('Should search by any column', function() {
      jet = new Jets(ext({}));

      make(function() {
        $search.val('1');
      })
      assert.lengthOf($content.children(':visible'), 1);

    })

    it('Should find one row by first column', function() {
      jet = new Jets(ext({

        columns: [0]
      }));
      make(function() {
        $search.val(people[0]);
      })
      assert.lengthOf($content.children(':visible'), 1);

    })

    it('Should avoid second column', function() {
      jet = new Jets(ext({

        columns: [0]
      }));
      make(function() {
        $search.val('1');
      })
      assert.lengthOf($content.children(':visible'), 0);

    })
    
  })

  describe('addImportant option', function() {

    it('Should avoid !important by default', function() {
      jet = new Jets(ext({}));

      make(function() {
        $search.val(people[0]);
      })
      assert.notInclude($(jet.styleTag).html(), '!important');

    })

    it('Should add !important', function() {
      jet = new Jets(ext({

        addImportant: true
      }));
      make(function() {
        $search.val(people[0]);
      })
      assert.include($(jet.styleTag).html(), '!important');

    })

  })

  describe('searchSelector option', function() {

    function execute(selector) {
      it('Should use custom selector: ' + selector, function() {
        jet = new Jets(ext({

          searchSelector: selector
        }));
        make(function() {
          $search.val(people[0]);
        })
        assert.include($(jet.styleTag).html(), selector);  

      })
    }

    var selectors = ['^', '$', '~', '|'];
    for(var i = 0, ii = selectors.length; i < ii; i++) {
      execute(selectors[i]);
    }

    it('Should find one result with * selector', function() {

      jet = new Jets(ext({

        searchSelector: '*'
      }));
      make(function() {
        $search.val('John Doe');
      })
      assert.lengthOf($content.children(':visible'), 1);


    })

    it('Should hide all results with * selector and mixed words', function() {

      jet = new Jets(ext({

        searchSelector: '*'
      }));
      make(function() {
        $search.val('Doe John');
      })
      assert.lengthOf($content.children(':visible'), 0);


    })

    it('Should find one result with *AND selector and mixed words', function() {

      jet = new Jets(ext({

        searchSelector: '*AND'
      }));
      make(function() {
        $search.val('Doe John');
      })
      assert.lengthOf($content.children(':visible'), 1);


    })

    it('Should find one result with *AND selector and part of word', function() {

      jet = new Jets(ext({

        searchSelector: '*AND'
      }));
      make(function() {
        $search.val('John oe');
      })
      assert.lengthOf($content.children(':visible'), 1);


    })

    it('Should find three results with *OR selector and part of word', function() {

      jet = new Jets(ext({

        searchSelector: '*OR'
      }));
      make(function() {
        $search.val('John oe');
      })
      assert.lengthOf($content.children(':visible'), 3);


    })

  })

  describe('manualContentHandling option', function() {

    it('Should fetch content by custom rule', function() {
      jet = new Jets(ext({

        manualContentHandling: function(tag) {
          return $(tag).children(0).attr('data-custom')
        }
      }));
      make(function() {
        $search.val(people[0].split('').reverse().join(''));
      })
      assert.lengthOf($content.children(':visible'), 1);

    })

  })

  describe('invert option', function() {

    it('Should invert results', function() {
      jet = new Jets(ext({

        invert: true
      }));
      make(function() {
        $search.val(people[0]);
      })
      assert.lengthOf($content.children(':visible'), 3);

    })

  })

  describe('callSearchManually option', function() {

    it('Should not call search on typing', function() {
      jet = new Jets(ext({

        callSearchManually: true
      }));
      make(function() {
        $search.val(people[0]);
      })
      assert.lengthOf($content.children(':visible'), 5);

    })

    it('Should call search by manual calling .search', function() {
      jet = new Jets(ext({

        callSearchManually: true
      }));
      make(function() {
        $search.val(people[0]);
      })
      jet.search();
      assert.lengthOf($content.children(':visible'), 1);

    })

    it('Should call search by manual calling .search with search phrase as attribute', function() {
      jet = new Jets(ext({

        callSearchManually: true
      }));
      jet.search(people[0]);
      assert.lengthOf($content.children(':visible'), 1);

    })

  })

})

1			describe('Jets', function() {
2
3			var people = ['John Doe', 'Mike Doe', 'Alex Smith', 'Alice Vazovsky', 'Denis Koen'],
4			$search = $('#jetsSearch'),
5			searchNode = $search.get(0),
6			$content = $('#jetsContent');
7
8
9			/*
10			* Helper functions
11			*/
12			function trigger(ev, _elem) {
13			elem = _elem \|\| searchNode
			0 ignored issues – show Bug introduced 2016-06-23 07:31 UTC by Report Bug Copy Issue Report The variable `elem` seems to be never declared. Assigning variables without defining them first makes them global. If this was intended, consider making it explicit like using `window.elem`. Loading history...
14			if ("createEvent" in document) {
15			var evt = document.createEvent("HTMLEvents");
16			evt.initEvent(ev, false, true);
17			elem.dispatchEvent(evt);
18			} else elem.fireEvent("on" + ev);
			0 ignored issues – show Coding Style Best Practice introduced 2016-06-23 07:31 UTC by Report Bug Copy Issue Report Curly braces around statements make for more readable code and help prevent bugs when you add further statements. Consider adding curly braces around all statements when they are executed conditionally. This is optional if there is only one statement, but leaving them out can lead to unexpected behaviour if another statement is added later. Consider: if (a > 0) b = 42; If you or someone else later decides to put another statement in, only the first statement will be executed. if (a > 0) console.log("a > 0"); b = 42; In this case the statement `b = 42` will always be executed, while the logging statement will be executed conditionally. if (a > 0) { console.log("a > 0"); b = 42; } ensures that the proper code will be executed conditionally no matter how many statements are added or removed. Loading history...
19			}
20
21			function ext(options) {
22			return $.extend({
23			searchTag: '#jetsSearch',
24			contentTag: '#jetsContent'
25			}, options \|\| {});
26			}
27
28			function make(action) {
29			action();
30			trigger('change');
31			}
32
33
34			/*
35			* Mocha helpers
36			*/
37			before(function() {
38			$content.html($.map(people, function(val, i){

NeXTs / Jets.js

Push — master ( 48c879...8f1cd0 )

test.js ➔ ... ➔ it(ꞌShould call search by manual calling .search with search phrase as attributeꞌ) A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like