diff options
| -rw-r--r-- | lib/evernote/client.rb | 7 | ||||
| -rw-r--r-- | lib/evernote/user_store.rb | 6 | ||||
| -rw-r--r-- | spec/evernote/client_spec.rb | 9 | ||||
| -rw-r--r-- | spec/evernote/user_store_spec.rb | 4 | 
4 files changed, 8 insertions, 18 deletions
| diff --git a/lib/evernote/client.rb b/lib/evernote/client.rb index e6962dc..1522d59 100644 --- a/lib/evernote/client.rb +++ b/lib/evernote/client.rb @@ -1,7 +1,6 @@ -require 'forwardable' -  module Evernote    class Client +    attr_reader :client      THRIFT_DEFAULTS = {        :transport => Thrift::HTTPClientTransport @@ -11,9 +10,5 @@ module Evernote        thrift_opts = THRIFT_DEFAULTS.merge(thrift_client_options)        @client = ThriftClient.new(klass, url, thrift_opts)      end - -    def method_missing(name, *args, &block) -      @client.send(name, *args, &block) -    end    end  end diff --git a/lib/evernote/user_store.rb b/lib/evernote/user_store.rb index 2b60756..b9ccc17 100644 --- a/lib/evernote/user_store.rb +++ b/lib/evernote/user_store.rb @@ -25,7 +25,11 @@ module Evernote      end      def version_valid? -      @client.checkVersion("Ruby EDAMTest", Evernote::EDAM::UserStore::EDAM_VERSION_MAJOR, Evernote::EDAM::UserStore::EDAM_VERSION_MINOR) +      checkVersion("Ruby EDAMTest", Evernote::EDAM::UserStore::EDAM_VERSION_MAJOR, Evernote::EDAM::UserStore::EDAM_VERSION_MINOR) +    end + +    def method_missing(name, *args, &block) +      @client.client.send(name, *args, &block)      end    end  end diff --git a/spec/evernote/client_spec.rb b/spec/evernote/client_spec.rb index 2a75a74..5d3d08c 100644 --- a/spec/evernote/client_spec.rb +++ b/spec/evernote/client_spec.rb @@ -8,13 +8,4 @@ describe "Evernote::Client" do      Evernote::Client.new(klass, "https://www.example.com")    end - -  it "delegates method calls to the underlying ThriftClient" do -    client = Evernote::Client.new(Evernote::EDAM::UserStore::UserStore::Client, "https://www.example.com") -    internal_client = client.instance_variable_get(:@client) -    internal_client.stub(:checkVersion) -    internal_client.should_receive(:checkVersion) - -    client.checkVersion -  end  end diff --git a/spec/evernote/user_store_spec.rb b/spec/evernote/user_store_spec.rb index 6196ffe..7b0127e 100644 --- a/spec/evernote/user_store_spec.rb +++ b/spec/evernote/user_store_spec.rb @@ -7,14 +7,14 @@ describe "Evernote::UserStore" do    end    it "initializes an Evernote::Client and validate the client code version" do -    client = mock("client", :checkVersion => true) +    client = mock("Evernote::Client", :client => mock("ThriftClient", :checkVersion => true))      Evernote::Client.should_receive(:new).with(Evernote::EDAM::UserStore::UserStore::Client, "https://sandbox.evernote.com/edam/user", {}).and_return(client)      Evernote::UserStore.new("https://sandbox.evernote.com/edam/user", @opts, @thrift_client_opts)    end    it "should raise an error on init if the version is not up to date" do -    client = mock("client", :checkVersion => false) +    client = mock("Evernote::Client", :client => mock("ThriftClient", :checkVersion => false))      Evernote::Client.should_receive(:new).with(Evernote::EDAM::UserStore::UserStore::Client, "https://sandbox.evernote.com/edam/user", {}).and_return(client)      lambda { | 
