Skip to content
This repository was archived by the owner on May 29, 2019. It is now read-only.

Commit 758860d

Browse files
icfantvFoxandxss
authored andcommitted
chore(buttons): add missing unit test for btn-radio
- added missing unit test to test for string literal values on btn-radio directive Closes #4083
1 parent 74a8be4 commit 758860d

File tree

1 file changed

+19
-0
lines changed

1 file changed

+19
-0
lines changed

Diff for: src/buttons/test/buttons.spec.js

+19
Original file line numberDiff line numberDiff line change
@@ -212,6 +212,25 @@ describe('buttons', function () {
212212
expect(btns.eq(1)).not.toHaveClass('active');
213213
});
214214

215+
it('should handle string values in btn-radio value', function () {
216+
$scope.model = 'Two';
217+
var btns = compileButtons('<button ng-model="model" btn-radio="\'One\'">click1</button><button ng-model="model" btn-radio="\'Two\'">click2</button>', $scope);
218+
219+
expect(btns.eq(0)).not.toHaveClass('active');
220+
expect(btns.eq(1)).toHaveClass('active');
221+
222+
btns.eq(0).click();
223+
expect(btns.eq(0)).toHaveClass('active');
224+
expect(btns.eq(1)).not.toHaveClass('active');
225+
expect($scope.model).toEqual('One');
226+
227+
$scope.$digest();
228+
229+
expect(btns.eq(0)).toHaveClass('active');
230+
expect(btns.eq(1)).not.toHaveClass('active');
231+
expect($scope.model).toEqual('One');
232+
});
233+
215234
describe('uncheckable', function () {
216235
//model -> UI
217236
it('should set active class based on model', function () {

0 commit comments

Comments
 (0)