reve 0.0.130 → 0.0.131

data/Rakefile

@@ -23,7 +23,7 @@ if File.exists?('.git')
       s.rubyforge_project = "reve"
       s.author = "Lisa Seelye"
       s.email = "lisa@thedoh.com"
-      s.homepage = "http://revetrac.crudvision.com"
+      s.homepage = "http://github.com/lisa/reve"
       s.platform = Gem::Platform::RUBY
       s.summary = "Reve is a Ruby library to interface with the Eve Online API"
       s.files = FileList["Rakefile","LICENSE", "lib/**/*.rb","reve.rb","tester.rb","init.rb"].to_a

data/lib/reve.rb

@@ -112,7 +112,7 @@ module Reve
 
     attr_accessor :key, :userid, :charid
     attr_accessor :http_user_agent, :save_path
-    attr_reader :current_time, :cached_until, :last_hash
+    attr_reader :current_time, :cached_until, :last_hash, :reve_version
     
     # Create a new API instance.
     # current_time and cached_until are meaningful only for the LAST call made.
@@ -127,13 +127,14 @@ module Reve
       @charid = (charid || "").to_s
       
       @save_path = nil
-
-      @http_user_agent = "Reve"
+      
       @max_tries = 3
 
       @current_time = nil
       @cached_until = nil
       @last_hash = nil
+      @reve_version = File.read(File.join(File.dirname(__FILE__),'../','VERSION'))
+      @http_user_agent = "Reve v#{@reve_version}; http://github.com/lisa/reve"
     end
     # Save XML to this directory with the format:
     # :save_path/:userid/:method/:expires_at_in_unixtime.xml
@@ -1030,7 +1031,7 @@ module Reve
         opts.merge({ :version => 2, :url => nil }) #the uri bit will now ignored in format_url_request
         req_args =  format_url_request(opts)
         req = Net::HTTP::Get.new(source.path + req_args)
-        req['User-Agent'] = @http_referer_agent || "Reve"
+        req['User-Agent'] = @http_referer_agent || "Reve v#{@reve_version}; http://github.com/lisa/reve"
         
         res = nil
         response = nil

data/test/test_reve.rb

@@ -1252,11 +1252,14 @@ class TestReve < Test::Unit::TestCase
     assert_equal real,time
   end
   
-  protected
-  def get_api(userid = nil, apikey = nil, charid = nil)
-    api = Reve::API.new(userid, apikey, charid)
-    api.save_path = nil
-    api
+  # It's useful to know the version and we'll stick it in the user agent
+  # now as well.
+  def test_reve_version
+    # Path to Reve version is ../VERSION. We rely on File.read here and in the
+    # class so it's kind of crummy.
+    version = File.read(File.join(File.dirname(__FILE__),'../','VERSION'))
+    assert_equal(@api.reve_version, version)
+    assert_equal("Reve v#{version}; http://github.com/lisa/reve", @api.http_user_agent)
   end
   
   # no need to test corporate cos they're the same.
@@ -1308,4 +1311,11 @@ class TestReve < Test::Unit::TestCase
     end
   end
 
+  #### All tests above this method.
+  protected
+  def get_api(userid = nil, apikey = nil, charid = nil)
+    api = Reve::API.new(userid, apikey, charid)
+    api.save_path = nil
+    api
+  end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: reve
 version: !ruby/object:Gem::Version 
-  hash: 283
+  hash: 281
   prerelease: false
   segments: 
   - 0
   - 0
-  - 130
-  version: 0.0.130
+  - 131
+  version: 0.0.131
 platform: ruby
 authors: 
 - Lisa Seelye
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-08-21 00:00:00 -04:00
+date: 2010-08-22 00:00:00 -04:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -175,7 +175,7 @@ files:
 - test/xml/wallet_balance.xml
 - test/xml/wallet_journal.xml
 has_rdoc: true
-homepage: http://revetrac.crudvision.com
+homepage: http://github.com/lisa/reve
 licenses: []
 
 post_install_message: