fix: Throw better error message if version parsing fails (#821)

* Throw better error message if version parsing fails.

* Cleaner fix.

* Update targeted HTTP endpoint.

* Add dummy server route for new endpoint.

Author: justinkambic

__tests__/utils/kibana-test-server.ts

@@ -34,6 +34,9 @@ export const createKibanaTestServer = async (kibanaVersion: string) => {
   server.route('/s/dummy/api/status', (req, res) =>
     res.end(JSON.stringify({ version: { number: kibanaVersion } }))
   );
+  server.route('/s/dummy/api/stats', (req, res) =>
+    res.end(JSON.stringify({ kibana: { version: kibanaVersion } }))
+  );
   // Legacy
   server.route(
     '/s/dummy/api/synthetics/service/project/monitors',

src/push/kibana_api.ts

@@ -124,8 +124,8 @@ export async function bulkDeleteMonitors(
 }
 
 type StatusResponse = {
-  version: {
-    number: string;
+  kibana: {
+    version: string;
   };
 };
 
@@ -135,7 +135,8 @@ export async function getVersion(options: PushOptions) {
     method: 'GET',
     auth: options.auth,
   });
-  return data.version.number;
+
+  return data.kibana.version;
 }
 
 export type LegacyAPISchema = {

src/push/utils.ts

@@ -64,7 +64,7 @@ export function generateURL(options: PushOptions, operation: Operation) {
   const url = removeTrailingSlash(options.url);
   switch (operation) {
     case 'status':
-      return `${url}/s/${options.space}/api/status`;
+      return `${url}/s/${options.space}/api/stats`;
     case 'bulk_get':
     case 'bulk_update':
     case 'bulk_delete':