lib/hockeyhelper/client.rb in hockeyhelper-0.0.2 vs lib/hockeyhelper/client.rb in hockeyhelper-0.0.3

@@ -1,59 +1,62 @@
 require_relative 'app'
 require_relative 'networking'
+require_relative 'paging_array'
 
 module Hockey
 
   # HockeyApp API Client for Application
   class Client
 
     #
-    def initialize(token, debug:false)
-      @net = Networking.new token, debug:debug
-      @apps = nil
-      @teams = nil
+    def initialize(token, debug: false, network: nil)
+      @net = network || Networking.new(token, debug:debug)
+      @cached_apps = nil
     end
 
-    # return Array of App objects
-    def apps
-      return @apps if @apps
+    # List all apps for the logged user, including owned apps, developer apps, member apps, and tester apps on HockeyApp.
+    #
+    # @return [Array<App>] an array of {App} objects.
+    # @param page [Fixnum] optional, used for pagination
+    def apps(page: 1)
+      @cached_apps ||= []
 
-      obj = @net.get_object '/api/2/apps'
-
-      @apps = []
-      obj['apps'].each do |hashobj|
-        @apps << App.create_from(hashobj, @net)
+      if @cached_apps.empty?
+        obj = @net.get_object '/api/2/apps'
+        obj['apps'].each do |hashobj|
+          @cached_apps << App.create_from(hashobj, @net)
+        end
       end
 
-      @apps
+      apps = PagingArray.new
+      apps.replace(@cached_apps[(page - 1) * apps.per_page, apps.per_page])
+      apps.update_page_with(page, @cached_apps.size)
+
+      apps
     end
 
-    # return Array of Team objects
-    def teams
-      return @teams if @teams
+    # List all teams for an account.
+    #
+    # @return [Hockey::PagingArray<Team>] an array of {Team} objects
+    # @param page [Fixnum] optional, used for pagination
+    def teams(page: 1)
+      teams = PagingArray.new
 
-      @teams = []
-      page = 1
-
-      while true
-        obj = @net.get_object('/api/2/teams') do |req|
-          req.params[:page] = page
-        end
-        obj['teams'].each do |hashobj|
-          @teams << Team.create_from(hashobj, @net)
-        end
-
-        total = obj['total_pages'].to_i
-        break unless page < total
-
-        page = page + 1
+      obj = @net.get_object('/api/2/teams') do |req|
+        req.params[:page] = page
       end
+      obj['teams'].each do |hashobj|
+        teams << Team.create_from(hashobj, @net)
+      end
 
-      @teams
+      teams.update_page(obj)
+
+      teams
     end
 
-    # create new app on HockeyApp
-    def new_app(title:title, bundle_identifier:bundle_identifier, platform: 'iOS')
+    # Create a new app without uploading a file on HockeyApp.
+    # return an App object.
+    def new_app(title: '', bundle_identifier: '', platform: 'iOS')
       obj = @net.post_object '/api/2/apps/new', {:title=>title, :bundle_identifier=>bundle_identifier, :platform=>platform, :release_type=>0}
 
       app = App.create_from(obj, @net)
 
       app