hotwire_native_rails 0.3.1 → 0.3.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: db9feb69df016d6781d12df5d095422b9984afe20e0128add4ec5db096bc9232
-  data.tar.gz: ae82da4ac986d04d552d47207b7c93dc75fea6cde2e38944f8b9926997d89b3e
+  metadata.gz: b7c5dccf379ea2df338bbcb79be305de69d6bddf6d18b90aa385b3b5cfa02c0a
+  data.tar.gz: 735bb16d4b1edea360e76be697e2b2205fed6bb3cc600a1d701f62d2bfcf50b9
 SHA512:
-  metadata.gz: 18302cf1e8af968d95a23685db7d7020375060bc7a22b628b146d3596791fcdd18409f818d04456594b030d2ab40bc98f90b4d6c1d08fd3108448929155c16f8
-  data.tar.gz: 1385c781224c90744b7eeea4b7fa5690524768cd5fc21db85085ac1725b42ae0f901bc433a80082771ba88e1ca04f1a1c8918dbbcf692e87f82d015cc48a9bfb
+  metadata.gz: f85af13e81abf134367debe8dd96ba2159dd291daf517c142c97784515df219cf8f4e8388ac56e9fbbacbbf455a00edd44f5124e08400a3ea45f5261f51bd1f3
+  data.tar.gz: 4f17e15e2067bdef0aa9db0c9295410f19315ba7159e761d5e323f5e29d32154f2ed788f03bd9186d265495158a44cfce5c7694bd1a7d22903642f4b42e3e0cc

data/hotwire_native_rails.gemspec

@@ -18,7 +18,7 @@ Gem::Specification.new do |spec|
 
   # Specify which files should be added to the gem when it is released.
   # The `git ls-files -z` loads the files in the RubyGem that have been added into git.
-  spec.files         = Dir.chdir(File.expand_path('..', __FILE__)) do
-    `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
+  spec.files = Dir.chdir(File.expand_path('..', __FILE__)) do
+    `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) || f.end_with?('.gem') }
   end
 end

data/lib/generators/hotwire_native/hotwire_native_generator.rb

@@ -9,11 +9,13 @@ class HotwireNativeGenerator < Rails::Generators::Base
     # helpers
     copy_file "helpers/hotwire_native_helper.rb", "app/helpers/hotwire_native_helper.rb"
     copy_file "test_unit/hotwire_native_helper_test.rb", "test/helpers/hotwire_native_helper_test.rb"
+    copy_file "test_unit/hotwire_native_controller_test.rb", "test/controllers/hotwire_native_controller_test.rb"
 
     # routes
     copy_file "routes/hotwire_native.rb", "config/routes/hotwire_native.rb"
-    copy_file "controllers/hotwire_native/v1/android/path_configuration_controller.rb", "app/controllers/hotwire_native/v1/android/path_configuration_controller.rb"
-    copy_file "controllers/hotwire_native/v1/ios/path_configuration_controller.rb", "app/controllers/hotwire_native/v1/ios/path_configuration_controller.rb"
+    copy_file "controllers/hotwire_native/v1/android/path_configurations_controller.rb", "app/controllers/hotwire_native/v1/android/path_configurations_controller.rb"
+    copy_file "controllers/hotwire_native/v1/ios/path_configurations_controller.rb", "app/controllers/hotwire_native/v1/ios/path_configurations_controller.rb"
+    copy_file "test_unit/path_configurations_controller_test.rb", "test/controllers/path_configurations_controller_test.rb"
 
     # :native request variant
     copy_file "controllers/concerns/device_format.rb", "app/controllers/concerns/device_format.rb"

data/lib/generators/hotwire_native/templates/controllers/concerns/device_format.rb

@@ -8,6 +8,6 @@ module DeviceFormat
   private
 
   def set_variant
-    return request.variant = :mobile if turbo_native_app? || browser.device.mobile?
+    request.variant = :mobile if turbo_native_app? || browser.device.mobile?
   end
 end

data/lib/generators/hotwire_native/templates/helpers/hotwire_native_helper.rb

@@ -37,37 +37,37 @@ module HotwireNativeHelper
   end
 
   # override link_to to not open internal links in in-app browser on native app
-  def link_to(name = nil, options = nil, html_options = {}, &block)
+  def link_to(name = nil, options = nil, html_options = {}, &)
     html_options[:target] = '' if turbo_native_app? && internal_url?(url_for(options))
-    super(name, options, html_options, &block)
+    super
   end
 
   # https://github.com/joemasilotti/daily-log/blob/main/rails/app/helpers/form_helper.rb
   class BridgeFormBuilder < ActionView::Helpers::FormBuilder
     def submit(value = nil, options = {})
       options[:data] ||= {}
-      options["data-bridge--form-target"] = "submit"
-      options[:class] = [options[:class], "turbo-native:hidden"].compact
-      super(value, options)
+      options['data-bridge--form-target'] = 'submit'
+      options[:class] = [options[:class], 'turbo-native:hidden'].compact
+      super
     end
   end
 
-  def bridge_form_with(*, **options, &block)
+  def bridge_form_with(*, **options, &)
     options[:html] ||= {}
     options[:html][:data] ||= {}
     options[:html][:data] = options[:html][:data].merge(bridge_form_data)
 
     options[:builder] = BridgeFormBuilder
 
-    form_with(*, **options, &block)
+    form_with(*, **options, &)
   end
 
   private
 
   def bridge_form_data
     {
-      controller: "bridge--form",
-      action: "turbo:submit-start->bridge--form#submitStart turbo:submit-end->bridge--form#submitEnd"
+      controller: 'bridge--form',
+      action: 'turbo:submit-start->bridge--form#submitStart turbo:submit-end->bridge--form#submitEnd'
     }
   end
 

data/lib/generators/hotwire_native/templates/test_unit/hotwire_native_controller_test.rb

@@ -0,0 +1,15 @@
+require "test_helper"
+
+class HotwireNativeTest < ActionDispatch::IntegrationTest
+  test 'zoom is disabled for native' do
+    get root_path, headers: { HTTP_USER_AGENT: 'Hotwire Native' }
+    assert_response :success
+    assert_match(/user-scalable=0/, response.body)
+  end
+
+  test 'zoom is enabled for web' do
+    get root_path
+    assert_response :success
+    assert_no_match(/user-scalable=0/, response.body)
+  end
+end

data/lib/generators/hotwire_native/templates/test_unit/hotwire_native_helper_test.rb

@@ -1,54 +1,53 @@
 require 'test_helper'
 
 class HotwireNativeHelperTest < ActionView::TestCase
-  # add more tests
   setup do
     def root_url
-      "http://test.host/"
-    end    
+      'http://test.host/'
+    end
   end
 
-  test "link_to removes target for internal links in turbo native app" do
+  test 'link_to removes target for internal links in turbo native app' do
     def turbo_native_app?
       true
     end
 
-    result = link_to("Home", "/foo/bar", target: "_blank")
-    assert_no_match /target="_blank"/, result
+    result = link_to('Home', '/foo/bar', target: '_blank', rel: 'noopener')
+    assert_no_match(/target="_blank"/, result)
   end
 
-  test "link_to keeps target for external links in turbo native app" do
+  test 'link_to keeps target for external links in turbo native app' do
     def turbo_native_app?
       true
     end
-    
-    result = link_to("External", "http://external.com", target: "_blank")
-    assert_match /target="_blank"/, result
+
+    result = link_to('External', 'http://external.com', target: '_blank', rel: 'noopener')
+    assert_match(/target="_blank"/, result)
   end
 
-  test "link_to keeps target for internal links when not in turbo native app" do
+  test 'link_to keeps target for internal links when not in turbo native app' do
     def turbo_native_app?
       false
     end
-    
-    result = link_to("Home", root_url, target: "_blank")
-    assert_match /target="_blank"/, result
+
+    result = link_to('Home', root_url, target: '_blank', rel: 'noopener')
+    assert_match(/target="_blank"/, result)
   end
 
-  test "link_to keeps target for external links when not in turbo native app" do
+  test 'link_to keeps target for external links when not in turbo native app' do
     def turbo_native_app?
       false
     end
 
-    result = link_to("External", "http://external.com", target: "_blank")
-    assert_match /target="_blank"/, result
+    result = link_to('External', 'http://external.com', target: '_blank', rel: 'noopener')
+    assert_match(/target="_blank"/, result)
   end
 
-  test "internal_url? returns true for internal links" do
-    assert internal_url?("http://test.host/some_path")
+  test 'internal_url? returns true for internal links' do
+    assert internal_url?('http://test.host/some_path')
   end
 
-  test "internal_url? returns false for external links" do
-    assert_not internal_url?("http://external.com")
+  test 'internal_url? returns false for external links' do
+    assert_not internal_url?('http://external.com')
   end
-end
+end

data/lib/generators/hotwire_native/templates/test_unit/path_configurations_controller_test.rb

@@ -0,0 +1,13 @@
+require "test_helper"
+
+class PathConfigurationsControllerTest < ActionDispatch::IntegrationTest
+  test 'android' do
+    get '/hotwire_native/v1/android/path_configuration'
+    assert_response :success
+  end
+
+  test 'ios' do
+    get '/hotwire_native/v1/ios/path_configuration'
+    assert_response :success
+  end
+end

data/lib/hotwire_native_rails/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module HotwireNativeRails
-  VERSION = "0.3.1"
+  VERSION = "0.3.2"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hotwire_native_rails
 version: !ruby/object:Gem::Version
-  version: 0.3.1
+  version: 0.3.2
 platform: ruby
 authors:
 - Yaro Shm
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-11-30 00:00:00.000000000 Z
+date: 2024-12-01 00:00:00.000000000 Z
 dependencies: []
 description:
 email:
@@ -29,8 +29,8 @@ files:
 - hotwire_native_rails.gemspec
 - lib/generators/hotwire_native/hotwire_native_generator.rb
 - lib/generators/hotwire_native/templates/controllers/concerns/device_format.rb
-- lib/generators/hotwire_native/templates/controllers/hotwire_native/v1/android/path_configuration_controller.rb
-- lib/generators/hotwire_native/templates/controllers/hotwire_native/v1/ios/path_configuration_controller.rb
+- lib/generators/hotwire_native/templates/controllers/hotwire_native/v1/android/path_configurations_controller.rb
+- lib/generators/hotwire_native/templates/controllers/hotwire_native/v1/ios/path_configurations_controller.rb
 - lib/generators/hotwire_native/templates/helpers/hotwire_native_helper.rb
 - lib/generators/hotwire_native/templates/javascript/controllers/bridge/button_controller.js
 - lib/generators/hotwire_native/templates/javascript/controllers/bridge/form_controller.js
@@ -39,7 +39,9 @@ files:
 - lib/generators/hotwire_native/templates/javascript/controllers/bridge/overflow_menu_controller.js
 - lib/generators/hotwire_native/templates/javascript/controllers/bridge/review_prompt_controller.js
 - lib/generators/hotwire_native/templates/routes/hotwire_native.rb
+- lib/generators/hotwire_native/templates/test_unit/hotwire_native_controller_test.rb
 - lib/generators/hotwire_native/templates/test_unit/hotwire_native_helper_test.rb
+- lib/generators/hotwire_native/templates/test_unit/path_configurations_controller_test.rb
 - lib/hotwire_native_rails.rb
 - lib/hotwire_native_rails/version.rb
 homepage: https://github.com/yshmarov/hotwire_native_rails