motion 0.5.0 → 0.7.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 579385926d481a6978aa354e54e7b63cdd92f8bb37dc60b49be0f1530371c0d5
-  data.tar.gz: 9d37a416852746fb7b75fd74a3ff972419baddf822b1283c42b86b18be3a0b64
+  metadata.gz: b6798d3da253c0dde5bf0255304659122bfad61dadd649ef39d2cbe8e8b79eb2
+  data.tar.gz: c989839489a975789356ab25f200ade4e36402fc4f928c6168fff57c41b2fba9
 SHA512:
-  metadata.gz: 0bcc311420b91f6479e1a8e9e45680e9854f5f719dc74ef7f25eeda5595e960f3d1082bf84d5b42c6e1bd01b61c24f788ef34df4322bee95d8162e4b732c08d1
-  data.tar.gz: d3d8ead03222c53a801116cc20197860cad5bd8aaa5534b728cc37a32ca00b231fbafdee1fa6e649f004f9b897eef1826a5b8c5d8317d1b3fd11da5586776b43
+  metadata.gz: 28d9a85fc5707916722ffaf38400c63cf4f0e1653f91250e6b93f38db7a1a4998d6f8f07dbc87a52b3b2d80cdcf4bae5f8c387b4dd9bf3958c38afada7221f3c
+  data.tar.gz: 9f06cc7db3f7645fd3451f7ca00a38c2b00a8a4e962bd3638ae89957dc5ea69cad468b0bda1eb780795544f194f051c0628475ae74d56d1ce235913b6af8bf91

data/lib/motion/channel.rb

@@ -13,7 +13,7 @@ module Motion
     ACTION_METHODS = Set.new(["process_motion"]).freeze
     private_constant :ACTION_METHODS
 
-    # Don't use the ActionCable huertistic for deciding what actions can be
+    # Don't use the ActionCable heuristic for deciding what actions can be
     # called from JavaScript. Instead, hard-code the list so we can make other
     # methods public without worrying about them being called from JavaScript.
     def self.action_methods
@@ -84,7 +84,7 @@ module Motion
       @log_helper ||= LogHelper.for_channel(self)
     end
 
-    # Memoize the renderer on the connection so that it can be shared accross
+    # Memoize the renderer on the connection so that it can be shared across
     # all components. `ActionController::Renderer` is already thread-safe and
     # designed to be reused.
     def renderer

data/lib/motion/component/rendering.rb

@@ -28,7 +28,9 @@ module Motion
         @assets_environment
       ].freeze
 
-      private_constant :STATE_EXCLUDED_IVARS
+      STATE_IVAR_OBFUSCATION_PREFIX = "@__vc_"
+
+      private_constant :STATE_EXCLUDED_IVARS, :STATE_IVAR_OBFUSCATION_PREFIX
 
       def rerender!
         @_awaiting_forced_rerender = true
@@ -69,6 +71,7 @@ module Motion
 
       def marshal_dump
         (instance_variables - STATE_EXCLUDED_IVARS)
+          .reject { |ivar| ivar.start_with? STATE_IVAR_OBFUSCATION_PREFIX }
           .map { |ivar| [ivar, instance_variable_get(ivar)] }
           .to_h
       end

data/lib/motion/element.rb

@@ -22,6 +22,10 @@ module Motion
       raw["value"]
     end
 
+    def checked?
+      raw["checked"]
+    end
+
     def attributes
       raw.fetch("attributes", {})
     end

data/lib/motion/errors.rb

@@ -68,7 +68,7 @@ module Motion
   class InvalidComponentStateError < ComponentError; end
 
   class UnrepresentableStateError < InvalidComponentStateError
-    def initialize(component, cause)
+    def initialize(component, cause, failing_ivars)
       super(
         component,
         "Some state prevented `#{component.class}` from being serialized " \
@@ -79,6 +79,9 @@ module Motion
         "\n" \
         "The specific error from `Marshal.dump` was: #{cause}\n" \
         "\n" \
+        "unserializable ivars: #{failing_ivars.join(", ")}\n"\
+        "\n" \
+        "\n" \
         "Hint: Ensure that any exotic state variables in " \
         "`#{component.class}` are removed or replaced."
       )
@@ -114,7 +117,7 @@ module Motion
         "of the application to be mounted because new code with old state " \
         "can lead to unpredictable and unsafe behavior.\n" \
         "\n" \
-        "Hint: If you would like to allow this component to surive " \
+        "Hint: If you would like to allow this component to survive " \
         "deployments, consider providing an alternative implimentation for " \
         "`#{component.class}.upgrade_from`."
       )

data/lib/motion/serializer.rb

@@ -61,10 +61,22 @@ module Motion
 
     private
 
-    def dump(component)
+    def dump!(component)
       Marshal.dump(component)
+    end
+
+    def dump(component)
+      dump!(component)
     rescue TypeError => e
-      raise UnrepresentableStateError.new(component, e.message)
+      failing_ivars = []
+
+      component.marshal_dump.each do |ivar_name, ivar_value|
+        dump!(ivar_value)
+      rescue TypeError
+        failing_ivars << ivar_name
+      end
+
+      raise UnrepresentableStateError.new(component, e.message, failing_ivars)
     end
 
     def load(state)

data/lib/motion/test_helpers.rb

@@ -4,6 +4,14 @@ require "motion"
 
 module Motion
   module TestHelpers
+    class MockComponent < ViewComponent::Base
+      include Motion::Component
+    end
+
+    def callback_stub(method_name = :bound)
+      Motion::Callback.new(MockComponent.new, method_name)
+    end
+
     def assert_motion(component, motion_name)
       assert motion?(component, motion_name)
     end
@@ -26,8 +34,37 @@ module Motion
       end
     end
 
+    def process_broadcast(component, method_name, msg)
+      callback = component.bind(method_name)
+      component.process_broadcast(callback.broadcast, msg)
+    end
+
+    def assert_timer(component, method_name, interval)
+      assert_equal interval, component.periodic_timers[method_name.to_s]
+    end
+
+    def refute_timer(component, method_name)
+      refute timer?(component, method_name)
+    end
+
+    def timer?(component, method_name)
+      component.periodic_timers.include?(method_name.to_s)
+    end
+
     def motion_event(attributes = {})
-      Motion::Event.new(ActiveSupport::JSON.decode(attributes.to_json))
+      Motion::Event.new(ActiveSupport::JSON.decode(attributes.to_json)).tap do |event|
+        set_form_data(event, attributes)
+      end
+    end
+
+    private
+
+    def set_form_data(event, attrs)
+      return event unless attrs.fetch(:element, nil)
+
+      form_data = attrs.dig(:element, :formData) || {}
+      params = ActionController::Parameters.new(form_data)
+      event.element.instance_variable_set(:@form_data, params)
     end
   end
 end

data/lib/motion/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Motion
-  VERSION = "0.5.0"
+  VERSION = "0.7.1"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: motion
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.7.1
 platform: ruby
 authors:
 - Alec Larsen
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-03-11 00:00:00.000000000 Z
+date: 2023-01-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
@@ -31,14 +31,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.1'
+        version: '5.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.1'
+        version: '5.2'
 - !ruby/object:Gem::Dependency
   name: lz4-ruby
   requirement: !ruby/object:Gem::Requirement
@@ -102,7 +102,7 @@ metadata:
   source_code_uri: https://github.com/unabridged/motion
 post_install_message: |
   Friendly reminder: When updating the motion gem, don't forget to update the
-  NPM package as well (`bin/yarn add '@unabridged/motion@0.5.0'`).
+  NPM package as well (`bin/yarn add '@unabridged/motion@0.7.1'`).
 rdoc_options: []
 require_paths:
 - lib
@@ -117,7 +117,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.2.22
 signing_key:
 specification_version: 4
 summary: Reactive frontend UI components for Rails in pure Ruby.