Quand tu fais ça : employeeList[i].salary = salaryAugmentation(employeeList[i],100);
Ce qui appelle ça : this.salary = this.salary + augmentation;
return this.salary;
C'est absurde. Dans la fonction SalaryAugmentation, tu calcules le nouveau salaire et tu modifies du coup l'attribut salaire de l'employé en question... Mais l'appel de la fonction se fait avec un signe "=", donc tu modifies cette valeur une deuxième fois...
De même, tu passes un employé en paramètre, et la fonction s'appelle sur le même employé ? du coup le paramètre n'est pas utilisé.
Demande toi ce que tu préfères faire : Soit ta fonction ne prend en paramètre que l'augmentation (int), et tu utilises this, du coup tu appelles en faisant employe.salaryAugmentation(100).
Ou alors ta fonction prend l'employé et l'augmentation en paramètre, donc tu l'appelles comme ça : salaryAugmentation(employe28, 250);
Et dedans tu fais "employe.salary += augmentation;".
Dernière chose si c'est pas trop pour toi, l'accès direct et public aux attributs est fortement déconseillé en POO, c'est-à-dire que les attributs doivent être private et non public, et être modifiable et consultables via des méthodes set et get pour chaque attribut (quand c'est nécessaire bien sûr).