Browse Source

benchmark: don't check wrk in non-http benchmark

When running a non-http benchmark, there is no need the check for the
wrk tool so move the wrk check into the http method.

PR-URL: https://github.com/iojs/io.js/pull/1368
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
v1.8.0-commit
Jackson Tian 10 years ago
committed by Ben Noordhuis
parent
commit
d2b62a4973
  1. 3
      benchmark/common.js

3
benchmark/common.js

@ -16,7 +16,6 @@ exports.PORT = process.env.PORT || 12346;
// If this is the main module, then run the benchmarks // If this is the main module, then run the benchmarks
if (module === require.main) { if (module === require.main) {
hasWrk();
var type = process.argv[2]; var type = process.argv[2];
var testFilter = process.argv[3]; var testFilter = process.argv[3];
if (!type) { if (!type) {
@ -92,7 +91,6 @@ function Benchmark(fn, options) {
var self = this; var self = this;
hasWrk();
process.nextTick(function() { process.nextTick(function() {
self._run(); self._run();
}); });
@ -100,6 +98,7 @@ function Benchmark(fn, options) {
// benchmark an http server. // benchmark an http server.
Benchmark.prototype.http = function(p, args, cb) { Benchmark.prototype.http = function(p, args, cb) {
hasWrk();
var self = this; var self = this;
var regexp = /Requests\/sec:[ \t]+([0-9\.]+)/; var regexp = /Requests\/sec:[ \t]+([0-9\.]+)/;
var url = 'http://127.0.0.1:' + exports.PORT + p; var url = 'http://127.0.0.1:' + exports.PORT + p;

Loading…
Cancel
Save