they-yahoo_stock 1.0.8

data/spec/yahoo_stock/history_spec.rb

@@ -0,0 +1,75 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe YahooStock::History do
+  
+  describe ".new" do
+    
+    it "should raise error when no parameter is passed during object initialization" do
+      lambda { YahooStock::History.new 
+        }.should raise_error(ArgumentError)
+    end
+    
+    it "should raise error when the parameter is not a hash" do
+      lambda { YahooStock::History.new(nil) 
+        }.should raise_error(YahooStock::History::HistoryError, 'A hash of start_date, end_date and stock_symbol is expected as parameters')
+    end
+    
+    it "should inititalize the history interface" do
+      options = {}
+      YahooStock::Interface::History.should_receive(:new).with(options)
+      YahooStock::History.new({})
+    end
+    
+  end
+  
+  describe "find" do
+    before(:each) do
+      @interface = stub('YahooStock::Interface::History')
+      YahooStock::Interface::History.stub!(:new).and_return(@interface)
+      @interface.stub!(:values)
+      @history = YahooStock::History.new(:stock_symbol => 'a symbol', :start_date => Date.today-3, :end_date => Date.today-1)
+    end
+    
+    it "should find values using interface" do
+      @interface.should_receive(:values).and_return('data')
+      @history.find
+    end
+    
+    it "should remove the first line from the data as it is just the headings" do
+      data = 'data returned'
+      @interface.stub!(:values).and_return(data)
+      data.should_receive(:sub).with(/Date.*\s/,'')
+      @history.find
+    end
+  end
+  
+  describe "data_attributes" do
+    before(:each) do
+      @history = YahooStock::History.new(:stock_symbol => 'a symbol', :start_date => Date.today-3, :end_date => Date.today-1)
+      @data = "Date, price, etc \n23, 34, 44\n\n222,234,2"
+      @history.stub!(:values_with_header).and_return(@data)
+    end
+    
+    it "should return nil if history values are not present" do
+      @history.stub!(:values_with_header)
+      @history.data_attributes.should eql(nil)
+    end
+    
+    it "should return only header values if history values are present" do
+      @history.data_attributes.should eql(["Date","price","etc"])
+    end
+    
+  end
+  
+  describe "values_with_header" do
+    
+    it "should get values from the interface" do
+      @interface = stub('YahooStock::Interface::History')
+      YahooStock::Interface::History.stub!(:new).and_return(@interface)
+      @history = YahooStock::History.new(:stock_symbol => 'a symbol', :start_date => Date.today-3, :end_date => Date.today-1)
+      @interface.should_receive(:values)
+      @history.values_with_header
+    end
+  end
+  
+end

data/spec/yahoo_stock/interface/history_spec.rb

@@ -0,0 +1,317 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
+
+describe YahooStock::Interface::History do
+  
+  describe ".new" do
+    
+    it "should raise error when no parameter is passed during object initialization" do
+      lambda { YahooStock::Interface::History.new 
+        }.should raise_error(ArgumentError)
+    end
+    
+    it "should raise an error if stock symbol key is not present in parameter hash" do
+      lambda { YahooStock::Interface::History.new(:start_date => Date.today-7) 
+        }.should raise_error(YahooStock::Interface::History::HistoryError, ':stock_symbol key is not present in the parameter hash')
+    end
+    
+    it "should raise error when stock_symbol value in hash is nil" do
+      lambda { YahooStock::Interface::History.new(:stock_symbol => nil)
+         }.should raise_error(YahooStock::Interface::History::HistoryError, ':stock_symbol value cannot be nil or blank')
+    end
+    
+    it "should raise error when stock_symbol value in hash is blank" do
+      lambda { YahooStock::Interface::History.new(:stock_symbol => '')
+         }.should raise_error(YahooStock::Interface::History::HistoryError, ':stock_symbol value cannot be nil or blank')
+    end
+    
+    it "should raise error when start date key is not present in parameter hash" do
+      lambda { YahooStock::Interface::History.new(:stock_symbol => 'd') }.should raise_error(YahooStock::Interface::History::HistoryError, ':start_date key is not present in the parameter hash')
+    end
+    
+    it "should raise error when start_date value is nil" do
+      lambda { YahooStock::Interface::History.new(:stock_symbol => 'd', :start_date => nil) }.should raise_error(YahooStock::Interface::History::HistoryError, ':start_date value cannot be blank')
+    end
+    
+    it "should raise error when start_date value is blank" do
+      lambda { YahooStock::Interface::History.new(:stock_symbol => 'd', :start_date => '') }.should raise_error(YahooStock::Interface::History::HistoryError, ':start_date value cannot be blank')
+    end
+    
+    it "should raise error when start date value is not of type date" do
+      lambda { YahooStock::Interface::History.new(:stock_symbol => 'd', :start_date => '23/3/2009') }.should raise_error(YahooStock::Interface::History::HistoryError, 'Start date must be of type Date')
+    end
+    
+    it "should raise error when end date key is not present in parameter hash" do
+      lambda { YahooStock::Interface::History.new(:stock_symbol => 'd', :start_date => Date.today-1) }.should raise_error(YahooStock::Interface::History::HistoryError, ':end_date key is not present in the parameter hash')
+    end
+    
+    it "should raise error when end_date value is nil" do
+      lambda { YahooStock::Interface::History.new(:stock_symbol => 'd', 
+                                                  :start_date => Date.today-1, 
+                                                  :end_date => nil) }.should raise_error(YahooStock::Interface::History::HistoryError, ':end_date value cannot be blank')
+    end
+    
+    it "should raise error when end_date value is blank" do
+      lambda { YahooStock::Interface::History.new(:stock_symbol => 'd', 
+                                                  :start_date => Date.today-1, 
+                                                  :end_date => '') }.should raise_error(YahooStock::Interface::History::HistoryError, ':end_date value cannot be blank')
+    end
+    
+    it "should raise error when end date value is not of type date" do
+      lambda { YahooStock::Interface::History.new(:stock_symbol => 'd', 
+                                                  :start_date => Date.today-1, 
+                                                  :end_date => 'sds') }.should raise_error(YahooStock::Interface::History::HistoryError, 'End date must be of type Date')
+    end
+    
+    it "should raise error when start date is not less than today" do
+      lambda { YahooStock::Interface::History.new(:stock_symbol => 'd', 
+                                                  :start_date => Date.today, 
+                                                  :end_date => Date.today) 
+        }.should raise_error(YahooStock::Interface::History::HistoryError, 'Start date must be in the past')
+    end
+    
+    it "should raise error when start date is greater than end date" do
+      lambda { YahooStock::Interface::History.new(:stock_symbol => 'd', 
+                                                  :start_date => Date.today-7, 
+                                                  :end_date => Date.today-8) 
+        }.should raise_error(YahooStock::Interface::History::HistoryError, 'End date must be greater than the start date')
+    end
+    
+    it "should not raise error when start date is in the past, end date is greater than start date and all relevant keys are present with right type" do
+      lambda { YahooStock::Interface::History.new(:stock_symbol => 'd', 
+                                                  :start_date => Date.today-7, 
+                                                  :end_date => Date.today-1) 
+        }.should_not raise_error
+      
+    end
+    
+    it "should by default set type to daily if no value provided for it" do
+      @history = YahooStock::Interface::History.new(:stock_symbol => 'd', 
+                                                    :start_date => Date.today-7, 
+                                                    :end_date => Date.today-1)
+      @history.type.should eql(:daily)
+    end
+    
+    it "should raise invalid keys error when an invalid key is passed in the parameter" do
+      lambda { YahooStock::Interface::History.new(:stock_symbol => 'd', 
+                                                  :start_date => Date.today-7, 
+                                                  :end_date => Date.today-1, :boom => 1) }.should raise_error(YahooStock::Interface::History::HistoryError, "An invalid key 'boom' is passed in the parameters. Allowed keys are stock_symbol, start_date, end_date, type")
+    end
+    
+    it "should raise error when type is neither :daily, :weekly, :monthly or :dividend" do
+      lambda { YahooStock::Interface::History.new(:stock_symbol => 'd', 
+                                                  :start_date => Date.today-7, 
+                                                  :end_date => Date.today-1,
+                                                  :type => :yearly)
+                                        }.should raise_error(YahooStock::Interface::History::HistoryError, "Allowed values for type are daily, weekly, monthly, dividend")
+      
+    end
+    
+    it "should not raise error when type is :daily" do
+      lambda { YahooStock::Interface::History.new(:stock_symbol => 'd', 
+                                                  :start_date => Date.today-7, 
+                                                  :end_date => Date.today-1,
+                                                  :type => :daily)
+                                        }.should_not raise_error
+      
+    end
+  end
+  
+  describe "setters" do
+    before(:each) do
+      @history = YahooStock::Interface::History.new(:stock_symbol => 'd', 
+                                                    :start_date => Date.today-7, 
+                                                    :end_date => Date.today-1)
+    end
+    
+    it "should raise error when stock symbol is being set to nil" do
+      lambda { @history.stock_symbol = nil 
+        }.should raise_error(YahooStock::Interface::History::HistoryError, ':stock_symbol value cannot be nil or blank')
+    end
+    
+    it "should raise error when stock symbol is being set to an empty string" do
+      lambda { @history.stock_symbol = '' 
+        }.should raise_error(YahooStock::Interface::History::HistoryError, ':stock_symbol value cannot be nil or blank')
+    end
+    
+    it "should set the new stock symbol value when it is valid" do
+      @history.stock_symbol = 'new_val'
+      @history.stock_symbol.should eql('new_val') 
+    end
+    
+    it "should raise error when start date type is not a date" do
+      lambda { @history.start_date = '21/3/2009' 
+        }.should raise_error(YahooStock::Interface::History::HistoryError, 'Start date must be of type Date')
+    end
+    
+    it "should raise error when start date is not in the past" do
+      lambda { @history.start_date = Date.today
+        }.should raise_error(YahooStock::Interface::History::HistoryError, 'Start date must be in the past')
+    end
+    
+    it "should set the start date when start date is valid" do
+      s_date = Date.today-1
+      @history.start_date = s_date
+      @history.start_date.should eql(s_date)
+    end
+    
+    it "should raise error when end date type is not a date" do
+      lambda { @history.end_date = '21/3/2009' 
+        }.should raise_error(YahooStock::Interface::History::HistoryError, 'End date must be of type Date')
+    end
+    
+    it "should raise error when end date type is not in the past" do
+      lambda { @history.end_date = Date.today
+        }.should raise_error(YahooStock::Interface::History::HistoryError, 'End date must be in the past')
+    end
+    
+    it "should raise error when end date type is not in the past" do
+      e_date = Date.today-1
+      @history.end_date = e_date
+      @history.end_date.should eql(e_date)
+    end
+    
+    it "should raise error when settin type other than :daily, :monthly or :weekly" do
+      lambda { @history.type = :yearly }.should raise_error(YahooStock::Interface::History::HistoryError)
+    end
+    
+    it "should set the type" do
+      @history.type = :daily
+      @history.type.should eql(:daily)
+    end
+    
+    it "should set the type default to daily if not set" do
+      @history.type.should eql(:daily)
+    end
+  end
+  
+  describe "uri" do
+    before(:each) do
+      @history = YahooStock::Interface::History.new(:stock_symbol => 'symbol', 
+                                                    :start_date => Date.today-7, 
+                                                    :end_date => Date.today-1)
+    end
+    
+    it "should get the type to generate url" do
+      @history.should_receive(:type)
+      @history.uri
+    end
+    
+    it "should get the day of the start date" do
+      @history.start_date.should_receive(:day)
+      @history.uri
+    end
+    
+    it "should get the day of the end date" do
+      @history.end_date.should_receive(:day)
+      @history.uri
+    end
+    
+    it "should get the year of the start date" do
+      @history.start_date.should_receive(:year)
+      @history.uri
+    end
+    
+    it "should get the day of the end date" do
+      @history.start_date.should_receive(:year)
+      @history.uri
+    end
+    
+    it "should get the month of the start and end date" do
+      @history.should_receive(:sprintf).exactly(2)
+      @history.uri
+    end
+    
+    it "should have the base url" do
+      @history.uri.should =~ /http:\/\/ichart.finance.yahoo.com\/table.csv?/
+    end
+    
+    it "should have parameter 'a' with month -1 " do
+      month = sprintf("%02d", @history.start_date.month-1)
+      @history.uri.should =~ /a=#{month}/
+    end
+    
+    it "should have parameter 'b' with start date day " do
+      @history.uri.should =~ /b=#{@history.start_date.day}/
+    end
+    
+    it "should have parameter 'c' with start date year " do
+      @history.uri.should =~ /c=#{@history.start_date.year}/
+    end
+    
+    it "should have parameter 'd' with end date month -1 " do
+      month = sprintf("%02d", @history.end_date.month-1)
+      @history.uri.should =~ /d=#{month}/
+    end
+    
+    it "should have parameter 'e' with end date day " do
+      @history.uri.should =~ /e=#{@history.end_date.day}/
+    end
+    
+    it "should have parameter 'f' with end date year " do
+      @history.uri.should =~ /c=#{@history.end_date.year}/
+    end
+    
+    it "should have parameter 'g' with type" do
+      @history.uri.should =~ /g=d/
+    end
+    
+    it "should have the parameter 's' with stock symbol" do
+      @history.uri.should =~ /s=symbol/
+    end
+    
+  end
+  
+  describe "get" do
+    before(:each) do
+      @history = YahooStock::Interface::History.new(:stock_symbol => 'd', 
+                                                    :start_date => Date.today-7, 
+                                                    :end_date => Date.today-1)
+      response = stub('HTTP')
+      Net::HTTP.stub!(:get_response).and_return(response)
+      response.stub!(:code).and_return('200')
+      response.stub!(:body)
+    end
+    
+    it "should check the stock symbol" do
+      @history.should_receive(:stock_symbol).times.at_least(2).and_return('symbol')
+      @history.get
+    end
+    
+    it "should check the end date" do
+      @history.should_receive(:end_date).times.at_least(1).and_return(Date.today-1)
+      @history.get
+    end
+    
+    it "should check the start date" do
+      @history.should_receive(:start_date).times.at_least(1).and_return(Date.today-7)
+      @history.get
+    end
+    
+    it "should raise error if start date is not present" do
+      @history.stub!(:start_date)
+      lambda { @history.get }.should raise_error('Cannot send request unless all parameters, ie stock_symbol, start and end date are present')
+    end
+    
+    it "should raise error if end date is not present" do
+      @history.stub!(:end_date)
+      lambda { @history.get }.should raise_error('Cannot send request unless all parameters, ie stock_symbol, start and end date are present')
+    end
+    
+    it "should raise error if stock symbol is not present" do
+      @history.stub!(:stock_symbol)
+      lambda { @history.get }.should raise_error('Cannot send request unless all parameters, ie stock_symbol, start and end date are present')
+    end
+    
+    it "should send the http request to yahoo" do
+      Net::HTTP.should_receive(:get_response)
+      @history.get
+    end
+    
+    it "should parse the url" do
+      URI.should_receive(:parse)
+      @history.get
+    end
+    
+  end
+  
+end

data/spec/yahoo_stock/interface/quote_spec.rb

@@ -0,0 +1,414 @@
+require File.dirname(__FILE__) + '/../../spec_helper'
+
+describe YahooStock::Interface::Quote::Quote do
+  
+  before(:each) do
+    @interface = YahooStock::Interface::Quote.new(:stock_symbols => ['sym1'], :read_parameters => [:param1])
+    @allowed_parameters = [:param1, :param2, :param3, :zxy, :xyz, :abc, :def]
+  end
+  
+  describe ".new" do
+    
+    it "should add new parameters after current parametr and sort the url parameters" do
+      @interface.stub!(:allowed_parameters).and_return(@allowed_parameters)
+      @interface.add_parameters :zxy, :xyz
+      @interface.yahoo_url_parameters.should eql([:param1, :xyz, :zxy])
+    end
+    
+    it "should sort the url parameters" do
+      @interface.stub!(:allowed_parameters).and_return(@allowed_parameters)
+      @interface.add_parameters :def, :abc
+      @interface.yahoo_url_parameters.should eql([:abc, :def, :param1])
+    end
+    
+    it "should raise InterfaceError when stock params hash is nil" do
+      lambda { YahooStock::Interface::Quote.new(nil)
+       }.should raise_error(YahooStock::Interface::Quote::QuoteError, 'You must pass a hash of stock symbols and the data you would like to see')
+    end
+    
+    it "should raise ArgumentError when no parameter is passed" do
+      lambda { YahooStock::Interface::Quote.new }.should raise_error(ArgumentError)  
+    end
+    
+    it "should raise InterfaceError when stock symbols hash is nil" do
+      lambda { YahooStock::Interface::Quote.new(:stock_symbols => nil) }.should raise_error(YahooStock::Interface::Quote::QuoteError, 'No stocks passed')
+    end
+    
+    it "should raise InterfaceError when stock symbols parameter hash is an array of zero elements" do
+      lambda { YahooStock::Interface::Quote.new(:stock_symbols => []) }.should raise_error(YahooStock::Interface::Quote::QuoteError, 'No stocks passed')
+    end
+    
+    it "should raise InterfaceError when read parameters hash is nil" do
+      lambda { YahooStock::Interface::Quote.new(:stock_symbols => 'sym', :read_parameters => nil) }.should raise_error(YahooStock::Interface::Quote::QuoteError,'Read parameters are not provided')
+    end
+    
+    it "should raise InterfaceError when read parameters hash is a zero element array" do
+      lambda { YahooStock::Interface::Quote.new(:stock_symbols => 'sym', :read_parameters => []) }.should raise_error(YahooStock::Interface::Quote::QuoteError,'Read parameters are not provided')
+    end
+    
+    it "should assign appropriate values to the stock symbols accessor" do
+      interface = YahooStock::Interface::Quote.new(:stock_symbols => ['sym'], :read_parameters => [:param1])
+      interface.stock_symbols.should eql(['sym'])
+    end
+    
+    it "should assign appropriate values to the yahoo url parameters accessor" do
+      interface = YahooStock::Interface::Quote.new(:stock_symbols => ['sym'], :read_parameters => [:param1])
+      interface.yahoo_url_parameters.should eql([:param1])
+    end
+    
+    it "should have the base url" do
+      interface = YahooStock::Interface::Quote.new(:stock_symbols => ['sym'], :read_parameters => [:param1])
+      interface.base_url = 'http://download.finance.yahoo.com/d/quotes.csv'
+    end
+    
+    it "should add the self as an observer to reset the values if any symbol or parameter is modified" do
+      interface = YahooStock::Interface::Quote.new(:stock_symbols => ['sym'], :read_parameters => [:param1])
+      interface.count_observers.should eql(1)
+    end
+    
+    it "should not have zero observer" do
+      interface = YahooStock::Interface::Quote.new(:stock_symbols => ['sym'], :read_parameters => [:param1])
+      interface.count_observers.should_not eql(0)
+    end
+  end
+  
+  describe "scrip_symbols=" do
+    
+    it "should add the new symbol to the existing stock symbol array" do
+      @interface.stock_symbols = 'sym2'
+      @interface.stock_symbols.should include('sym2','sym1')
+    end
+    
+    it "should not add duplicate symbols more than once" do
+      @interface.stock_symbols = 'sym2'
+      @interface.stock_symbols = 'sym2'
+      @interface.stock_symbols.size.should eql(2)
+    end
+    
+    it "should not run observers if symbols havent change" do
+      @interface.stock_symbols = 'sym1'
+      @interface.should_not_receive(:run_observers)
+    end
+    
+    it "should run observers if stock symbols have changed" do
+      @interface.stock_symbols.should eql(['sym1'])
+      @interface.should_receive(:run_observers)
+      @interface.stock_symbols = 'sym2'
+      @interface.stock_symbols.should include('sym2','sym1')
+    end
+  end
+  
+  describe "yahoo_url_parameters=" do
+    before(:each) do
+      @interface.stub!(:allowed_parameters).and_return(@allowed_parameters)
+    end
+    it "should raise error when an invalid parameter is passed" do
+      lambda { 
+        @interface.yahoo_url_parameters = :random_param 
+        }.should raise_error(YahooStock::Interface::Quote::QuoteError, "Interface parameter random_param is not a valid parameter.")
+    end
+    
+    it "should add the new param to the existing params array" do
+      @interface.yahoo_url_parameters = :param2
+      @interface.yahoo_url_parameters.should include(:param1, :param2)
+    end
+    
+    it "should not add duplicate params more than once" do
+      @interface.yahoo_url_parameters = :param1
+      @interface.yahoo_url_parameters = :param1
+      @interface.yahoo_url_parameters.size.should eql(1)
+    end
+    
+    it "should not run observers if parameters havent change" do
+      @interface.yahoo_url_parameters = :param1
+      @interface.should_not_receive(:run_observers)
+    end
+    
+    it "should run observers if stock symbols have changed" do
+      @interface.yahoo_url_parameters.should eql([:param1])
+      @interface.should_receive(:run_observers)
+      @interface.yahoo_url_parameters = :param2
+      @interface.yahoo_url_parameters.should include(:param1, :param2)
+    end
+    
+  end
+  
+  describe "uri" do
+    before(:each) do
+      @interface.stub!(:yahoo_url_parameters).and_return([:param1])
+      @interface.stub!(:parameters).and_return({:param1 => 's', :param2 => 'w', :param3 => 't'})
+    end
+    
+    it "should raise InterfaceError if the passed parameter is not present in the parameter list" do
+      interface = YahooStock::Interface::Quote.new(:stock_symbols => ['sym'], :read_parameters => [:param4, :param5])
+      lambda { interface.uri }.should raise_error(YahooStock::Interface::Quote::QuoteError, "The parameters 'param4, param5' are not valid. Please check using YahooStock::Interface::Quote#allowed_parameters or YahooStock::Quote#valid_parameters")
+    end
+    
+    it "should not raise any error if parameters passed are correct" do
+      lambda { @interface.uri }.should_not raise_error
+    end
+    
+    it "should raise error if stock symbols accessor is cleared and empty" do
+      @interface.stock_symbols.clear
+      lambda { @interface.uri }.should raise_error(YahooStock::Interface::Quote::QuoteError, "You must add atleast one stock symbol to get stock data")
+    end
+    
+    it "should raise error if url parameters are cleared and empty" do
+      @interface.yahoo_url_parameters.clear
+      lambda { @interface.uri }.should raise_error(YahooStock::Interface::Quote::QuoteError, "You must add atleast one parameter to get stock data")
+    end
+    
+    it "should create the uri with base url" do
+      @interface.uri.should =~ /http:\/\/download.finance.yahoo.com\/d\/quotes.csv?/
+    end
+    
+    it "should create the uri with stock symbol parameter" do
+      @interface.uri.should =~ /s=sym1/
+    end
+    
+    it "should create the uri with parameter code" do
+      @interface.uri.should =~ /f=s/
+    end
+    
+    it "should have an & to join symbols and data parameter" do
+      @interface.uri.should =~ /&/
+    end
+    
+    it "should create the uri with 3 symbols" do
+      @interface.stub!(:stock_symbols).and_return(['sym1','sym2', 'sym3'])
+      @interface.uri.should  =~ /s=sym1\+sym2\+sym3/
+    end
+    
+    it "should create the uri with 3 parameters" do
+      @interface.stub!(:yahoo_url_parameters).and_return([:param1,:param2, :param3])
+      @interface.uri.should  =~ /f=swt/
+    end
+    
+  end
+  
+  describe "get" do
+    before(:each) do
+      @response = stub('HTTP Response')
+      @response.stub!(:code).and_return('200')
+      @response.stub!(:body)
+      URI.stub!(:parse)
+      Net::HTTP.stub!(:get_response).and_return(@response)
+      @interface.stub!(:uri)
+    end
+    
+    it "should use uri to get the content" do
+      @interface.should_receive(:uri).at_least(2)
+      @interface.get
+    end
+    
+    it "should get response for the uri" do
+      Net::HTTP.should_receive(:get_response).and_return(@response)
+      @interface.get
+    end
+    
+    it "should parse the uri" do
+      URI.should_receive(:parse)
+      @interface.get
+    end
+    
+    it "should check the response code" do
+      @response.should_receive(:code)
+      @interface.get
+    end
+    
+    it "should get the body of the response if returned code is 200, ie success" do
+      @response.stub!(:code).and_return('200')
+      @response.should_receive(:body)
+      @interface.get
+    end
+    
+  end
+  
+  describe "remove_parameters" do
+    
+    it "should remove the parameters if they are present" do
+      params = [:test1, :test2, :test3]
+      @interface = YahooStock::Interface::Quote.new(:stock_symbols => ['sym1'], :read_parameters => params)
+      @interface.remove_parameters(:test1, :test2)
+      @interface.yahoo_url_parameters.should eql([:test3])
+    end
+    
+    it "should raise an error when removing a parameter that is not present" do
+      lambda { @interface.remove_parameters(:test1) }.should raise_error(YahooStock::Interface::Quote::QuoteError, "Parameter test1 is not present in current list")
+    end
+  end
+  
+  describe "add_parameters" do
+    
+    it "should raise an error when the parameter is not a valid parameter" do
+      lambda { @interface.add_parameters('test1') }.should raise_error(YahooStock::Interface::Quote::QuoteError, "Interface parameter test1 is not a valid parameter.")
+    end
+    
+    it "should add the parameter if its a valid parameter" do
+      params = [:test1, :test2, :test3]
+      @interface.stub!(:allowed_parameters).and_return(params)
+      @interface.add_parameters(:test1, :test2)
+      @interface.yahoo_url_parameters.should include(:test1, :test2)
+    end
+    
+    it "should not add the parameter more than once in the parameter list and silently ignore it" do
+      params = [:test1, :test2, :test3]
+      @interface.stub!(:allowed_parameters).and_return(params)
+      @interface.add_parameters(:test1, :test2)
+      @interface.yahoo_url_parameters.should include(:test1, :test2)
+      @interface.yahoo_url_parameters.size.should eql(3)
+      @interface.add_parameters(:test1, :test2)
+      @interface.yahoo_url_parameters.should include(:test1, :test2)
+      @interface.yahoo_url_parameters.size.should eql(3)
+    end
+  end
+  
+  describe "remove_symbols" do
+    
+    it "should remove the symbols if they are present" do
+      symbols = ['test1', 'test2', 'test3']
+      @interface.stub!(:stock_symbols).and_return(symbols)
+      @interface.remove_symbols('test1', 'test2')
+      @interface.stock_symbols.should eql(['test3'])
+    end
+    
+    it "should raise an error when removing a symbol that is not present" do
+      lambda { @interface.remove_symbols('test1') }.should raise_error(YahooStock::Interface::Quote::QuoteError, "Cannot remove stock symbol test1 as it is currently not present.")
+    end
+  end
+  
+  describe "add_symbols" do
+    
+    it "should add the symbol" do
+      symbols = ['test1', 'test2', 'test3']
+      @interface.stub!(:stock_symbols).and_return(symbols)
+      @interface.add_symbols('test1', 'test2')
+      @interface.stock_symbols.should include('test1', 'test2')
+    end
+    
+    it "should not add the symbol more than once in the symbol list and silently ignore the duplicate ones" do
+      @interface.add_symbols('test1', 'test2')
+      @interface.stock_symbols.should include('test1', 'test2')
+      @interface.stock_symbols.size.should eql(3)
+      @interface.add_symbols('test1', 'test2')
+      @interface.stock_symbols.should include('test1', 'test2')
+      @interface.stock_symbols.size.should eql(3)
+    end
+  end
+   
+  describe "allowed_parameters" do
+    
+    it "should find all parameters" do
+      @interface.should_receive(:parameters).and_return({})
+      @interface.allowed_parameters
+    end
+    
+    it "should return the keys of all parameters" do
+      parameter_hash = {}
+      @interface.stub!(:parameters).and_return(parameter_hash)
+      parameter_hash.should_receive(:keys)
+      @interface.allowed_parameters
+    end
+    
+  end
+  
+  describe "add_standard_params" do
+    
+    it "should get the keys for standard parameters" do
+      YahooStock::Interface::Quote::STD_PARAMETERS.should_receive(:keys).and_return([])
+      @interface.add_standard_params
+    end
+    
+    it "should add each parameter" do
+      keys = [:a_key, :b_key]
+      YahooStock::Interface::Quote::STD_PARAMETERS.stub!(:keys).and_return(keys)
+      @interface.should_receive(:add_parameters).with(:a_key)
+      @interface.should_receive(:add_parameters).with(:b_key)
+      @interface.add_standard_params
+    end
+    
+    it "should add grouped parameters" do
+      @interface.should_receive(:add_grouped_parameters)
+      @interface.add_extended_params
+    end
+   
+  end
+  
+  describe "add_extended_params" do
+    
+    it "should get the keys for extended parameters" do
+      YahooStock::Interface::Quote::EXTENDED_PARAMETERS.should_receive(:keys).and_return([])
+      @interface.add_extended_params
+    end
+    
+    it "should add each parameter" do
+      keys = [:a_key, :b_key]
+      YahooStock::Interface::Quote::EXTENDED_PARAMETERS.stub!(:keys).and_return(keys)
+      @interface.should_receive(:add_parameters).with(:a_key)
+      @interface.should_receive(:add_parameters).with(:b_key)
+      @interface.add_extended_params
+    end
+    
+    it "should add grouped parameters" do
+      @interface.should_receive(:add_grouped_parameters)
+      @interface.add_extended_params
+    end
+   
+  end
+  
+  describe "add_realtime_params" do
+    
+    it "should get the keys for realtime parameters" do
+      YahooStock::Interface::Quote::REALTIME_PARAMETERS.should_receive(:keys).and_return([])
+      @interface.add_realtime_params
+    end
+    
+    it "should add each parameter" do
+      keys = [:a_key, :b_key]
+      YahooStock::Interface::Quote::REALTIME_PARAMETERS.stub!(:keys).and_return(keys)
+      @interface.should_receive(:add_parameters).with(:a_key)
+      @interface.should_receive(:add_parameters).with(:b_key)
+      @interface.add_realtime_params
+    end
+    
+    it "should add grouped parameters" do
+      @interface.should_receive(:add_grouped_parameters)
+      @interface.add_extended_params
+    end
+   
+  end
+  
+  describe "clear_symbols" do
+    it "should get all stock symbols" do
+      @interface.should_receive(:stock_symbols).and_return([])
+      @interface.clear_symbols
+    end
+
+    it "should clear all stock symbols" do
+      sym = []
+      @interface.stub!(:stock_symbols).and_return(sym)
+      sym.should_receive(:clear)
+      @interface.clear_symbols
+    end
+
+    it "should run the observers" do
+      @interface.should_receive(:run_observers)
+      @interface.clear_symbols
+    end
+  end
+
+  describe "clear_parameters" do
+    
+    it "should clear all yahoo url parameters" do
+      @interface.yahoo_url_parameters.should_not eql([])
+      @interface.clear_parameters
+      @interface.yahoo_url_parameters.should eql([])
+    end
+    
+    it "should run the observers" do
+      @interface.should_receive(:run_observers)
+      @interface.clear_parameters
+    end
+  end
+  
+end