mjml-rails 4.6.0 → 4.7.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e001eb3b8f9abfa49b6fbf3e0e10d655fdeb38e68da79f55b348977f4a7082b8
-  data.tar.gz: 7d78f4466486513af5d364c7ca3e5146724a6e9fc785b26cdf66ae5c626138fb
+  metadata.gz: c0a8228a5acfd5b16debb4c6a1bdcf9a7b6ee3efd48b4004ac63b1cb94814f7e
+  data.tar.gz: 1d34c76fff5718954beeccaeb0b5cef3876362ffe11d9b1b7338bcb72fc843cf
 SHA512:
-  metadata.gz: 534c1c2975ff87a7e5bfed79bcbf19ec0d00ca16cf4a871faac3c9c89a348452aed2afb6197629598f473fd84943d67b64581a75139ca7e608ebcf4878004615
-  data.tar.gz: 6b58532184a1d750fc2d4ae76508a52a3b86578b82fa501a6c4a01c7e9f03856fa2ac48522b0744536e573468233e7f6f54e6a8bd234dff6b58f58e02879ebd5
+  metadata.gz: bb7051418c7a08f8c8927abd5d87ad1569ce366d319f683a7bf9d195263e596cca7a159a964d4d4593463308e25e38510b80dddf74df19913f9ff008b9a9a7f4
+  data.tar.gz: 373e935513e3a3079273505731d272f8d031549e7dc2b6a64b8c1cd688340db0f3439cb1719529406ce755a38e079477b8b58e28701b18cf06b8befca3578abb

data/README.md

@@ -2,14 +2,14 @@
 
 [![Build Status](https://github.com/sighmon/mjml-rails/workflows/test/badge.svg?branch=master)](https://github.com/sighmon/mjml-rails/actions?query=workflow%3Atest+branch%3Amaster) [![Gem Version](https://badge.fury.io/rb/mjml-rails.svg)](https://badge.fury.io/rb/mjml-rails)
 
-**MJML-Rails** allows you to render HTML e-mails from an [MJML](https://mjml.io) template.
+**MJML-Rails** allows you to render HTML emails from an [MJML](https://mjml.io) template.
 
 **Note**: As of MJML 4.3.0 you can no longer use `<mj-text>` directly inside an `<mj-body>`, wrap it in `<mj-section><mj-column>`.
 
 An example template might look like:
 
 ```erb
-<!-- ./app/views/user_mailer/email.html.mjml -->
+<!-- ./app/views/user_mailer/user_signup_confirmation.mjml -->
 <mjml>
   <mj-head>
     <mj-preview>Hello World</mj-preview>
@@ -18,7 +18,7 @@ An example template might look like:
     <mj-section>
       <mj-column>
         <mj-text>Hello World</mj-text>
-        <%= render partial: "info" %>
+        <%= render partial: "info", formats: [:html] %>
       </mj-column>
     </mj-section>
   </mj-body>
@@ -80,52 +80,61 @@ MJML-Rails falls back to a global installation of MJML but it is strongly recomm
 
 You'll need at least Node.js version 6 for MJML to function properly.
 
-If you're using ```:haml``` or any other Rails template language, create an initializer to set it up:
+## Configuration
 
-```ruby
-# config/initializers/mjml.rb
-Mjml.setup do |config|
-  config.template_language = :erb # :erb (default), :slim, :haml, or any other you are using
-end
-```
+MJML-Rails has the following settings with defaults:
+
+- `template_language: :erb`
+
+   ERB can be used inside MJML templates by default. Possible values are all template languages that you have installed, e.g. `:haml` or `:slim`.
+
+   **Note:** If you’re using Haml/Slim layouts, please don’t put `<mjml>` in comments in your partial. Read more at [#34](https://github.com/sighmon/mjml-rails/issues/34).
+
+- `raise_render_exception: true`
+
+   Exceptions will be raised and passed to your application by default.
+
+   Beware that setting it to `false` leads to an empty html email when an exception is raised, so only set this to `false` if you do not rely on html (e.g. you have a text fallback for your emails).
+
+- `minify: false`
+
+- `beautify: true`
+
+- `validation_level: "strict"`
+
+   MJML-Rails will raise an exception on any template error by default.
+
+   If set to `soft`, MJML will render invalid templates and ignore invalid parts. This means in case of an invalid template those invalid parts will be missing from the output.
+
+   See [MJML validation documentation](https://github.com/mjmlio/mjml/tree/master/packages/mjml-validator#validating-mjml) for all possible values.
 
-**Note:** If you’re using Haml/Slim layouts, please don’t put `<mjml>` in comments in your partial. Read more: [#34](https://github.com/sighmon/mjml-rails/issues/34).
+- `mjml_binary: nil`
+
+   This can be used to specify the path to a custom MJML binary if it is not detected automatically (shouldn't be needed).
+
+- `mjml_binary_version_support: "4."`
+
+   MJML-Rails checks the version of the MJML binary and fails if it does not start with this version, e.g. if an old version is installed by accident.
 
-If there are configurations you'd like change:
-- render errors: defaults to `true` (errors raised)
-- minify: defaults to `false` (not minified)
-- beautify: defaults to `true` (beautified)
-- validation_level: defaults to `strict` (abort on any template error, see [MJML validation documentation](https://github.com/mjmlio/mjml/tree/master/packages/mjml-validator#validating-mjml) for possible values)
 
 ```ruby
 # config/initializers/mjml.rb
 Mjml.setup do |config|
-  # ignore errors silently
+  # Use :haml as a template language
+  config.template_language = :haml
+
+  # Ignore errors silently
   config.raise_render_exception = false
 
-  # optimize the size of your email
+  # Optimize the size of your emails
   config.beautify = false
   config.minify = true
 
-  # render illformed MJML templates, not recommended
+  # Render MJML templates with errors
   config.validation_level = "soft"
-end
-```
-
-You can optionally provide a custom MJML binary if MJML-Rails cannot find it:
 
-```ruby
-# config/initializers/mjml.rb
-Mjml.setup do |config|
+  # Use custom MJML binary with custom version
   config.mjml_binary = "/path/to/custom/mjml"
-end
-```
-
-If you’d like to specify a different MJML version, set it like this:
-
-```ruby
-# config/initializers/mjml.rb
-Mjml.setup do |config|
   config.mjml_binary_version_supported = "3.3.5"
 end
 ```

data/lib/generators/mjml/mailer/mailer_generator.rb

@@ -1,9 +1,11 @@
-require "rails/generators/erb/mailer/mailer_generator"
+# frozen_string_literal: true
+
+require 'rails/generators/erb/mailer/mailer_generator'
 
 module Mjml
   module Generators
     class MailerGenerator < Erb::Generators::MailerGenerator
-      source_root File.expand_path("../templates", __FILE__)
+      source_root File.expand_path('templates', __dir__)
 
       private
 
@@ -26,8 +28,8 @@ module Mjml
       def filename_with_extensions(name, file_format = format)
         # Due to MJML single-pass processing nature
         # layout files MUST have .mjml extension, but views/templates cannot
-        is_layout_file = name.in?([:layout, "mailer"])
-        [name, file_format, is_layout_file ? handler : view_handler].compact.join(".")
+        is_layout_file = name.in?([:layout, 'mailer'])
+        [name, file_format, is_layout_file ? handler : view_handler].compact.join('.')
       end
     end
   end

data/lib/mjml/handler.rb

@@ -0,0 +1,43 @@
+# frozen_string_literal: true
+
+require 'action_view'
+require 'action_view/template'
+require 'rails/version'
+
+module Mjml
+  class Handler
+    def template_handler
+      @template_handler ||= ActionView::Template.registered_template_handler(Mjml.template_language)
+    end
+
+    # Optional second source parameter to make it work with Rails >= 6:
+    # Beginning with Rails 6 template handlers get the source of the template as the second
+    # parameter.
+    def call(template, source = nil)
+      compiled_source = compile_source(source, template)
+
+      # Per MJML v4 syntax documentation[0] valid/render'able document MUST start with <mjml> root tag
+      # If we get here and template source doesn't start with one it means
+      # that we are rendering partial named according to legacy naming convention (partials ending with '.mjml')
+      # Therefore we skip MJML processing and return raw compiled source. It will be processed
+      # by MJML library when top-level layout/template is rendered
+      #
+      # [0] - https://github.com/mjmlio/mjml/blob/master/doc/components_1.md#mjml
+      if /<mjml.*?>/i.match?(compiled_source)
+        "Mjml::Parser.new(begin;#{compiled_source};end).render.html_safe"
+      else
+        compiled_source
+      end
+    end
+
+    private
+
+    def compile_source(source, template)
+      if Rails::VERSION::MAJOR >= 6
+        template_handler.call(template, source)
+      else
+        template_handler.call(template)
+      end
+    end
+  end
+end

data/lib/mjml/parser.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Mjml
   class Parser
     class ParseError < StandardError; end
@@ -7,8 +9,9 @@ module Mjml
     # Create new parser
     #
     # @param input [String] The string to transform in html
-    def initialize input
+    def initialize(input)
       raise Mjml.mjml_binary_error_string unless Mjml.valid_mjml_binary
+
       @input = input
     end
 
@@ -16,29 +19,39 @@ module Mjml
     #
     # @return [String]
     def render
-      in_tmp_file = Tempfile.open(["in", ".mjml"]) do |file|
+      in_tmp_file = Tempfile.open(['in', '.mjml']) do |file|
         file.write(input)
         file # return tempfile from block so #unlink works later
       end
       run(in_tmp_file.path, Mjml.beautify, Mjml.minify, Mjml.validation_level)
-    rescue
+    rescue StandardError
       raise if Mjml.raise_render_exception
-      ""
+
+      ''
     ensure
-      in_tmp_file.unlink
+      in_tmp_file&.unlink
     end
 
     # Exec mjml command
     #
     # @return [String] The result as string
-    def run(in_tmp_file, beautify=true, minify=false, validation_level="strict")
-      Tempfile.create(["out", ".html"]) do |out_tmp_file|
-        command = "-r #{in_tmp_file} -o #{out_tmp_file.path} --config.beautify #{beautify} --config.minify #{minify} --config.validationLevel #{validation_level}"
+    # rubocop:disable Style/OptionalBooleanParameter: Fixing this offense would imply a change in the public API.
+    def run(in_tmp_file, beautify = true, minify = false, validation_level = 'strict')
+      Tempfile.create(['out', '.html']) do |out_tmp_file|
+        command = "-r #{in_tmp_file} -o #{out_tmp_file.path} " \
+                  "--config.beautify #{beautify} --config.minify #{minify} --config.validationLevel #{validation_level}"
         _, stderr, status = Mjml.run_mjml(command)
-        raise ParseError.new(stderr.chomp) unless status.success?
-        Mjml.logger.warn(stderr.chomp) unless stderr.blank?
+
+        unless status.success?
+          # The process status ist quite helpful in case of dying processes without STDERR output.
+          # Node exit codes are documented here: https://node.readthedocs.io/en/latest/api/process/#exit-codes
+          raise ParseError, "#{stderr.chomp}\n(process status: #{status})"
+        end
+
+        Mjml.logger.warn(stderr.chomp) if stderr.present?
         out_tmp_file.read
       end
     end
+    # rubocop:enable Style/OptionalBooleanParameter
   end
 end

data/lib/mjml/railtie.rb

@@ -1,11 +1,13 @@
+# frozen_string_literal: true
+
 module Mjml
   class Railtie < Rails::Railtie
     config.mjml = Mjml
-    config.app_generators.mailer :template_engine => :mjml
+    config.app_generators.mailer template_engine: :mjml
 
-    initializer "mjml-rails.register_template_handler" do
+    initializer 'mjml-rails.register_template_handler' do
       ActionView::Template.register_template_handler :mjml, Mjml::Handler.new
-      Mime::Type.register_alias "text/html", :mjml
+      Mime::Type.register_alias 'text/html', :mjml
     end
 
     config.to_prepare do

data/lib/mjml/version.rb

@@ -1,4 +1,6 @@
+# frozen_string_literal: true
+
 module Mjml
-	# Version number no longer matches MJML.io version
-  VERSION = "4.6.0"
+  # Version number no longer matches MJML.io version
+  VERSION = '4.7.2'
 end

data/lib/mjml-rails.rb

@@ -1 +1,5 @@
-require "mjml"
+# rubocop:disable Naming/FileName
+# frozen_string_literal: true
+
+require 'mjml'
+# rubocop:enable Naming/FileName

data/lib/mjml.rb

@@ -1,10 +1,12 @@
-require "rails/version"
-require "action_view"
-require "action_view/template"
-require "mjml/mjmltemplate"
-require "mjml/railtie"
-require "rubygems"
-require "open3"
+# frozen_string_literal: true
+
+require 'rubygems'
+require 'open3'
+
+require 'mjml/handler'
+require 'mjml/parser'
+
+require 'mjml/railtie' if defined?(Rails)
 
 module Mjml
   mattr_accessor \
@@ -19,13 +21,14 @@ module Mjml
 
   mattr_writer :valid_mjml_binary
 
-  @@template_language = :erb
-  @@raise_render_exception = true
-  @@mjml_binary_version_supported = "4."
-  @@mjml_binary_error_string = "Couldn't find the MJML #{Mjml.mjml_binary_version_supported} binary.. have you run $ npm install mjml?"
-  @@beautify = true
-  @@minify = false
-  @@validation_level = "strict"
+  self.template_language = :erb
+  self.raise_render_exception = true
+  self.mjml_binary_version_supported = '4.'
+  self.mjml_binary_error_string = "Couldn't find the MJML #{Mjml.mjml_binary_version_supported} binary.." \
+                                  ' have you run $ npm install mjml?'
+  self.beautify = true
+  self.minify = false
+  self.validation_level = 'strict'
 
   def self.check_version(bin)
     stdout, _, status = run_mjml('--version', mjml_bin: bin)
@@ -39,11 +42,11 @@ module Mjml
   end
 
   def self.valid_mjml_binary
-    @@valid_mjml_binary ||=
-      check_for_custom_mjml_binary ||
-      check_for_yarn_mjml_binary ||
-      check_for_npm_mjml_binary ||
-      check_for_global_mjml_binary
+    self.valid_mjml_binary = @@valid_mjml_binary ||
+                             check_for_custom_mjml_binary ||
+                             check_for_yarn_mjml_binary ||
+                             check_for_npm_mjml_binary ||
+                             check_for_global_mjml_binary
 
     return @@valid_mjml_binary if @@valid_mjml_binary
 
@@ -52,21 +55,23 @@ module Mjml
 
   def self.check_for_custom_mjml_binary
     if const_defined?('BIN') && Mjml::BIN.present?
-      logger.warn('Setting `Mjml::BIN` is deprecated and will be removed in a future version! Please use `Mjml.mjml_binary=` instead.')
+      logger.warn('Setting `Mjml::BIN` is deprecated and will be removed in a future version! ' \
+                  'Please use `Mjml.mjml_binary=` instead.')
       self.mjml_binary = Mjml::BIN
       remove_const 'BIN'
     end
 
-    return unless mjml_binary.present?
+    return if mjml_binary.blank?
 
     return mjml_binary if check_version(mjml_binary)
 
-    raise "MJML.mjml_binary is set to '#{mjml_binary}' but MJML-Rails could not validate that it is a valid MJML binary. Please check your configuration."
+    raise "MJML.mjml_binary is set to '#{mjml_binary}' but MJML-Rails could not validate that " \
+          'it is a valid MJML binary. Please check your configuration.'
   end
 
   def self.check_for_yarn_mjml_binary
     yarn_bin = `which yarn`.chomp
-    return unless yarn_bin.present?
+    return if yarn_bin.blank?
 
     mjml_bin = "#{yarn_bin} run mjml"
     return mjml_bin if check_version(mjml_bin)
@@ -96,38 +101,8 @@ module Mjml
   end
 
   def self.discover_mjml_bin
-    logger.warn('`Mjml.discover_mjml_bin` is deprecated and has no effect anymore! Please use `Mjml.mjml_binary=` to set a custom MJML binary.')
-  end
-
-  class Handler
-    def template_handler
-      @_template_handler ||= ActionView::Template.registered_template_handler(Mjml.template_language)
-    end
-
-    # Optional second source parameter to make it work with Rails >= 6:
-    # Beginning with Rails 6 template handlers get the source of the template as the second
-    # parameter.
-    def call(template, source = nil)
-      compiled_source =
-        if Rails::VERSION::MAJOR >= 6
-          template_handler.call(template, source)
-        else
-          template_handler.call(template)
-        end
-
-      # Per MJML v4 syntax documentation[0] valid/render'able document MUST start with <mjml> root tag
-      # If we get here and template source doesn't start with one it means
-      # that we are rendering partial named according to legacy naming convention (partials ending with '.mjml')
-      # Therefore we skip MJML processing and return raw compiled source. It will be processed
-      # by MJML library when top-level layout/template is rendered
-      #
-      # [0] - https://github.com/mjmlio/mjml/blob/master/doc/guide.md#mjml
-      if compiled_source =~ /<mjml(.+)?>/i
-        "Mjml::Mjmltemplate.to_html(begin;#{compiled_source};end).html_safe"
-      else
-        compiled_source
-      end
-    end
+    logger.warn('`Mjml.discover_mjml_bin` is deprecated and has no effect anymore! ' \
+                'Please use `Mjml.mjml_binary=` to set a custom MJML binary.')
   end
 
   def self.setup
@@ -139,7 +114,7 @@ module Mjml
 
     def logger
       @logger ||= Logger.new($stdout).tap do |log|
-        log.progname = self.name
+        log.progname = name
       end
     end
   end

data/test/generator_test.rb

@@ -1,26 +1,28 @@
-require "test_helper"
-require "generators/mjml/mailer/mailer_generator"
+# frozen_string_literal: true
+
+require 'test_helper'
+require 'generators/mjml/mailer/mailer_generator'
 
 class GeneratorTest < Rails::Generators::TestCase
   tests Mjml::Generators::MailerGenerator
-  destination File.expand_path("../tmp", __FILE__)
+  destination File.expand_path('tmp', __dir__)
   setup :prepare_destination
 
-  test "assert all views are properly created with given name" do
-    run_generator %w(notifier foo bar baz)
+  test 'assert all views are properly created with given name' do
+    run_generator %w[notifier foo bar baz]
 
-    assert_file "app/views/layouts/mailer.html.mjml" do |mailer|
-      assert_match "<mjml>", mailer
-      assert_match "<mj-body>", mailer
-      assert_match "<%= yield %>", mailer
+    assert_file 'app/views/layouts/mailer.html.mjml' do |mailer|
+      assert_match '<mjml>', mailer
+      assert_match '<mj-body>', mailer
+      assert_match '<%= yield %>', mailer
     end
 
-    assert_file "app/views/notifier_mailer/foo.html.erb" do |template|
-      assert_match "<%= @greeting %>", template
-      assert_match "app/views/notifier_mailer/foo.html.erb", template
+    assert_file 'app/views/notifier_mailer/foo.html.erb' do |template|
+      assert_match '<%= @greeting %>', template
+      assert_match 'app/views/notifier_mailer/foo.html.erb', template
     end
 
-    assert_file "app/views/notifier_mailer/bar.html.erb"
-    assert_file "app/views/notifier_mailer/baz.html.erb"
+    assert_file 'app/views/notifier_mailer/bar.html.erb'
+    assert_file 'app/views/notifier_mailer/baz.html.erb'
   end
 end

data/test/mjml_test.rb

@@ -1,14 +1,16 @@
-require "test_helper"
+# frozen_string_literal: true
+
+require 'test_helper'
 
 class NotifierMailer < ActionMailer::Base
-  self.view_paths = File.expand_path("../views", __FILE__)
+  self.view_paths = File.expand_path('views', __dir__)
 
-  layout "default"
+  layout 'default'
 
   def inform_contact(recipient)
     @recipient = recipient
 
-    mail(to: @recipient, from: "app@example.com") do |format|
+    mail(to: @recipient, from: 'app@example.com') do |format|
       format.text
       format.html
     end
@@ -17,7 +19,7 @@ class NotifierMailer < ActionMailer::Base
   def invalid_template(recipient)
     @recipient = recipient
 
-    mail(to: @recipient, from: "app@example.com") do |format|
+    mail(to: @recipient, from: 'app@example.com') do |format|
       format.html
       format.text
     end
@@ -25,91 +27,90 @@ class NotifierMailer < ActionMailer::Base
 end
 
 class NoLayoutMailer < ActionMailer::Base
-  self.view_paths = File.expand_path("../views", __FILE__)
+  self.view_paths = File.expand_path('views', __dir__)
 
   layout nil
 
   def inform_contact(recipient)
     @recipient = recipient
 
-    mail(to: @recipient, from: "app@example.com") do |format|
-      format.mjml
-    end
+    mail(to: @recipient, from: 'app@example.com', &:mjml)
   end
 
   def with_owa(recipient)
     @recipient = recipient
 
-    mail(to: @recipient, from: "app@example.com") do |format|
-      format.mjml
-    end
+    mail(to: @recipient, from: 'app@example.com', &:mjml)
   end
 end
 
 class NotifierMailerTest < ActiveSupport::TestCase
-  test "MJML layout based multipart email is generated correctly" do
-    email = NotifierMailer.inform_contact("user@example.com")
+  test 'MJML layout based multipart email is generated correctly' do
+    email = NotifierMailer.inform_contact('user@example.com')
 
-    assert_equal "multipart/alternative", email.mime_type
+    assert_equal 'multipart/alternative', email.mime_type
 
     # To debug tests:
     # Mjml.logger.info email.mime_type
     # Mjml.logger.info email.to_s
     # Mjml.logger.info email.html_part.body
 
-    refute email.html_part.body.match(%r{</?mj.+?>})
-    assert email.html_part.body.match(/<body>/)
+    assert_not email.html_part.body.match(%r{</?mj.+?>})
+    assert email.html_part.body.include?('<body')
     assert email.html_part.body.match(/Hello, user@example.com!/)
-    assert email.html_part.body.match(%r{<h2>We inform you about something</h2>})
+    assert email.html_part.body.include?('<h2>We inform you about something</h2>')
     assert email.html_part.body.match(%r{<a href="https://www.example.com">this link</a>})
-    assert email.html_part.body.match(/tracking-code-123/)
+    assert email.html_part.body.include?('tracking-code-123')
 
-    assert email.text_part.body.match(/We inform you about something/)
+    assert email.text_part.body.include?('We inform you about something')
     assert email.text_part.body.match(%r{Please visit https://www.example.com})
   end
 
-  test "Invalid template raises error with validation level strict" do
+  test 'Invalid template raises error with validation level strict' do
     with_settings(validation_level: 'strict') do
-      email = NotifierMailer.invalid_template("user@example.com")
+      email = NotifierMailer.invalid_template('user@example.com')
       assert_raise(ActionView::Template::Error) { email.html_part.body.to_s }
     end
   end
 
-  test "Invalid template gets compiled with validation level soft" do
+  test 'Invalid template gets compiled with validation level soft' do
+    # suppress warning of MJML binary
+    Mjml.logger.stubs(:warn)
+
     with_settings(validation_level: 'soft') do
-      email = NotifierMailer.invalid_template("user@example.com")
-      assert email.text_part.body.match(/This is valid/)
-      assert email.html_part.body.match(/This is valid/)
-      refute email.html_part.body.match(/This is invalid/)
+      email = NotifierMailer.invalid_template('user@example.com')
+      assert email.text_part.body.include?('This is valid')
+      assert email.html_part.body.include?('This is valid')
+      assert_not email.html_part.body.include?('This is invalid')
     end
   end
 end
 
 class NotifierMailerTest < ActiveSupport::TestCase
-  test "old mjml-rails configuration style MJML template is rendered correctly" do
-    email = NoLayoutMailer.inform_contact("user@example.com")
+  test 'old mjml-rails configuration style MJML template is rendered correctly' do
+    email = NoLayoutMailer.inform_contact('user@example.com')
 
-    assert_equal "text/html", email.mime_type
+    assert_equal 'text/html', email.mime_type
 
-    refute email.body.match(%r{</?mj.+?>})
-    assert email.body.match(/<body>/)
+    assert_not email.body.match(%r{</?mj.+?>})
+    assert email.body.include?('<body')
     assert email.body.match(/Welcome, user@example.com!/)
-    assert email.body.match(%r{<h2>We inform you about something</h2>})
+    assert email.body.include?('<h2>We inform you about something</h2>')
     assert email.body.match(%r{<a href="https://www.example.com">this link</a>})
-    refute email.body.match(/tracking-code-123/)
+    assert_not email.body.include?('tracking-code-123')
   end
 
-  test "old mjml-rails MJML template with owa is rendered correctly" do
-    email = NoLayoutMailer.with_owa("user@example.com")
+  test 'old mjml-rails MJML template with owa is rendered correctly' do
+    email = NoLayoutMailer.with_owa('user@example.com')
 
-    assert_equal "text/html", email.mime_type
+    assert_equal 'text/html', email.mime_type
 
-    refute email.body.match(%r{</?mj.+?>})
-    assert email.body.match(/<body>/)
+    assert_not email.body.match(%r{</?mj.+?>})
+    assert email.body.include?('<body')
     assert email.body.match(/Welcome, user@example.com!/)
-    assert email.body.match(%r{<h2>We inform you about something</h2>})
+    assert email.body.include?('<h2>We inform you about something</h2>')
     assert email.body.match(%r{<a href="https://www.example.com">this link</a>})
-    refute email.body.match(/tracking-code-123/)
+    assert_not email.body.include?('tracking-code-123')
   end
 end
 
@@ -135,13 +136,16 @@ describe Mjml do
     it 'raises an error if mjml_binary is invalid' do
       Mjml.mjml_binary = 'some custom value'
       err = expect { Mjml.valid_mjml_binary }.must_raise(StandardError)
-      expect(err.message).must_match(/MJML\.mjml_binary is set to 'some custom value' but MJML-Rails could not validate that it is a valid MJML binary/)
+      assert(err.message.start_with?("MJML.mjml_binary is set to 'some custom value' " \
+                                     'but MJML-Rails could not validate that it is a valid MJML binary'))
     end
 
     it 'honors old Mjml::BIN way of setting custom binary' do
-      Mjml::BIN = 'set by old way'
+      silence_warnings { Mjml::BIN = 'set by old way' }
+      Mjml.logger.expects(:warn).with(regexp_matches(/Setting `Mjml::BIN` is deprecated/))
       err = expect { Mjml.valid_mjml_binary }.must_raise(StandardError)
-      expect(err.message).must_match(/MJML\.mjml_binary is set to 'set by old way' but MJML-Rails could not validate that it is a valid MJML binary/)
+      assert(err.message.start_with?("MJML.mjml_binary is set to 'set by old way' " \
+                                     'but MJML-Rails could not validate that it is a valid MJML binary'))
     end
 
     it 'ignores empty Mjml::BIN' do
@@ -149,7 +153,8 @@ describe Mjml do
       Mjml.mjml_binary = 'set by mjml_binary'
 
       err = expect { Mjml.valid_mjml_binary }.must_raise(StandardError)
-      expect(err.message).must_match(/MJML\.mjml_binary is set to 'set by mjml_binary' but MJML-Rails could not validate that it is a valid MJML binary/)
+      assert(err.message.start_with?("MJML.mjml_binary is set to 'set by mjml_binary' " \
+                                     'but MJML-Rails could not validate that it is a valid MJML binary'))
     end
   end
 end

data/test/parser_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 describe Mjml::Parser do
@@ -5,7 +7,7 @@ describe Mjml::Parser do
   let(:parser) { Mjml::Parser.new(input) }
 
   describe '#render' do
-    describe 'when execption is raised' do
+    describe 'when exception is raised' do
       let(:custom_error_class) { Class.new(StandardError) }
       let(:error) { custom_error_class.new('custom error') }
 
@@ -25,6 +27,14 @@ describe Mjml::Parser do
           expect(parser.render).must_equal ''
         end
       end
+
+      it 'raises unshadowed exception in case on an `Tempfile` error' do
+        my_tempfile_error_class = Class.new StandardError
+        with_settings(raise_render_exception: true) do
+          Tempfile.stubs(:open).raises(my_tempfile_error_class)
+          expect { parser.render }.must_raise(my_tempfile_error_class)
+        end
+      end
     end
 
     describe 'can read beautify, minify, and validation_level configs' do
@@ -56,9 +66,16 @@ describe Mjml::Parser do
 
   describe '#run' do
     describe 'when shell command failed' do
+      let(:error_msg) do
+        expect { parser.run '/tmp/non_existent_file.mjml' }.must_raise(Mjml::Parser::ParseError).message
+      end
+
       it 'raises exception' do
-        err = expect { parser.run "/tmp/non_existent_file.mjml" }.must_raise(Mjml::Parser::ParseError)
-        expect(err.message).must_include 'Command line error'
+        expect(error_msg).must_include 'Command line error'
+      end
+
+      it 'includes process status' do
+        expect(error_msg).must_match(/\(process status: .+\)/)
       end
     end
   end

data/test/test_helper.rb

@@ -1,21 +1,24 @@
-require "rubygems"
-require "bundler"
+# frozen_string_literal: true
+
+require 'rubygems'
+require 'bundler'
 Bundler.setup
 
-require "minitest/autorun"
-require "active_support/test_case"
+require 'minitest/autorun'
+require 'active_support/test_case'
 
-require "action_mailer"
-require "rails/railtie"
-require "rails/generators"
-require "rails/generators/test_case"
+require 'action_mailer'
+require 'rails/railtie'
+require 'rails/generators'
+require 'rails/generators/test_case'
 require 'mocha/minitest'
+require 'byebug'
 
 # require "minitest/reporters"
 # Minitest::Reporters.use!
 
-$:.unshift File.expand_path("../../lib", __FILE__)
-require "mjml"
+$LOAD_PATH.unshift File.expand_path('../lib', __dir__)
+require 'mjml'
 Mjml::Railtie.run_initializers
 
 ActiveSupport::TestCase.test_order = :sorted if ActiveSupport::TestCase.respond_to? :test_order=
@@ -27,18 +30,16 @@ ActionMailer::Base.delivery_method = :test
 ActionMailer::Base.perform_deliveries = true
 
 def with_settings(settings)
-  original_settings =
-    settings.each_with_object({}) do |(key, _), agg|
-      agg[key] = Mjml.public_send(key)
-    end
-
-  settings.each do |key, value|
-    Mjml.public_send("#{key}=", value)
-  end
-
+  original_settings = settings.each_with_object({}) { |(key, _), agg| agg[key] = Mjml.public_send(key) }
+  settings.each { |key, value| Mjml.public_send("#{key}=", value) }
   yield
 ensure
-  original_settings.each do |key, value|
-    Mjml.public_send("#{key}=", value)
-  end
+  original_settings.each { |key, value| Mjml.public_send("#{key}=", value) }
 end
+
+# Suppress all ruby warnings of the mail gem, see:
+# * https://github.com/mikel/mail/issues/1424
+# * https://github.com/mikel/mail/issues/1384
+# * https://github.com/mikel/mail/pull/1162
+require 'warning'
+Warning.ignore(//, %r{.*gems/mail.*/lib/mail/})

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mjml-rails
 version: !ruby/object:Gem::Version
-  version: 4.6.0
+  version: 4.7.2
 platform: ruby
 authors:
 - Simon Loffler
@@ -11,32 +11,130 @@ bindir: bin
 cert_chain:
 - |
   -----BEGIN CERTIFICATE-----
-  MIIEKDCCApCgAwIBAgIBATANBgkqhkiG9w0BAQsFADAfMR0wGwYDVQQDDBR5b3Vy
-  L0RDPWVtYWlsL0RDPWNvbTAeFw0yMDExMjIwNTA5MjJaFw0yMTExMjIwNTA5MjJa
-  MB8xHTAbBgNVBAMMFHlvdXIvREM9ZW1haWwvREM9Y29tMIIBojANBgkqhkiG9w0B
-  AQEFAAOCAY8AMIIBigKCAYEAmYphpc2KJNqmKAAKTE53F59mnVv2aW8r5NFDZqG0
-  DF6V8N5rdWanaMLlo1HyU9r7D1f/h7O/LnjZemvgH6MROd19Qe64GIdIPRXC/ZxM
-  cTBNC7GMoZf9LomBmg8sXnoL0Z2enUP6BV716I00EecxrEb8drtvZlRwjUgRcZrv
-  iEaanYYp1Sw3+koyOsof/xYb3Pn2TAh9UE58lbVDpf88zc74ij3Vs3+LW/k+jZzh
-  pzo7qq47XeVNGWtVDGm3dk8QLrIyQdk28B2ZRJ+HeuLEgx7ASDYFVZc05QlAjehS
-  4k37CIgF0ODDnscoSNNb6toFzaiD/kU9+ManRj871qOAMDlcX6cM4Wl08xfXZsxh
-  VgmIv7vFoSkBDw6t3xrHGD1HiYYQl2tFQsS9D/X9I+1ArChtUTSxEyN6VGP1YVd6
-  jsf4A/eMNRO6+udEQjYHontemi1xFeeyb6PDbP14oLMOls73fKvGZ3eW5uO3qCOb
-  tKhPFou/w1GgU3vR3O26Q3EZAgMBAAGjbzBtMAkGA1UdEwQCMAAwCwYDVR0PBAQD
-  AgSwMB0GA1UdDgQWBBTx6I3ZLm1G7VCtTJWWSKR2qv6x7TAZBgNVHREEEjAQgQ55
-  b3VyQGVtYWlsLmNvbTAZBgNVHRIEEjAQgQ55b3VyQGVtYWlsLmNvbTANBgkqhkiG
-  9w0BAQsFAAOCAYEAKCpNu5+MmYNrb+oOws9m8mghO4KDO53EQsfHtRcpsBxzwzV3
-  MLO4el5fMiAiv2PrULEWVyEoe3k65bEiSEQP7m7w02B02OGugXNzwbetG56gogFy
-  aJ4VJ95aiPEwDmGORLg7RmZcL/07ikDfb96ebPn3v2REYN2lWrqu4fT1MdTmA+hZ
-  vhfqJeSuWM5wNsWxA66SgbQe8BMHcsqV1CTG+Ir8FIAHzfa+c7CNke/28htRKYJV
-  6+lk55Ueov5TjjgLTUvjdqbAEtyCZpgxe2e0o3xqKh5d5YjWVQ4mSNvBsqK7UXOx
-  MGsePVBV9lnSFEJkGB3iOvavQSVUvqybF+yk6DrJR9iZtKCvAqd96PjkcFoM19dG
-  5ofv88TRZwXM9lWn4UKuekSSF1ElH3UBVDbH4zEHaOzrOvgfnZw3VteDigwfmmwF
-  lsAqKbu4nrHhtGhkwdYbflf2URJTUxXGptrnPYV7okmEg4rsykK3RAsZ6kMNdKmx
-  DcQ7RSt1TU5eck6c
+  MIIEPDCCAqSgAwIBAgIBATANBgkqhkiG9w0BAQsFADAkMSIwIAYDVQQDDBlzaWdo
+  bW9uL0RDPXNpZ2htb24vREM9Y29tMB4XDTIxMTIxODA1MDcyM1oXDTIyMTIxODA1
+  MDcyM1owJDEiMCAGA1UEAwwZc2lnaG1vbi9EQz1zaWdobW9uL0RDPWNvbTCCAaIw
+  DQYJKoZIhvcNAQEBBQADggGPADCCAYoCggGBALvTbM9xGlGQ9pByLEAJjAqc6U76
+  64uCOSIkgIFbC4iQdU5vzB/w4Y+mjgkH4oMsuW2NQfFBV+xI3jxQA0dN4qMgHooB
+  AWT/CSk0AAqTY1qCfCSASHLUWb5ZxURvsHmlXX/HCLX9bPcaYB0nQ+VT/u0lCicZ
+  beXMczHsNAj8f3ZJWDR9mw5OYc6cdGHM7Se3sqrmEsMYL9s8d08tChDcllxKV7cE
+  r2zb0oPrRFoYpakOIz1ViCl4KqBpQsDqXbwEw1kDCkKMXGHCsmDKML55kXsVVu4Q
+  9vqGvtEuvseWsDpKb6Psy4nJoktswhotUf21dNVM9k8ufXNaWvZed+cYnBFPLdQV
+  daaEqh6hIC/yUP98D5u+xmTYLRQQEsjhkbdfoLmB8UOwrsEZu79TXN6Iq4MZqmBe
+  qkCTlXOMaO/EbWVRRvs64iibC/boVVlXu6RZCVNyzXEvYxg/zyl0zjS8KPR8Vf4f
+  RKOUVKJCcL/nW4VZ1mvmJtiXI3eARaJdrBEjjQIDAQABo3kwdzAJBgNVHRMEAjAA
+  MAsGA1UdDwQEAwIEsDAdBgNVHQ4EFgQUC9Ln2Rotb61VZFq2rxBPP7sJIiUwHgYD
+  VR0RBBcwFYETc2lnaG1vbkBzaWdobW9uLmNvbTAeBgNVHRIEFzAVgRNzaWdobW9u
+  QHNpZ2htb24uY29tMA0GCSqGSIb3DQEBCwUAA4IBgQBN7i1ChHKDNa5ceCXUXdJ/
+  ZQnVQTx2KMvWwZ6qpkIWFbzwwL5w/xQdhQbLMjuaie5X1ZYOB9753ipw3tvl7iJz
+  vHEdTcOWAzMzHytySVcSa2DhanDopC+hrO5N+GoQ+5X9CN+xKG1Png4ho4FsuUtI
+  fESG8Kb48iNlpLGbfcM8Nqm/mYk/Xwqh3aSVb2E1NVUNr3QW84xB/xmzy2d9qNBm
+  0GecfnBmH+ARmR9Qk9H7YjtnhQv1W2wOBiz1k6GeO8wMLKRRHVI2TGcPR8AqhQGd
+  ejWL8tjMzIw5ggxro3WDklV1hn0iFSeWXNWd9tN40PjLYZqZ3krQ2wWgcgw/9eC9
+  yVr+12iQ7XbbPqlveO/FhP4gLa8e2q1TgWjYAXIaM61em83Dlm/3p75ch0YCB3X5
+  R9NsaxhBfawJEJcYoPZflGkLjJU8pjSuvIW5+rNgAiqY8D37hTtZMu/n2Fz8Qp24
+  7EZetZmcvBzARf8vQSJjga3y0Bftk8u7LmblOEWddzE=
   -----END CERTIFICATE-----
-date: 2020-11-29 00:00:00.000000000 Z
-dependencies: []
+date: 2022-01-21 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: byebug
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: mocha
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.4.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.4.0
+- !ruby/object:Gem::Dependency
+  name: rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.23.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.23.0
+- !ruby/object:Gem::Dependency
+  name: rubocop-performance
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.12.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.12.0
+- !ruby/object:Gem::Dependency
+  name: rubocop-rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.12.4
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.12.4
+- !ruby/object:Gem::Dependency
+  name: warning
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.2.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.2.1
 description: Render MJML + ERb template views in Rails
 email: sighmon@sighmon.com
 executables: []
@@ -50,14 +148,13 @@ files:
 - lib/generators/mjml/mailer/templates/view.html.erb
 - lib/mjml-rails.rb
 - lib/mjml.rb
-- lib/mjml/mjmltemplate.rb
+- lib/mjml/handler.rb
 - lib/mjml/parser.rb
 - lib/mjml/railtie.rb
 - lib/mjml/version.rb
 - test/generator_test.rb
 - test/mjml_test.rb
 - test/parser_test.rb
-- test/template_test.rb
 - test/test_helper.rb
 homepage: https://github.com/sighmon/mjml-rails
 licenses:
@@ -71,14 +168,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.5'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.4
+rubygems_version: 3.3.5
 signing_key:
 specification_version: 4
 summary: MJML + ERb templates
@@ -86,5 +183,4 @@ test_files:
 - test/generator_test.rb
 - test/parser_test.rb
 - test/mjml_test.rb
-- test/template_test.rb
 - test/test_helper.rb

data/lib/mjml/mjmltemplate.rb

@@ -1,12 +0,0 @@
-require "mjml/parser"
-
-module Mjml
-  mattr_accessor :processing_options, :renderer
-  @@processing_options = {}
-
-  class Mjmltemplate
-    def self.to_html(compiled_source)
-      Mjml::Parser.new(compiled_source).render.html_safe
-    end
-  end
-end

data/test/template_test.rb

@@ -1,9 +0,0 @@
-require "test_helper"
-
-class MjmlTest < ActiveSupport::TestCase
-
-  test 'with Mjmltemplate processor' do
-    assert_not_equal "<mj-body></mj-body>", Mjml::Mjmltemplate.to_html("<mjml><mj-body><mj-section><mj-column></mj-column></mj-section></mj-body></mjml>").strip
-  end
-
-end