aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDavid Gil <dgilperez@gmail.com>2018-12-14 21:24:03 +0100
committerDavid Gil <dgilperez@gmail.com>2018-12-14 21:24:03 +0100
commitd872de3c69a8762834d206d2f080550c39143080 (patch)
tree72a910858482fe9cd3e3ba093d4d962afbca2270
parentd6621cc2659127217c3c4a128f1ba76cf5c231e8 (diff)
downloadrails-d872de3c69a8762834d206d2f080550c39143080.tar.gz
rails-d872de3c69a8762834d206d2f080550c39143080.tar.bz2
rails-d872de3c69a8762834d206d2f080550c39143080.zip
Add Mailbox and MailboxTest generators
-rw-r--r--.gitignore3
-rw-r--r--lib/rails/generators/mailbox/USAGE12
-rw-r--r--lib/rails/generators/mailbox/mailbox_generator.rb38
-rw-r--r--lib/rails/generators/mailbox/templates/application_mailbox.rb.tt5
-rw-r--r--lib/rails/generators/mailbox/templates/mailbox.rb.tt8
-rw-r--r--lib/rails/generators/mailbox/templates/mailer_test.rb.tt13
-rw-r--r--test/generators/mailbox_generator_test.rb77
7 files changed, 156 insertions, 0 deletions
diff --git a/.gitignore b/.gitignore
index 4862d67b18..ead1a2d308 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,2 +1,5 @@
.byebug_history
*.sqlite3-journal
+.ruby-version
+.ruby-gemset
+/tmp/
diff --git a/lib/rails/generators/mailbox/USAGE b/lib/rails/generators/mailbox/USAGE
new file mode 100644
index 0000000000..d679dd63ae
--- /dev/null
+++ b/lib/rails/generators/mailbox/USAGE
@@ -0,0 +1,12 @@
+Description:
+============
+ Stubs out a new mailbox class in app/mailboxes and invokes your template
+ engine and test framework generators.
+
+Example:
+========
+ rails generate mailbox inbox
+
+ creates a InboxMailbox class and test:
+ Mailbox: app/mailboxes/inbox_mailbox.rb
+ Test: test/mailboxes/inbox_mailbox_test.rb
diff --git a/lib/rails/generators/mailbox/mailbox_generator.rb b/lib/rails/generators/mailbox/mailbox_generator.rb
new file mode 100644
index 0000000000..7fcdfd3384
--- /dev/null
+++ b/lib/rails/generators/mailbox/mailbox_generator.rb
@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+module Rails
+ module Generators
+ class MailboxGenerator < NamedBase
+ source_root File.expand_path("templates", __dir__)
+
+ argument :actions, type: :array, default: [:process], banner: "method method"
+
+ def check_class_collision
+ class_collisions "#{class_name}Mailbox", "#{class_name}MailboxTest"
+ end
+
+ def create_mailbox_file
+ template "mailbox.rb", File.join("app/mailboxes", class_path, "#{file_name}_mailbox.rb")
+
+ in_root do
+ if behavior == :invoke && !File.exist?(application_mailbox_file_name)
+ template "application_mailbox.rb", application_mailbox_file_name
+ end
+ end
+
+ template "mailer_test.rb", File.join('test/mailboxes', class_path, "#{file_name}_mailbox_test.rb")
+ end
+
+ hook_for :test_framework
+
+ private
+ def file_name # :doc:
+ @_file_name ||= super.sub(/_mailbox\z/i, "")
+ end
+
+ def application_mailbox_file_name
+ "app/mailboxes/application_mailbox.rb"
+ end
+ end
+ end
+end
diff --git a/lib/rails/generators/mailbox/templates/application_mailbox.rb.tt b/lib/rails/generators/mailbox/templates/application_mailbox.rb.tt
new file mode 100644
index 0000000000..be51eb3639
--- /dev/null
+++ b/lib/rails/generators/mailbox/templates/application_mailbox.rb.tt
@@ -0,0 +1,5 @@
+# frozen_string_literal: true
+
+class ApplicationMailbox < ActionMailbox::Base
+ # routing /something/i => :somewhere
+end
diff --git a/lib/rails/generators/mailbox/templates/mailbox.rb.tt b/lib/rails/generators/mailbox/templates/mailbox.rb.tt
new file mode 100644
index 0000000000..9788bd9bb4
--- /dev/null
+++ b/lib/rails/generators/mailbox/templates/mailbox.rb.tt
@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+class <%= class_name %>Mailbox < ApplicationMailbox
+<% actions.each do |action| -%>
+ def <%= action %>
+ end
+<% end -%>
+end
diff --git a/lib/rails/generators/mailbox/templates/mailer_test.rb.tt b/lib/rails/generators/mailbox/templates/mailer_test.rb.tt
new file mode 100644
index 0000000000..41749808e3
--- /dev/null
+++ b/lib/rails/generators/mailbox/templates/mailer_test.rb.tt
@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+require "test_helper"
+
+class <%= class_name %>MailboxTest < ActionMailbox::TestCase
+ # test "receive mail" do
+ # receive_inbound_email_from_mail \
+ # to: '"someone" <someone@example.com>,
+ # from: '"else" <else@example.com>',
+ # subject: "Hello world!",
+ # body: "Hello?"
+ # end
+end
diff --git a/test/generators/mailbox_generator_test.rb b/test/generators/mailbox_generator_test.rb
new file mode 100644
index 0000000000..9295c2d8c8
--- /dev/null
+++ b/test/generators/mailbox_generator_test.rb
@@ -0,0 +1,77 @@
+# frozen_string_literal: true
+require "test_helper"
+require "rails/generators/mailbox/mailbox_generator"
+
+class MailboxGeneratorTest < Rails::Generators::TestCase
+ destination File.expand_path("../../tmp", File.dirname(__FILE__))
+ setup :prepare_destination
+ tests Rails::Generators::MailboxGenerator
+
+ arguments ["inbox"]
+
+ def test_mailbox_skeleton_is_created
+ run_generator
+ assert_file "app/mailboxes/inbox_mailbox.rb" do |mailbox|
+ assert_match(/class InboxMailbox < ApplicationMailbox/, mailbox)
+ assert_match(/def process/, mailbox)
+ assert_no_match(%r{# routing /something/i => :somewhere}, mailbox)
+ end
+
+ assert_file "app/mailboxes/application_mailbox.rb" do |mailbox|
+ assert_match(/class ApplicationMailbox < ActionMailbox::Base/, mailbox)
+ assert_match(%r{# routing /something/i => :somewhere}, mailbox)
+ assert_no_match(/def process/, mailbox)
+ end
+ end
+
+ def test_check_class_collision
+ Object.send :const_set, :InboxMailbox, Class.new
+ content = capture(:stderr) { run_generator }
+ assert_match(/The name 'InboxMailbox' is either already used in your application or reserved/, content)
+ ensure
+ Object.send :remove_const, :InboxMailbox
+ end
+
+ def test_invokes_default_test_framework
+ run_generator %w(inbox foo bar)
+ assert_file "test/mailboxes/inbox_mailbox_test.rb" do |test|
+ assert_match(/class InboxMailboxTest < ActionMailbox::TestCase/, test)
+ assert_match(/# test "receive mail" do/, test)
+ assert_match(/# to: '"someone" <someone@example.com>,/, test)
+ end
+ end
+
+ def test_check_test_class_collision
+ Object.send :const_set, :InboxMailboxTest, Class.new
+ content = capture(:stderr) { run_generator }
+ assert_match(/The name 'InboxMailboxTest' is either already used in your application or reserved/, content)
+ ensure
+ Object.send :remove_const, :InboxMailboxTest
+ end
+
+ def test_actions_are_turned_into_methods
+ run_generator %w(inbox foo bar)
+
+ assert_file "app/mailboxes/inbox_mailbox.rb" do |mailbox|
+ assert_instance_method :foo, mailbox
+ assert_instance_method :bar, mailbox
+ end
+ end
+
+ def test_mailbox_on_revoke
+ run_generator
+ run_generator ["inbox"], behavior: :revoke
+
+ assert_no_file "app/mailboxes/inbox.rb"
+ end
+
+ def test_mailbox_suffix_is_not_duplicated
+ run_generator ["inbox_mailbox"]
+
+ assert_no_file "app/mailboxes/inbox_mailbox_mailbox.rb"
+ assert_file "app/mailboxes/inbox_mailbox.rb"
+
+ assert_no_file "test/mailboxes/inbox_mailbox_mailbox_test.rb"
+ assert_file "test/mailboxes/inbox_mailbox_test.rb"
+ end
+end