Como ya te dijo @Riven en comentarios, no puedes devolver 2 valores y `return max, min;` podría generar error de sintaxis o devolver solo un valor. Debes corregir encerrando ambos valores entre corchetes para que sea un arreglo válido `return [max, min];`. Pero no están en el orden correcto de acuerdo al enunciado:

> debe devolver un arreglo que contenga el menor número del arreglo 'numeros' en la posición cero y el mayor número del arreglo 'numeros' en la posición 1

No es necesario usar ciclos para obtener los elementos, puedes hacer uso de funciones matemáticas como [Math.min()](https://developer.mozilla.org/es/docs/Web/JavaScript/Reference/Global_Objects/Math/min) para obtener el menor y [Math.max()](https://developer.mozilla.org/es/docs/Web/JavaScript/Reference/Global_Objects/Math/max) para el máximo.

El "único detalle" es que no puedes enviar el arreglo `numeros` como parámetro, tienes que [desestructurarlo](https://developer.mozilla.org/es/docs/Web/JavaScript/Reference/Operators/Destructuring_assignment) para que todos los elementos se tomen como parámetros individuales.

<!-- begin snippet: js hide: false console: true babel: false -->

<!-- language: lang-js -->

    function menorMayor(numeros) {
        return [Math.min(...numeros), Math.max(...numeros)];
    }

    console.log(menorMayor([4, 6, 1, 7, 15]));

<!-- end snippet -->