travis 1.5.6.travis.297.4 → 1.5.6.travis.299.4

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    NmI2MGJjYjg5ZDA0OTVhNWUwMWQ2OTY3YWU1NDM1Mzg4MDI2ZjRhOA==
+    NjA0ZDQ3MjhlOGUzMzY3NmEzY2MxNTI3ZTZhNzg0YTkwM2Q3MjQ3YQ==
   data.tar.gz: !binary |-
-    ZWQ5NmNmYzk2ZDBmMzhiZWM3MjJhNmM0MTFkOTgwM2FhODliYjI3Mg==
+    NTUxNjZiNTcxMjg1Y2VkZWQ3OGRmOWIwM2I2NjdlMTllM2FkZDY3Nw==
 !binary "U0hBNTEy":
   metadata.gz: !binary |-
-    NWRiN2VjNzkyOTI5Y2RhOTY2ODRhNDQ1NjI1YjRlMjlhY2IwZjBhODUxNjVk
-    MWVlNTI1ZTdlMjcxNTAwYWE3YTg2NzFkZWQyMTZkMjMzZWRmYTU4ZTBlYzgy
-    MmY1NDQzMGIzYzU3MmViMWRjY2ZmOTgzZTU0NzU1ZTViNmMyMmQ=
+    ZjNjZTI4MWM4ZGJhNzY0MjFjZDc2ZDA4NmQ1NzliYzFjYjI2OTU0NmQ2OTVj
+    NTIxNTIzNjY2OTIzZmJjZWU1Zjg0MDdiMDcwNTVlMWQ1NjBlYzgxN2Q2ZTYy
+    NTY4ZGM2MWZiNTU3Zjg5ZjZhZGQzZDU3Y2FkZDg2NmQyYTBkNGY=
   data.tar.gz: !binary |-
-    MTNhZGNlNzU5YzE5MjcyMzVhYTNiZDNmYmRhNTZmZWRmOTBmZDQ2NmNhYWQz
-    YTU0NTdhZGE0ZGM2NGVkMzZlOTIwZjJjOGU2NDkxYzYxMmVkYTczYjkyN2I2
-    Mjg2OWFlNDYyNmQ1MzU1Y2JmYmZmMmRmODU3NzgzNTJlZTI4ZTg=
+    ODk1YTBmZWRjMTg2NjI1NGM2ZTI5MzY0Yzg2ZTI4Mjk3ODYzMjRkZjEzYzRi
+    Y2MzNmUzODU5ZDNjN2M5M2Q5MGE0OGExNWUwNzhhMWIzMTcxZjI5NzNiYTEy
+    OTk2YTBmYWEzYjViOGRjMDVkOTIwMmJjODNlMjRhMzY1NTdlMWI=

data/README.md

@@ -897,6 +897,9 @@ Travis::Repository.find('rails/rails')            # find by slug
 Travis::Repository.find(891)                      # find by id
 Travis::Repository.find_all(owner_name: 'rails')  # all repos in the rails organization
 Travis::Repository.current                        # repos that see some action right now
+
+# all repos with the same owner as the repo with id 891
+Travis::Repository.find(891).owner.repositories
 ```
 
 Once you have a repository, you can for instance encrypt some strings with its private key:
@@ -1250,6 +1253,11 @@ If you have the old `travis-cli` gem installed, you should `gem uninstall travis
 
 * Use new API for fetching a single branch for Repository#branch. This also circumvents the 25 branches limit.
 * Start publishing gem prereleases after successful builds.
+* Add `account` method for fetching a single account to `Travis::Client::Methods`.
+* Allow creating account objects for any account, not just these the user is part of. Add `Account#member?` to check for membership.
+* Add `Account#repositories` to load all repos for a given account.
+* Add `Repository#owner_name` and `Repository#owner` to load the account owning a repository.
+* Add `Repository#member?` to check if the current user is a member of a repository.
 
 **1.5.5** (October 2, 2013)
 

data/lib/travis/client/account.rb

@@ -9,10 +9,38 @@ module Travis
       many :accounts
 
       inspect_info :login
+      id_field :login
+
+      def self.cast_id(id)
+        String(id)
+      end
+
+      def self.id?(object)
+        object.is_a? String
+      end
 
       def subscribed
-        attributes.fetch('subscribed') { true }
+        load_attribute('subscribed') { true } if member?
+      end
+
+      def repos_count
+        load_attribute("repos_count") { repositories.count }
       end
+
+      def repositories
+        attributes['repositories'] ||= session.repos(:owner_name => login)
+      end
+
+      def member?
+        session.accounts.include? self
+      end
+
+      private
+
+        def load_attribute(name, &block)
+          session.accounts if missing? name
+          block ? attributes.fetch(name.to_s, &block) : attributes[name.to_s]
+        end
     end
   end
 end

data/lib/travis/client/entity.rb

@@ -18,7 +18,7 @@ module Travis
       end
 
       def self.subclass_for(key)
-        MAP.fetch(key)
+        MAP.fetch(key.to_s)
       end
 
       def self.aka(*names)
@@ -73,6 +73,17 @@ module Travis
         Integer(id)
       end
 
+      def self.id?(object)
+        object.is_a? Integer
+      end
+
+      def self.id_field(key = nil)
+        @id_field = key.to_s if key
+        @id_field || superclass.id_field
+      end
+
+      id_field :id
+
       def initialize(session, id)
         @attributes = {}
         @session    = session

data/lib/travis/client/error.rb

@@ -7,5 +7,8 @@ module Travis
 
     class NotFound < Error
     end
+
+    class NotLoggedIn < NotFound
+    end
   end
 end

data/lib/travis/client/methods.rb

@@ -61,6 +61,12 @@ module Travis
 
       def user
         session.find_one(User)
+      rescue NotFound
+        raise NotLoggedIn, 'currently not logged in'
+      end
+
+      def account(name)
+        session.find_one(Account, name)
       end
 
       def accounts

data/lib/travis/client/repository.rb

@@ -166,6 +166,18 @@ module Travis
         end
       end
 
+      def member?
+        session.user.repositories.include? self
+      end
+
+      def owner_name
+        slug[/^[^\/]+/]
+      end
+
+      def owner
+        session.account(owner_name)
+      end
+
       private
 
         def state

data/lib/travis/client/session.rb

@@ -83,18 +83,18 @@ module Travis
       end
 
       def find_one(entity, id = nil)
-        raise Travis::Error, "cannot fetch #{entity}" unless entity.respond_to?(:many) and entity.many
-        return create_entity(entity, "id" => id) if id.is_a? Integer
+        raise Travis::Client::Error, "cannot fetch #{entity}" unless entity.respond_to?(:many) and entity.many
+        return create_entity(entity, entity.id_field => id) if entity.id? id
         cached(entity, :by, id) { fetch_one(entity, id) }
       end
 
       def find_many(entity, args = {})
-        raise Travis::Error, "cannot fetch #{entity}" unless entity.respond_to?(:many) and entity.many
+        raise Travis::Client::Error, "cannot fetch #{entity}" unless entity.respond_to?(:many) and entity.many
         cached(entity, :many, args) { fetch_many(entity, args) }
       end
 
       def find_one_or_many(entity, args = nil)
-        raise Travis::Error, "cannot fetch #{entity}" unless entity.respond_to?(:many) and entity.many
+        raise Travis::Client::Error, "cannot fetch #{entity}" unless entity.respond_to?(:many) and entity.many
         cached(entity, :one_or_many, args) do
           path       = "/#{entity.many}"
           path, args = "#{path}/#{args}", {} unless args.is_a? Hash
@@ -213,8 +213,8 @@ module Travis
         end
 
         def create_entity(type, data)
-          data   = { "id" => data } if Integer === data or String === data
-          id     = type.cast_id(data.fetch('id'))
+          data   = { type.id_field => data } if type.id? data
+          id     = type.cast_id(data.fetch(type.id_field))
           entity = cached(type, :id, id) { type.new(self, id) }
           entity.update_attributes(data)
           entity

data/lib/travis/client/worker.rb

@@ -16,6 +16,10 @@ module Travis
       one  :worker
       many :workers
 
+      def self.id?(object)
+        object.is_a? String
+      end
+
       def payload=(value)
         set_attribute(:payload, session.load(value))
       end

data/spec/client/account_spec.rb

@@ -1,12 +1,32 @@
 require 'spec_helper'
 
 describe Travis::Client::Account do
-  let(:session) { Travis::Client.new }
-  subject { session.accounts.first }
-  its(:id) { should be == 123 }
-  its(:name) { should be == 'Konstantin Haase' }
-  its(:login) { should be == 'rkh' }
-  its(:type) { should be == 'user' }
-  its(:repos_count) { should be == 200 }
-  its(:inspect) { should be == "#<Travis::Client::Account: rkh>" }
+  context "from all accounts" do
+    let(:session) { Travis::Client.new }
+    subject { session.accounts.first }
+    its(:name) { should be == 'Konstantin Haase' }
+    its(:login) { should be == 'rkh' }
+    its(:type) { should be == 'user' }
+    its(:repos_count) { should be == 200 }
+    its(:inspect) { should be == "#<Travis::Client::Account: rkh>" }
+  end
+
+  context "known account" do
+    let(:session) { Travis::Client.new }
+    subject { session.account('rkh') }
+    its(:name) { should be == 'Konstantin Haase' }
+    its(:login) { should be == 'rkh' }
+    its(:type) { should be == 'user' }
+    its(:repos_count) { should be == 200 }
+    its(:inspect) { should be == "#<Travis::Client::Account: rkh>" }
+  end
+
+  context "known account" do
+    let(:session) { Travis::Client.new }
+    subject { session.account('foo') }
+    its(:name) { should be_nil }
+    its(:login) { should be == 'foo' }
+    its(:type) { should be_nil }
+    its(:inspect) { should be == "#<Travis::Client::Account: foo>" }
+  end
 end

data/spec/client/repository_spec.rb

@@ -1,7 +1,9 @@
 require 'spec_helper'
 
 describe Travis::Client::Repository do
-  subject { Travis::Client.new.repo('rails/rails') }
+  let(:session) { Travis::Client.new }
+  subject { session.repo('rails/rails') }
+
   its(:slug) { should be == 'rails/rails' }
   its(:description) { should_not be_empty }
   its(:last_build_id) { should be == 4125095 }
@@ -15,6 +17,8 @@ describe Travis::Client::Repository do
   its(:last_build) { should be_a(Travis::Client::Build) }
   its(:color) { should be == 'red' }
   its(:github_language) { should be == 'Ruby' }
+  its(:owner_name) { should be == 'rails' }
+  its(:owner) { should be == session.account("rails") }
 
   it { should_not be_pending  }
   it { should     be_started  }

data/spec/support/fake_api.rb

@@ -708,8 +708,7 @@ module Travis
 
         get '/accounts/' do
           {"accounts"=>
-          [{'id' => 123,
-            'name' => "Konstantin Haase",
+          [{'name' => "Konstantin Haase",
             'login' => 'rkh',
             'type' => 'user',
             'repos_count' => 200

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: travis
 version: !ruby/object:Gem::Version
-  version: 1.5.6.travis.297.4
+  version: 1.5.6.travis.299.4
 platform: ruby
 authors:
 - Konstantin Haase
@@ -25,7 +25,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-10-03 00:00:00.000000000 Z
+date: 2013-10-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday