zoomus 0.1.0 → 0.1.2

data/README.md

@@ -16,6 +16,17 @@ Or install it yourself as:
 
     $ gem install zoomus
 
+## Available actions
+
+- user/create
+- user/custcreate
+- user/list
+
+- user/create
+- user/delete
+- user/list
+- user/update
+
 ## Usage
 
     require 'zoomus'

data/lib/zoomus/actions/meeting.rb

@@ -3,24 +3,30 @@ module Zoomus
     module Meeting
 
       def meeting_list(*args)
-        options = extract_options!(args)
-        require_params(:host_id, options)
-        parse_response self.class.post("/meeting/list", :query => options)
+        options = Utils.extract_options!(args)
+        Utils.require_params(:host_id, options)
+        Utils.parse_response self.class.post("/meeting/list", :query => options)
       end
 
       def meeting_create(*args)
-        options = extract_options!(args)
-        require_params([:host_id, :topic, :type], options)
-        parse_response self.class.post("/meeting/create", :query => options)
+        options = Utils.extract_options!(args)
+        Utils.require_params([:host_id, :topic, :type], options)
+        Utils.parse_response self.class.post("/meeting/create", :query => options)
       end
 
       def meeting_update(*args)
-        options = extract_options!(args)
-        require_params([:id, :host_id, :topic, :type], options)
-        parse_response self.class.post("/meeting/update", :query => options)
+        options = Utils.extract_options!(args)
+        Utils.require_params([:id, :host_id, :topic, :type], options)
+        Utils.parse_response self.class.post("/meeting/update", :query => options)
       end
 
-      Utils::define_bang_methods(self)
+      def meeting_delete(*args)
+        options = Utils.extract_options!(args)
+        Utils.require_params([:id, :host_id], options)
+        Utils.parse_response self.class.post("/meeting/delete", :query => options)
+      end
+
+      Utils.define_bang_methods(self)
 
     end
   end

data/lib/zoomus/actions/user.rb

@@ -3,22 +3,22 @@ module Zoomus
     module User
 
       def user_list
-        parse_response self.class.post("/user/list")
+        Utils.parse_response self.class.post("/user/list")
       end
 
       def user_create(*args)
-        options = extract_options!(args)
-        require_params([:type, :email], options)
-        parse_response self.class.post("/user/create", :query => options)
+        options = Utils.extract_options!(args)
+        Utils.require_params([:type, :email], options)
+        Utils.parse_response self.class.post("/user/create", :query => options)
       end
 
       def user_custcreate(*args)
-        options = extract_options!(args)
-        require_params([:type, :email], options)
-        parse_response self.class.post("/user/custcreate", :query => options)
+        options = Utils.extract_options!(args)
+        Utils.require_params([:type, :email], options)
+        Utils.parse_response self.class.post("/user/custcreate", :query => options)
       end
 
-      Utils::define_bang_methods(self)
+      Utils.define_bang_methods(self)
 
     end
   end

data/lib/zoomus/client.rb

@@ -5,8 +5,6 @@ module Zoomus
   class Client
 
     include HTTParty
-
-    include Utils
     include Actions::User
     include Actions::Meeting
 
@@ -14,13 +12,14 @@ module Zoomus
 
     def initialize(*args)
 
-      options = extract_options!(args)
-
-      raise argument_error("api_key and api_secret") unless options[:api_key] and options[:api_secret]
+      options = Utils.extract_options!(args)
 
-      self.class.default_params :api_key => options[:api_key],
-                                :api_secret => options[:api_secret]
+      raise Utils.argument_error("api_key and api_secret") unless options[:api_key] &&
+                                                                  options[:api_secret]
+      self.class.default_params(
+        :api_key    => options[:api_key],
+        :api_secret => options[:api_secret]
+      )
     end
-
   end
 end

data/lib/zoomus/utils.rb

@@ -1,47 +1,48 @@
 module Zoomus
-  module Utils
+  class Utils
 
-    private
+    class << self
 
-    def argument_error(name)
-      name ? ArgumentError.new("You must provide #{name}") : ArgumentError
-    end
+      def argument_error(name)
+        name ? ArgumentError.new("You must provide #{name}") : ArgumentError.new
+      end
 
-    def raise_if_error!(response)
-      if response["error"]
-        raise Error.new(response["error"]["message"])
-      else
-        response
+      def raise_if_error!(response)
+        if response["error"]
+          raise Error.new(response["error"]["message"])
+        else
+          response
+        end
       end
-    end
 
-    def parse_response(http_response)
-      response = http_response.parsed_response
-      # Mocked response returns a string
-      response.kind_of?(Hash) ? response : JSON.parse(response)
-    end
+      def parse_response(http_response)
+        response = http_response.parsed_response
+        # Mocked response returns a string
+        response.kind_of?(Hash) ? response : JSON.parse(response)
+      end
 
-    def require_params(params, options)
-      params = [params] unless params.is_a? Array
-      params.each do |param|
-        unless options[param]
-          raise argument_error(param.to_s)
-          break
+      def require_params(params, options)
+        params = [params] unless params.is_a? Array
+        params.each do |param|
+          unless options[param]
+            raise argument_error(param.to_s)
+            break
+          end
         end
       end
-    end
 
-    # Dinamically defines bang methods for Actions modules
-    def self.define_bang_methods(klass)
-      klass.instance_methods.each do |m|
-        klass.send(:define_method, "#{m}!") do |*args|
-          raise_if_error! send(m, *args)
+      # Dinamically defines bang methods for Actions modules
+      def define_bang_methods(klass)
+        klass.instance_methods.each do |m|
+          klass.send(:define_method, "#{m}!") do |*args|
+            Utils.raise_if_error! send(m, *args)
+          end
         end
       end
-    end
 
-    def extract_options!(array)
-      array.last.is_a?(::Hash) ? array.pop : {}
+      def extract_options!(array)
+        array.last.is_a?(::Hash) ? array.pop : {}
+      end
     end
   end
 end

data/spec/fixtures/error.json

@@ -0,0 +1,6 @@
+{
+  "error": {
+    "message": "Invalid api key or secret",
+    "code": 200
+  }
+}

data/spec/fixtures/meeting_delete.json

@@ -0,0 +1,4 @@
+{
+  "id": "252482092",
+  "deleted_at": "2013-04-05T15:50:47Z"
+}

data/spec/lib/zoomus/actions/meeting/create_spec.rb

@@ -9,7 +9,8 @@ describe Zoomus::Actions::Meeting do
 
   describe "#meeting_create action" do
     before :each do
-      stub_request(:post, zoomus_url("/meeting/create")).to_return(:body => json_response("meeting_create"))
+      stub_request(:post, zoomus_url("/meeting/create")).
+        to_return(:body => json_response("meeting_create"))
     end
 
     it "requires a 'host_id' argument" do
@@ -49,4 +50,18 @@ describe Zoomus::Actions::Meeting do
       expect(res["join_url"]).to_not be_nil
     end
   end
+
+  describe "#meeting_create! action" do
+    before :each do
+      stub_request(:post, zoomus_url("/meeting/create")).
+        to_return(:body => json_response("error"))
+    end
+
+    it "raises Zoomus::Error exception" do
+      expect{ @zc.meeting_create!(
+                :host_id => @host_id,
+                :type => 1,
+                :topic => "Foo")}.to raise_error(Zoomus::Error)
+    end
+  end
 end

data/spec/lib/zoomus/actions/meeting/delete_spec.rb

@@ -0,0 +1,50 @@
+require 'spec_helper'
+
+describe Zoomus::Actions::Meeting do
+
+  before :all do
+    @zc = zoomus_client
+    @host_id = "ufR93M2pRyy8ePFN92dttq"
+    @id = "252482092"
+  end
+
+  describe "#meeting_delete action" do
+    before :each do
+      stub_request(:post, zoomus_url("/meeting/delete")).to_return(:body => json_response("meeting_delete"))
+    end
+
+    it "requires a 'host_id' argument" do
+      expect{@zc.meeting_delete(:id => @id)}.to raise_error(ArgumentError)
+    end
+
+    it "requires a 'id' argument" do
+      expect{@zc.meeting_delete(:host_id => @host_id)}.to raise_error(ArgumentError)
+    end
+
+    it "returns a hash" do
+      expect(@zc.meeting_delete(:host_id => @host_id,
+                                :id => @id)).to be_kind_of(Hash)
+    end
+
+    it "returns id and deleted at attributes" do
+      res = @zc.meeting_delete(:host_id => @host_id,
+                               :id => @id)
+
+      expect(res["id"]).to eq(@id)
+      expect(res["deleted_at"]).to eq("2013-04-05T15:50:47Z")
+    end
+  end
+
+  describe "#meeting_delete! action" do
+    before :each do
+      stub_request(:post, zoomus_url("/meeting/delete")).
+        to_return(:body => json_response("error"))
+    end
+
+    it "raises Zoomus::Error exception" do
+      expect{ @zc.meeting_delete!(
+                :host_id => @host_id,
+                :id => @id)}.to raise_error(Zoomus::Error)
+    end
+  end
+end

data/spec/lib/zoomus/actions/meeting/list_spec.rb

@@ -28,4 +28,15 @@ describe Zoomus::Actions::Meeting do
       expect(@zc.meeting_list(:host_id => @host_id)["meetings"]).to be_kind_of(Array)
     end
   end
+
+  describe "#meeting_list! action" do
+    before :each do
+      stub_request(:post, zoomus_url("/meeting/list")).
+        to_return(:body => json_response("error"))
+    end
+
+    it "raises Zoomus::Error exception" do
+      expect{ @zc.meeting_list!(:host_id => @host_id)}.to raise_error(Zoomus::Error)
+    end
+  end
 end

data/spec/lib/zoomus/actions/meeting/update_spec.rb

@@ -46,4 +46,19 @@ describe Zoomus::Actions::Meeting do
       expect(res["updated_at"]).to eq("2013-02-25T15:52:38Z")
     end
   end
+
+  describe "#meeting_update! action" do
+    before :each do
+      stub_request(:post, zoomus_url("/meeting/update")).
+        to_return(:body => json_response("error"))
+    end
+
+    it "raises Zoomus::Error exception" do
+      expect{ @zc.meeting_update!(
+                :host_id => @host_id,
+                :id => @id,
+                :type => 1,
+                :topic => "Foo")}.to raise_error(Zoomus::Error)
+    end
+  end
 end

data/spec/lib/zoomus/actions/user/create_spec.rb

@@ -38,4 +38,20 @@ describe Zoomus::Actions::User do
       expect(res["type"]).to eq(1)
     end
   end
+
+  describe "#user_create! action" do
+    before :each do
+      stub_request(:post, zoomus_url("/user/create")).
+        to_return(:body => json_response("error"))
+    end
+
+    it "raises Zoomus::Error exception" do
+      expect{ @zc.user_create!(
+                :email => "foo@bar.com",
+                :first_name => "Foo",
+                :last_name => "Bar",
+                :type => 1)}.to raise_error(Zoomus::Error)
+    end
+  end
+
 end

data/spec/lib/zoomus/actions/user/custcreate_spec.rb

@@ -12,18 +12,17 @@ describe Zoomus::Actions::User do
     end
 
     it "requires email param" do
-      expect{@zc.user_custcreate(:type => 1, :password => "lolcat")}.to raise_error(ArgumentError)
+      expect{@zc.user_custcreate(:type => 1)}.to raise_error(ArgumentError)
     end
 
     it "requires type param" do
-      expect{@zc.user_custcreate(:email => "foo@bar.com", :password => "lolcat")}.to raise_error(ArgumentError)
+      expect{@zc.user_custcreate(:email => "foo@bar.com")}.to raise_error(ArgumentError)
     end
 
     it "returns a hash" do
       expect(@zc.user_custcreate(:email => "foo@bar.com",
                                  :first_name => "Foo",
                                  :last_name => "Bar",
-                                 :password => "lolcat",
                                  :type => 1)).to be_kind_of(Hash)
     end
 
@@ -31,7 +30,6 @@ describe Zoomus::Actions::User do
       res = @zc.user_custcreate(:email => "foo@bar.com",
                                 :first_name => "Foo",
                                 :last_name => "Bar",
-                                :password => "lolcat",
                                 :type => 1)
 
       expect(res["email"]).to eq("foo@bar.com")
@@ -40,4 +38,19 @@ describe Zoomus::Actions::User do
       expect(res["type"]).to eq(1)
     end
   end
+
+  describe "#user_custcreate! action" do
+    before :each do
+      stub_request(:post, zoomus_url("/user/custcreate")).
+        to_return(:body => json_response("error"))
+    end
+
+    it "raises Zoomus::Error exception" do
+      expect{ @zc.user_custcreate!(
+                :email => "foo@bar.com",
+                :first_name => "Foo",
+                :last_name => "Bar",
+                :type => 1)}.to raise_error(Zoomus::Error)
+    end
+  end
 end

data/spec/lib/zoomus/actions/user/list_spec.rb

@@ -23,4 +23,15 @@ describe Zoomus::Actions::User do
       expect(@zc.user_list["users"]).to be_kind_of(Array)
     end
   end
+
+  describe "#user_list! action" do
+    before :each do
+      stub_request(:post, zoomus_url("/user/list")).
+        to_return(:body => json_response("error"))
+    end
+
+    it "raises Zoomus::Error exception" do
+      expect{ @zc.user_list! }.to raise_error(Zoomus::Error)
+    end
+  end
 end

data/spec/lib/zoomus/utils_spec.rb

@@ -2,14 +2,42 @@ require 'spec_helper'
 
 describe Zoomus::Utils do
 
-  describe "#argument_error"
+  before(:all) do
+    class Utils < Zoomus::Utils; end
+  end
 
-  describe "#raise_if_error!"
+  describe "#argument_error" do
+    it "raises ArgumentError" do
+      expect(Utils.argument_error("foo")).to be_instance_of(ArgumentError)
+    end
+  end
 
-  describe "#parse_response"
+  describe "#raise_if_error!" do
+    it "raises Zoomus::Error if error is present" do
+      response = {'error' => { 'message' => 'lol error'}}
+      expect{Utils.raise_if_error!(response)}.to raise_error(Zoomus::Error)
+    end
 
-  describe "#require_params"
+    it "does not raise Zoomus::Error if error is not present" do
+      response = {}
+      expect{Utils.raise_if_error!(response)}.to_not raise_error(Zoomus::Error)
+    end
+  end
 
-  describe "#extract_options!"
+  describe "#require_params" do
+    it "raises ArgumentError if the param is not present" do
+      expect{Utils.require_params(:foo, {:bar => 'bar'})}.to raise_error(ArgumentError)
+    end
 
+    it "does not raise ArgumentError if the param is present" do
+      expect{Utils.require_params(:foo, {:foo => 'foo'})}.to_not raise_error(ArgumentError)
+    end
+  end
+
+  describe "#extract_options!" do
+    it "converts array to hash options" do
+      args = [{:foo => 'foo'}, {:bar => 'bar'}, {:zemba => 'zemba'}]
+      expect(Utils.extract_options!(args)).to be_kind_of(Hash)
+    end
+  end
 end

data/zoomus.gemspec

@@ -16,5 +16,5 @@ Gem::Specification.new do |gem|
   gem.test_files    = gem.files.grep(%r{^(test|spec|features)/})
   gem.name          = 'zoomus'
   gem.require_paths = ['lib']
-  gem.version       = '0.1.0'
+  gem.version       = '0.1.2'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: zoomus
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.2
   prerelease: 
 platform: ruby
 authors:
@@ -13,7 +13,7 @@ date: 2013-04-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty
-  requirement: &69721570 !ruby/object:Gem::Requirement
+  requirement: &78305960 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *69721570
+  version_requirements: *78305960
 - !ruby/object:Gem::Dependency
   name: json
-  requirement: &69721110 !ruby/object:Gem::Requirement
+  requirement: &78305650 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,7 +32,7 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *69721110
+  version_requirements: *78305650
 description: A Ruby wrapper for zoom.us API v1
 email:
 - collsmaxim@gmail.com
@@ -52,13 +52,16 @@ files:
 - lib/zoomus/error.rb
 - lib/zoomus/interface.rb
 - lib/zoomus/utils.rb
+- spec/fixtures/error.json
 - spec/fixtures/meeting_create.json
+- spec/fixtures/meeting_delete.json
 - spec/fixtures/meeting_list.json
 - spec/fixtures/meeting_update.json
 - spec/fixtures/user_create.json
 - spec/fixtures/user_custcreate.json
 - spec/fixtures/user_list.json
 - spec/lib/zoomus/actions/meeting/create_spec.rb
+- spec/lib/zoomus/actions/meeting/delete_spec.rb
 - spec/lib/zoomus/actions/meeting/list_spec.rb
 - spec/lib/zoomus/actions/meeting/update_spec.rb
 - spec/lib/zoomus/actions/user/create_spec.rb
@@ -93,13 +96,16 @@ signing_key:
 specification_version: 3
 summary: zoom.us API wrapper
 test_files:
+- spec/fixtures/error.json
 - spec/fixtures/meeting_create.json
+- spec/fixtures/meeting_delete.json
 - spec/fixtures/meeting_list.json
 - spec/fixtures/meeting_update.json
 - spec/fixtures/user_create.json
 - spec/fixtures/user_custcreate.json
 - spec/fixtures/user_list.json
 - spec/lib/zoomus/actions/meeting/create_spec.rb
+- spec/lib/zoomus/actions/meeting/delete_spec.rb
 - spec/lib/zoomus/actions/meeting/list_spec.rb
 - spec/lib/zoomus/actions/meeting/update_spec.rb
 - spec/lib/zoomus/actions/user/create_spec.rb