Merge pull request #61 from Fullscreen/funky-metrics-216

Funky metrics 216

CHANGELOG.md

@@ -6,6 +6,10 @@ For more information about changelogs, check
 [Keep a Changelog](http://keepachangelog.com) and
 [Vandamme](http://tech-angels.github.io/vandamme).
 
+## 0.2.8 - 2016/12/12
+
+* [BUGFIX] Change the way that "like counts" are parsed now that Facebook has changed the structure of the HTML and can include multiple "likecounts" elements in the same page.
+
 ## 0.2.7 - 2016/08/18
 
 * [ENHANCEMENT] When Funky makes a request to an HTML page, Funky would now raise `Funky::ConnectionError` should the following errors occur:

lib/funky/html/parser.rb

@@ -23,7 +23,7 @@ def extract_views_from(html)
       end
 
       def extract_likes_from(html)
-        html.match /"likecount":(.*?),/
+        html.match(/"likecount":(\d+),"likecountreduced"/)
         matched_count $1
       end
 

lib/funky/version.rb

@@ -1,3 +1,3 @@
 module Funky
-  VERSION = "0.2.7"
+  VERSION = "0.2.8"
 end

spec/videos/video_spec.rb

@@ -61,6 +61,12 @@
       include_examples 'check id and counters'
     end
 
+    context 'given an existing video ID of a video with more than 800 likes was passed' do
+      let(:video_id) { existing_video_id }
+
+      it { expect(video.like_count).to be > 800 }
+    end
+
     context 'given a non-existing video ID was passed' do
       let(:video_id) { unknown_video_id }