Hay algunos errores en su código que podrían estar causando el comportamiento inesperado.

• Como tiene i++ dentro de la cabeza del bucle for, no necesita incluirlo en el cuerpo.
• Dado que el segundo parámetro de slice no es inclusivo, según las condiciones de su prueba, en realidad desea dividir a i + 2 ;

Código completo:

 function slice(array) { for (let i = 0; i < array.length; i++) { if (array[i] !== null) return array.slice(0, i + 2); } return array; } console.log(slice([null, null, 'a', 'b', 'c'])); // => [null, null, a, b] console.log(slice([null, 'a', 'b', 'c'])); // => [null, a, b] console.log(slice([null, null])); // => [null, null]

Podría tomar un cierre sobre un contador para los siguientes valores y filtrar la matriz.

 const slice = array => array.filter( (c => v => v === null || c && c--) (2) ); console.log(...slice([null, null, 'a', 'b', 'c', 'd', 'e'])); // [null, null, a, b] console.log(...slice([null, null, 'a', 'b', 'c'])); // [null, null, a, b] console.log(...slice([null, 'a', 'b', 'c'])); // [null, a, b] console.log(...slice([null, null])); // [null, null]

Me parece que, según sus ejemplos, solo necesitaba cortar una matriz sin el último elemento de la matriz. Desea cambiar la primera matriz con nulos, a, b y c a otra con los mismos nulos, a y b.

 [null, null, a, b, c] => [null, null, a, b]

Entonces, si ese es el caso, ¿por qué no simplemente hacer estallar la matriz?

 let arr = [null, null, a, b, c]; arr.pop();

arr es ahora [nulo, nulo, a, b];

Para el caso especial en el que solo se deben eliminar los valores no nulos y, por lo tanto, devolver la misma matriz si contiene todos los valores nulos, puede simplemente abrir, inspeccionar y, si es nulo, volver a la matriz original nuevamente.

 let arr = [null, null, a, b, c]; if(arr.pop() === null) arr.push(null);

¡Y eso es todo!

Debería aclarar lo que quiere decir con i + 1. Parece que desea dividir una nueva matriz con todos los valores nulos y un elemento después. Pero no puedo responder a eso porque no estoy seguro.