remove the metrics sendernlf/remove-metrics

PR-URL: https://github.com/npm/cli/pull/2382
Credit: @nlf
Close: #2382
Reviewed-by: @isaacs

2 files changed, 0 insertions, 10 deletions

diff --git a/test/lib/utils/error-handler.js b/test/lib/utils/error-handler.js
index 2dc116a4d..0b896fee4 100644
--- a/test/lib/utils/error-handler.js
+++ b/test/lib/utils/error-handler.js
@@ -78,10 +78,6 @@ const npmlog = {
   },
 }
 
-const metrics = {
-  stop: () => null,
-}
-
 // overrides OS type/release for cross platform snapshots
 const os = require('os')
 os.type = () => 'Foo'
@@ -124,7 +120,6 @@ const mocks = {
     summary: [['ERR', err.message]],
     detail: [['ERR', err.message]],
   }),
-  '../../../lib/utils/metrics.js': metrics,
   '../../../lib/utils/cache-file.js': cacheFile,
 }
 
diff --git a/test/lib/utils/flat-options.js b/test/lib/utils/flat-options.js
index ee7620fa7..6f580fabc 100644
--- a/test/lib/utils/flat-options.js
+++ b/test/lib/utils/flat-options.js
@@ -34,8 +34,6 @@ class MockConfig {
       cache: 'cache',
       'node-version': '1.2.3',
       global: 'global',
-      'metrics-registry': 'metrics-registry',
-      'send-metrics': 'send-metrics',
       registry: 'registry',
       access: 'access',
       'always-auth': 'always-auth',
@@ -299,15 +297,12 @@ t.test('various default values and falsey fallbacks', t => {
   const npm = new Mocknpm({
     'script-shell': false,
     registry: 'http://example.com',
-    'metrics-registry': null,
     searchlimit: 0,
     'save-exact': false,
     'save-prefix': '>=',
   })
   const opts = flatOptions(npm)
   t.equal(opts.scriptShell, undefined, 'scriptShell is undefined if falsey')
-  t.equal(opts.metricsRegistry, 'http://example.com',
-    'metricsRegistry defaults to registry')
   t.equal(opts.search.limit, 20, 'searchLimit defaults to 20')
   t.equal(opts.savePrefix, '>=', 'save-prefix respected if no save-exact')
   t.equal(opts.scope, '', 'scope defaults to empty string')