las clases no están hechas para ser elegidas al azar... para eso está el array D: pero aquí un workaround
(te faltó una coma en linea 9)
var conteo = 0;
var azar = 0;
var ataque;
for(ataque in AttackType.Water) conteo++;
var azar = parseInt(Math.random() * conteo);
for(ataque in AttackType.Water){
if(azar--==0)break;
}
alert(ataque)
no se si quieres eliminar el primero (name:"Water"), pero si es así azar es 1+Math.random() * (conteo-1)
espero que sirva