🌐 AI搜索 & 代理 主页
Skip to content

Ajax: Execute JSONP error script responses #4773

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
mgol merged 10 commits into from
Aug 25, 2020
Merged

Ajax: Execute JSONP error script responses #4773

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
5e7fcc7
The fix works for me such that parses the jsonp response
Aug 20, 2020
2c84715
Fix linting issues
Aug 20, 2020
ad1e4c7
Appears do not need new mock and fix up the test
Aug 20, 2020
bd0dde8
Cant use const -> use var instead
Aug 20, 2020
c178cdb
MIssed a const
Aug 20, 2020
06c3070
Fix linting issue
Aug 20, 2020
bc7de3d
Failure suppose to be error I think
Aug 20, 2020
f09715f
Remove before send since seems to be causing issues with out tests
Aug 20, 2020
f5cb586
Try not using strictEqual and just equal
Aug 20, 2020
3ee656e
Need to use deep equal
Aug 21, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Try not using strictEqual and just equal 
It looks like it should be passing
  • Loading branch information
Dallas Fraser committed Aug 20, 2020
commit f5cb5869443fb8d26c2c23519368b184b1059d4c
2 changes: 1 addition & 1 deletion test/unit/ajax.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -814,7 +814,7 @@ QUnit.module( "ajax", {
// error is the significant assertion
error: function( xhr ) {
var expected = { "status": 404, "msg": "Not Found" };
assert.strictEqual( xhr.responseJSON, expected, testMsg );
assert.equal( xhr.responseJSON, expected, testMsg );
}
};
} );
Expand Down