From 797d9ee9249c55eb28e0eacb4f2d128c6381b78f Mon Sep 17 00:00:00 2001
From: Rich Trott <rtrott@gmail.com>
Date: Sat, 7 Jan 2017 20:12:08 -0800
Subject: [PATCH] lib: refactor crypto cipher/hash/curve getters

* refactor internal util.filterDuplicateStrings() to eliminate unused
  code paths
* `.indexOf()` -> `.includes()` in test
* more concise arrow functions

PR-URL: https://github.com/nodejs/node/pull/10682
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michal Zasso <targos@protonmail.com>
---
 lib/crypto.js                              | 18 +++++++++---------
 lib/internal/util.js                       | 10 ++--------
 test/parallel/test-crypto-authenticated.js |  2 +-
 3 files changed, 12 insertions(+), 18 deletions(-)

diff --git a/lib/crypto.js b/lib/crypto.js
index aa5a7d0cc97579..b16334461a1e51 100644
--- a/lib/crypto.js
+++ b/lib/crypto.js
@@ -633,17 +633,17 @@ exports.randomBytes = exports.pseudoRandomBytes = randomBytes;
 
 exports.rng = exports.prng = randomBytes;
 
-exports.getCiphers = internalUtil.cachedResult(() => {
-  return internalUtil.filterDuplicateStrings(getCiphers());
-});
+exports.getCiphers = internalUtil.cachedResult(
+  () => internalUtil.filterDuplicateStrings(getCiphers())
+);
 
-exports.getHashes = internalUtil.cachedResult(() => {
-  return internalUtil.filterDuplicateStrings(getHashes());
-});
+exports.getHashes = internalUtil.cachedResult(
+  () => internalUtil.filterDuplicateStrings(getHashes())
+);
 
-exports.getCurves = internalUtil.cachedResult(() => {
-  return internalUtil.filterDuplicateStrings(getCurves());
-});
+exports.getCurves = internalUtil.cachedResult(
+  () => internalUtil.filterDuplicateStrings(getCurves())
+);
 
 Object.defineProperty(exports, 'fips', {
   get: getFipsCrypto,
diff --git a/lib/internal/util.js b/lib/internal/util.js
index 8162434da7cfca..bf8f947661d763 100644
--- a/lib/internal/util.js
+++ b/lib/internal/util.js
@@ -109,20 +109,14 @@ exports.assertCrypto = function(exports) {
 // Filters duplicate strings. Used to support functions in crypto and tls
 // modules. Implemented specifically to maintain existing behaviors in each.
 exports.filterDuplicateStrings = function filterDuplicateStrings(items, low) {
-  if (!Array.isArray(items))
-    return [];
-  const len = items.length;
-  if (len <= 1)
-    return items;
   const map = new Map();
-  for (var i = 0; i < len; i++) {
+  for (var i = 0; i < items.length; i++) {
     const item = items[i];
     const key = item.toLowerCase();
     if (low) {
       map.set(key, key);
     } else {
-      if (!map.has(key) || map.get(key) <= item)
-        map.set(key, item);
+      map.set(key, item);
     }
   }
   return Array.from(map.values()).sort();
diff --git a/test/parallel/test-crypto-authenticated.js b/test/parallel/test-crypto-authenticated.js
index 4bed5aa665b64d..ee67250ff35223 100644
--- a/test/parallel/test-crypto-authenticated.js
+++ b/test/parallel/test-crypto-authenticated.js
@@ -312,7 +312,7 @@ const ciphers = crypto.getCiphers();
 for (const i in TEST_CASES) {
   const test = TEST_CASES[i];
 
-  if (ciphers.indexOf(test.algo) === -1) {
+  if (!ciphers.includes(test.algo)) {
     common.skip('unsupported ' + test.algo + ' test');
     continue;
   }