Automatically convert before and after arguments of Strava::Api::Clie…

…nt#athlete_activities from Time to Integer.

CHANGELOG.md

@@ -1,5 +1,6 @@
 ### 0.3.2 (Next)
 
+* Automatically convert `before` and `after` arguments of `Strava::Api::Client#athlete_activities` from `Time` to `Integer` - [@dblock](https://github.com/dblock).
 * Your contribution here.
 
 ### 0.3.1 (2018/12/05)

lib/strava/api/endpoints/activities.rb

@@ -91,6 +91,9 @@ def activity_laps(id_or_options, options = {})
         #   Number of items per page. Defaults to 30.
         #
         def athlete_activities(options = {}, &block)
+          options = options.dup if options.key?(:after) || options.key?(:before)
+          options[:after] = options[:after].to_i if options[:after]
+          options[:before] = options[:before].to_i if options[:before]
           paginate 'athlete/activities', options, Strava::Models::Activity, &block
         end
 

spec/strava/api/client/endpoints/activities/athlete_activities_spec.rb

@@ -2,9 +2,9 @@
 
 RSpec.describe 'Strava::Api::Client#athlete_activities' do
   include_context 'API client'
-  describe '#athlete_activities', vcr: { cassette_name: 'client/athlete_activities' } do
-    let(:athlete_activities) { client.athlete_activities }
-    it 'returns athlete activities' do
+  describe '#athlete_activities' do
+    it 'returns athlete activities', vcr: { cassette_name: 'client/athlete_activities' } do
+      athlete_activities = client.athlete_activities
       expect(athlete_activities).to be_a Enumerable
       expect(athlete_activities.count).to eq 30
       activity = athlete_activities.first
@@ -13,6 +13,11 @@
       expect(activity.map).to be_a Strava::Models::Map
       expect(activity.start_date).to be_a Time
     end
+    it 'returns athlete activities for December 2018 only', vcr: { cassette_name: 'client/athlete_activities_december_2018' } do
+      athlete_activities = client.athlete_activities(before: Time.parse('2019/1/1'), after: Time.parse('2018/11/1'))
+      expect(athlete_activities).to be_a Enumerable
+      expect(athlete_activities.count).to eq 8
+    end
   end
   describe 'paginated #athlete_activities', vcr: { cassette_name: 'client/all_athlete_activities' } do
     let(:athlete_activities) do