RSpec集成测试时ApplicationController中定义的实例变量返回nil问题求助

你遇到的报错本质是测试环境运行时@test = User.find_by(id: 1)返回了nil，后续调用@test.value时触发了空对象方法调用异常，和控制器实例变量的访问权限无关，也不需要手动在测试里定义控制器实例变量。
Rails测试套件默认开启了事务回滚/数据库清空机制，每次运行测试前都会清空数据库数据，你开发/生产环境存在的id为1的用户，在测试环境中默认不存在，才会触发这个问题。

1. 优先添加空值防护（必做） #

即使在生产环境，也存在对应数据被误删的风险，先给方法加空值兼容逻辑，用Ruby的安全导航运算符&.即可：

def do_something_on_start
  @test = User.find_by(id: 1)
  @enabled = @test&.value == "true"
  @start_date = @test&.start_date
end

如果该用户是业务必须存在的核心数据，可以把find_by换成find，找不到时会直接抛出ActiveRecord::RecordNotFound异常，比空对象调用异常更方便排查问题。

2. 测试环境预置对应数据 #

根据你测试套件使用的工具，选择对应方式预置id为1的用户数据即可：

• 如果你用FactoryBot管理测试数据：在spec/rails_helper.rb中添加全局前置钩子，每次运行feature spec前自动创建对应数据

RSpec.configure do |config|
  config.before(:each, type: :feature) do
    User.find_or_create_by(id: 1) do |user|
      # 补充User模型的所有必填字段
      user.value = "false"
      user.start_date = Date.today
    end
  end
end

• 如果你用Rails默认的fixtures管理测试数据：直接在test/fixtures/users.yml（RSpec为spec/fixtures/users.yml）中定义固定id的用户即可，测试运行时会自动加载该数据。

3. 可选优化建议 #

不要在业务代码里硬写固定id查询数据，如果你用这个用户存储全局配置，可以单独抽离SystemConfig模型存储全局配置项，或者给User模型加个专属方法查询该固定用户，方便后续维护：

# app/models/user.rb
class User < ApplicationRecord
  def self.default_config_user
    find_by(id: 1)
  end
end

# 控制器内调用
@test = User.default_config_user

内容的提问来源于stack exchange，提问作者fev3r