leaderbrag 1.0.0 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7579c259982dedf4121354fd0cdcdd697e2c76ff
-  data.tar.gz: 6c98eebdb490302afcde4ef45289516ad51421e4
+  metadata.gz: a290db98c22b0314fc48551b7d319ea8b3a59817
+  data.tar.gz: e8209617515bc0e5d8341149e2ad9c2e1e5044c9
 SHA512:
-  metadata.gz: 6d7b685b826f17d073c2c923e3790ef1f1ab66c552f50ee928ae1c4c43feec0377ea734063c80395b9b594b2a163d774c50ca7b8711b1f1cf4d7bcda0909daa9
-  data.tar.gz: b30eacd0ffd35c3540d24dd41e7702055405ab06c4c88c1e2ea83cecb860bee89445c5ebdaca81baba3d41a7adf7bdf377e65fb1d02f7c413f1db90e9c7a74dc
+  metadata.gz: cae3372bfd3175481c84b7c900e950cfd194985b6a3266d9f208cc0d2c3814a3db0c1d76fc4218e16e584d55f7357bfe0bfe4b44a9f89a3b811bc2f3e850b69c
+  data.tar.gz: 4cc9b47a6969af72376c5126728bb6f5944e892a8bfc6068008467df75aa144fb667290fa8538216977bbee4f3d22186c1f089b957e9f42b18f442eda79d0dc1

data/.rubocop_todo.yml

@@ -1,6 +1,6 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2019-06-21 22:32:50 -0500 using RuboCop version 0.71.0.
+# on 2019-06-22 22:54:35 -0500 using RuboCop version 0.71.0.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
@@ -12,19 +12,30 @@ Lint/HandleExceptions:
   Exclude:
     - 'Rakefile'
 
-# Offense count: 2
+# Offense count: 6
 Metrics/AbcSize:
-  Max: 44
+  Max: 43
 
 # Offense count: 1
+# Configuration parameters: CountComments.
+Metrics/ClassLength:
+  Max: 139
+
+# Offense count: 3
 Metrics/CyclomaticComplexity:
-  Max: 11
+  Max: 10
 
-# Offense count: 1
+# Offense count: 5
 # Configuration parameters: CountComments, ExcludedMethods.
 Metrics/MethodLength:
-  Max: 38
+  Max: 33
 
-# Offense count: 1
+# Offense count: 3
 Metrics/PerceivedComplexity:
-  Max: 12
+  Max: 10
+
+# Offense count: 1
+# Configuration parameters: MinBodyLength.
+Style/GuardClause:
+  Exclude:
+    - 'lib/leaderbrag/cli.rb'

data/.travis.yml

@@ -2,3 +2,5 @@ language: ruby
 before_install:
   - gem update --system
   - gem install bundler
+services:
+  - redis-server

data/lib/leaderbrag/cli.rb

@@ -13,39 +13,91 @@ module Leaderbrag
     @leader = nil
     def initialize(args = [], local_options = {}, config = {})
       super
+      if ENV['XMLSTATS_CONTACT_INFO'].nil?
+        warn 'Please set XMLSTATS_CONTACT_INFO environment variable.'
+        exit 60
+      elsif ENV['XMLSTATS_API_KEY'].nil?
+        warn 'Please set XMLSTATS_API_KEY environment variable'
+        exit 70
+      end
       @leader = Leaderbrag::Leader.new
     end
 
     desc 'board', 'Lists all baseball teams with their standings'
+    method_option(:sort_by_league, aliases: '-l',
+                                   desc: 'Sort by league.', default: false,
+                                   type: :boolean)
+    method_option(:only_league, aliases: '-L', type: :string,
+                                desc:
+                           'Only show results for this league.',
+                                enum: %w[AL NL])
+    method_option(:sort_by_division, aliases: '-d',
+                                     desc: 'Sort by league and division.'\
+                                     ' Supercedes -l.',
+                                     type: :boolean)
+    method_option(:only_division, aliases: '-D', type: :string,
+                                  desc:
+                          'Only show results for this division.',
+                                  enum: %w[E C W])
     def board
       puts 'Team Name'.ljust(23) +
-           'Team ID               Rank  Win%  League  Div'
-      puts '-'.ljust(68, '-')
-      @leader.standings.each_with_index do |team, i|
+           'Team ID               Win%  Rank  League Rank  Div Rank'
+      puts '-'.ljust(78, '-')
+      teams = @leader.filter(options[:sort_by_league],
+                             options[:sort_by_division],
+                             options[:only_league],
+                             options[:only_division])
+      teams.each do |team|
         puts "#{team.first_name} #{team.last_name}:".ljust(23) +
-             team.team_id.ljust(22) + (i + 1).to_s.ljust(6) +
-             team.win_percentage.ljust(6) +
-             team.conference.ljust(8) + team.division
+             team.team_id.ljust(22) + team.win_percentage.ljust(7) +
+             @leader.overall_rank(team).to_s.ljust(7) +
+             team.conference.ljust(6) +
+             @leader.league_rank(team).to_s.ljust(6) +
+             team.division.ljust(4) + team.rank.to_s
       end
     end
 
     desc 'find', 'Finds the best team in baseball'
     method_option(:quiet, aliases: '-q',
-                          desc: 'Do not print results', default: false)
+                          desc: 'Do not print results.', default: false)
     method_option(:stats, aliases: '-s',
-                          desc: 'Include team stats in output',
+                          desc: 'Include team stats in output.',
                           default: false)
+    method_option(:league, aliases: '-l', type: :string,
+                           desc: 'Find the league leader rather than overall'\
+                           ' best team.',
+                           enum: %w[AL NL])
+    method_option(:division, aliases: '-d', type: :string,
+                             desc: 'Find the division leader rather than '\
+                             'overall best team. Requires use of --league.',
+                             enum: %w[E C W])
     def find
-      myteam = @leader.standings[0]
+      if !options[:division].nil? && options[:league].nil?
+        warn 'League must be specified.'
+        exit 80
+      end
+      s_league = (options[:league].nil? ? nil : options[:league])
+      s_division = (options[:division].nil? ? nil : options[:division])
+      teams = @leader.filter(!options[:league].nil?, !options[:division].nil?,
+                             s_league, s_division)
+      myteam = teams[0]
       brag(myteam, options[:quiet], options[:stats])
     end
 
     desc 'is TEAM', 'asserts that TEAM is the best team in baseball'
-    method_option(:quiet, aliases: '-q',
+    method_option(:quiet, aliases: '-q', type: :boolean,
                           desc: 'Do not print results', default: false)
-    method_option(:stats, aliases: '-s',
+    method_option(:stats, aliases: '-s', type: :boolean,
                           desc: 'Include team stats in output',
                           default: false)
+    method_option(:league, aliases: '-l', type: :boolean,
+                           desc: 'Check leadership of team\'s league. '\
+                           'Only affects exit code.',
+                           default: false)
+    method_option(:division, aliases: '-d', type: :boolean,
+                             desc: 'Check leadership of team\'s division.'\
+                             'Only affects the exit code.',
+                             default: false)
     def is?(team_id)
       myteam = @leader.team(team_id)
       if myteam.nil?
@@ -53,6 +105,13 @@ module Leaderbrag
         exit 50
       end
       brag(myteam, options[:quiet], options[:stats])
+      if @leader.overall_leader?(myteam) ||
+         (options[:league] && @leader.league_leader?(myteam)) ||
+         (options[:division] && @leader.division_leader?(myteam))
+        exit 0
+      else
+        exit myteam.rank
+      end
     end
 
     private
@@ -93,11 +152,6 @@ module Leaderbrag
           puts label.ljust(30) + myteam.send(field).to_s
         end
       end
-      if @leader.overall_leader?(myteam)
-        exit 0
-      else
-        exit myteam.rank
-      end
     end
   end
 end

data/lib/leaderbrag/leader.rb

@@ -66,5 +66,30 @@ module Leaderbrag
     def overall_rank(team)
       @standings.find_index(team) + 1
     end
+
+    def filter(sb_league = false, sb_division = false,
+               s_conference = nil, s_division = nil)
+      view = standings.sort_by do |team|
+        sort_items = []
+        sort_items << team.conference if sb_league
+        sort_items << team.conference if sb_division
+        sort_items << team.division if sb_division
+        sort_items << league_rank(team) if sb_league
+        sort_items << team.rank if sb_division
+        sort_items << overall_rank(team) if !sb_league && !sb_division
+        sort_items
+      end
+      unless s_conference.nil?
+        view.select! do |team|
+          team.conference == s_conference
+        end
+      end
+      unless s_division.nil?
+        view.select! do |team|
+          team.division == s_division
+        end
+      end
+      view
+    end
   end
 end

data/lib/leaderbrag/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Leaderbrag
-  VERSION = '1.0.0'
+  VERSION = '1.1.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: leaderbrag
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.1.0
 platform: ruby
 authors:
 - David Schlenk
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-06-22 00:00:00.000000000 Z
+date: 2019-06-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: redis