test/fixtures/object/bookmark.xml @@ -13,7 +13,8 @@ require 'gluestick/object' module Gluestick class << self extend Forwardable - def_delegators :client, :login + attr_reader :username + def_delegators :client, :login, :logout, :username def client @client = Gluestick::Client.instance unless @client lib/gluestick.rb @@ -7,6 +7,8 @@ module Gluestick base_uri API_URI format :xml + attr_reader :username + def login(username, password) basic_auth = { :username => username, :password => password } response = unauthenticated_get("/user/validate", :basic_auth => basic_auth) lib/gluestick/client.rb @@ -1,9 +1,12 @@ module Gluestick class Object extend LazyLoader - - private_class_method :new - attr_reader :objectKey, :title, :image, :link, :type + attr_reader :objectKey + lazy_load [:title, :image, :link, :type], :get + + def initialize(objectKey) + @objectKey = objectKey + end def self.get(objectId) begin @@ -18,15 +21,18 @@ module Gluestick raise TypeError if not response.instance_of?(Gluestick::AdaptiveBlueResponse) object_xml = response.response['object'] - object = create_for_type(object_xml['type']) + type = object_xml['type'] + objectKey = object_xml['objectKey'] + object = create_for_type(objectKey, type) assign_variables_from_response(object, response) object end def self.from_interaction(response) + objectKey = response['objectKey'] type = response['category'] - object = create_for_type(type) + object = create_for_type(objectKey, type) %w[title image objectKey].each do |property| object.instance_variable_set("@#{property}", response[property]) @@ -53,33 +59,34 @@ module Gluestick response.response['links']['link'] end - protected - - def self.create - new + def glue_object? + false end - def self.create_for_type(type) + protected + + def self.create_for_type(objectKey, type) # Would've extracted this out into a constant, but the classes # are not defined and throws NameErrors @@categories = { - :books => Gluestick::BookObject, - :electronics => Gluestick::ElectronicObject, - :movie_stars => Gluestick::MovieStarObject, - :movies => Gluestick::MovieObject, - :music => Gluestick::MusicObject, - :recording_artists => Gluestick::RecordingArtistObject, - :restaurants => Gluestick::RestaurantObject, - :stocks => Gluestick::StockObject, - :tv_shows => Gluestick::TVShowObject, - :video_games => Gluestick::VideoGameObject, - :wines => Gluestick::WineObject + :books => Gluestick::BookObject, + :electronics => Gluestick::ElectronicObject, + :movie_stars => Gluestick::MovieStarObject, + :movies => Gluestick::MovieObject, + :music => Gluestick::MusicObject, + :recording_artists => Gluestick::RecordingArtistObject, + :restaurants => Gluestick::RestaurantObject, + :stocks => Gluestick::StockObject, + :tv_shows => Gluestick::TVShowObject, + :video_games => Gluestick::VideoGameObject, + :wines => Gluestick::WineObject, + :bookmarks => Gluestick::BookmarkObject } if @@categories.has_key?(type.to_sym) - @@categories[type.to_sym].create + @@categories[type.to_sym].new(objectKey) else - create + new(objectKey) end end @@ -90,39 +97,45 @@ module Gluestick end end - class BookObject < Object + class GlueObject < Object + def glue_object?; true; end + end + + class BookObject < GlueObject end - class ElectronicObject < Object + class ElectronicObject < GlueObject end - class MovieStarObject < Object + class MovieStarObject < GlueObject end - class MovieObject < Object + class MovieObject < GlueObject lazy_load [:director, :year, :starring], :get end - class MusicObject < Object + class MusicObject < GlueObject end - class RecordingArtistObject < Object + class RecordingArtistObject < GlueObject end - class RestaurantObject < Object + class RestaurantObject < GlueObject end - class StockObject < Object + class StockObject < GlueObject end - class TVShowObject < Object + class TVShowObject < GlueObject end - class VideoGameObject < Object + class VideoGameObject < GlueObject end - class WineObject < Object + class WineObject < GlueObject end + class BookmarkObject < Object + end end lib/gluestick/object.rb @@ -9,8 +9,51 @@ class ObjectTest < Test::Unit::TestCase Gluestick::Object.from_object(response) end - should "not be able to instantiate an interaction" do + should "not be able to instantiate without objectKey" do lambda { Gluestick::Object.new }.should raise_error + + object = Gluestick::Object.new("movies/slumdog_millionaire/danny_boyle") + object.should be_instance_of(Gluestick::Object) + end + + context "object instantiated" do + context "with a valid key" do + setup do + stub_get("/object/users?objectId=movies%2Fslumdog_millionaire%2Fdanny_boyle", "object/get.xml") + @object = Gluestick::Object.get("movies/slumdog_millionaire/danny_boyle") + end + + should "be a glue object" do + @object.should be_glue_object + @object.should be_instance_of(Gluestick::MovieObject) + end + end + + context "with a bad key" do + setup do + stub_get("/object/get?objectId=blah", "errors/invalid_object.xml") + @object = Gluestick::Object.new("blah") + end + + should "throw an invalid object error when getting a lazy loaded attribute" do + lambda { @object.title }.should raise_error(Gluestick::InvalidObject) + end + end + + context "with a URL not Glue Object" do + setup do + stub_get("/object/get?objectId=http%3A%2F%2Fwww.cnn.com%2F", "object/bookmark.xml") + @object = Gluestick::Object.get("http://www.cnn.com/") + end + + should "be a bookmark object" do + @object.should be_instance_of(Gluestick::BookmarkObject) + end + + should "not be a glue object" do + @object.should_not be_glue_object + end + end end should "be able to generate objects from factories" do test/gluestick/object_test.rb @@ -2,9 +2,12 @@ require 'test_helper' class GluestickTest < Test::Unit::TestCase +should "respond to client attributes" do + Gluestick.should respond_to(:client, :login, :username) +end + context "client" do should "have a client" do - Gluestick.should respond_to(:client) Gluestick.client.should_not be_nil end @@ -13,9 +16,30 @@ context "client" do end end -context "login" do - should "respond to login" do - Gluestick.should respond_to(:login) +context "username" do + context "logged in" do + setup do + stub_login + end + + should "have a username" do + Gluestick.username.should_not be_nil + Gluestick.username.should == "username" + end + end + + context "not logged in" do + setup do + Gluestick.logout if Gluestick.logged_in? + end + + teardown do + stub_login if not Gluestick.logged_in? + end + + should "not have a username" do + Gluestick.username.should be_nil + end end end test/gluestick_test.rb a992121536e7eb32ce305ad697cf41ef196e6ddf bdotdub bdotdub@gmail.com http://github.com/bdotdub/gluby/commit/01c943ad9a0ecc4839a8797a230f38624a8adb3e 01c943ad9a0ecc4839a8797a230f38624a8adb3e 2009-05-30T14:20:55-07:00 2009-05-30T14:20:55-07:00 added bookmark objects for urls that aren't glue objects. added Object.new, though I'm not sure I'm happy with it. added logout and username accessor for Gluestick bea5366ae9baabb595f43df97561bca634518011 bdotdub bdotdub@gmail.com