Changeset 5980 for sites/trunk/api.wordpress.org/public_html/core/browse-happy/1.0/tests/phpunit/tests/browse-happy.php

Timestamp:

09/29/2017 07:20:28 PM (3 years ago)

Author:

coffee2code

Message:

Browse Happy API: Improve handling of unknown browsers.

File:

• sites/trunk/api.wordpress.org/public_html/core/browse-happy/1.0/tests/phpunit/tests/browse-happy.php (modified) (2 diffs)

sites/trunk/api.wordpress.org/public_html/core/browse-happy/1.0/tests/phpunit/tests/browse-happy.php

@@ -450 +450 @@
             ],
 
+            // Unknown
+
+            [
+                'Dalvik/1.6.0 (Linux; U; Android 4.1.1; BroadSign Xpress 1.0.14 B- (720) Build/JRO03H)',
+                'Android unknown',
+            ],
+            [ // on Galaxy SIII
+                'Dalvik/1.6.0 (Linux; U; Android 4.4.2; SCH-I535 Build/KOT49H)',
+                'Android unknown',
+            ],
+            [
+                'Mozilla/5.0 (iPhone; CPU iPhone OS 9_3_2 like Mac OS X) AppleWebKit/601.1.46 (KHTML, like Gecko) Mobile/13F69 [FBAN/FBIOS;FBAV/59.0.0.51.142;FBBV/33266808;FBRV/0;FBDV/iPhone7,1;FBMD/iPhone;FBSN/iPhone OS;FBSV/9.3.2;FBSS/3;FBCR/Telkomsel;FBID/phone;FBLC/en_US;FBOP/5] evaliant',
+                'iPhone unknown', // Actually: iPhone Facebook App 59.0.0.51.142
+            ],
+            [
+                'Nokia-MIT-Browser/3.0',
+                'unknown', // It's really a Nokia Browser, but not critical to recognize as such.
+            ],
+
         ];
     }
@@ -463 +482 @@
         $result = $parsed['platform'] . ' ' . $parsed['name'] . ' ' . $parsed['version'];
 
-        $this->assertEquals( $expected, $result );
+        $this->assertEquals( $expected, trim( $result ) );
     }