PaymentRequest Mojo bindings.

third_party/WebKit/Source/modules/payments/PaymentRequestTest.cpp

+// 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.
+
+#include "modules/payments/PaymentRequest.h"
+
+#include "bindings/core/v8/ExceptionState.h"
+#include "bindings/core/v8/ScriptState.h"
+#include "core/dom/ExceptionCode.h"
+#include "core/testing/DummyPageHolder.h"
+#include "modules/payments/MockPaymentRequest.h"
+#include "testing/gtest/include/gtest/gtest.h"
+#include "wtf/OwnPtr.h"
+
+namespace blink {
+namespace {
+
+class PaymentRequestTest : public testing::Test {
+public:
+    PaymentRequestTest()
+        : m_page(DummyPageHolder::create())
+    {
+        setSecurityOrigin("https://www.example.com/");
+    }
+
+    ~PaymentRequestTest() override {}
+
+    ScriptState* getScriptState() { return ScriptState::forMainWorld(m_page->document().frame()); }
+    ExceptionState& getExceptionState() { return m_exceptionState; }
+
+    void setSecurityOrigin(const String& securityOrigin)
+    {
+        m_page->document().setSecurityOrigin(SecurityOrigin::create(KURL(KURL(), securityOrigin)));
+    }
+
+private:
+    OwnPtr<DummyPageHolder> m_page;
+    TrackExceptionState m_exceptionState;
+};
+
+TEST_F(PaymentRequestTest, NoExceptionWithValidData)
+{
+    PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), MockPaymentRequest::buildDetails(), getExceptionState());
+
+    EXPECT_FALSE(getExceptionState().hadException());
+}
+
+TEST_F(PaymentRequestTest, SecureContextRequired)
+{
+    setSecurityOrigin("http://www.example.com");
+
+    PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), MockPaymentRequest::buildDetails(), getExceptionState());
+
+    EXPECT_TRUE(getExceptionState().hadException());
+    EXPECT_EQ(SecurityError, getExceptionState().code());
+}
+
+TEST_F(PaymentRequestTest, SupportedMethodListRequired)
+{
+    PaymentRequest::create(getScriptState(), Vector<String>(), MockPaymentRequest::buildDetails(), getExceptionState());
+
+    EXPECT_TRUE(getExceptionState().hadException());
+    EXPECT_EQ(V8TypeError, getExceptionState().code());
+}
+
+TEST_F(PaymentRequestTest, ItemListRequired)
+{
+    PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), PaymentDetails(), getExceptionState());
+
+    EXPECT_TRUE(getExceptionState().hadException());
+    EXPECT_EQ(V8TypeError, getExceptionState().code());
+}
+
+TEST_F(PaymentRequestTest, AtLeastOnePaymentDetailsItemRequired)
+{
+    PaymentDetails emptyItems;
+    emptyItems.setItems(HeapVector<PaymentItem>());
+
+    PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), emptyItems, getExceptionState());
+
+    EXPECT_TRUE(getExceptionState().hadException());
+    EXPECT_EQ(V8TypeError, getExceptionState().code());
+}
+
+TEST_F(PaymentRequestTest, NullShippingOptionWhenNoOptionsAvailable)
+{
+    PaymentDetails details = MockPaymentRequest::buildDetails();
+    details.setShippingOptions(HeapVector<ShippingOption>());
+
+    PaymentRequest* request = PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), details, getExceptionState());
+
+    EXPECT_TRUE(request->shippingOption().isNull());
+}
+
+TEST_F(PaymentRequestTest, NullShippingOptionWhenMultipleOptionsAvailable)
+{
+    PaymentDetails details = MockPaymentRequest::buildDetails();
+
+    PaymentRequest* request = PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), details, getExceptionState());
+
+    EXPECT_LE(2U, details.shippingOptions().size());

[Marijn Kruisselbrink, 2016/03/25 22:46:25]

This is testing that MockPaymentRequest returns data that is suitable for this
test? I suppose that's fine, but maybe move the check to just after getting the
PaymentDetails creating, to make clear it isn't actually testing anything
related to creating the PaymentRequest.

[please use gerrit instead, 2016/03/29 22:15:44]

Done.

+    EXPECT_TRUE(request->shippingOption().isNull());
+}
+
+TEST_F(PaymentRequestTest, SelectSingleAvailableShippingOption)
+{
+    CurrencyAmount amount;
+    amount.setCurrencyCode("USD");
+    amount.setValue("10.00");
+    ShippingOption option;
+    option.setAmount(amount);
+    option.setId("standard");
+    option.setLabel("Standard shipping");
+    PaymentDetails details = MockPaymentRequest::buildDetails();
+    details.setShippingOptions(HeapVector<ShippingOption>(1, option));
+
+    PaymentRequest* request = PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), details, getExceptionState());
+
+    EXPECT_EQ("standard", request->shippingOption());
+}
+
+TEST_F(PaymentRequestTest, AbortWithoutShowShouldThrow)
+{
+    PaymentRequest* request = PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), MockPaymentRequest::buildDetails(), getExceptionState());
+    EXPECT_FALSE(getExceptionState().hadException());
+
+    request->abort(getExceptionState());
+    EXPECT_TRUE(getExceptionState().hadException());
+}
+
+class DetailsTestCase {
+public:
+    DetailsTestCase(DetailToChange detail, DataToChange data, ModificationType modType, const char* valueToUse, bool expectException = false, ExceptionCode expectedExceptionCode = 0)
+        : m_detail(detail)
+        , m_data(data)
+        , m_modType(modType)
+        , m_valueToUse(valueToUse)
+        , m_expectException(expectException)
+        , m_expectedExceptionCode(expectedExceptionCode)
+    {
+    }
+
+    ~DetailsTestCase() {}
+
+    PaymentDetails buildDetails() const
+    {
+        return MockPaymentRequest::buildDetails(m_detail, m_data, m_modType, m_valueToUse);
+    }
+
+    bool expectException() const
+    {
+        return m_expectException;
+    }
+
+    ExceptionCode getExpectedExceptionCode() const
+    {
+        return m_expectedExceptionCode;
+    }
+
+private:
+    DetailToChange m_detail;
+    DataToChange m_data;
+    ModificationType m_modType;
+    const char* m_valueToUse;
+    bool m_expectException;
+    ExceptionCode m_expectedExceptionCode;
+};
+
+class PaymentRequestDetailsTest : public PaymentRequestTest, public testing::WithParamInterface<DetailsTestCase> {
+};
+
+TEST_P(PaymentRequestDetailsTest, ValidatesDetails)
+{
+    PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), GetParam().buildDetails(), getExceptionState());
+
+    EXPECT_EQ(GetParam().expectException(), getExceptionState().hadException());
+    if (GetParam().expectException())
+        EXPECT_EQ(GetParam().getExpectedExceptionCode(), getExceptionState().code());
+}
+
+INSTANTIATE_TEST_CASE_P(RequiredData,
+    PaymentRequestDetailsTest,
+    testing::Values(
+        DetailsTestCase(DetailItem, DataAmount, RemoveKey, "", true, V8TypeError),
+        DetailsTestCase(DetailItem, DataCurrencyCode, RemoveKey, "", true, V8TypeError),
+        DetailsTestCase(DetailItem, DataId, RemoveKey, "", true, V8TypeError),
+        DetailsTestCase(DetailItem, DataLabel, RemoveKey, "", true, V8TypeError),
+        DetailsTestCase(DetailShippingOption, DataAmount, RemoveKey, "", true, V8TypeError),
+        DetailsTestCase(DetailShippingOption, DataCurrencyCode, RemoveKey, "", true, V8TypeError),
+        DetailsTestCase(DetailShippingOption, DataId, RemoveKey, "", true, V8TypeError),
+        DetailsTestCase(DetailShippingOption, DataLabel, RemoveKey, "", true, V8TypeError)));
+
+INSTANTIATE_TEST_CASE_P(ValidCurrencyCodeFormat,
+    PaymentRequestDetailsTest,
+    testing::Values(
+        DetailsTestCase(DetailItem, DataCurrencyCode, OverwriteValue, "USD"),
+        DetailsTestCase(DetailShippingOption, DataCurrencyCode, OverwriteValue, "USD")));
+
+INSTANTIATE_TEST_CASE_P(InvalidCurrencyCodeFormat,
+    PaymentRequestDetailsTest,
+    testing::Values(
+        DetailsTestCase(DetailItem, DataCurrencyCode, OverwriteValue, "US1", true, V8TypeError),
+        DetailsTestCase(DetailItem, DataCurrencyCode, OverwriteValue, "US", true, V8TypeError),
+        DetailsTestCase(DetailItem, DataCurrencyCode, OverwriteValue, "USDO", true, V8TypeError),
+        DetailsTestCase(DetailItem, DataCurrencyCode, OverwriteValue, "usd", true, V8TypeError),
+        DetailsTestCase(DetailItem, DataCurrencyCode, OverwriteValue, "", true, V8TypeError),
+        DetailsTestCase(DetailShippingOption, DataCurrencyCode, OverwriteValue, "US1", true, V8TypeError),
+        DetailsTestCase(DetailShippingOption, DataCurrencyCode, OverwriteValue, "US", true, V8TypeError),
+        DetailsTestCase(DetailShippingOption, DataCurrencyCode, OverwriteValue, "USDO", true, V8TypeError),
+        DetailsTestCase(DetailShippingOption, DataCurrencyCode, OverwriteValue, "usd", true, V8TypeError),
+        DetailsTestCase(DetailShippingOption, DataCurrencyCode, OverwriteValue, "", true, V8TypeError)));
+
+INSTANTIATE_TEST_CASE_P(ValidAmountFormat,
+    PaymentRequestDetailsTest,
+    testing::Values(
+        DetailsTestCase(DetailItem, DataAmount, OverwriteValue, "0"),
+        DetailsTestCase(DetailItem, DataAmount, OverwriteValue, "-0"),
+        DetailsTestCase(DetailItem, DataAmount, OverwriteValue, "1"),
+        DetailsTestCase(DetailItem, DataAmount, OverwriteValue, "10"),
+        DetailsTestCase(DetailItem, DataAmount, OverwriteValue, "-3"),
+        DetailsTestCase(DetailItem, DataAmount, OverwriteValue, "10.99"),
+        DetailsTestCase(DetailItem, DataAmount, OverwriteValue, "-3.00"),
+        DetailsTestCase(DetailItem, DataAmount, OverwriteValue, "01234567890123456789.0123456789"),
+        DetailsTestCase(DetailItem, DataAmount, OverwriteValue, "01234567890123456789012345678.9"),
+        DetailsTestCase(DetailItem, DataAmount, OverwriteValue, "012345678901234567890123456789"),
+        DetailsTestCase(DetailItem, DataAmount, OverwriteValue, "-01234567890123456789.0123456789"),
+        DetailsTestCase(DetailItem, DataAmount, OverwriteValue, "-01234567890123456789012345678.9"),
+        DetailsTestCase(DetailItem, DataAmount, OverwriteValue, "-012345678901234567890123456789"),
+        DetailsTestCase(DetailShippingOption, DataAmount, OverwriteValue, "0"),
+        DetailsTestCase(DetailShippingOption, DataAmount, OverwriteValue, "-0"),
+        DetailsTestCase(DetailShippingOption, DataAmount, OverwriteValue, "1"),
+        DetailsTestCase(DetailShippingOption, DataAmount, OverwriteValue, "10"),
+        DetailsTestCase(DetailShippingOption, DataAmount, OverwriteValue, "-3"),
+        DetailsTestCase(DetailShippingOption, DataAmount, OverwriteValue, "10.99"),
+        DetailsTestCase(DetailShippingOption, DataAmount, OverwriteValue, "-3.00"),
+        DetailsTestCase(DetailShippingOption, DataAmount, OverwriteValue, "01234567890123456789.0123456789"),
+        DetailsTestCase(DetailShippingOption, DataAmount, OverwriteValue, "01234567890123456789012345678.9"),
+        DetailsTestCase(DetailShippingOption, DataAmount, OverwriteValue, "012345678901234567890123456789"),
+        DetailsTestCase(DetailShippingOption, DataAmount, OverwriteValue, "-01234567890123456789.0123456789"),
+        DetailsTestCase(DetailShippingOption, DataAmount, OverwriteValue, "-01234567890123456789012345678.9"),
+        DetailsTestCase(DetailShippingOption, DataAmount, OverwriteValue, "-012345678901234567890123456789")));
+
+INSTANTIATE_TEST_CASE_P(InvalidAmountFormat,
+    PaymentRequestDetailsTest,
+    testing::Values(
+        DetailsTestCase(DetailItem, DataAmount, OverwriteValue, "", true, V8TypeError),
+        DetailsTestCase(DetailItem, DataAmount, OverwriteValue, "-", true, V8TypeError),
+        DetailsTestCase(DetailItem, DataAmount, OverwriteValue, "notdigits", true, V8TypeError),
+        DetailsTestCase(DetailItem, DataAmount, OverwriteValue, "ALSONOTDIGITS", true, V8TypeError),
+        DetailsTestCase(DetailItem, DataAmount, OverwriteValue, "10.", true, V8TypeError),
+        DetailsTestCase(DetailItem, DataAmount, OverwriteValue, ".99", true, V8TypeError),
+        DetailsTestCase(DetailItem, DataAmount, OverwriteValue, "-10.", true, V8TypeError),
+        DetailsTestCase(DetailItem, DataAmount, OverwriteValue, "10-", true, V8TypeError),
+        DetailsTestCase(DetailItem, DataAmount, OverwriteValue, "1-0", true, V8TypeError),
+        DetailsTestCase(DetailItem, DataAmount, OverwriteValue, "1.0.0", true, V8TypeError),
+        DetailsTestCase(DetailItem, DataAmount, OverwriteValue, "1/3", true, V8TypeError)));
+
+} // namespace
+} // namespace blink