VK::Client#offline?, some fixes and docs

lib/vkontakte_api/client.rb

@@ -44,7 +44,7 @@ def initialize(token = nil)
         # token is an OAuth2::AccessToken
         @token      = token.token
         @user_id    = token.params['user_id']
-        @expires_at = Time.at(token.expires_at)
+        @expires_at = Time.at(token.expires_at) unless token.expires_at.nil?
       else
         # token is a String or nil
         @token = token
@@ -56,10 +56,18 @@ def authorized?
       !@token.nil?
     end
 
+    # Did the token already expire.
     def expired?
-      @expires_at < Time.now
+      @expires_at && @expires_at < Time.now
     end
 
+    # Is the token permanent.
+    def offline?
+      @expires_at.nil?
+    end
+
+    # Access rights of this token.
+    # @return [Array] An array of symbols representing the access rights.
     def scope
       SCOPE.reject do |access_scope, mask|
         (settings & mask).zero?

spec/vkontakte_api/client_spec.rb

@@ -41,6 +41,20 @@
     end
   end
 
+  describe "#authorized?" do
+    context "with an unauthorized client" do
+      it "returns false" do
+        VkontakteApi::Client.new.should_not be_authorized
+      end
+    end
+
+    context "with an authorized client" do
+      it "returns true" do
+        VkontakteApi::Client.new(@string_token).should be_authorized
+      end
+    end
+  end
+
   describe "#expired?" do
     context "with an expired token" do
       before(:each) do
@@ -62,18 +76,38 @@
         @client.should_not be_expired
       end
     end
+
+    context "with a String token" do
+      before(:each) do
+        @client = VkontakteApi::Client.new(@string_token)
+      end
+
+      it "returns false" do
+        @client.should_not be_expired
+      end
+    end
   end
 
-  describe "#authorized?" do
-    context "with an unauthorized client" do
+  describe "#offline?" do
+    context "with a usual token" do
       it "returns false" do
-        VkontakteApi::Client.new.should_not be_authorized
+        VkontakteApi::Client.new(@oauth2_token).should_not be_offline
       end
     end
 
-    context "with an authorized client" do
+    context "with an offline token" do
+      before(:each) do
+        @oauth2_token.stub(:expires_at).and_return(nil)
+      end
+
       it "returns true" do
-        VkontakteApi::Client.new(@string_token).should be_authorized
+        VkontakteApi::Client.new(@oauth2_token).should be_offline
+      end
+    end
+
+    context "with a String token" do
+      it "returns true" do
+        VkontakteApi::Client.new(@string_token).should be_offline
       end
     end
   end