Merge pull request #198 from joaocv3/fix-vimeo-specs

Fix vimeo specs
thibaudgg · Sep 28, 2020 · eba0e8c · eba0e8c
2 parents 67e76e0 + 81a3aa1
commit eba0e8c
Show file tree

Hide file tree

Showing 77 changed files with 4,199 additions and 3,743 deletions.
diff --git a/lib/video_info/providers/vimeo_scraper.rb b/lib/video_info/providers/vimeo_scraper.rb
@@ -122,12 +122,22 @@ def thumbnail_large
 
       def view_count
         if available?
-          json_info['interactionCount']
+          user_interaction_count(interaction_type: "http://schema.org/WatchAction")
         end
       end
 
       private
 
+      def user_interaction_count(interaction_type:)
+        interaction_statistic.find do |stat|
+          stat["interactionType"] == interaction_type
+        end["userInteractionCount"]
+      end
+
+      def interaction_statistic
+        json_info["interactionStatistic"]
+      end
+
       def json_info
         @json_info ||= JSON.parse(data.css('script').detect do |n|
           type = n.attr('type')

diff --git a/...eoInfo_Providers_Vimeo/_available_/with_password_required_video/_available_/1_2_3_1_1.yml b/...eoInfo_Providers_Vimeo/_available_/with_password_required_video/_available_/1_2_3_1_1.yml
diff --git a/...eoInfo_Providers_Vimeo/_available_/with_password_required_video/_available_/2_2_3_1_1.yml b/...eoInfo_Providers_Vimeo/_available_/with_password_required_video/_available_/2_2_3_1_1.yml
diff --git a/...roviders_Vimeo/_available_/with_this_video_does_not_exist_video/_available_/1_2_2_1_1.yml b/...roviders_Vimeo/_available_/with_this_video_does_not_exist_video/_available_/1_2_2_1_1.yml
diff --git a/...roviders_Vimeo/_available_/with_this_video_does_not_exist_video/_available_/2_2_2_1_1.yml b/...roviders_Vimeo/_available_/with_this_video_does_not_exist_video/_available_/2_2_2_1_1.yml
diff --git a/...assettes/VideoInfo_Providers_Vimeo/_available_/with_valid_video/_available_/1_2_1_1_1.yml b/...assettes/VideoInfo_Providers_Vimeo/_available_/with_valid_video/_available_/1_2_1_1_1.yml
diff --git a/...assettes/VideoInfo_Providers_Vimeo/_available_/with_valid_video/_available_/2_2_1_1_1.yml b/...assettes/VideoInfo_Providers_Vimeo/_available_/with_valid_video/_available_/2_2_1_1_1.yml