Merge pull request #287 from sam-github/lrtime-optional

Do not export lrtime if it is not implemented

Author: sjanuary

index.js

@@ -241,9 +241,7 @@ module.exports.monitor = function() {
     return this.api;
 };
 
-module.exports.lrtime = function() {
-    return agent.lrtime();
-};
+module.exports.lrtime = agent.lrtime;
 
 module.exports.configure = function(options) {
     options = options || {};
@@ -262,4 +260,4 @@ module.exports.setJSONProfilingMode = function(val) {
 
 module.exports.getJSONProfilingMode = function() {
     return jsonProfilingMode;
-}
+}

tests/api_tests.js

@@ -19,11 +19,22 @@ var monitor = app.appmetrics.monitor();
 app.appmetrics.enable("profiling");
 
 var tap = require('tap');
-tap.plan(5); // NOTE: This needs to be updated when tests are added/removed
+tap.plan(6); // NOTE: This needs to be updated when tests are added/removed
 tap.tearDown(function(){
 	app.endRun();
 });
 
+tap.test('lrtime is a function or undefined', function(t) {
+  var lrtime = app.appmetrics.lrtime;
+
+  if (lrtime) {
+    t.doesNotThrow(lrtime, 'callable, not just present');
+  } else {
+    t.notEqual(process.platform, 'linux', 'lrtime mandatory on linux');
+  }
+  t.end();
+});
+
 var completedTests = {}; //Stores which tests have been run, ensures single run per test
 
 monitor.on('cpu', function(data) {
@@ -394,4 +405,4 @@ function runEventLoopTests(elData, t){
 		t.ok(elData.latency[elem] <= 5000,
 			   "Contains " + elem + " latency value less that 5 seconds");
 	}
-}
+}