Почему переменные экземпляра иногда используются в тестах RSpec?

Question

Я заметил примеры использования переменной экземпляра в модульных тестов RSpec и т.д.

Что-то вроде этого:

it 'should update something' do 
    @user = user(:userone) 
    @attr = { 
    :name => 'this', 
    :phone => 'that' 
    } 
    put :update, :id => @user.id, :user => @attr 

    @user.reload 
    expect(response.status).to eq(200) 
    expect(@user.name).to eq('this') 
    expect(@user.phone).to eq('that') 
end 

Почему бы просто не использовать локальные переменные, если не устанавливать переменную в пределах before :each и т.д.?

Answer 1

Общая проблема заключается в том, что переменные в тестах (например, переменные в любой программе) должны обладать наименьшим охватом. Это упрощает чтение для читателей, поскольку это минимизирует количество кода, о котором они должны думать, и часто позволяет время выполнения быть более эффективным. Поэтому, если переменная экземпляра (или в текущей RSpec, переменная let) может быть локальной (без дублирующего кода или потери эффективности), она должна быть.

Назад, когда наилучший доступный способ передачи данных из блока before в пример (блок it) был в переменной экземпляра, я всегда видел спецификации, подобные вашему примеру. Иногда они были написаны неопытными программистами, которые видели множество переменных экземпляра, используемых в спецификациях, и немыслимо имитировали это. (Я все еще вижу ту же ошибку в контроллерах Rails.) Иногда они возникали из-за того, что кто-то вложил блок перед блоком, но не стал беспокоиться об изменении переменных экземпляра, которые больше не нужны, чтобы быть переменными экземпляра в locals.

В эти дни RSpec предоставляет let. Одна из причин, почему let лучше, чем переменная экземпляра, заключается в том, что переменная let выглядит как locals, поэтому вам не нужно менять их, если вы меняете локальную на переменную let или наоборот. Тем не менее, я все еще часто вижу проблему тестовых данных, которая используется только в одном примере, определяемом переменной let, когда она может и должна быть локальной. Подобно чрезмерному использованию переменных экземпляра, иногда это происходит от неопытных программистов, которые немыслимо имитируют примеры, а иногда и после сбоя при очистке после изменения теста делает ненужной переменную let.