Merge pull request #1771 from Safelite/defect/CSR-1982
CSR-1982 type check in handleIFrameContentWindowMessage on payment page
This commit is contained in:
commit
c9b9831acd
2 changed files with 12 additions and 10 deletions
|
|
@ -512,7 +512,7 @@ describe("payment.vue", () => {
|
|||
});
|
||||
});
|
||||
|
||||
describe("handleIFrameContentWindwMessage", () => {
|
||||
describe("handleIFrameContentWindowMessage", () => {
|
||||
test("Navigates back if afterpay is closed", () => {
|
||||
// Arrange
|
||||
const event = {
|
||||
|
|
@ -524,7 +524,7 @@ describe("payment.vue", () => {
|
|||
wrapper.vm.backButtonAction = jest.fn();
|
||||
|
||||
// Act
|
||||
wrapper.vm.handleIFrameContentWindwMessage(event);
|
||||
wrapper.vm.handleIFrameContentWindowMessage(event);
|
||||
|
||||
// Assert
|
||||
expect(wrapper.vm.backButtonAction).toBeCalled();
|
||||
|
|
@ -539,7 +539,7 @@ describe("payment.vue", () => {
|
|||
const wrapper = setupMocks({});
|
||||
|
||||
// Act
|
||||
wrapper.vm.handleIFrameContentWindwMessage(event);
|
||||
wrapper.vm.handleIFrameContentWindowMessage(event);
|
||||
|
||||
// Assert
|
||||
expect(wrapper.vm.shouldBlockInteraction).toBe(true);
|
||||
|
|
|
|||
|
|
@ -643,17 +643,19 @@ export default {
|
|||
|
||||
this.submitHopForm();
|
||||
},
|
||||
handleIFrameContentWindwMessage(event) {
|
||||
if (event.data.indexOf("afterpayClosed") > -1) {
|
||||
this.backButtonAction();
|
||||
}
|
||||
if (event.data.indexOf("creditCardSubmit") > -1) {
|
||||
this.setUIBlock(true);
|
||||
handleIFrameContentWindowMessage(event) {
|
||||
if (typeof event.data === "string") {
|
||||
if (event.data.indexOf("afterpayClosed") > -1) {
|
||||
this.backButtonAction();
|
||||
}
|
||||
if (event.data.indexOf("creditCardSubmit") > -1) {
|
||||
this.setUIBlock(true);
|
||||
}
|
||||
}
|
||||
},
|
||||
setIFrameListener() {
|
||||
window.addEventListener("message", (event) =>
|
||||
this.handleIFrameContentWindwMessage(event)
|
||||
this.handleIFrameContentWindowMessage(event)
|
||||
);
|
||||
},
|
||||
setUIBlock(val) {
|
||||
|
|
|
|||
Loading…
Reference in a new issue