Skip to content
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

Jump to bottom

fix(package): rename repository URL #828

Merged
merged 2 commits into from
Jan 14, 2024
Merged

fix(package): rename repository URL #828

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
79a69b6
Update package.json
9renpoto Dec 6, 2023
2cba6d4
si/visionmedia/ladjs/
9renpoto Jan 14, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@
"main": "index.js",
"repository": {
"type": "git",
"url": "https://github.com/visionmedia/supertest.git"
"url": "https://github.com/ladjs/supertest.git"
},
"scripts": {
"coverage": "nyc report --reporter=text-lcov > coverage.lcov",
Expand Down
8 changes: 4 additions & 4 deletions test/supertest.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -765,9 +765,9 @@ describe('request(app)', function () {
});
});

// this scenario should never happen after https://github.com/visionmedia/supertest/pull/767
// this scenario should never happen after https://github.com/ladjs/supertest/pull/767
// meant for test coverage for lib/test.js#287
// https://github.com/visionmedia/supertest/blob/e064b5ae71e1dfa3e1a74745fda527ac542e1878/lib/test.js#L287
// https://github.com/ladjs/supertest/blob/e064b5ae71e1dfa3e1a74745fda527ac542e1878/lib/test.js#L287
it('_assertFunction should catch and return error', function (done) {
const error = new Error('failed');
const returnedError = get
Expand Down Expand Up @@ -1306,7 +1306,7 @@ describe('request.get(url).query(vals) works as expected', function () {
.get('/')
// This expect should never get called, but exposes this issue with other
// errors being obscured by the response assertions
// https://github.com/visionmedia/supertest/issues/352
// https://github.com/ladjs/supertest/issues/352
.expect(200)
.end(function (err, res) {
should.exist(err);
Expand All @@ -1323,7 +1323,7 @@ describe('request.get(url).query(vals) works as expected', function () {
// this scenario should never happen
// there shouldn't be any res if there is an err
// meant for test coverage for lib/test.js#169
// https://github.com/visionmedia/supertest/blob/5543d674cf9aa4547927ba6010d31d9474950dec/lib/test.js#L169
// https://github.com/ladjs/supertest/blob/5543d674cf9aa4547927ba6010d31d9474950dec/lib/test.js#L169
it('handles unknown errors (err with res)', function (done) {
const app = express();

Expand Down