Question

我为存储键/值对的表构建ActiveRecord模型

示例 -

|------------------------------|
| KEY      | VALUE             |
|----------|-------------------|
| LOCATION | San Francisco, CA |
| TITLE    | Manager           |
| LOCATION | New York City, NY |
|------------------------------|

这是模特 -

class CompanyEnum < ActiveRecord::Base
  KEYS = [:title, :department, :location]
  KEYS_ENUM = KEYS.map(&:to_s).map(&:upcase)

  # `key` column must be one of the above - LOCATION, DEPARTMENT, or TITLE
  validates(:key, inclusion: KEYS_ENUM, allow_nil: false)

  # `value` can be anything, but must be unique for a given key (ignoring case) 
  validates(
    :value,
    uniqueness: { scope: :key, case_sensitive: false },
    allow_nil: false
  )
end

我使用shoulda matchers为这些验证编写规范。所以在我的spec文件中我有以下两个规格 -

describe "validations" do
  it { should_not allow_value(nil).for(:key) }
  it { should_not allow_value(nil).for(:value) }
end

我的问题是:key的第一次验证通过，但:value的第二次验证失败。根据模型定义，两者都使用相同的allow_nil: false选项。

1) CompanyEnum validations value should not allow value to be set to nil
   Failure/Error: it { should_not allow_value(nil).for(:value) }
     Expected errors when value is set to nil,
     got no errors
   # ./spec/models/company_enum_spec.rb:13:in `block (4 levels) in <top (required)>'
   # ./spec/support/analytics.rb:4:in `block (2 levels) in <top (required)>'

将allow_nil: false与uniqueness:和scope: optoins一起使用是否存在任何逻辑问题？或者是否与我命名实际列:key和:value有关（因为这些列似乎足以与其他方法冲突）？

谢谢！

Answer 1

关于您的唯一性验证的

allow_nil: false并不意味着nil不是允许的值。这有点误导。

正如您所知，默认情况下，唯一性验证以这种方式工作（根据您使用的代码判断）：

如果有两个记录具有相同的key和value值，则第二个记录与第一个记录相比应该无效（假设数据库中存在第一个记录）。
如果有两个记录具有不同的key和value值，则两个记录都应该有效。

那么allow_nil做了什么？让我们看看the docs要说的内容：

:allow_nil - 如果设置为true，如果属性为nil（默认为false），则会跳过此验证。

因此，如果为true，则allow_nil允许两个记录与正在验证的属性的nil值共存（在本例中为value）。

但您已指定allow_nil: false，这意味着此选项无论如何都不适用。

总之，您allow_value的第一次使用失败，因为nil不是key的有效值（包含验证失败）。但是，第二种用法通过，因为nil是value的有效值（前提是没有现有记录也value为nil。

我的观点是，如果你真的想测试你的验证，你应该使用直接对应这些验证的匹配器：

it do
  should validate_inclusion_of(:key).
    in_array(["TITLE", "DEPARTMENT", "LOCATION"]).
    allow_nil
end

it do
  should validate_uniqueness_of(:value).
    scoped_to(:key).
    case_insensitive
end

使用shoulda匹配器将`allow_nil`与`uniqueness`和`scope`结合起来测试

1 个答案: