Fixed addOptionGroup method (typo) + added tests (#85).

Author: brianreavis

src/selectize.js

@@ -978,7 +978,7 @@ $.extend(Selectize.prototype, {
 	 */
 	addOptionGroup: function(id, data) {
 		this.optgroups[id] = data;
-		this.trigger('optgroup_add', value, data);
+		this.trigger('optgroup_add', id, data);
 	},
 
 	/**

test/api.js

@@ -138,6 +138,21 @@
 			});
 		});
 
+		describe('addOptionGroup()', function() {
+			before(function() {
+				test = setup_test('<select>', {valueField: 'value', labelField: 'value'});
+			});
+			after(function() {
+				test.teardown();
+			});
+			it('should register group', function() {
+				var data = {label: 'Group Label'};
+				test.selectize.addOptionGroup('group_id', data);
+				expect(test.selectize.optgroups).to.have.property('group_id');
+				expect(test.selectize.optgroups['group_id']).to.eql(data);
+			});
+		});
+
 		describe('addOption()', function() {
 			before(function() {
 				test = setup_test('<select>', {valueField: 'value', labelField: 'value'});

test/events.js

@@ -87,6 +87,34 @@ describe('Events', function() {
 		});
 	});
 
+	describe('optgroup_add', function() {
+		beforeEach(function() {
+			test = setup_test('<select><option value="a" selected></option><option value="b" selected></option><option value="c"></option></select>', {});
+		});
+		afterEach(function() {
+			test.teardown();
+		});
+		it('should be triggered', function(done) {
+			test.selectize.on('optgroup_add', function() { done(); });
+			test.selectize.addOptionGroup('id', {label: 'Group'});
+		});
+		it('should contain optgroup id', function(done) {
+			test.selectize.on('optgroup_add', function(id, data) {
+				expect(id).to.be.equal('id');
+				done();
+			});
+			test.selectize.addOptionGroup('id', {label: 'Group'});
+		});
+		it('should contain outgroup data', function(done) {
+			var optgroup = {label: 'Group'};
+			test.selectize.on('optgroup_add', function(id, data) {
+				expect(data).to.eql(optgroup);
+				done();
+			});
+			test.selectize.addOptionGroup('id', optgroup);
+		});
+	});
+
 	describe('option_add', function() {
 		beforeEach(function() {
 			test = setup_test('<select><option value="a" selected></option><option value="b" selected></option><option value="c"></option></select>', {});
@@ -110,7 +138,7 @@ describe('Events', function() {
 		it('should contain option data', function(done) {
 			var option = {value: 'e'};
 			test.selectize.on('option_add', function(value, data) {
-				expect(data).to.be.equal(data);
+				expect(option).to.eql(data);
 				done();
 			});
 			test.selectize.addOption(option);