omnievent 0.1.0.pre4 → 0.1.0.pre5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 38717b9e58eab673d2701d9bd2fb29a47b39820614cf9b46ab2dfde98a940d7f
-  data.tar.gz: 8f750f1bf92f031b333bb50a39e1dd8f355d641549ed271c9c0017689676c63c
+  metadata.gz: 5fe7bcccfdc17aa7d63acd4c44143de5006e7cee9c142078656751bc3a5641fe
+  data.tar.gz: 7be7b177ae90c1eb8d3490e6563956c7d3f1ac702d59e108058c1eac70d4f696
 SHA512:
-  metadata.gz: 4e5224989a966288fb06247ca102f5a93491703df0180e69532f3baf228488e3cee2872b244248418108717430413745e1c083526cb344bf59cd6536eafab2cc
-  data.tar.gz: 5941e388487c35a43935675dc12a5a8af99756b86d98d053d3aed128b915d3032d12e2ced512889525c9a00fd0aa0326a294ee9e1c8810e389d1527b60a060d9
+  metadata.gz: ba6ca77f917e16246ec0bb3d80186b9ae3fb515b4b4bc1947688cd5f2cd51bbf06abe88adaa557448f00c89fd12467c5bebf9f04852f6d92cae6773a12060230
+  data.tar.gz: 43ebe3948021f40fa946c2bc5a7d200da65fd96754425bda2d378e6fa06dbdb94bfe1a051c53ed3605edfe9593b6a17ede5448065e61c696e5e1097a0d1f6150

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    omnievent (0.1.0.pre3)
+    omnievent (0.1.0.pre5)
       hashie (>= 3.4.6)
       iso-639 (~> 0.3.5)
       tzinfo (~> 2.0)
@@ -21,7 +21,7 @@ GEM
     hashie (5.0.0)
     i18n (1.12.0)
       concurrent-ruby (~> 1.0)
-    iso-639 (0.3.5)
+    iso-639 (0.3.6)
     json (2.6.2)
     minitest (5.16.3)
     parallel (1.22.1)

data/lib/omnievent/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module OmniEvent
-  VERSION = "0.1.0.pre4"
+  VERSION = "0.1.0.pre5"
 end

data/lib/omnievent.rb

@@ -57,44 +57,38 @@ module OmniEvent
 
     # List events.
     def list_events(provider = nil, opts = {})
-      validate_provider_arg(provider)
+      raise ArgumentError, "You need to pass a provider name as the first argument." unless provider
 
       strategy_instance(provider).request(:list_events, opts)
     end
 
     # Create event.
     def create_event(provider = nil, opts = {})
-      validate_event_args(provider, opts)
+      raise ArgumentError, "You need to pass a provider name as the first argument." unless provider
+      raise ArgumentError, "You need to pass an :event in opts." unless opts_has_event?(opts)
+      raise ArgumentError, "Event is not valid." unless opts[:event].valid?
 
       strategy_instance(provider).request(:create_event, opts)
     end
 
     # Update event.
     def update_event(provider = nil, opts = {})
-      validate_event_args(provider, opts)
+      raise ArgumentError, "You need to pass a provider name as the first argument." unless provider
+      raise ArgumentError, "You need to pass an :event in opts." unless opts_has_event?(opts)
 
       strategy_instance(provider).request(:update_event, opts)
     end
 
     # Destroy event.
     def destroy_event(provider = nil, opts = {})
-      validate_event_args(provider, opts)
-
-      strategy_instance(provider).request(:destroy_event, opts)
-    end
-
-    def validate_provider_arg(provider)
       raise ArgumentError, "You need to pass a provider name as the first argument." unless provider
-    end
+      raise ArgumentError, "You need to pass an :event in opts." unless opts_has_event?(opts)
 
-    def validate_event_args(provider, opts)
-      validate_provider_arg(provider)
-
-      raise ArgumentError, "You need to pass a valid as :event in opts." unless opts_have_valid_event?(opts)
+      strategy_instance(provider).request(:destroy_event, opts)
     end
 
-    def opts_have_valid_event?(opts)
-      opts[:event].present? && opts[:event].is_a?(OmniEvent::EventHash) && opts[:event].valid?
+    def opts_has_event?(opts)
+      opts[:event]&.is_a?(OmniEvent::EventHash)
     end
 
     def strategy_instance(provider)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: omnievent
 version: !ruby/object:Gem::Version
-  version: 0.1.0.pre4
+  version: 0.1.0.pre5
 platform: ruby
 authors:
 - Angus McLeod