Skip to content

Commit e8afafc

Browse files
committed
add another route-handler test
1 parent 2a9f7a9 commit e8afafc

File tree

2 files changed

+25
-1
lines changed

2 files changed

+25
-1
lines changed

javascript/ql/test/library-tests/frameworks/NodeJSLib/createServer.js

Lines changed: 18 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,4 +12,21 @@ const tls = require('tls');
1212

1313
const server = (isSecure ? tls : net).createServer(options, (socket) => {
1414
socket.on("data", (data) => {})
15-
});
15+
});
16+
17+
18+
const http = require("http");
19+
20+
(function () {
21+
function MyApp(data) {this.data = data};
22+
MyApp.prototype.getRequestHandler = function () {
23+
return this.handleRequest.bind(this)
24+
}
25+
MyApp.prototype.handleRequest = function (req, res) {
26+
res.end(this.data);
27+
}
28+
29+
var app = new MyApp("data");
30+
31+
const srv = http.createServer(app.getRequestHandler());
32+
})();

javascript/ql/test/library-tests/frameworks/NodeJSLib/tests.expected

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ test_isCreateServer
22
| createServer.js:2:1:2:42 | https.c ... es) {}) |
33
| createServer.js:3:1:3:45 | https.c ... es) {}) |
44
| createServer.js:4:1:4:47 | require ... => {}) |
5+
| createServer.js:31:17:31:58 | http.cr ... dler()) |
56
| src/http.js:4:14:10:2 | http.cr ... foo;\\n}) |
67
| src/http.js:12:1:16:2 | http.cr ... r");\\n}) |
78
| src/http.js:57:1:57:31 | http.cr ... dler()) |
@@ -87,6 +88,7 @@ test_RouteSetup_getServer
8788
| createServer.js:2:1:2:42 | https.c ... es) {}) | createServer.js:2:1:2:42 | https.c ... es) {}) |
8889
| createServer.js:3:1:3:45 | https.c ... es) {}) | createServer.js:3:1:3:45 | https.c ... es) {}) |
8990
| createServer.js:4:1:4:47 | require ... => {}) | createServer.js:4:1:4:47 | require ... => {}) |
91+
| createServer.js:31:17:31:58 | http.cr ... dler()) | createServer.js:31:17:31:58 | http.cr ... dler()) |
9092
| src/http.js:4:14:10:2 | http.cr ... foo;\\n}) | src/http.js:4:14:10:2 | http.cr ... foo;\\n}) |
9193
| src/http.js:12:1:16:2 | http.cr ... r");\\n}) | src/http.js:12:1:16:2 | http.cr ... r");\\n}) |
9294
| src/http.js:57:1:57:31 | http.cr ... dler()) | src/http.js:57:1:57:31 | http.cr ... dler()) |
@@ -114,6 +116,7 @@ test_ServerDefinition
114116
| createServer.js:2:1:2:42 | https.c ... es) {}) |
115117
| createServer.js:3:1:3:45 | https.c ... es) {}) |
116118
| createServer.js:4:1:4:47 | require ... => {}) |
119+
| createServer.js:31:17:31:58 | http.cr ... dler()) |
117120
| src/http.js:4:14:10:2 | http.cr ... foo;\\n}) |
118121
| src/http.js:12:1:16:2 | http.cr ... r");\\n}) |
119122
| src/http.js:57:1:57:31 | http.cr ... dler()) |
@@ -204,6 +207,10 @@ test_RouteSetup_getARouteHandler
204207
| createServer.js:2:1:2:42 | https.c ... es) {}) | createServer.js:2:20:2:41 | functio ... res) {} |
205208
| createServer.js:3:1:3:45 | https.c ... es) {}) | createServer.js:3:23:3:44 | functio ... res) {} |
206209
| createServer.js:4:1:4:47 | require ... => {}) | createServer.js:4:31:4:46 | (req, res) => {} |
210+
| createServer.js:31:17:31:58 | http.cr ... dler()) | createServer.js:22:41:24:5 | return of anonymous function |
211+
| createServer.js:31:17:31:58 | http.cr ... dler()) | createServer.js:23:16:23:33 | this.handleRequest |
212+
| createServer.js:31:17:31:58 | http.cr ... dler()) | createServer.js:23:16:23:44 | this.ha ... d(this) |
213+
| createServer.js:31:17:31:58 | http.cr ... dler()) | createServer.js:31:35:31:57 | app.get ... ndler() |
207214
| src/http.js:4:14:10:2 | http.cr ... foo;\\n}) | src/http.js:4:32:10:1 | functio ... .foo;\\n} |
208215
| src/http.js:12:1:16:2 | http.cr ... r");\\n}) | src/http.js:12:19:16:1 | functio ... ar");\\n} |
209216
| src/http.js:57:1:57:31 | http.cr ... dler()) | src/http.js:54:1:56:1 | return of function getHandler |

0 commit comments

Comments
 (0)