Bom dia, Pedro! Tudo bem?
Está correto o funcionamento mas você não precisaria fazer um novo if que confere a mesma coisa que o outro, você pode fazer esse break em seguida do alert:
if( loginCadastrado == loginInformado && senhaCadastrada == senhaInformada ) {
alert("Bem-vindo ao sistema " + loginInformado);
break;
} else {
alert("Login inválido. Tente novamente");
}
Dessa maneira ele vai ter o mesmo resultado (parar de perguntar o login depois de acertar). Também vai diminuir o número de linhas e o código fica mais limpo.
Mas em geral, testei aqui e está funcionando direito e o código segue tudo proposto a ensinar até agora no curso. Parabéns pela dedicação!
Qualquer coisa pode contar conosco. Bons estudos!