diff options
Diffstat (limited to 'chromium/components/payments/core/payments_validators_unittest.cc')
| -rw-r--r-- | chromium/components/payments/core/payments_validators_unittest.cc | 45 |
1 files changed, 12 insertions, 33 deletions
diff --git a/chromium/components/payments/core/payments_validators_unittest.cc b/chromium/components/payments/core/payments_validators_unittest.cc index 4e459b92878..c10085b6564 100644 --- a/chromium/components/payments/core/payments_validators_unittest.cc +++ b/chromium/components/payments/core/payments_validators_unittest.cc @@ -11,14 +11,11 @@ namespace payments { namespace { struct CurrencyCodeTestCase { - CurrencyCodeTestCase(const char* code, - const char* system, - bool expected_valid) - : code(code), system(system), expected_valid(expected_valid) {} + CurrencyCodeTestCase(const char* code, bool expected_valid) + : code(code), expected_valid(expected_valid) {} ~CurrencyCodeTestCase() {} const char* code; - const char* system; bool expected_valid; }; @@ -33,25 +30,17 @@ const char* longString2048() { return long_string; } -const char* longString2049() { - static char long_string[2050]; - for (int i = 0; i < 2049; i++) - long_string[i] = 'a'; - long_string[2049] = '\0'; - return long_string; -} - TEST_P(PaymentsCurrencyValidatorTest, IsValidCurrencyCodeFormat) { std::string error_message; EXPECT_EQ(GetParam().expected_valid, payments::PaymentsValidators::IsValidCurrencyCodeFormat( - GetParam().code, GetParam().system, &error_message)) + GetParam().code, &error_message)) << error_message; EXPECT_EQ(GetParam().expected_valid, error_message.empty()) << error_message; EXPECT_EQ(GetParam().expected_valid, payments::PaymentsValidators::IsValidCurrencyCodeFormat( - GetParam().code, GetParam().system, nullptr)); + GetParam().code, nullptr)); } INSTANTIATE_TEST_CASE_P( @@ -66,24 +55,14 @@ INSTANTIATE_TEST_CASE_P( // by [[ISO4217]], however any string of at most 2048 // characters is considered valid in other currencySystem. Returns false // if currency |code| is too long (greater than 2048). - CurrencyCodeTestCase("USD", "urn:iso:std:iso:4217", true), - CurrencyCodeTestCase("US1", "http://www.example.com", true), - CurrencyCodeTestCase("US1", "urn:iso:std:iso:4217", false), - CurrencyCodeTestCase("US", "http://www.example.com", true), - CurrencyCodeTestCase("US", "urn:iso:std:iso:4217", false), - CurrencyCodeTestCase("USDO", "http://www.example.com", true), - CurrencyCodeTestCase("USDO", "urn:iso:std:iso:4217", false), - CurrencyCodeTestCase("usd", "http://www.example.com", true), - CurrencyCodeTestCase("usd", "urn:iso:std:iso:4217", false), - CurrencyCodeTestCase("ANYSTRING", "http://www.example.com", true), - CurrencyCodeTestCase("ANYSTRING", "urn:iso:std:iso:4217", false), - CurrencyCodeTestCase("", "http://www.example.com", true), - CurrencyCodeTestCase("", "urn:iso:std:iso:4217", false), - CurrencyCodeTestCase(longString2048(), "http://www.example.com", true), - CurrencyCodeTestCase(longString2048(), "urn:iso:std:iso:4217", false), - CurrencyCodeTestCase(longString2049(), - "http://www.example.com", - false))); + CurrencyCodeTestCase("USD", true), + CurrencyCodeTestCase("US1", false), + CurrencyCodeTestCase("US", false), + CurrencyCodeTestCase("USDO", false), + CurrencyCodeTestCase("usd", false), + CurrencyCodeTestCase("ANYSTRING", false), + CurrencyCodeTestCase("", false), + CurrencyCodeTestCase(longString2048(), false))); struct TestCase { TestCase(const char* input, bool expected_valid) |