PaymentRequest.abort() should return a promise.

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/JSONValuesForV8.h"
+#include "bindings/core/v8/ScriptFunction.h"
 #include "bindings/core/v8/ScriptState.h"
 #include "bindings/modules/v8/V8PaymentResponse.h"
 #include "core/dom/ExceptionCode.h"
-#include "core/testing/DummyPageHolder.h"
-#include "modules/payments/CurrencyAmount.h"
+#include "modules/payments/MockFunction.h"
 #include "modules/payments/PaymentAddress.h"
 #include "modules/payments/PaymentDetailsTestHelper.h"
-#include "modules/payments/PaymentItem.h"
-#include "modules/payments/ShippingOption.h"
+#include "modules/payments/PaymentRequestTestBase.h"
 #include "platform/heap/HeapAllocator.h"
-#include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
-#include "wtf/OwnPtr.h"
 #include <utility>
 
 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;
+class PaymentRequestTest : public PaymentRequestTestBase {
 };
 
 TEST_F(PaymentRequestTest, NoExceptionWithValidData)
 {
     PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), buildPaymentDetailsForTest(), getExceptionState());
 
     EXPECT_FALSE(getExceptionState().hadException());
 }
 
 TEST_F(PaymentRequestTest, SecureContextRequired)
@@ skipping 127 matching lines @@
     shippingOptions[1].setSelected(true);
     details.setShippingOptions(shippingOptions);
     PaymentOptions options;
     options.setRequestShipping(true);
 
     PaymentRequest* request = PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), details, options, getExceptionState());
 
     EXPECT_EQ("express", request->shippingOption());
 }
 
-TEST_F(PaymentRequestTest, AbortWithoutShowShouldThrow)
-{
-    PaymentRequest* request = PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), buildPaymentDetailsForTest(), getExceptionState());
-    EXPECT_FALSE(getExceptionState().hadException());
-
-    request->abort(getExceptionState());
-    EXPECT_TRUE(getExceptionState().hadException());
-}
-
-class MockFunction : public ScriptFunction {
-public:
-    static v8::Local<v8::Function> noExpectations(ScriptState* scriptState)
-    {
-        MockFunction* self = new MockFunction(scriptState);
-        return self->bindToV8Function();
-    }
-
-    static v8::Local<v8::Function> expectCall(ScriptState* scriptState)
-    {
-        MockFunction* self = new MockFunction(scriptState);
-        EXPECT_CALL(*self, call(testing::_));
-        return self->bindToV8Function();
-    }
-
-    static v8::Local<v8::Function> expectNoCall(ScriptState* scriptState)
-    {
-        MockFunction* self = new MockFunction(scriptState);
-        EXPECT_CALL(*self, call(testing::_)).Times(0);
-        return self->bindToV8Function();
-    }
-
-private:
-    explicit MockFunction(ScriptState* scriptState)
-        : ScriptFunction(scriptState)
-    {
-        ON_CALL(*this, call(testing::_)).WillByDefault(testing::ReturnArg<0>());
-    }
-
-    MOCK_METHOD1(call, ScriptValue(ScriptValue));
-};
-
 class PaymentResponseFunction : public ScriptFunction {
 public:
     static v8::Local<v8::Function> create(ScriptState* scriptState, ScriptValue* outValue)
     {
         PaymentResponseFunction* self = new PaymentResponseFunction(scriptState, outValue);
         return self->bindToV8Function();
     }
 
     ScriptValue call(ScriptValue value) override
     {
         DCHECK(!value.isEmpty());
         *m_value = value;
         return value;
     }
 
 private:
     PaymentResponseFunction(ScriptState* scriptState, ScriptValue* outValue)
         : ScriptFunction(scriptState)
         , m_value(outValue)
     {
         DCHECK(m_value);
     }
 
     ScriptValue* m_value;
 };
 
-TEST_F(PaymentRequestTest, CanAbortAfterShow)
-{
-    ScriptState::Scope scope(getScriptState());
-    PaymentRequest* request = PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), buildPaymentDetailsForTest(), getExceptionState());
-    EXPECT_FALSE(getExceptionState().hadException());
-
-    request->show(getScriptState()).then(MockFunction::expectNoCall(getScriptState()), MockFunction::noExpectations(getScriptState()));
-    request->abort(getExceptionState());
-
-    EXPECT_FALSE(getExceptionState().hadException());
-}
-
 TEST_F(PaymentRequestTest, RejectShowPromiseOnInvalidShippingAddress)
 {
     ScriptState::Scope scope(getScriptState());
     PaymentRequest* request = PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), buildPaymentDetailsForTest(), getExceptionState());
     EXPECT_FALSE(getExceptionState().hadException());
 
     request->show(getScriptState()).then(MockFunction::expectNoCall(getScriptState()), MockFunction::expectCall(getScriptState()));
 
     static_cast<mojom::blink::PaymentRequestClient*>(request)->OnShippingAddressChange(mojom::blink::PaymentAddress::New());
 }
@@ skipping 27 matching lines @@
     static_cast<mojom::blink::PaymentRequestClient*>(request)->OnPaymentResponse(std::move(response));
 }
 
 TEST_F(PaymentRequestTest, RejectShowPromiseWithRequestShippingFalseAndShippingAddressExistsInResponse)
 {
     ScriptState::Scope scope(getScriptState());
     PaymentOptions options;
     options.setRequestShipping(false);
     PaymentRequest* request = PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), buildPaymentDetailsForTest(), options, getExceptionState());
     EXPECT_FALSE(getExceptionState().hadException());
-    mojom::blink::PaymentAddressPtr shippingAddress = mojom::blink::PaymentAddress::New();
-    shippingAddress->country = "US";
-    shippingAddress->language_code = "en";
-    shippingAddress->script_code = "Latn";
+    mojom::blink::PaymentResponsePtr response = mojom::blink::PaymentResponse::New();
+    response->shipping_address = mojom::blink::PaymentAddress::New();
+    response->shipping_address->country = "US";
+    response->shipping_address->language_code = "en";
+    response->shipping_address->script_code = "Latn";
 
     request->show(getScriptState()).then(MockFunction::expectNoCall(getScriptState()), MockFunction::expectCall(getScriptState()));
 
-    static_cast<mojom::blink::PaymentRequestClient*>(request)->OnShippingAddressChange(std::move(shippingAddress));
+    static_cast<mojom::blink::PaymentRequestClient*>(request)->OnPaymentResponse(std::move(response));
 }
 
 TEST_F(PaymentRequestTest, ResolveShowPromiseWithRequestShippingTrueAndValidShippingAddressInResponse)
 {
     ScriptState::Scope scope(getScriptState());
     PaymentOptions options;
     options.setRequestShipping(true);
     PaymentRequest* request = PaymentRequest::create(getScriptState(), Vector<String>(1, "foo"), buildPaymentDetailsForTest(), options, getExceptionState());
     EXPECT_FALSE(getExceptionState().hadException());
     mojom::blink::PaymentResponsePtr response = mojom::blink::PaymentResponse::New();
@@ skipping 214 matching lines @@
         "{\"id\": \"fast\", \"label\": \"Fast\", \"amount\": {\"currency\": \"USD\", \"value\": \"50.00\"}, \"selected\": true}]}";
 
     request->onUpdatePaymentDetails(ScriptValue::from(getScriptState(), fromJSONString(getScriptState(), detail, getExceptionState())));
     EXPECT_FALSE(getExceptionState().hadException());
 
     EXPECT_EQ("fast", request->shippingOption());
 }
 
 } // namespace
 } // namespace blink