Rspec-rails: Attendez-vous à ce que l'action change l'attribut d'objet en quelque chose

Le problème avec ce code est que reload est envoyé avant le changement.
Donc en gros ça :

expect { update_user }.to change(user.reload, :name).to 'Kaka Ruto'

s'étend à :

object = user.reload
old_value = object.send(:name)
update_user
new_value = object.send(:name)
expect(new_value).not_to eq(old_value) # i.e. "change"
expect(new_value).to eq('Kaka Ruto')

Ici, nous avons deux problèmes. La première est que reload est exécuté avant le changement. Vous vous attendiez probablement à ce qu'il se recharge après le changement.
Et une autre est que vous devez faire un rechargement, quelque chose dans votre code au lieu d'utiliser le modèle le trouve par id et le réinstancie à partir de zéro, donc celui instancié en test est désynchronisé avec celui du code testé vous obligeant à recharger.

Je vous propose d'utiliser :

expect { update_user }.to change { user.reload.name }.to 'Kaka Ruto'

Cela a un inconvénient que reload est exécuté deux fois, avant le changement et après.
J'ai vu quelque chose comme change_reloaded(model, attribute_name) matcher qui enregistre la valeur pour comparaison avant le changement, puis recharge et compare après le changement.
Cependant, le problème que update_user charge le modèle à partir de la base de données persiste.

N'hésitez pas à rouvrir si vous estimez que rspec-rails est incomplet ou présente un défaut.

Merci @pirj ! Cela aide en effet.

Vous êtes toujours les bienvenus!

Pour éviter le double rechargement vous pouvez aussi faire :

expect { update_user; user.reload }.to change { user.name }.to 'Kaka Ruto'

@JonRowe chouette !