mirror of
https://github.com/jquery/jquery.git
synced 2024-11-23 02:54:22 +00:00
Tests: Make the beforeunload event tests work regardless of extensions
Some browser extensions, like React DevTools, send messages to the content area. Since our beforeunload event test listens for all messages, it used to catch those as well, failing the test. Add a `source` field to the payload JSON and check for it before treating the message as coming from our own test to make sure the test passes even with such browser extensions installed. Closes gh-5478
This commit is contained in:
parent
7cdd837423
commit
399a78ee9f
@ -4,6 +4,7 @@
|
||||
<script>
|
||||
function report( event ) {
|
||||
var payload = {
|
||||
source: "jQuery onbeforeunload iframe test",
|
||||
event: event.type
|
||||
};
|
||||
return parent.postMessage( JSON.stringify( payload ), "*" );
|
||||
|
@ -1448,13 +1448,22 @@ QUnit[ /(ipad|iphone|ipod)/i.test( navigator.userAgent ) ? "skip" : "test" ](
|
||||
var done = assert.async();
|
||||
|
||||
window.onmessage = function( event ) {
|
||||
try {
|
||||
var payload = JSON.parse( event.data );
|
||||
|
||||
// Ignore unrelated messages
|
||||
if ( payload.source === "jQuery onbeforeunload iframe test" ) {
|
||||
assert.ok( payload.event, "beforeunload", "beforeunload event" );
|
||||
|
||||
iframe.remove();
|
||||
window.onmessage = null;
|
||||
done();
|
||||
}
|
||||
} catch ( e ) {
|
||||
|
||||
// Messages may come from other sources, like browser extensions;
|
||||
// some may not be valid JSONs and thus cannot be `JSON.parse`d.
|
||||
}
|
||||
};
|
||||
|
||||
iframe.appendTo( "#qunit-fixture" );
|
||||
|
Loading…
Reference in New Issue
Block a user