Skip to content
This repository has been archived by the owner on Jul 12, 2024. It is now read-only.

Spec to test Rails 5.1 and convert to request spec #296

Merged
merged 6 commits into from
Apr 23, 2018

Conversation

lulalala
Copy link
Contributor

  1. Allow to test Rails 5.1, which in turn need to have spec changed to use params to pass parameters.
  2. Convert controller spec to request spec.
  3. Convert deprecated call to success? to successful?

before(:each) do
fake_adapter = double(normalize_params: normalized_params)
Griddler.adapter_registry.register(:one_that_works, fake_adapter)
Griddler.configuration.email_service = :one_that_works
end

describe 'POST create', type: :controller do
let(:path) { "/email_processor" }

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Prefer single-quoted strings when you don't need string interpolation or special symbols.

@@ -1,17 +1,19 @@
require 'spec_helper'

describe Griddler::EmailsController, :type => :controller do
RSpec.describe "Receiving Email", :type => :request do

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Prefer single-quoted strings when you don't need string interpolation or special symbols.
Use the new Ruby 1.9 hash syntax.

@lulalala
Copy link
Contributor Author

Feel free to give opinion on file name change, etc.

@wingrunr21 wingrunr21 merged commit 543033a into thoughtbot:master Apr 23, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants