mjml-rails 4.15.1 → 4.16.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9c6559e7522a96885f084a543108e00fc80bf778dd371edbf6a67ffb20c8269a
-  data.tar.gz: e14f43e0de4ad7524c1774784552299fd9689b1643b9504e73f44bed92677c2e
+  metadata.gz: 2765f5756d63843512d96e8c0dca3d5ae41c132e2e527ffd1d161e963ff39e8c
+  data.tar.gz: d7c19dae4feafb3593e4f478463897f1310a0950414811fef87de94323ec8e4d
 SHA512:
-  metadata.gz: fe6820680ae3e42c303ac19d687b2fd01b7c36eb16f9deb590a34e3fe0eb6fd55774f09f95514a46239d233c2209b73247f0e8c577a2c3902e509596c0f8e7ac
-  data.tar.gz: 79ef029fc1a7cecdc35bb3e7681edb865968e5263e110bf7a0b1a87316a6d403e60d6e8383936fda29d6494e118cbf64b8756599933291b6711a63de08fd0548
+  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/version.rb

@@ -2,5 +2,5 @@
 
 module Mjml
   # Version number no longer matches MJML.io version
-  VERSION = '4.15.1'
+  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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mjml-rails
 version: !ruby/object:Gem::Version
-  version: 4.15.1
+  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-07 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