Skip to content

Commit b89d941

Browse files
rodyhaddadIgorMinar
rodyhaddad
authored and
IgorMinar
committed
style(parseSpec): make jshint happy
1 parent 07fa87a commit b89d941

File tree

1 file changed

+65
-35
lines changed

1 file changed

+65
-35
lines changed

‎test/ng/parseSpec.js

+65-35
Original file line numberDiff line numberDiff line change
@@ -703,7 +703,7 @@ describe('parser', function() {
703703
scope.fn = Function.prototype.call;
704704

705705
expect(function() {
706-
scope.$eval('$eval.call()')
706+
scope.$eval('$eval.call()');
707707
}).toThrowMinErr(
708708
'$parse', 'isecff', 'Referencing call, apply or bind in Angular expressions is disallowed! ' +
709709
'Expression: $eval.call()');
@@ -719,13 +719,13 @@ describe('parser', function() {
719719
scope.apply = Function.prototype.apply;
720720

721721
expect(function() {
722-
scope.$eval('$eval.apply()')
722+
scope.$eval('$eval.apply()');
723723
}).toThrowMinErr(
724724
'$parse', 'isecff', 'Referencing call, apply or bind in Angular expressions is disallowed! ' +
725725
'Expression: $eval.apply()');
726726

727727
expect(function() {
728-
scope.$eval('apply()')
728+
scope.$eval('apply()');
729729
}).toThrowMinErr(
730730
'$parse', 'isecff', 'Referencing call, apply or bind in Angular expressions is disallowed! ' +
731731
'Expression: apply()');
@@ -735,13 +735,13 @@ describe('parser', function() {
735735
scope.bind = Function.prototype.bind;
736736

737737
expect(function() {
738-
scope.$eval('$eval.bind()')
738+
scope.$eval('$eval.bind()');
739739
}).toThrowMinErr(
740740
'$parse', 'isecff', 'Referencing call, apply or bind in Angular expressions is disallowed! ' +
741741
'Expression: $eval.bind()');
742742

743743
expect(function() {
744-
scope.$eval('bind()')
744+
scope.$eval('bind()');
745745
}).toThrowMinErr(
746746
'$parse', 'isecff', 'Referencing call, apply or bind in Angular expressions is disallowed! ' +
747747
'Expression: bind()');
@@ -840,97 +840,127 @@ describe('parser', function() {
840840
describe('Disallowed fields', function() {
841841
it('should NOT allow access or invocation of __defineGetter__', function() {
842842
expect(function() {
843-
scope.$eval('{}.__defineGetter__'); }).toThrowMinErr('$parse', 'isecfld');
843+
scope.$eval('{}.__defineGetter__');
844+
}).toThrowMinErr('$parse', 'isecfld');
844845
expect(function() {
845-
scope.$eval('{}.__defineGetter__("a", "".charAt)'); }).toThrowMinErr('$parse', 'isecfld');
846+
scope.$eval('{}.__defineGetter__("a", "".charAt)');
847+
}).toThrowMinErr('$parse', 'isecfld');
846848

847849
expect(function() {
848-
scope.$eval('{}["__defineGetter__"]'); }).toThrowMinErr('$parse', 'isecfld');
850+
scope.$eval('{}["__defineGetter__"]');
851+
}).toThrowMinErr('$parse', 'isecfld');
849852
expect(function() {
850-
scope.$eval('{}["__defineGetter__"]("a", "".charAt)'); }).toThrowMinErr('$parse', 'isecfld');
853+
scope.$eval('{}["__defineGetter__"]("a", "".charAt)');
854+
}).toThrowMinErr('$parse', 'isecfld');
851855

852856
scope.a = "__define";
853857
scope.b = "Getter__";
854858
expect(function() {
855-
scope.$eval('{}[a + b]'); }).toThrowMinErr('$parse', 'isecfld');
859+
scope.$eval('{}[a + b]');
860+
}).toThrowMinErr('$parse', 'isecfld');
856861
expect(function() {
857-
scope.$eval('{}[a + b]("a", "".charAt)'); }).toThrowMinErr('$parse', 'isecfld');
862+
scope.$eval('{}[a + b]("a", "".charAt)');
863+
}).toThrowMinErr('$parse', 'isecfld');
858864
});
859865

860866
it('should NOT allow access or invocation of __defineSetter__', function() {
861867
expect(function() {
862-
scope.$eval('{}.__defineSetter__'); }).toThrowMinErr('$parse', 'isecfld');
868+
scope.$eval('{}.__defineSetter__');
869+
}).toThrowMinErr('$parse', 'isecfld');
863870
expect(function() {
864-
scope.$eval('{}.__defineSetter__("a", "".charAt)'); }).toThrowMinErr('$parse', 'isecfld');
871+
scope.$eval('{}.__defineSetter__("a", "".charAt)');
872+
}).toThrowMinErr('$parse', 'isecfld');
865873

866874
expect(function() {
867-
scope.$eval('{}["__defineSetter__"]'); }).toThrowMinErr('$parse', 'isecfld');
875+
scope.$eval('{}["__defineSetter__"]');
876+
}).toThrowMinErr('$parse', 'isecfld');
868877
expect(function() {
869-
scope.$eval('{}["__defineSetter__"]("a", "".charAt)'); }).toThrowMinErr('$parse', 'isecfld');
878+
scope.$eval('{}["__defineSetter__"]("a", "".charAt)');
879+
}).toThrowMinErr('$parse', 'isecfld');
870880

871881
scope.a = "__define";
872882
scope.b = "Setter__";
873883
expect(function() {
874-
scope.$eval('{}[a + b]'); }).toThrowMinErr('$parse', 'isecfld');
884+
scope.$eval('{}[a + b]');
885+
}).toThrowMinErr('$parse', 'isecfld');
875886
expect(function() {
876-
scope.$eval('{}[a + b]("a", "".charAt)'); }).toThrowMinErr('$parse', 'isecfld');
887+
scope.$eval('{}[a + b]("a", "".charAt)');
888+
}).toThrowMinErr('$parse', 'isecfld');
877889
});
878890

879891
it('should NOT allow access or invocation of __lookupGetter__', function() {
880892
expect(function() {
881-
scope.$eval('{}.__lookupGetter__'); }).toThrowMinErr('$parse', 'isecfld');
893+
scope.$eval('{}.__lookupGetter__');
894+
}).toThrowMinErr('$parse', 'isecfld');
882895
expect(function() {
883-
scope.$eval('{}.__lookupGetter__("a")'); }).toThrowMinErr('$parse', 'isecfld');
896+
scope.$eval('{}.__lookupGetter__("a")');
897+
}).toThrowMinErr('$parse', 'isecfld');
884898

885899
expect(function() {
886-
scope.$eval('{}["__lookupGetter__"]'); }).toThrowMinErr('$parse', 'isecfld');
900+
scope.$eval('{}["__lookupGetter__"]');
901+
}).toThrowMinErr('$parse', 'isecfld');
887902
expect(function() {
888-
scope.$eval('{}["__lookupGetter__"]("a")'); }).toThrowMinErr('$parse', 'isecfld');
903+
scope.$eval('{}["__lookupGetter__"]("a")');
904+
}).toThrowMinErr('$parse', 'isecfld');
889905

890906
scope.a = "__lookup";
891907
scope.b = "Getter__";
892908
expect(function() {
893-
scope.$eval('{}[a + b]'); }).toThrowMinErr('$parse', 'isecfld');
909+
scope.$eval('{}[a + b]');
910+
}).toThrowMinErr('$parse', 'isecfld');
894911
expect(function() {
895-
scope.$eval('{}[a + b]("a")'); }).toThrowMinErr('$parse', 'isecfld');
912+
scope.$eval('{}[a + b]("a")');
913+
}).toThrowMinErr('$parse', 'isecfld');
896914
});
897915

898916
it('should NOT allow access or invocation of __lookupSetter__', function() {
899917
expect(function() {
900-
scope.$eval('{}.__lookupSetter__'); }).toThrowMinErr('$parse', 'isecfld');
918+
scope.$eval('{}.__lookupSetter__');
919+
}).toThrowMinErr('$parse', 'isecfld');
901920
expect(function() {
902-
scope.$eval('{}.__lookupSetter__("a")'); }).toThrowMinErr('$parse', 'isecfld');
921+
scope.$eval('{}.__lookupSetter__("a")');
922+
}).toThrowMinErr('$parse', 'isecfld');
903923

904924
expect(function() {
905-
scope.$eval('{}["__lookupSetter__"]'); }).toThrowMinErr('$parse', 'isecfld');
925+
scope.$eval('{}["__lookupSetter__"]');
926+
}).toThrowMinErr('$parse', 'isecfld');
906927
expect(function() {
907-
scope.$eval('{}["__lookupSetter__"]("a")'); }).toThrowMinErr('$parse', 'isecfld');
928+
scope.$eval('{}["__lookupSetter__"]("a")');
929+
}).toThrowMinErr('$parse', 'isecfld');
908930

909931
scope.a = "__lookup";
910932
scope.b = "Setter__";
911933
expect(function() {
912-
scope.$eval('{}[a + b]'); }).toThrowMinErr('$parse', 'isecfld');
934+
scope.$eval('{}[a + b]');
935+
}).toThrowMinErr('$parse', 'isecfld');
913936
expect(function() {
914-
scope.$eval('{}[a + b]("a")'); }).toThrowMinErr('$parse', 'isecfld');
937+
scope.$eval('{}[a + b]("a")');
938+
}).toThrowMinErr('$parse', 'isecfld');
915939
});
916940

917941
it('should NOT allow access to __proto__', function() {
918942
expect(function() {
919-
scope.$eval('{}.__proto__'); }).toThrowMinErr('$parse', 'isecfld');
943+
scope.$eval('{}.__proto__');
944+
}).toThrowMinErr('$parse', 'isecfld');
920945
expect(function() {
921-
scope.$eval('{}.__proto__.foo = 1'); }).toThrowMinErr('$parse', 'isecfld');
946+
scope.$eval('{}.__proto__.foo = 1');
947+
}).toThrowMinErr('$parse', 'isecfld');
922948

923949
expect(function() {
924-
scope.$eval('{}["__proto__"]'); }).toThrowMinErr('$parse', 'isecfld');
950+
scope.$eval('{}["__proto__"]');
951+
}).toThrowMinErr('$parse', 'isecfld');
925952
expect(function() {
926-
scope.$eval('{}["__proto__"].foo = 1'); }).toThrowMinErr('$parse', 'isecfld');
953+
scope.$eval('{}["__proto__"].foo = 1');
954+
}).toThrowMinErr('$parse', 'isecfld');
927955

928956
scope.a = "__pro";
929957
scope.b = "to__";
930958
expect(function() {
931-
scope.$eval('{}[a + b]'); }).toThrowMinErr('$parse', 'isecfld');
959+
scope.$eval('{}[a + b]');
960+
}).toThrowMinErr('$parse', 'isecfld');
932961
expect(function() {
933-
scope.$eval('{}[a + b].foo = 1'); }).toThrowMinErr('$parse', 'isecfld');
962+
scope.$eval('{}[a + b].foo = 1');
963+
}).toThrowMinErr('$parse', 'isecfld');
934964
});
935965
});
936966

0 commit comments

Comments
 (0)