mirror of
https://github.com/discourse/discourse.git
synced 2024-11-22 11:23:25 +08:00
DEV: use REPORT_REQUESTS=1 to find all requests
We can use this to profile our pretender and ensure nothing is superfluous and nothing is missing
This commit is contained in:
parent
37bf38f801
commit
8a50ab45aa
|
@ -24,6 +24,8 @@ task "qunit:test", [:timeout, :qunit_path] do |_, args|
|
|||
abort "Yarn is not installed. Download from https://yarnpkg.com/lang/en/docs/install/"
|
||||
end
|
||||
|
||||
report_requests = ENV['REPORT_REQUESTS'] == "1"
|
||||
|
||||
system("yarn install --dev")
|
||||
|
||||
# ensure we have this port available
|
||||
|
@ -63,10 +65,12 @@ task "qunit:test", [:timeout, :qunit_path] do |_, args|
|
|||
options[arg] = ENV[arg.upcase] if ENV[arg.upcase].present?
|
||||
end
|
||||
|
||||
if options.present?
|
||||
cmd += "?#{options.to_query.gsub('+', '%20').gsub("&", '\\\&')}"
|
||||
if report_requests
|
||||
options['report_requests'] = '1'
|
||||
end
|
||||
|
||||
cmd += "?#{options.to_query.gsub('+', '%20').gsub("&", '\\\&')}"
|
||||
|
||||
if args[:timeout].present?
|
||||
cmd += " #{args[:timeout]}"
|
||||
end
|
||||
|
|
|
@ -117,7 +117,17 @@ QUnit.testStart(function(ctx) {
|
|||
return body;
|
||||
};
|
||||
|
||||
if (QUnit.config.logAllRequests) {
|
||||
server.handledRequest = function(verb, path, request) {
|
||||
console.log("REQ: " + verb + " " + path);
|
||||
};
|
||||
}
|
||||
|
||||
server.unhandledRequest = function(verb, path) {
|
||||
if (QUnit.config.logAllRequests) {
|
||||
console.log("REQ: " + verb + " " + path + " missing");
|
||||
}
|
||||
|
||||
const error =
|
||||
"Unhandled request in test environment: " + path + " (" + verb + ")";
|
||||
window.console.error(error);
|
||||
|
|
|
@ -112,6 +112,13 @@ async function runAllTests() {
|
|||
await Runtime.evaluate({
|
||||
expression: `(${qunit_script})()`
|
||||
});
|
||||
|
||||
if (args[0].indexOf("report_requests=1") > -1) {
|
||||
await Runtime.evaluate({
|
||||
expression: "QUnit.config.logAllRequests = true"
|
||||
});
|
||||
}
|
||||
|
||||
const timeout = parseInt(args[1] || 300000, 10);
|
||||
var start = Date.now();
|
||||
|
||||
|
|
Loading…
Reference in New Issue
Block a user