RSpec + объект формы + простая форма дает неопределенные методы - PullRequest
Новая
       10

RSpec + объект формы + простая форма дает неопределенные методы

0 голосов
/ 19 мая 2018

Я пытаюсь сделать некоторые насмешки в спецификации для контроллера.Я написал код, который прекрасно работает для случая, когда я публикую действительную статью, но когда я пытаюсь указать, что она не должна быть сохранена, я сталкиваюсь с ошибками.

Мой код:

Форма статьи: 

class ArticleForm
  include ActiveModel::Model
  delegate :title, :body, :author_id, :tags, :id, :persisted?, :new_record?, to: :article
  attr_accessor :article, :tags_string
  validates :title, :body, :tags, presence: true
  validates_length_of :title, within: 8..512
  validates_length_of :body, within: 8..2048
  validate :validate_prohibited_words

  def initialize(article = Article.new)
    @article = article
  end

  def save(article_params)
    assign_params_to_article(article_params)

    if valid?
      @article.tags.each(&:save!)
      @article.save!
      true
    else
      false
    end
  end
...
end

Контроллер статей (только действие создания): 

  def create
    @article_form = ArticleForm.new
    if @article_form.save(article_params)
      flash[:notice] = 'You have added a new article.'
      redirect_to @article_form.article
    else
      flash[:danger] = 'Failed to add new article.'
      render :new
    end
  end

_form: 

= simple_form_for @article_form,
  url: (@article_form.article.new_record? ? articles_path : article_path(@article_form.article) ) do |f|
  = f.input :title, label: "Article title:"
  = f.input :body, label: "Body of the article:", as: :text, input_html: { :style => 'height: 200px' }
  = f.input :tags_string, label: "Tags:", input_html: { value: f.object.all_tags }
  = f.button :submit, 'Send!'

Спецификация контроллера статьи: 

require 'rails_helper'

RSpec.describe ArticlesController, type: :controller do
  render_views

  let!(:user) { create(:user) }
  let!(:tag) { create(:tag) }
  let(:tags_string) { 'test tag' }
  let!(:article) { create(:article, :with_comments, tags: [tag], author_id: user.id) }


  context 'user logged in' do
    before { sign_in(user) }

    describe 'POST artictles#create' do
      let(:article_form) { instance_double(ArticleForm) }
      let(:form_params) do
        {
          article_form:
          {
            title: 'title',
            body: 'body',
            tags_string: tags_string
          }
        }
      end

      context 'user adds valid article' do
        it 'redirects to new article', :aggregate_failures do
          expect(ArticleForm).to receive(:new).and_return(article_form)
          expect(article_form).to receive(:save).with(hash_including(:author_id, form_params[:article_form]))
                                                .and_return(true)
          allow(article_form).to receive(:article) { article }

          post :create, params: form_params
          expect(response).to redirect_to(article)
        end
      end

      context 'user adds invalid article' do
        it 'renders new form', :aggregate_failures do
          expect(ArticleForm).to receive(:new).and_return(article_form)
          allow(article_form).to receive(:article) { article }

          expect(article_form).to receive(:save).with(hash_including(:author_id, form_params[:article_form]))
                                                .and_return(false)

          post :create, params: form_params
          expect(response).to render_template(:new)
        end
      end
    end
  end
end

Публикация правильных сообщений работает нормально, это ошибка, которую я получаю на 'недействительном сообщении':

Сбои:

1) Пользователь ArticlesController, вошедший в артикулы POST # создает пользователя, добавляет недействительную статью, отображает новую форму Ошибка 1 и 1 другая ошибка: 

 1.1) Failure/Error: = simple_form_for @article_form,
        #<InstanceDouble(ArticleForm) (anonymous)> received unexpected message :model_name with (no args)
      # ./app/views/articles/_form.html.haml:2:in `_app_views_articles__form_html_haml__2443549359101958040_47338976410880'
      # ./app/views/articles/new.html.haml:2:in `_app_views_articles_new_html_haml___678894721646621807_47338976253400'
      # ./app/controllers/articles_controller.rb:26:in `create'
      # ./spec/controllers/articles_controller_spec.rb:51:in `block (5 levels) in <top (required)>'

 1.2) Failure/Error: = simple_form_for @article_form,

      ActionView::Template::Error:
        undefined method `param_key' for #<Array:0x0000561bedbcd888>
      # ./app/views/articles/_form.html.haml:2:in `_app_views_articles__form_html_haml__2443549359101958040_47338976410880'
      # ./app/views/articles/new.html.haml:2:in `_app_views_articles_new_html_haml___678894721646621807_47338976253400'
      # ./app/controllers/articles_controller.rb:26:in `create'
      # ./spec/controllers/articles_controller_spec.rb:51:in `block (5 levels) in <top (required)>'
      # ------------------
      # --- Caused by: ---
      # NoMethodError:
      #   undefined method `param_key' for #<Array:0x0000561bedbcd888>
      #   ./app/views/articles/_form.html.haml:2:in `_app_views_articles__form_html_haml__2443549359101958040_47338976410880'

Я сделал несколько попыток добавить отсутствующие методы, разрешивобъект, чтобы получить его, но хорошо ли это делать?Я должен был бы разрешить каждый отдельный вызов позже (когда я разрешаю param_keys, он запрашивает все значения для _form - title, body и tags).Есть ли способ заставить его работать без указания всех методов построчно?

1 Ответ

0 голосов
/ 21 мая 2018

Тесты контроллера в Rails используются для "функционального" тестирования .Это означает, что они тестируют несколько уровней запроса, который отправляется приложению.

Таким образом, вы пытаетесь проверить, что все части, участвующие в обработке запроса, работают.Из-за этого использование тестов и заглушек нежелательно в таких тестах, потому что вы пытаетесь сделать тест как можно более реальным.Я бы порекомендовал создать экземпляр реального ArticleForm объекта вместо создания экземпляра double. 

let(:article_form) { ArticleForm.new(article) }

Таким образом, вы также тестируете экземпляр ArticleForm.В некоторых случаях заглушка или насмешка могут иметь смысл и в тесте контроллера, но в вашем случае экземпляр article_form находится в центре теста, поэтому использование для него макета означает огромную работу, и тест будетусложняемся.

Если вы хотите попробовать макет, лучше всего начать с , например, посмотреть спецификации .

...