Rspec、模型加载顺序、固定装置和named_scope 挑战

Question

我有一些球员，球员有交易状态。我认为我会很聪明并拥有一个单独的 TradeState 模型，以便玩家拥有 trade_state_id （玩家只能处于一次一种贸易状态）。

现在，通过使用命名范围然后通过说“Player.active”来获取所有活跃玩家将很方便。为此，我需要获取与“active”匹配的 TradeState 记录的 ID，因此我在 Player 类中提出了这个：

named_scope :active, :conditions => {:trade_state_id => TradeState.active.first.id}

在脚本/控制台中测试时，这就像一个魅力，但当我去测试。我正在使用 RSpec，但我怀疑这不相关。当我运行最简单的测试时，我收到以下错误：

“为 nil 调用 id，这会错误地为 4”

据我所知，测试框架正在按字母顺序加载和解析模型。该框架解析 Player 模型中的 name_scope 调用，并尽职尽责地查找第一个处于活动状态的 TradeState 记录的 id。然而，该模型尚未处理且尚未准备好，因此会出现有关获取 nil id 的错误。

起初我以为是因为 trade_states 表中可能没有任何记录，所以我在 before(:each) 块中创建并保存了我需要的 trade_states，但这不起作用。然后我制作了一些装置并尝试加载它们，但这不起作用。

这看起来合理吗？还有其他解释吗？解决方法怎么样？我可以尝试模拟 TradeState 对象，然后我会尝试一下。

非常感谢您抽出时间。

Answer 1

您提到的这段代码在播放器“加载”时执行。因此，即使您在 before(:each) 块中创建设置数据，该模型也可能在此之前加载。要解决该问题，您可以尝试以下操作：

named_scope :active, lambda { {:conditions => {:trade_state_id => TradeState.active.first.id} } }

这可确保每当您调用named_scope时都会调用TradeState.active。

但是通过调用 TradeState.active.first 可以实现什么目的。 做到这一点：

:conditions => {:trade_state_id => TradeState::ACTIVE}

相反，您可以通过这种很好的方式在 TradeState 模型上使用以下代码来

class TradeState < ActiveRecord::Base
  def self.const_missing(sym)
    const_set(sym, find_by_name(sym.to_s))
  end
end

Answer 2

这实际上是一个潜在的“预期”结果。如果没有活跃玩家，则

TradeState.active

为空，因此

TradeState.active.first

为 nil，

从而

TradeState.active.first.id

在 nil 对象上调用 Object::id 。

您可以尝试：

TradeState.active.empty? ? 0 : TradeState.active.first[:id]

在任何情况下，请确保包含 trade_states 以确保它们存在于连接中。

Answer 3

不幸的是，尽管所有这些建议都非常好，但在运行测试用例时没有任何可靠的工作。起初，我放弃了所有这些代码，但我必须编写的代码吸引力太低，以至于我开始重新思考这个问题。

因为我们将通过大量的交易状态来查找玩家，所以在数字交易状态列上建立索引是一个明确的优先事项。然而，没有任何内容表明贸易状态需要是一个表格，因为只定义了几个贸易状态，并且定义的贸易状态不太可能有太大的波动。

因此，我使 TradeState 成为一个独立的类，不再从 ActiveRecord 派生，并定义了我需要的常量。

感谢大家的想法和解决方案。