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: #1368 Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Jeremiah Senkpiel <[email protected]>
nodejs · Apr 9, 2015 · d2b62a4 · d2b62a4
1 parent b464d46
commit d2b62a4
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/benchmark/common.js b/benchmark/common.js
@@ -16,7 +16,6 @@ exports.PORT = process.env.PORT || 12346;
 
 // If this is the main module, then run the benchmarks
 if (module === require.main) {
-  hasWrk();
   var type = process.argv[2];
   var testFilter = process.argv[3];
   if (!type) {
@@ -92,14 +91,14 @@ function Benchmark(fn, options) {
 
   var self = this;
 
-  hasWrk();
   process.nextTick(function() {
     self._run();
   });
 }
 
 // benchmark an http server.
 Benchmark.prototype.http = function(p, args, cb) {
+  hasWrk();
   var self = this;
   var regexp = /Requests\/sec:[ \t]+([0-9\.]+)/;
   var url = 'http://127.0.0.1:' + exports.PORT + p;