Increase test coverage: Array.prototype.splice

Branch coverage:
   Before: 56/78
   After: 78/78

Also corrected a mistake, that "else if" branch is not needed, should be replaced with an "else".

JerryScript-DCO-1.0-Signed-off-by: Mate Dabis [email protected]

Author: matedabis

jerry-core/ecma/builtin-objects/ecma-builtin-array-prototype.c

@@ -1378,8 +1378,9 @@ ecma_builtin_array_prototype_object_splice (ecma_value_t this_arg, /**< this arg
       }
     }
     /* 13. */
-    else if (item_count > delete_count)
+    else
     {
+      JERRY_ASSERT (item_count > delete_count);
       /* 13.b */
       for (k = len - delete_count; k > start  && ecma_is_value_empty (ret_value); k--)
       {

tests/jerry/array-prototype-splice.js

@@ -175,3 +175,174 @@ try {
   assert (e.message === "foo");
   assert (e instanceof ReferenceError);
 }
+
+try {
+  Array.prototype.splice.call(undefined);
+  assert(false);
+} catch(e) {
+  assert(e instanceof TypeError);
+}
+
+try {
+  var o = {};
+  Object.defineProperty(o, 'toString', { 'get' : function() { throw new ReferenceError("foo"); } });
+  var a = { length : o }
+  Array.prototype.splice.call(a, function() { })
+  assert(false);
+} catch(e) {
+  assert(e instanceof ReferenceError);
+}
+
+try {
+  var o = {};
+  Object.defineProperty(o, 'toString', { 'get' : function() { throw new ReferenceError("foo"); } });
+  [1,2].splice(o)
+  assert(false);
+} catch(e) {
+  assert(e instanceof ReferenceError);
+}
+
+try {
+  var o = {};
+  Object.defineProperty(o, 'toString', { 'get' : function() { throw new ReferenceError("foo"); } });
+  [1,2].splice(1, o)
+  assert(false);
+} catch(e) {
+  assert(e instanceof ReferenceError);
+}
+
+try {
+  var a = [1,5, ,2]
+  Object.defineProperty(a, '2', { 'get' : function() { throw new ReferenceError("foo"); } })
+  a.splice(1, 7, 7, 7,7,7)
+  assert(false)
+} catch(e) {
+  assert(e instanceof ReferenceError);
+}
+
+var a = {length : 13}
+Array.prototype.splice.call(a, function () {delete a})
+assert(a.length === 0)
+
+try {
+  var a = [1,5,6,7,8,5]
+  Object.defineProperty(a, '1', { 'get' : function() { throw new ReferenceError("foo"); } })
+  Object.defineProperty(a, '0', { 'get' : function() { throw new ReferenceError("foo"); } })
+  Object.defineProperty(a, '2', { 'get' : function() { throw new ReferenceError("foo"); } })
+  Array.prototype.splice.call(a, 0, 3)
+  assert(false)
+} catch(e) {
+  assert(e instanceof ReferenceError);
+}
+
+try {
+  f = function() {delete arr[3]; arr.length = 13; Object.defineProperty(arr, '5', function() { })};
+  obj = {get: f, valueOf : f, toString: f}
+  arr = [1, 2, obj, 4, 5];
+  Object.defineProperty(arr, '2',{ 'get' : f } )
+  print(arr)
+  arr.splice(1, 4, obj);
+  print(arr)
+} catch(e) {}
+
+try {
+  f = function() { for(var i=0; i<arr.length; i++) { delete arr[i]} };
+  obj = {get: f, valueOf : f, toString: f}
+  arr = [1, 2, obj, 4, 5];
+  Object.defineProperty(arr, '0', { 'get' : f })
+  Object.defineProperty(arr, '1', { 'get' : f })
+  Object.defineProperty(arr, '2', { 'get' : f })
+  Object.defineProperty(arr, '3', { 'get' : f })
+  Object.defineProperty(arr, '4', { 'get' : f })
+  Object.defineProperty(arr, '5', { 'get' : f })
+  arr.splice(1, 3, obj);
+} catch(e) {}
+
+try {
+  f = function() { throw new TypeError;};
+  obj = {get: f, valueOf : f, toString: f}
+  arr = [1, 2, obj, 4, 5];
+  Object.defineProperty(arr, '4', { 'get' : f } )
+  arr.splice(1, 3, obj);
+} catch(e) {}
+
+try {
+  f = function() { for(var i=0; i<arr.length; i++) { delete arr[i]} };
+  f1 = function() { throw new TypeError;};
+  f2 = function() { delete arr[2];};
+  f3 = function() { delete arr[3];};
+  f4 = function() { delete arr[4];};
+  arr = [1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12];
+  delete arr[2];
+  delete arr[3];
+  Object.defineProperty(arr, '0', { 'get' : f })
+  Object.defineProperty(arr, '1', { 'get' : f })
+  Object.defineProperty(arr, '2', { 'get' : f })
+  Object.defineProperty(arr, '3', { 'get' : f })
+  Object.defineProperty(arr, '8', { 'get' : f2 })
+  Object.defineProperty(arr, '9', { 'get' : f3 })
+  Object.defineProperty(arr, '10', { 'get' : f4 })
+  arr.splice(1, 7, 5);
+} catch(e) {}
+
+try {
+  f = function() { for(var i=0; i<arr.length; i++) { delete arr[i]} };
+  f1 = function() { throw new TypeError;};
+  f2 = function() { delete arr[2];};
+  f3 = function() { delete arr[3];};
+  f4 = function() { delete arr[4];};
+  arr = [1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24];
+  delete arr[2];
+  delete arr[3];
+  Object.defineProperty(arr, '0', { 'get' : f })
+  Object.defineProperty(arr, '1', { 'get' : f })
+  Object.defineProperty(arr, '2', { 'get' : f })
+  Object.defineProperty(arr, '3', { 'get' : f })
+  Object.defineProperty(arr, '8', { 'get' : f2 })
+  Object.defineProperty(arr, '9', { 'get' : f3 })
+  Object.defineProperty(arr, '10', { 'get' : f4 })
+  arr.splice(1, 7, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5);
+} catch(e) {}
+
+try {
+  var a = [1, 5, 6, 7, 8, 5]
+  Object.defineProperty(a, '1', { 'get' : function() { throw new ReferenceError("foo"); } })
+  Object.defineProperty(a, '0', { 'get' : function() { throw new ReferenceError("foo"); } })
+  Object.defineProperty(a, '2', { 'get' : function() { throw new ReferenceError("foo"); } })
+  Array.prototype.splice.call(a, 0, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3)
+  assert(false)
+} catch(e) {
+  assert(e instanceof ReferenceError);
+}
+
+try {
+  f = function() {  for(var i=0; i<arr.length; i++) { delete arr[i]} };
+  arr = [1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24];
+  delete arr[22];
+  delete arr[23];
+  Object.defineProperty(arr, '21',{ 'get' : f })
+  Object.defineProperty(arr, '22',{ 'get' : f })
+  Object.defineProperty(arr, '23',{ 'get' : f })
+  arr.splice(1, 7, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5);
+} catch(e) {}
+
+try {
+  f = function () { throw new TypeError;};
+  arr = [1, 2, 4, 5];
+  Object.defineProperty(arr, '4',{ 'get' : f })
+  arr.splice(1, 2, 3, 4, 5, 6, 7, 8, 9, 10);
+} catch (e) {}
+
+try {
+  f = function () { delete arr[23];}
+  arr = [1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24];
+  delete arr[23];
+  Object.defineProperty(arr, '23',{ 'get' : f })
+  arr.splice(1, 7, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5);
+} catch (e) {}
+
+try {
+  arr = [];
+  Object.defineProperty(arr, 'length', { value : 999, writable: false })
+  arr.splice(1, 2, 4, 5);
+} catch(e) {}