@cloudcare/guance-front-tools 1.0.13 → 1.0.16

package/test/cli.test.mjs

@@ -4,7 +4,7 @@ import fs from 'node:fs'
 import os from 'node:os'
 import path from 'node:path'
 import { spawnSync } from 'node:child_process'
-import { fileURLToPath } from 'node:url'
+import { fileURLToPath, pathToFileURL } from 'node:url'
 
 import { covert } from '../lib/scripts/grafana-covert-to-guance-core.js'
 import {
@@ -15,6 +15,10 @@ const __filename = fileURLToPath(import.meta.url)
 const __dirname = path.dirname(__filename)
 const projectRoot = path.resolve(__dirname, '..')
 const cliPath = path.join(projectRoot, 'cli.js')
+const skillCliPath = path.join(
+  projectRoot,
+  'skills/grafana-to-guance-dashboard/scripts/convert-grafana-dashboard.mjs'
+)
 const grafanaInputPath = path.join(projectRoot, 'lib/example/test2.json')
 const grafanaLoadInputPath = path.join(projectRoot, 'lib/example/grafana2.json')
 
@@ -26,6 +30,14 @@ function runCli(args, options = {}) {
   })
 }
 
+function runSkillCli(args, options = {}) {
+  return spawnSync(process.execPath, [skillCliPath, ...args], {
+    cwd: projectRoot,
+    encoding: 'utf-8',
+    ...options,
+  })
+}
+
 function createTempDir() {
   return fs.mkdtempSync(path.join(os.tmpdir(), 'guance-front-tools-cli-'))
 }
@@ -39,6 +51,47 @@ test('covert delegates to the shared converter implementation', () => {
   assert.deepEqual(actual, expected)
 })
 
+test('published esm converter entry does not depend on skills path', async () => {
+  const esmCorePath = path.join(projectRoot, 'lib/esm/scripts/grafana-covert-to-guance-core.js')
+  const esmCoreSource = fs.readFileSync(esmCorePath, 'utf-8')
+  const esmConverterCorePath = path.join(projectRoot, 'lib/esm/scripts/convert-grafana-dashboard-core.js')
+  const esmConverterCoreSource = fs.readFileSync(esmConverterCorePath, 'utf-8')
+
+  assert.match(esmCoreSource, /convert-grafana-dashboard-core\.js/)
+  assert.doesNotMatch(esmCoreSource, /skills\/grafana-to-guance-dashboard/)
+  assert.doesNotMatch(esmConverterCoreSource, /from 'fs'|from "fs"|from 'path'|from "path"|from 'url'|from "url"|from 'ajv'|from "ajv"/)
+  assert.doesNotMatch(esmConverterCoreSource, /\?\.|\?\?/)
+
+  const publishedModule = await import(pathToFileURL(esmCorePath).href)
+  const grafanaDashboard = JSON.parse(fs.readFileSync(grafanaLoadInputPath, 'utf-8'))
+  const actual = publishedModule.covert(grafanaDashboard)
+  const expected = convertDashboard(grafanaDashboard)
+
+  assert.deepEqual(actual, expected)
+})
+
+test('skill converter script ships wrapper and core files without importing lib paths', () => {
+  const skillWrapperPath = path.join(
+    projectRoot,
+    'skills/grafana-to-guance-dashboard/scripts/convert-grafana-dashboard.mjs'
+  )
+  const skillCorePath = path.join(
+    projectRoot,
+    'skills/grafana-to-guance-dashboard/scripts/convert-grafana-dashboard-core.js'
+  )
+  const skillWrapperSource = fs.readFileSync(skillWrapperPath, 'utf-8')
+  const skillCoreSource = fs.readFileSync(skillCorePath, 'utf-8')
+
+  assert.doesNotMatch(skillWrapperSource, /lib\/scripts\/convert-grafana-dashboard/)
+  assert.match(skillWrapperSource, /import \{ convertDashboard \} from '\.\/convert-grafana-dashboard-core\.js'/)
+  assert.match(skillWrapperSource, /^#!\/usr\/bin\/env node/m)
+  assert.match(skillWrapperSource, /export function main\(/)
+  assert.match(skillWrapperSource, /export function validateDashboardFile\(/)
+  assert.match(skillCoreSource, /export function convertDashboard\(/)
+  assert.doesNotMatch(skillWrapperSource, /\?\.|\?\?/)
+  assert.doesNotMatch(skillCoreSource, /\?\.|\?\?/)
+})
+
 test('shared converter infers units from query text when panel unit is missing', () => {
   const grafanaDashboard = JSON.parse(fs.readFileSync(grafanaLoadInputPath, 'utf-8'))
 
@@ -371,3 +424,72 @@ test('cli writes default output file to current working directory', () => {
   const actual = JSON.parse(fs.readFileSync(outPath, 'utf-8'))
   assert.deepEqual(actual, expected)
 })
+
+test('skill cli converts and validates a grafana dashboard with explicit output path', () => {
+  const tempDir = createTempDir()
+  const outPath = path.join(tempDir, 'skill', 'guance-dashboard.json')
+  const expected = convertDashboard(JSON.parse(fs.readFileSync(grafanaInputPath, 'utf-8')))
+
+  const result = runSkillCli([
+    '--input',
+    grafanaInputPath,
+    '--output',
+    outPath,
+    '--validate',
+  ])
+
+  assert.equal(result.status, 0, result.stderr || result.stdout)
+  assert.match(result.stdout, /Validated /)
+  assert.equal(fs.existsSync(outPath), true)
+
+  const actual = JSON.parse(fs.readFileSync(outPath, 'utf-8'))
+  assert.deepEqual(actual, expected)
+})
+
+test('skill cli supports guance promql compatible output', () => {
+  const tempDir = createTempDir()
+  const outPath = path.join(tempDir, 'skill', 'guance-promql.json')
+  const expected = convertDashboard(JSON.parse(fs.readFileSync(grafanaInputPath, 'utf-8')), {
+    guancePromqlCompatible: true,
+  })
+
+  const result = runSkillCli([
+    '--input',
+    grafanaInputPath,
+    '--output',
+    outPath,
+    '--validate',
+    '--guance-promql-compatible',
+  ])
+
+  assert.equal(result.status, 0, result.stderr || result.stdout)
+  assert.match(result.stdout, /Validated /)
+  assert.equal(fs.existsSync(outPath), true)
+
+  const actual = JSON.parse(fs.readFileSync(outPath, 'utf-8'))
+  assert.deepEqual(actual, expected)
+})
+
+test('skill cli supports keep grafana meta output', () => {
+  const tempDir = createTempDir()
+  const outPath = path.join(tempDir, 'skill', 'guance-keep-meta.json')
+  const expected = convertDashboard(JSON.parse(fs.readFileSync(grafanaInputPath, 'utf-8')), {
+    keepGrafanaMeta: true,
+  })
+
+  const result = runSkillCli([
+    '--input',
+    grafanaInputPath,
+    '--output',
+    outPath,
+    '--validate',
+    '--keep-grafana-meta',
+  ])
+
+  assert.equal(result.status, 0, result.stderr || result.stdout)
+  assert.match(result.stdout, /Validated /)
+  assert.equal(fs.existsSync(outPath), true)
+
+  const actual = JSON.parse(fs.readFileSync(outPath, 'utf-8'))
+  assert.deepEqual(actual, expected)
+})