Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3940)

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/physicalweb/PwsClientImplTest.java

Issue 2721083002: Move PwsClientImplTest to junit (Closed)
Patch Set: Update commit message Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/javatests/src/org/chromium/chrome/browser/physicalweb/PwsClientImplTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/physicalweb/PwsClientImplTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/physicalweb/PwsClientImplTest.java
deleted file mode 100644
index 22e97da16791576b3500aa316e6da3bef5481379..0000000000000000000000000000000000000000
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/physicalweb/PwsClientImplTest.java
+++ /dev/null
@@ -1,163 +0,0 @@
-// Copyright 2016 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-package org.chromium.chrome.browser.physicalweb;
-
-import android.content.Context;
-import android.support.test.filters.SmallTest;
-import android.test.InstrumentationTestCase;
-import android.text.TextUtils;
-
-import org.chromium.base.LocaleUtils;
-
-import java.util.Locale;
-
-/**
- * Tests for the PwsClientImpl class.
- */
-public class PwsClientImplTest extends InstrumentationTestCase {
- private PwsClientImpl mPwsClientImpl;
-
- @Override
- protected void setUp() throws Exception {
- super.setUp();
- Context context = getInstrumentation().getTargetContext().getApplicationContext();
- mPwsClientImpl = new PwsClientImpl(context);
- }
-
- @SmallTest
- public void testUserAgentNonEmpty() {
- assertFalse(TextUtils.isEmpty(mPwsClientImpl.getUserAgent()));
- }
-
- @SmallTest
- public void testLanguageTagIsIncludedInAcceptLanguageHeader() {
- String defaultLocaleString = LocaleUtils.getDefaultLocaleString();
- String[] languageTags = defaultLocaleString.split(",");
-
- // Ensure Accept-Language contains the full language tag.
- String acceptLanguage = mPwsClientImpl.updateAcceptLanguage();
- for (String tag : languageTags) {
- assertTrue(acceptLanguage.contains(tag));
- // Ensure Accept-Language also contains the language code by itself.
- String languageCode;
- if (tag.length() == 2 || tag.length() == 3) {
- languageCode = tag;
- } else if (tag.charAt(2) == '-') {
- languageCode = tag.substring(0, 2);
- } else { // length of the language code is 3.
- languageCode = tag.substring(0, 3);
- }
- assertTrue(acceptLanguage.startsWith(languageCode + ",")
- || acceptLanguage.contains(languageCode + ";")
- || acceptLanguage.equals(languageCode));
- }
- }
-
- @SmallTest
- public void testLanguageTagIsPrepended() {
- Locale locale = new Locale("en", "GB");
- String defaultLocale = LocaleUtils.toLanguageTag(locale);
- String languageList = "fr-CA,fr-FR,fr";
-
- // Should prepend the language tag "en-GB" as well as the language code "en".
- String languageListWithTag =
- PwsClientImpl.prependToAcceptLanguagesIfNecessary(defaultLocale, languageList);
- assertEquals("en-GB,en,fr-CA,fr-FR,fr", languageListWithTag);
- }
-
- @SmallTest
- public void testLanguageOnlyTagIsPrepended() {
- Locale locale = new Locale("mas");
- String defaultLocale = LocaleUtils.toLanguageTag(locale);
- String languageList = "fr-CA,fr-FR,fr";
-
- // Should prepend the language code only language tag "aaa".
- String languageListWithTag =
- PwsClientImpl.prependToAcceptLanguagesIfNecessary(defaultLocale, languageList);
- assertEquals("mas,fr-CA,fr-FR,fr", languageListWithTag);
- }
-
- @SmallTest
- public void testSpecialLengthCountryCodeIsPrepended() {
- Locale locale = new Locale("es", "005");
- String defaultLocale = LocaleUtils.toLanguageTag(locale);
- String languageList = "fr-CA,fr-FR,fr";
-
- // Should prepend the language tag "aa-AAA" as well as the language code "aa".
- String languageListWithTag =
- PwsClientImpl.prependToAcceptLanguagesIfNecessary(defaultLocale, languageList);
- assertEquals("es-005,es,fr-CA,fr-FR,fr", languageListWithTag);
- }
-
- @SmallTest
- public void testMultipleLanguageTagIsPrepended() {
- String locale = "jp-JP,is-IS";
- String languageList = "en-US,en";
-
- // Should prepend the language tag "aa-AA" as well as the language code "aa".
- String languageListWithTag = PwsClientImpl.prependToAcceptLanguagesIfNecessary(locale,
- languageList);
- assertEquals("jp-JP,jp,is-IS,is,en-US,en", languageListWithTag);
-
- // Make sure the language code is only inserted after the last languageTag that
- // contains that language.
- locale = "jp-JP,fr-CA,fr-FR";
- languageListWithTag =
- PwsClientImpl.prependToAcceptLanguagesIfNecessary(locale, languageList);
- assertEquals("jp-JP,jp,fr-CA,fr-FR,fr,en-US,en", languageListWithTag);
- }
-
- @SmallTest
- public void testLanguageTagIsPrependedWhenListContainsLanguageCode() {
- Locale locale = new Locale("fr", "FR");
- String defaultLocale = LocaleUtils.toLanguageTag(locale);
- String languageList = "fr-CA,fr";
-
- // Should prepend the language tag "xx-XX" but not the language code "xx" as it's already
- // included at the end of the list.
- String languageListWithTag =
- PwsClientImpl.prependToAcceptLanguagesIfNecessary(defaultLocale, languageList);
- assertEquals("fr-FR,fr-CA,fr", languageListWithTag);
- }
-
- @SmallTest
- public void testLanguageTagNotPrependedWhenUnnecessary() {
- Locale locale = new Locale("fr", "CA");
- String defaultLocale = LocaleUtils.toLanguageTag(locale);
- String languageList = "fr-CA,fr-FR,fr";
-
- // Language list should be unmodified since the tag is already present.
- String languageListWithTag =
- PwsClientImpl.prependToAcceptLanguagesIfNecessary(defaultLocale, languageList);
- assertEquals(languageList, languageListWithTag);
- }
-
- @SmallTest
- public void testMultiLanguageTagNotPrependedWhenUnnecessary() {
- String locale = "fr-FR,is-IS";
- String languageList = "fr-FR,is-IS,fr,is";
-
- // Language list should be unmodified since the tag is already present. However, the order
- // changes because a language-code-only language tag is acceptable now.
- String languageListWithTag = PwsClientImpl.prependToAcceptLanguagesIfNecessary(locale,
- languageList);
- assertEquals(languageList, languageListWithTag);
- }
-
- @SmallTest
- public void testAcceptLanguageQvalues() {
- String languageList = "en-US,en-GB,en,jp-JP,jp";
-
- // Should insert q-values for each item except the first which implicitly has q=1.0.
- String acceptLanguage = PwsClientImpl.generateAcceptLanguageHeader(languageList);
- assertEquals("en-US,en-GB;q=0.8,en;q=0.6,jp-JP;q=0.4,jp;q=0.2", acceptLanguage);
-
- // When there are six or more items, the q-value should not go below 0.2.
- languageList = "mas,es,en,jp,ch,fr";
- acceptLanguage = PwsClientImpl.generateAcceptLanguageHeader(languageList);
- assertEquals("mas,es;q=0.8,en;q=0.6,jp;q=0.4,ch;q=0.2,fr;q=0.2",
- acceptLanguage);
- }
-}

Powered by Google App Engine
This is Rietveld 408576698