From fad524ed133af6f094357b94da54e616c7f943b6 Mon Sep 17 00:00:00 2001 From: "(quasar) nebula" Date: Sat, 25 Mar 2023 14:19:23 -0300 Subject: data steps: move data-tests outside tests These are mostly short REPL-like scripts for testing actual wiki data, not the codebase. They don't really belong in the repo at all, but actually cause trouble by living in the main tests directory! --- test/data-tests/index.js | 125 -------------------------- test/data-tests/test-no-short-tracks.js | 25 ------ test/data-tests/test-order-of-album-groups.js | 55 ------------ 3 files changed, 205 deletions(-) delete mode 100644 test/data-tests/index.js delete mode 100644 test/data-tests/test-no-short-tracks.js delete mode 100644 test/data-tests/test-order-of-album-groups.js (limited to 'test/data-tests') diff --git a/test/data-tests/index.js b/test/data-tests/index.js deleted file mode 100644 index 1b9ec990..00000000 --- a/test/data-tests/index.js +++ /dev/null @@ -1,125 +0,0 @@ -import chokidar from 'chokidar'; -import * as path from 'path'; -import {fileURLToPath} from 'url'; - -import {quickLoadAllFromYAML} from '../../src/data/yaml.js'; -import {isMain} from '../../src/util/node-utils.js'; -import {getContextAssignments} from '../../src/repl.js'; - -import { - color, - logError, - logInfo, - logWarn, - parseOptions, -} from '../../src/util/cli.js'; - -import { - bindOpts, - showAggregate, -} from '../../src/util/sugar.js'; - -async function main() { - const miscOptions = await parseOptions(process.argv.slice(2), { - 'data-path': { - type: 'value', - }, - }); - - const dataPath = miscOptions['data-path'] || process.env.HSMUSIC_DATA; - - if (!dataPath) { - logError`Expected --data-path option or HSMUSIC_DATA to be set`; - return; - } - - console.log(`HSMusic automated data tests`); - console.log(`${color.bright(color.yellow(`:star:`))} Now featuring quick-reloading! ${color.bright(color.cyan(`:earth:`))}`); - - // Watch adjacent files in data-tests directory - const metaPath = fileURLToPath(import.meta.url); - const metaDirname = path.dirname(metaPath); - const watcher = chokidar.watch(metaDirname); - - const wikiData = await quickLoadAllFromYAML(dataPath, { - showAggregate: bindOpts(showAggregate, { - showTraces: false, - }), - }); - - const context = await getContextAssignments({ - wikiData, - }); - - let resolveNext; - - const queue = []; - - watcher.on('all', (event, path) => { - if (!['add', 'change'].includes(event)) return; - if (path === metaPath) return; - if (resolveNext) { - resolveNext(path); - } else if (!queue.includes(path)) { - queue.push(path); - } - }); - - logInfo`Awaiting file changes.`; - - /* eslint-disable-next-line no-constant-condition */ - while (true) { - const testPath = (queue.length - ? queue.shift() - : await new Promise(resolve => { - resolveNext = resolve; - })); - - resolveNext = null; - - const shortPath = path.basename(testPath); - - logInfo`Path updated: ${shortPath} - running this test!`; - - let imp; - try { - imp = await import(`${testPath}?${Date.now()}`) - } catch (error) { - logWarn`Failed to import ${shortPath} - ${error.constructor.name} details below:`; - console.error(error); - continue; - } - - const {default: testFn} = imp; - - if (!testFn) { - logWarn`No default export for ${shortPath}`; - logWarn`Skipping this test for now!`; - continue; - } - - if (typeof testFn !== 'function') { - logWarn`Default export for ${shortPath} is ${typeof testFn}, not function`; - logWarn`Skipping this test for now!`; - continue; - } - - try { - await testFn(context); - } catch (error) { - showAggregate(error, { - pathToFileURL: f => path.relative(metaDirname, fileURLToPath(f)), - }); - } - } -} - -if (isMain(import.meta.url)) { - main().catch((error) => { - if (error instanceof AggregateError) { - showAggregate(error); - } else { - console.error(error); - } - }); -} diff --git a/test/data-tests/test-no-short-tracks.js b/test/data-tests/test-no-short-tracks.js deleted file mode 100644 index 76356099..00000000 --- a/test/data-tests/test-no-short-tracks.js +++ /dev/null @@ -1,25 +0,0 @@ -export default function({ - albumData, - getTotalDuration, -}) { - const shortAlbums = albumData - .filter(album => album.tracks.length > 1) - .map(album => ({ - album, - duration: getTotalDuration(album.tracks), - })) - .filter(album => album.duration) - .filter(album => album.duration < 60 * 15); - - if (!shortAlbums.length) return true; - - shortAlbums.sort((a, b) => a.duration - b.duration); - - console.log(`Found ${shortAlbums.length} short albums! Oh nooooooo!`); - console.log(`Here are the shortest 10:`); - for (const {duration, album} of shortAlbums.slice(0, 10)) { - console.log(`- (${duration}s)`, album); - } - - return false; -} diff --git a/test/data-tests/test-order-of-album-groups.js b/test/data-tests/test-order-of-album-groups.js deleted file mode 100644 index de2fcbed..00000000 --- a/test/data-tests/test-order-of-album-groups.js +++ /dev/null @@ -1,55 +0,0 @@ -import * as util from 'util'; - -export default function({ - albumData, - groupCategoryData, -}) { - const groupSchemaTemplate = [ - ['Projects beyond Homestuck', 'Fandom projects'], - ['Solo musicians', 'Fan-musician groups'], - ['HSMusic'], - ]; - - const groupSchema = - groupSchemaTemplate.map(names => names.flatMap( - name => groupCategoryData - .find(gc => gc.name === name) - .groups)); - - const badAlbums = albumData.filter(album => { - const groups = album.groups.slice(); - const disallowed = []; - for (const allowed of groupSchema) { - while (groups.length) { - if (disallowed.includes(groups[0])) - return true; - else if (allowed.includes(groups[0])) - groups.shift(); - else break; - } - disallowed.push(...allowed); - } - return false; - }); - - if (!badAlbums.length) return true; - - console.log(`Some albums don't list their groups in the right order:`); - for (const album of badAlbums) { - console.log('-', album); - for (const group of album.groups) { - console.log(` - ${util.inspect(group)}`) - } - } - - console.log(`Here's the group schema they should be updated to match:`); - for (const section of groupSchemaTemplate) { - if (section.length > 1) { - console.log(`- Groups from any of: ${section.join(', ')}`); - } else { - console.log(`- Groups from: ${section}`); - } - } - - return false; -} -- cgit 1.3.0-6-gf8a5