need to update extractor... not just supported version list to support

safari 5
jlong · Jun 17, 2010 · e19924c · e19924c
1 parent 74cfe67
commit e19924c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/lib/uaid/extractor.rb b/lib/uaid/extractor.rb
@@ -84,6 +84,7 @@ def version_extractions(product)
           'safari' => [
             ['3', /Version\/3/],
             ['4', /Version\/4/],
+            ['5', /Version\/5/],
           ],
           'mobilesafari' => [
             ['3', /Version\/3/],

diff --git a/spec/extractor_spec.rb b/spec/extractor_spec.rb
@@ -15,6 +15,8 @@
       "Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US) AppleWebKit/525.19 (KHTML, like Gecko) Chrome/1.0.154.36 Safari/525.19"],
     ["webkit", "safari", "4"] => [
       "Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10_5_6; en-us) AppleWebKit/528.16 (KHTML, like Gecko) Version/4.0 Safari/528.16"],
+    ["webkit", "safari", "5"] => [
+      "Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10_6_3; en-us) AppleWebKit/533.16 (KHTML, like Gecko) Version/5.0 Safari/533.16"],
     ["gecko", "firefox", "2"] => [
       "Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.8.1.14) Gecko/20080404 Firefox/2.0.0.14",
       "Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1.6) Gecko/20060601 Firefox/2.0.0.6 (Ubuntu-edgy)",