PTS component

Signed-off-by: Steven Hoffman <fustrate@me.com>

Author: Fustrate

lib/baseballbot/templates/blocks/pick_the_stick.rb

@@ -0,0 +1,51 @@
+# frozen_string_literal: true
+
+class Baseballbot
+  module Templates
+    module Blocks
+      class PickTheStick < Block
+        BASE_URL = 'https://www.pick-the-stick.com/api/standings?api_token=%<token>s&team=%<team_code>s&sort=desc'
+
+        include MarkdownHelpers
+
+        TABLE_HEADERS = [
+          ['Rank', :center],
+          ['User', :center],
+          ['Points', :center],
+          ['Total Picks', :center],
+          ['Position Change', :center]
+        ]
+
+        def render
+          return '[](/pickthestick "Team not configured")' unless @subreddit.team&.code
+
+          table(headers: TABLE_HEADERS, rows:)
+        end
+
+        protected
+
+        def rows
+          pick_the_stick_data.first(10).map do |entry|
+            [
+              entry['ranking'],
+              entry['username'],
+              entry['points'],
+              entry['total_picks'],
+              entry['position_change']
+            ]
+          end
+        end
+
+        def pick_the_stick_data
+          api_token = ENV.fetch('BASEBALLBOT_PTS_TOKEN', nil)
+
+          raise 'API token is required' unless api_token && !api_token.empty?
+
+          raise 'Team code is required' if team_code.nil? || team_code.empty?
+
+          JSON.parse(URI.parse(format(BASE_URL, api_token:, team_code:)).open.read)
+        end
+      end
+    end
+  end
+end

lib/baseballbot/templates/components/pick_the_stick.rb

@@ -4,24 +4,42 @@ class Baseballbot
   module Templates
     module Components
       class PickTheStick
-        BASE_URL = 'https://www.pick-the-stick.com/api/standings?api_token=%<token>s&team=%<team_code>s'
+        BASE_URL = 'https://www.pick-the-stick.com/api/standings?api_token=%<token>s&team=%<team_code>s&sort=desc'
 
         include MarkdownHelpers
 
+        TABLE_HEADERS = [
+          ['Rank', :center],
+          ['User', :center],
+          ['Points', :center],
+          ['Total Picks', :center],
+          ['Position Change', :center]
+        ]
+
         def initialize(subreddit)
           @subreddit = subreddit
         end
 
-        def current_standings
+        def to_s
           return '[](/pickthestick "Team not configured")' unless @subreddit.team&.code
 
-          @data = pick_the_stick_data
-
-          ''
+          table(headers: TABLE_HEADERS, rows:)
         end
 
         protected
 
+        def rows
+          pick_the_stick_data.first(10).map do |entry|
+            [
+              entry['ranking'],
+              entry['username'],
+              entry['points'],
+              entry['total_picks'],
+              entry['position_change']
+            ]
+          end
+        end
+
         def pick_the_stick_data
           api_token = ENV.fetch('BASEBALLBOT_PTS_TOKEN', nil)
 

spec/baseballbot/templates/components/pick_the_stick_spec.rb

@@ -0,0 +1,50 @@
+# frozen_string_literal: true
+
+RSpec.describe Baseballbot::Templates::Components::PickTheStick do
+  let(:bot) { Baseballbot.new(user_agent: 'Baseballbot Tests') }
+  let(:subreddit) do
+    Baseballbot::Subreddit.new(
+      { name: 'dodgers', team_code: 'LAD', team_id: 119, options: {} },
+      bot:,
+      bot_account: Baseballbot::Bot.new(bot:, name: 'RSpecTestBot', access: '')
+    )
+  end
+
+  before { stub_requests! with_response: true }
+
+  describe '#to_s' do
+    it 'returns a fallback markdown link when team is not configured' do
+      no_team_subreddit = Baseballbot::Subreddit.new(
+        { name: 'baseball', team_id: nil, options: {} },
+        bot:,
+        bot_account: Baseballbot::Bot.new(bot:, name: 'RSpecTestBot', access: '')
+      )
+
+      expect(described_class.new(no_team_subreddit).to_s).to eq '[](/pickthestick "Team not configured")'
+    end
+
+    it 'renders a standings table from pick the stick data' do
+      component = described_class.new(subreddit)
+
+      data = (1..11).map do |rank|
+        {
+          'ranking' => rank,
+          'username' => "user#{rank}",
+          'points' => 100 - rank,
+          'total_picks' => 20 + rank,
+          'position_change' => "+#{rank}"
+        }
+      end
+
+      allow(component).to receive(:pick_the_stick_data).and_return(data)
+
+      output = component.to_s
+
+      expect(output).to include('|Rank|User|Points|Total Picks|Position Change|')
+      expect(output).to include('|:-:|:-:|:-:|:-:|:-:|')
+      expect(output).to include('|1|user1|99|21|+1|')
+      expect(output).to include('|10|user10|90|30|+10|')
+      expect(output).not_to include('|11|user11|89|31|+11|')
+    end
+  end
+end