observers 0.2.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b1c16092762a682c96f1cc007988ec2406d3b2dd6ce91c1b793b2fbf4f95d628
-  data.tar.gz: 18045745a7cc775e61dd9afc1f46a80074565be6986a37ead5b30d49105a6e77
+  metadata.gz: 701e08478d6d96ff9be87ea056510e4dcf4995bc779c726d263dc9ba7d5803be
+  data.tar.gz: 1d9a10983efabeb4d7ac66c7d6df2f4d4204b947b85836369fdc93e9d91ffc57
 SHA512:
-  metadata.gz: 8c3eb3b00120bfcd498345c0f1ef44322212eacaa6856d794fb9e74044b7a26fa4a6502067cec87aaeff241b4835f5f5e0c048a80d59fe61e3dda277b84c03af
-  data.tar.gz: 843137754e96d5187b23f36fa72e2aa423cba6b9e0af82a99d3b84ef046bc7c2feb76fd3acfe0b3607ea55a6ef250d23ee0f90a211266bd551d5b2045211156f
+  metadata.gz: 2b1a653fa0e7cc33a70d121620c67141061618dc270cd6793b713b04822fe2ed32f0934c30b30079f190d8c3acfeb7aa90df086a4fe66fd3c09fc1e12adaa231
+  data.tar.gz: bcb0708e1be9ec611a28b887a3d18b3cba0798a05d4b8c9c3a502694aa3770e9a06f6cc1767d83aaa085900de6bb58d46e557edfc0dc9b80bd7ad68a7ac731af

data/lib/observables.rb

@@ -4,45 +4,44 @@ require_relative 'models/observable'
 
 module Observers
   class Observables
+    class MissingObservableError < StandardError; end
+
     class << self
       def observables
         @observables ||= {}
         @observables
       end
-
-      def reset
-        @observables = {}
-      end
+      alias all observables
 
       def upsert(key:)
         observables[key] = Observable.new(key:) if observables[key].nil?
         observables[key]
       end
 
+      def reset
+        @observables = {}
+      end
+
       def observe(key:, observer:)
         observable = observables[key] || upsert(key:)
         observable.add_observer(observer:)
       end
 
+      # @return: The last result of the trigger.
       def trigger(actionable:, key:)
         action, event = parse_actionable(actionable:)
 
-        observables[key].observers.each do |observer|
-          observer.trigger(action:, event:)
-        end
+        observable = observables[key]
+        raise MissingObservableError, "Observable key '#{key}' not found" if observable.nil?
 
-        nil # The trigger method is boring and uneventful, it fires events and if it doesn't complain then all is okay.
-      end
-
-      def take(actionable:, key:)
-        action, event = parse_actionable(actionable:)
+        last_result = nil
 
         observables[key].observers.each do |observer|
           result = observer.trigger(action:, event:)
-          return result unless result.nil?
+          last_result = result unless result.nil?
         end
 
-        nil # This is a bad day for the take method, one of the worst.
+        last_result
       end
 
       private

data/lib/observers.rb

@@ -4,23 +4,31 @@ require_relative 'observables'
 require_relative 'models/observer'
 
 module Observers
+  def self.included(klass)
+    klass.extend Observers
+  end
+
+  def observables
+    Observables.observables
+  end
+
   def observable(key = self)
     Observables.upsert(key:)
   end
 
   # TODO: Unit test order.
-  def observe(key, action = nil, order: Observables.observables.count)
-    observer = Observer.new(observer: self, action:, order:)
+  def observe(key, overridden_action: nil, order: Observables.observables.count)
+    observer = Observer.new(observer: self, action: overridden_action, order:)
     Observables.observe(key:, observer:)
   end
 
-  def trigger(actionable, key = nil)
-    Observables.trigger(actionable:, key: key || self)
-  end
+  # Returns the last observer with a non-nil return value.
+  def trigger(key = self, action: nil, event: nil)
+    raise ArgumentError, 'Action or event required' if action.nil? && event.nil?
 
-  # Returns the first observer with a non-nil return value.
-  # One day it may use ractors and be concurrent, if we can freeze the args.
-  def take(actionable, key = nil)
-    Observables.take(actionable:, key: key || self)
+    # TODO: Parsing logic can be simplified/removed now that action and event args are separated.
+    Observables.trigger(actionable: action || event, key:)
   end
+
+  # TODO: Provide a "pipe/port/take" method that uses ractors to be concurrent... if supplied with immutable Data?
 end

data/lib/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Observers
-  VERSION = '0.2.0'
+  VERSION = '0.3.0'
 end

metadata

@@ -1,14 +1,28 @@
 --- !ruby/object:Gem::Specification
 name: observers
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - maedi
 bindir: exe
 cert_chain: []
 date: 1980-01-02 00:00:00.000000000 Z
-dependencies: []
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: low_dependency
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Observe events for objects of any kind
 email:
 - maediprichard@gmail.com