onde esta errado ?

Bom dia Fábio beleza?

Então dando uma olhada o unico problema é vc usar a virgula para valores numéricos com casa decimal, o padrão é usar ponto nesses casos, ficando assim:

     var alturaInformada = 1.68;

obrigado

Oi Fábio, boa tarde.

Além do erro da vírgula, apontado pelo Rodrigo, vejo dois outros erros no seu código. O primeiro é apenas de ordem estrutural, ele não está afetando o resultado, mas segundo o professor o mais correto seria declarar todas as funções no início do código. Assim:

<script>

    function pula() {

        document.write("<br><br>");
    }

    function frase(texto) {

        document.write(texto);
        pula();
    }

    function calculaImc(peso, altura) {
         return peso /(altura *altura)
    }

O outro erro, esse sim afetando o resultado final, foi que você inverteu as informações na hora de passar o valor do peso e da altura.

Pensa que na Função calculaImc

 function calculaImc(peso, altura) 

invés de estar escrito Peso é X e invés de Altura é Y. E que, nesse caso, X sempre vai ser a primeira informação que você colocar entre parênteses para retornar. Então, sua função está basicamente assim :

function calculaImc (x , y ) {
   return x/ (y * y) ;
} 

Só que quando você foi criar a variável imc, você colocou

 var imc = calculaImc(alturaInformada, pesoInformado);

 ///lembrando, calculaImc é calcumaImc(x , y ) e x= peso / y = altura

ou seja, na variável imc seu X invés de receber um valor referente ao Peso, para realizar o cálculo da função calculaImc [ peso / (altura * altura) ], está recebendo o valor que seria para a altura.