Zadacha 7-2

Нет, это совершенно не так работает. Что ты на самом деле делаешь:

вызываешь метод login и передаёшь туда параметры params[:email] и params[:password]

а затем результат метода ты присваиваешь в переменную @user

и потом ты эту переменную никак не используешь.