Fix namespace handling

[jespirit]

I noticed these two lines were incorrect. The slider instance that is added/removed should be based on the namespace, either: slider or bootstrapSlider.

bootstrap-slider/src/js/bootstrap-slider.js

Line 995 in 88dfe2a

this.$element.removeData('slider');

bootstrap-slider/src/js/bootstrap-slider.js

Line 1064 in 88dfe2a

$.data(this.element, 'slider', this);

I added unit tests.

I also fixed some unit tests that were not properly destroying the <div> slider element. This caused some unit tests to fail when it used .siblings() to check the <div> slider element, but the method also collects more than 1 <div> if they exist, which was causing hard to track bugs.

Pull Requests

Please accompany all pull requests with the following (where appropriate):

• unit tests (we use Jasmine 2.x.x)
• Passes CI-server checks (runs automated tests, JS source-code linting, etc..). To run these on your local machine, type grunt test in your Terminal within the bootstrap-slider repository directory

[rovolution]

👍