feedBack/tests/js/v3_keep_practicing.test.js

// Pins the practice-aware library home in static/v3/songs.js:
//   - a "Repertoire" progress meter (mastered / total library songs), and
//   - a "Keep practicing" shelf (recently played, not yet mastered).
// Both reuse existing data (/api/stats/best already in state.accuracy, and
// /api/stats/recent) and are shown only on the unfiltered grid front door.
//
// Source-level only — same strategy as tests/js/v3_az_rail.test.js.

const { test } = require('node:test');
const assert = require('node:assert/strict');
const fs = require('node:fs');
const path = require('node:path');

const SONGS_JS = path.join(__dirname, '..', '..', 'static', 'v3', 'songs.js');
const src = fs.readFileSync(SONGS_JS, 'utf8');

test('repertoire uses the same mastery threshold as the green accuracy badge', () => {
    assert.match(src, /const\s+MASTERY_ACCURACY\s*=\s*0\.9/);
    assert.match(
        src,
        /function\s+_repertoireCounts[\s\S]*?v\s*>=\s*MASTERY_ACCURACY\s*\)\s*mastered\+\+;\s*else\s+learning\+\+/,
        'repertoire counts must bucket scored songs into mastered/learning at MASTERY_ACCURACY',
    );
});

test('the home is the unfiltered grid front door, local provider only', () => {
    assert.match(
        src,
        /function\s+libHomeVisible[\s\S]*?state\.view === 'grid'[\s\S]*?state\.provider === 'local'[\s\S]*?!state\.selectMode[\s\S]*?!state\.q[\s\S]*?activeFilterCount\(\)\s*===\s*0/,
        'libHomeVisible must require grid view, the local provider, no select mode, no search, no active filters',
    );
});

test('the shelf is recently-played, not-yet-mastered songs (per-song, deduped)', () => {
    assert.match(src, /\/api\/stats\/recent\?limit=/);
    // Mastery is gated on the per-SONG best (state.accuracy, what the badge
    // shows), not the per-arrangement recents row, and each filename appears
    // once — so no green-badged "keep practicing" card and no duplicates.
    assert.match(
        src,
        /const\s+best\s*=\s*acc\[r\.filename\][\s\S]*?best\s*>=\s*MASTERY_ACCURACY/,
        'the shelf must gate on the per-song best (state.accuracy) at MASTERY_ACCURACY',
    );
    assert.match(src, /seen\.has\(r\.filename\)/, 'the shelf must dedupe recents by filename');
});

test('the meter + shelf fetch together and a stale render is discarded', () => {
    assert.match(src, /Promise\.all\(\[[\s\S]*?library\/stats[\s\S]*?stats\/recent/,
        'the two reads must be issued together (Promise.all), not sequentially');
    assert.match(src, /_homeToken[\s\S]*?_homeToken !== myToken/,
        'a stale render must be superseded by a newer one via a token');
});

test('the repertoire denominator is the unfiltered library total', () => {
    assert.match(src, /\/api\/library\/stats\?provider='/);
    assert.match(src, /total_songs\s*\?\?\s*stats\.total/);
    assert.match(src, /Math\.round\(\(mastered\s*\/\s*total\)\s*\*\s*100\)/);
});

test('the home + #v3-lib-home host are wired into render and reload', () => {
    assert.match(src, /id="v3-lib-home"/, 'render() must include the #v3-lib-home host');
    assert.match(src, /function reload\s*\([\s\S]*?updateLibraryHome\(\)/,
        'reload() must refresh/toggle the home');
    assert.match(src, /function applyScoreRefresh[\s\S]*?renderLibraryHome\(\)/,
        'a new score must refresh the meter + shelf');
});

test('shelf cards play the song on click', () => {
    assert.match(
        src,
        /querySelectorAll\('\.v3-kp-card'\)[\s\S]*?window\.playSong\(enc\(fn\)/,
        'a shelf card click must call window.playSong with the recents filename',
    );
});