mjml-rails 4.15.0 → 4.16.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7a768bdc7c72cb474b41346d9f48685d46ac7eb477850f417eac944abcfe3289
-  data.tar.gz: ceca1bd32af0d391c92a5ff0a00a80751e34b069571fe57567fc3b1cfee656ff
+  metadata.gz: 2765f5756d63843512d96e8c0dca3d5ae41c132e2e527ffd1d161e963ff39e8c
+  data.tar.gz: d7c19dae4feafb3593e4f478463897f1310a0950414811fef87de94323ec8e4d
 SHA512:
-  metadata.gz: a2a6098dadd2d9004b7043903b61056da28d2d061bebb391c7ed9ab537f24879dac30b46d08388c41871a74c7a7e4e703d1b206e08f0980aeaeb006e3674ed7c
-  data.tar.gz: f626fc4c6141133b42cba395e137f557ed3259198a40ab7b445a8ec5b3521cbb74a5a499df5d3e0a176f5f3a8c10e5f933e37fa000d45957784c1d9869b4a778
+  metadata.gz: 5d8f725b93ec25cd3102309b453771157971bdf976d4f3f641ef51a78c90f678d518db51119110c1a5df53882e30814a16fcaf8ecef7eae07f59a05700df91ce
+  data.tar.gz: 36afdf274545cc6a8222ea86a03d9c020a7b248f8761454e73d4f68d515dfaeb665474a270e80f6ef4822e90d3a8b02275065d96f35fa75b5870cb0e039193c5

data/lib/mjml/handler.rb

@@ -17,7 +17,7 @@ module Mjml
       compiled_source = compile_source(source, template)
 
       parser_class = Mjml.use_mrml ? 'MrmlParser' : 'Parser'
-      template_path = template.virtual_path
+      template_path = template.respond_to?(:virtual_path) ? template.virtual_path : template.identifier
       # 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')

data/lib/mjml/mrml_parser.rb

@@ -2,27 +2,32 @@
 
 module Mjml
   class MrmlParser
-    attr_reader :input
+    attr_reader :template_path, :input
 
     # Create new parser
     #
+    # @param template_path [String] The path to the .mjml file
     # @param input [String] The string to transform in html
-    def initialize(input)
-      @input = input
+    def initialize(template_path, input)
+      @template_path = template_path
+      @input         = input
+      @with_cache    = Cache.new(template_path)
     end
 
     # Render mjml template
     #
     # @return [String]
     def render
-      MRML.to_html(input)
-    rescue NameError
-      Mjml.logger.fatal('MRML is not installed. Please add `gem "mrml"` to your Gemfile.')
-      raise
-    rescue StandardError
-      raise if Mjml.raise_render_exception
+      @with_cache.cache do
+        MRML.to_html(input)
+      rescue NameError
+        Mjml.logger.fatal('MRML is not installed. Please add `gem "mrml"` to your Gemfile.')
+        raise
+      rescue StandardError
+        raise if Mjml.raise_render_exception
 
-      ''
+        ''
+      end
     end
   end
 end

data/lib/mjml/version.rb

@@ -2,5 +2,5 @@
 
 module Mjml
   # Version number no longer matches MJML.io version
-  VERSION = '4.15.0'
+  VERSION = '4.16.0'
 end

data/test/handler_test.rb

@@ -0,0 +1,83 @@
+# frozen_string_literal: true
+
+require 'test_helper'
+
+class HandlerTest < ActiveSupport::TestCase
+  def setup
+    @handler = Mjml::Handler.new
+  end
+
+  test 'uses virtual_path when available' do
+    template = mock
+    template.stubs(:virtual_path).returns('/path/to/template')
+    template.stubs(:respond_to?).with(:virtual_path).returns(true)
+
+    compiled_source = '<mjml><mj-body><mj-text>Hello</mj-text></mj-body></mjml>'
+    @handler.stubs(:compile_source).returns(compiled_source)
+
+    result = @handler.call(template)
+
+    assert_includes result, '/path/to/template'
+  end
+
+  test 'falls back to identifier when virtual_path is not available' do
+    # Simulate ViewComponent::Template::DataWithSource behavior
+    template = mock
+    template.stubs(:identifier).returns('/path/to/component/template')
+    template.stubs(:respond_to?).with(:virtual_path).returns(false)
+
+    compiled_source = '<mjml><mj-body><mj-text>Hello</mj-text></mj-body></mjml>'
+    @handler.stubs(:compile_source).returns(compiled_source)
+
+    result = @handler.call(template)
+
+    assert_includes result, '/path/to/component/template'
+  end
+
+  test 'handles partials without mjml root tag' do
+    template = mock
+    template.stubs(:virtual_path).returns('/path/to/partial')
+    template.stubs(:respond_to?).with(:virtual_path).returns(true)
+
+    # Partial without <mjml> root tag
+    compiled_source = '<mj-text>This is a partial</mj-text>'
+    @handler.stubs(:compile_source).returns(compiled_source)
+
+    result = @handler.call(template)
+
+    # Should return the compiled source as-is for partials
+    assert_equal compiled_source, result
+  end
+
+  test 'processes full MJML documents' do
+    template = mock
+    template.stubs(:virtual_path).returns('/path/to/template')
+    template.stubs(:respond_to?).with(:virtual_path).returns(true)
+
+    compiled_source = '<mjml><mj-body><mj-text>Full document</mj-text></mj-body></mjml>'
+    @handler.stubs(:compile_source).returns(compiled_source)
+
+    result = @handler.call(template)
+
+    # Should return parser invocation for full MJML documents
+    assert_includes result, "Mjml::Parser.new('/path/to/template'"
+    assert_includes result, '.render.html_safe'
+  end
+
+  test 'uses MrmlParser when use_mrml is true' do
+    Mjml.stubs(:use_mrml).returns(true)
+
+    template = mock
+    template.stubs(:virtual_path).returns('/path/to/template')
+    template.stubs(:respond_to?).with(:virtual_path).returns(true)
+
+    compiled_source = '<mjml><mj-body><mj-text>Full document</mj-text></mj-body></mjml>'
+    @handler.stubs(:compile_source).returns(compiled_source)
+
+    result = @handler.call(template)
+
+    assert_includes result, "Mjml::MrmlParser.new('/path/to/template'"
+  ensure
+    Mjml.unstub(:use_mrml)
+  end
+end

data/test/mrml_parser_test.rb

@@ -3,7 +3,7 @@
 require 'test_helper'
 
 describe Mjml::MrmlParser do
-  let(:parser) { Mjml::MrmlParser.new(input) }
+  let(:parser) { Mjml::MrmlParser.new('test_template', input) }
 
   describe '#render' do
     describe 'when input is valid' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mjml-rails
 version: !ruby/object:Gem::Version
-  version: 4.15.0
+  version: 4.16.0
 platform: ruby
 authors:
 - Simon Loffler
@@ -35,7 +35,7 @@ cert_chain:
   Qqk2dn/fFE9vdcl9OaOw2Zizne/1VFL/jkzXLqnLgbKD/q129mVCBgw2CKYnQfMN
   4RwBjPyqnMxWnSq6Ycn7HdFEkgyf2cAxFfH5QtDsjEuca+/LAJMeAQ==
   -----END CERTIFICATE-----
-date: 2025-04-05 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: byebug
@@ -171,6 +171,7 @@ files:
 - lib/mjml/railtie.rb
 - lib/mjml/version.rb
 - test/generator_test.rb
+- test/handler_test.rb
 - test/mjml_test.rb
 - test/mrml_parser_test.rb
 - test/parser_test.rb
@@ -194,11 +195,12 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.2
+rubygems_version: 3.6.9
 specification_version: 4
 summary: MJML + ERb templates
 test_files:
 - test/generator_test.rb
+- test/handler_test.rb
 - test/mjml_test.rb
 - test/mrml_parser_test.rb
 - test/parser_test.rb