From 6276cb2e23375c1f2837b34bd9072ef804f996f1 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Micha=C5=82=20Go=C5=82e=CC=A8biowski-Owczarek?= Date: Mon, 23 Sep 2019 20:00:24 +0200 Subject: [PATCH] Tests: Move Android user agent detection above iOS, put Safari last Android was catching the generic iOS regex. Checking for Android first should eliminate that issue as iOS user agents don't contain the word "Android". Putting Safari last makes Android UAs not be tested against it. --- test/unit/support.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/test/unit/support.js b/test/unit/support.js index 2aad3a968..b280e364e 100644 --- a/test/unit/support.js +++ b/test/unit/support.js @@ -294,12 +294,12 @@ testIframe( expected = expectedMap.chrome; } else if ( /\b(?:9|10)\.\d+(\.\d+)* safari/i.test( userAgent ) ) { expected = expectedMap.safari_9_10; - } else if ( /\b\d+(\.\d+)+ safari/i.test( userAgent ) ) { - expected = expectedMap.safari; } else if ( /firefox\/(?:52|60)/i.test( userAgent ) ) { expected = expectedMap.firefox_60; } else if ( /firefox/i.test( userAgent ) ) { expected = expectedMap.firefox; + } else if ( /android 4\.[0-3]/i.test( userAgent ) ) { + expected = expectedMap.android; } else if ( /iphone os (?:9|10)_/i.test( userAgent ) ) { expected = expectedMap.ios_9_10; } else if ( /iphone os 8_/i.test( userAgent ) ) { @@ -308,8 +308,8 @@ testIframe( expected = expectedMap.ios_7; } else if ( /(?:iphone|ipad);.*(?:iphone)? os \d+_/i.test( userAgent ) ) { expected = expectedMap.ios; - } else if ( /android 4\.[0-3]/i.test( userAgent ) ) { - expected = expectedMap.android; + } else if ( /\b\d+(\.\d+)+ safari/i.test( userAgent ) ) { + expected = expectedMap.safari; } QUnit.test( "Verify that support tests resolve as expected per browser", function( assert ) { -- 2.39.5