From 0b531824e964f192ac931a904535055554f4f6cf Mon Sep 17 00:00:00 2001
From: Jonathan Budzenski <jon@budzenski.me>
Date: Thu, 23 Jul 2020 02:24:22 -0500
Subject: [PATCH] [keystore] use get_keystore in server cli (#72954)

* [keystore] use get_keystore in server cli

* temporily add docker build so this can be retested in original env

* Revert "temporily add docker build so this can be retested in original env"

This reverts commit 25f401aa2e1e669b8098ac67105682ad3f8e1095.
---
 src/cli/serve/read_keystore.js      |  7 +++----
 src/cli/serve/read_keystore.test.js | 16 +++++++++++-----
 2 files changed, 14 insertions(+), 9 deletions(-)

diff --git a/src/cli/serve/read_keystore.js b/src/cli/serve/read_keystore.js
index 962c708c0d8df..38d0e68bd5c4e 100644
--- a/src/cli/serve/read_keystore.js
+++ b/src/cli/serve/read_keystore.js
@@ -17,14 +17,13 @@
  * under the License.
  */
 
-import path from 'path';
 import { set } from '@elastic/safer-lodash-set';
 
 import { Keystore } from '../../legacy/server/keystore';
-import { getDataPath } from '../../core/server/path';
+import { getKeystore } from '../../cli_keystore/get_keystore';
 
-export function readKeystore(dataPath = getDataPath()) {
-  const keystore = new Keystore(path.join(dataPath, 'kibana.keystore'));
+export function readKeystore(keystorePath = getKeystore()) {
+  const keystore = new Keystore(keystorePath);
   keystore.load();
 
   const keys = Object.keys(keystore.data);
diff --git a/src/cli/serve/read_keystore.test.js b/src/cli/serve/read_keystore.test.js
index b77e51fc3033a..e5407b257a909 100644
--- a/src/cli/serve/read_keystore.test.js
+++ b/src/cli/serve/read_keystore.test.js
@@ -40,11 +40,17 @@ describe('cli/serve/read_keystore', () => {
     });
   });
 
-  it('uses data path provided', () => {
-    const keystoreDir = '/foo/';
-    const keystorePath = path.join(keystoreDir, 'kibana.keystore');
+  it('uses data path if provided', () => {
+    const keystorePath = path.join('/foo/', 'kibana.keystore');
 
-    readKeystore(keystoreDir);
-    expect(Keystore.mock.calls[0][0]).toEqual(keystorePath);
+    readKeystore(keystorePath);
+    expect(Keystore.mock.calls[0][0]).toContain(keystorePath);
+  });
+
+  it('uses the getKeystore path if not', () => {
+    readKeystore();
+    // we test exact path scenarios in get_keystore.test.js - we use both
+    // deprecated and new to cover any older local environments
+    expect(Keystore.mock.calls[0][0]).toMatch(/data|config/);
   });
 });