如何处理依赖对象（不知道如何表达）

Question

我正在学习 TDD 和 Rails。我正在伤害自己的“逻辑”问题。让我先介绍一个情况。

我们有 3 个模型：Project、Mission、Attachment。我是这样创建它们的

rails g model Mission
rails g model Project
rails g model Attachment title:string attachable:references{polymorphic}
rake db:migrate

是什么生成了这个schema.rb

ActiveRecord::Schema.define(version: 20180307200338) do
  create_table "attachments", force: :cascade do |t|
    t.string "attachable_type"
    t.integer "attachable_id"
    t.string "title"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.index ["attachable_type", "attachable_id"], name: "index_attachments_on_attachable_type_and_attachable_id"
  end
  create_table "missions", force: :cascade do |t|
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
  end
  create_table "projects", force: :cascade do |t|
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
  end
end

对于模型，我们有：

# app/models/mission.rb
class Mission < ApplicationRecord
  has_many :attachments, :as => :attachable
end

# app/models/project.rb
class Project < ApplicationRecord
  has_many :attachments, :as => :attachable
end

# app/models/attachment.rb
class Attachment < ApplicationRecord
  belongs_to :attachable, polymorphic: true
end

现在我们有了上下文，我想介绍一下我的“逻辑”问题。对我来说，像这样创建Attachment 是完全合乎逻辑的：

Mission.find(1).attachments.create(file: 'my-file.jpg')
Project.find(1).attachments.create(file: 'my-file.jpg')

但很难想象自己会创建这样的附件：

Attachment.create(attachable: Mission.find(1))
Attachment.create(attachable: Project.find(1))

在开始 TDD 之前，我从未想过这一点。我总是使用第一种方法，就是这样。但现在我正在为我的 Attachment 模型编写测试，我最终会做以下事情：

require 'test_helper'

class AttachmentTest < ActiveSupport::TestCase

  test "should be valid" do
    resource = build(:attachment)
    assert resource.valid?
  end

  # FIXME is 'attachable' the final name?
  test "should require attachable" do
    resource = build(:attachment, attachable: nil)
    assert resource.invalid?
  end

  test "should require file" do
    resource = build(:attachment, file: nil)
    assert resource.invalid?
  end
end

不知何故，我正在测试永远不会发生的场景build(:attachment)。

所以我的问题是：我是否应该辞职去想“我必须这样做才能证明Project.attachments.build() 会起作用”？

我也在测试Project 和Mission 之类的东西：

• 应该有很多附件
• 应该允许创建附件
• 应保存附件
• 应该检索附件

这变得令人困惑。

在测试方面，应该如何测试这种关系？

抱歉，这篇文章很长（可能）有点混乱。

编辑

问题奖励有没有办法简单地说“我们不能从附件创建并且必须通过关系”

Answer 1

在我看来，您正在尝试测试 Rails 中内置的功能：

• 应该有很多附件 = test has_many :attachments
• 应该允许创建附件 = testaccepts_nested_attributes_for :attachments
• 应该保存附件 = 测试 autosave 的 has_many
• 应该检索附件 = 再次测试has_many :attachments

还有，你自己说的：

不知何故，我正在测试永远不会发生的场景build(:attachment)。

那何必呢？

---

您想测试您的对象以确保它们响应规范。例如，如果你想确保Attachment 必须有 attachable，那么你可以这样写：

test 'must have attachable' do
  attachment = Attachment.new.tap(&:valid?)
  assert_includes(attachment.errors.details[:attachable], { error: :blank })
end

这是，我认为，确保您已对属性进行存在验证的最佳选择。