Fix #34: season incorrectly reports no seasons after reset.

CHANGELOG.md

@@ -1,5 +1,6 @@
 ### Changelog
 
+* [#34](https://github.com/dblock/slack-gamebot/issues/34): Fix: `season` incorrectly reports no seasons after reset - [@dblock](https://github.com/dblock).
 * Fix: correctly handle `user_change` event - [@dblock](https://github.com/dblock).
 * [#29](https://github.com/dblock/slack-gamebot/issues/29): Fix: `season` incorrectly reports number of players - [@dblock](https://github.com/dblock).
 * [#24](https://github.com/dblock/slack-gamebot/issues/24): Record game scores - [@dblock](https://github.com/dblock).

slack-gamebot/commands/season.rb

@@ -2,8 +2,10 @@ module SlackGamebot
   module Commands
     class Season < SlackRubyBot::Commands::Base
       def self.call(client, data, _match)
-        current_season = ::Season.new
-        if current_season.valid?
+        if ::Season.count > 0 && !::Challenge.current.any?
+          send_message_with_gif client, data.channel, 'No matches have been recorded.', 'history'
+        elsif ::Challenge.current.any?
+          current_season = ::Season.new
           send_message client, data.channel, current_season.to_s
         else
           send_message_with_gif client, data.channel, "There're no seasons.", %w(winter summer fall spring).sample

spec/slack-gamebot/commands/season_spec.rb

@@ -11,9 +11,17 @@
     before do
       2.times.map { Fabricate(:match) }
     end
-    it 'returns past seasons and current season' do
+    it 'returns current season' do
       current_season = Season.new
       expect(message: "#{SlackRubyBot.config.user} season").to respond_with_slack_message current_season.to_s
     end
+    context 'after reset' do
+      before do
+        ::Season.create!(created_by: User.first)
+      end
+      it 'returns current season' do
+        expect(message: "#{SlackRubyBot.config.user} season").to respond_with_slack_message 'No matches have been recorded.'
+      end
+    end
   end
 end