@@ -14,7 +14,7 @@ This gem handles some of the complexity in dealing with the netflix api (and OAu ## License -Copyright (c) 2008 <copyright holders> +Copyright (c) 2008 Rob Ares Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation README.markdown @@ -13,15 +13,18 @@ module Netflix # Client class responsible for setting up api calls. class Client - attr_accessor :consumer_key, :consumer_secret, :application_name + attr_accessor :consumer_key, :consumer_secret, :application_name, :api_version def initialize(consumer_key, consumer_secret, application_name) raise ArgumentError if consumer_key.nil? or consumer_secret.nil? or application_name.nil? @consumer_key = consumer_key @consumer_secret = consumer_secret @application_name = application_name - end - + @api_version = "1.0" + + @access_token = nil + @access_token_secret = nil + end # client.generate_request_token("http://cnn.com/") do |t, s, auth_url| # # save to db here. @@ -29,7 +32,7 @@ module Netflix # end def generate_request_token(callback_url, &blk) raise ArgumentError.new("a block is required as the last arguement") unless block_given? - # asser that we can do this here... + # assert that we can do this here... # populate oauth token # return token, secret @@ -67,7 +70,9 @@ module Netflix yield end - def go(method, path, arguments) + def go(method, path, expansions = [], arguments = {}) + raise NetflixClientError.new("An access token is required to make api calls") if @access_token.nil? or @access_token_secret.nil? + # fail if no access creds. # populate access token with instance vars. # delegate to the client class lib/netflix/client.rb @@ -1,3 +1,56 @@ -$:.unshift(File.dirname(__FILE__) + "/spec_helper") - +require File.dirname(__FILE__) + "/spec_helper" +require "client" +describe Netflix::Client, "When initially created" do + + before do + + end + + before(:each) do + end + + it "should define netflix OAuth endpoints" do + Netflix::OAUTH_ENDPOINTS.class.should == Hash + Netflix::OAUTH_ENDPOINTS.size.should == 3 + end + + it "should be createable" do + lambda {Netflix::Client.new("", "", "")} .should_not == nil + end + + it "should throw an ArgumentError if nil initializer args are passed" do + lambda {Netflix::Client.new(nil, nil, nil)}.should raise_error(ArgumentError) + end + + it "should respond to various accessors" do + client = Netflix::Client.new("", "", "") + client.should respond_to(:consumer_key) + client.should respond_to(:consumer_secret) + client.should respond_to(:application_name) + client.should respond_to(:api_version) + end + + it "should default the api version" do + client = Netflix::Client.new("", "", "") + client.api_version.should == "1.0" + end + + it "should respond to a #generate_request_token instance method and yield 3 variables" do + client = Netflix::Client.new("", "", "") + client.should respond_to(:generate_request_token) + end + it "should respond to a #generate_access_token instance method and yield 3 variables" do + client = Netflix::Client.new("", "", "") + client.should respond_to(:generate_access_token) + end + + it "should respond to a #from_access_token and yield nothing" do + client = Netflix::Client.new("", "", "") + client.should respond_to(:from_access_token) + end + + it "should raise NetflixClientError if no access token info is available when calling #go" do + lambda {Netflix::Client.new("", "", "").go(:get, "", [], {})}.should raise_error(Netflix::NetflixClientError) + end +end spec/netflix_spec.rb @@ -1,2 +1,3 @@ +$:.unshift(File.dirname(__FILE__) + "/../lib") $:.unshift(File.dirname(__FILE__) + "/../lib/netflix") -require "spec" +require "spec" \ No newline at end of file spec/spec_helper.rb b31a224fdea417b4ed1731e03b19d7cb74160f2b Rob Ares rob.ares@gmail.com http://github.com/rares/netflix/commit/1778f792fca2dc4d618d4b5bf58e4f6c4784e546 1778f792fca2dc4d618d4b5bf58e4f6c4784e546 2008-11-13T22:14:32-08:00 2008-11-13T22:14:32-08:00 initial specing cf29a324fd4791da639101dcdf2c8219213fb518 Rob Ares rob.ares@gmail.com