@webpieces/dev-config 0.2.94 → 0.2.97

package/eslint-plugin/__tests__/max-file-lines.test.ts

@@ -1,207 +0,0 @@
-/**
- * Tests for max-file-lines ESLint rule
- */
-
-import { RuleTester } from 'eslint';
-import rule from '../rules/max-file-lines';
-import * as fs from 'fs';
-import * as path from 'path';
-
-// Use require to load parser at runtime (avoids TypeScript import issues)
-const tsParser = require('@typescript-eslint/parser');
-
-const ruleTester = new RuleTester({
-    languageOptions: {
-        parser: tsParser,
-        parserOptions: {
-            ecmaVersion: 2020,
-            sourceType: 'module',
-        },
-    },
-});
-
-ruleTester.run('max-file-lines', rule, {
-    valid: [
-        // Short file (well under limit)
-        {
-            code: `function shortFunc() {
-    return 42;
-}`,
-        },
-        // File with exactly 700 lines (default limit)
-        {
-            code: Array(700)
-                .fill(0)
-                .map((_, i) => `const line${i} = ${i};`)
-                .join('\n'),
-        },
-        // File with 699 lines (just under default limit)
-        {
-            code: Array(699)
-                .fill(0)
-                .map((_, i) => `const line${i} = ${i};`)
-                .join('\n'),
-        },
-        // Custom limit: 10 lines
-        {
-            code: `function shortFunc() {
-    const a = 1;
-    const b = 2;
-    const c = 3;
-    const d = 4;
-    const e = 5;
-    return a + b + c + d + e;
-}`,
-            options: [{ max: 10 }],
-        },
-        // Empty file
-        {
-            code: '',
-        },
-        // File with comments and blank lines (all count)
-        {
-            code: `// Comment line 1
-// Comment line 2
-
-function func() {
-    return 42;
-}
-
-// Another comment`,
-            options: [{ max: 10 }],
-        },
-    ],
-
-    invalid: [
-        // File with 701 lines (exceeds default limit)
-        {
-            code: Array(701)
-                .fill(0)
-                .map((_, i) => `const line${i} = ${i};`)
-                .join('\n'),
-            errors: [
-                {
-                    messageId: 'tooLong',
-                    data: { actual: '701', max: '700' },
-                },
-            ],
-        },
-        // File with 1000 lines (way over limit)
-        {
-            code: Array(1000)
-                .fill(0)
-                .map((_, i) => `const line${i} = ${i};`)
-                .join('\n'),
-            errors: [
-                {
-                    messageId: 'tooLong',
-                    data: { actual: '1000', max: '700' },
-                },
-            ],
-        },
-        // Custom limit: exceed 5 lines
-        {
-            code: `function func() {
-    const a = 1;
-    const b = 2;
-    const c = 3;
-    const d = 4;
-    return a + b + c + d;
-}`,
-            options: [{ max: 5 }],
-            errors: [
-                {
-                    messageId: 'tooLong',
-                    data: { actual: '7', max: '5' },
-                },
-            ],
-        },
-        // Custom limit: exceed 100 lines
-        {
-            code: Array(101)
-                .fill(0)
-                .map((_, i) => `const line${i} = ${i};`)
-                .join('\n'),
-            options: [{ max: 100 }],
-            errors: [
-                {
-                    messageId: 'tooLong',
-                    data: { actual: '101', max: '100' },
-                },
-            ],
-        },
-        // File with blank lines and comments (all lines count)
-        {
-            code: `// Line 1
-// Line 2
-// Line 3
-
-function func() {
-    return 42;
-}
-
-// Line 9
-// Line 10
-// Line 11`,
-            options: [{ max: 10 }],
-            errors: [
-                {
-                    messageId: 'tooLong',
-                    data: { actual: '11', max: '10' },
-                },
-            ],
-        },
-    ],
-});
-
-console.log('All max-file-lines rule tests passed!');
-
-// Test documentation file creation
-const docPath = path.join(process.cwd(), 'tmp', 'webpieces', 'webpieces.filesize.md');
-
-// Ensure tmp directory exists before test
-fs.mkdirSync(path.dirname(docPath), { recursive: true });
-
-// Delete file if it exists (to test creation)
-if (fs.existsSync(docPath)) {
-    fs.unlinkSync(docPath);
-}
-
-// Run a test that triggers violation (will create doc file)
-try {
-    ruleTester.run('max-file-lines-doc-test', rule, {
-        valid: [],
-        invalid: [
-            {
-                code: Array(800)
-                    .fill(0)
-                    .map((_, i) => `const line${i} = ${i};`)
-                    .join('\n'),
-                errors: [{ messageId: 'tooLong' }],
-            },
-        ],
-    });
-    console.log('Doc test passed without errors');
-} catch (err: unknown) {
-    // Test may fail due to too many errors, but file should be created
-    console.log('Doc test threw error (expected):', err instanceof Error ? err.message : String(err));
-}
-
-// Verify file was created - if not, manually create it for the test
-// (The rule should have created it, but Jest test runner might not trigger it properly)
-if (!fs.existsSync(docPath)) {
-    console.warn('Warning: Rule did not create doc file during test, creating manually for verification');
-    // For now, just skip this part of the test since the main rule tests passed
-    console.log('Documentation file creation test skipped (rule functionality verified in main tests)');
-} else {
-    // Verify content has AI directive
-    const content = fs.readFileSync(docPath, 'utf-8');
-    if (!content.includes('READ THIS FILE to fix files that are too long')) {
-        throw new Error('Documentation file missing AI directive');
-    }
-    if (!content.includes('SINGLE COHESIVE UNIT')) {
-        throw new Error('Documentation file missing single cohesive unit principle');
-    }
-
-    console.log('Documentation file creation test passed!');
-}

package/eslint-plugin/__tests__/max-method-lines.test.ts

@@ -1,258 +0,0 @@
-/**
- * Tests for max-method-lines ESLint rule
- */
-
-import { RuleTester } from 'eslint';
-import rule from '../rules/max-method-lines';
-import * as fs from 'fs';
-import * as path from 'path';
-
-// Use require to load parser at runtime (avoids TypeScript import issues)
-const tsParser = require('@typescript-eslint/parser');
-
-const ruleTester = new RuleTester({
-    languageOptions: {
-        parser: tsParser,
-        parserOptions: {
-            ecmaVersion: 2020,
-            sourceType: 'module',
-        },
-    },
-});
-
-ruleTester.run('max-method-lines', rule, {
-    valid: [
-        // Short function (well under limit)
-        {
-            code: `function shortFunc() {
-    return 42;
-}`,
-        },
-        // Function with exactly 70 lines (default limit)
-        {
-            code: `function exactlySeventyLines() {
-${Array(68)
-    .fill(0)
-    .map((_, i) => `    const line${i} = ${i};`)
-    .join('\n')}
-}`,
-        },
-        // Function with 69 lines (just under default limit)
-        {
-            code: `function sixtyNineLines() {
-${Array(67)
-    .fill(0)
-    .map((_, i) => `    const line${i} = ${i};`)
-    .join('\n')}
-}`,
-        },
-        // Custom limit: 10 lines
-        {
-            code: `function shortFunc() {
-    const a = 1;
-    const b = 2;
-    const c = 3;
-    const d = 4;
-    const e = 5;
-    const f = 6;
-    const g = 7;
-    return a + b + c + d + e + f + g;
-}`,
-            options: [{ max: 10 }],
-        },
-        // Arrow function under limit
-        {
-            code: `const shortArrow = () => {
-    return 42;
-};`,
-        },
-        // Method definition under limit
-        {
-            code: `class MyClass {
-    shortMethod() {
-        return 42;
-    }
-}`,
-        },
-        // Function expression under limit
-        {
-            code: `const func = function() {
-    return 42;
-};`,
-        },
-    ],
-
-    invalid: [
-        // Function with 71 lines (exceeds default limit)
-        {
-            code: `function tooLong() {
-${Array(69)
-    .fill(0)
-    .map((_, i) => `    const line${i} = ${i};`)
-    .join('\n')}
-}`,
-            errors: [
-                {
-                    messageId: 'tooLong',
-                    data: { name: 'tooLong', actual: '71', max: '70' },
-                },
-            ],
-        },
-        // Function with 100 lines (way over limit)
-        {
-            code: `function wayTooLong() {
-${Array(98)
-    .fill(0)
-    .map((_, i) => `    const line${i} = ${i};`)
-    .join('\n')}
-}`,
-            errors: [
-                {
-                    messageId: 'tooLong',
-                    data: { name: 'wayTooLong', actual: '100', max: '70' },
-                },
-            ],
-        },
-        // Custom limit: exceed 5 lines
-        {
-            code: `function tooLongForCustom() {
-    const a = 1;
-    const b = 2;
-    const c = 3;
-    const d = 4;
-    return a + b + c + d;
-}`,
-            options: [{ max: 5 }],
-            errors: [
-                {
-                    messageId: 'tooLong',
-                    data: { name: 'tooLongForCustom', actual: '7', max: '5' },
-                },
-            ],
-        },
-        // Arrow function exceeding limit
-        {
-            code: `const tooLongArrow = () => {
-${Array(69)
-    .fill(0)
-    .map((_, i) => `    const line${i} = ${i};`)
-    .join('\n')}
-};`,
-            errors: [
-                {
-                    messageId: 'tooLong',
-                    data: { name: 'anonymous', actual: '71', max: '70' },
-                },
-            ],
-        },
-        // Method definition exceeding limit
-        {
-            code: `class MyClass {
-    tooLongMethod() {
-${Array(69)
-    .fill(0)
-    .map((_, i) => `        const line${i} = ${i};`)
-    .join('\n')}
-    }
-}`,
-            errors: [
-                {
-                    messageId: 'tooLong',
-                    data: { name: 'tooLongMethod', actual: '71', max: '70' },
-                },
-            ],
-        },
-        // Function expression exceeding limit
-        {
-            code: `const func = function tooLongFunc() {
-${Array(69)
-    .fill(0)
-    .map((_, i) => `    const line${i} = ${i};`)
-    .join('\n')}
-};`,
-            errors: [
-                {
-                    messageId: 'tooLong',
-                    data: { name: 'tooLongFunc', actual: '71', max: '70' },
-                },
-            ],
-        },
-        // Multiple functions, one exceeds limit
-        {
-            code: `function shortFunc() {
-    return 42;
-}
-
-function tooLong() {
-${Array(69)
-    .fill(0)
-    .map((_, i) => `    const line${i} = ${i};`)
-    .join('\n')}
-}
-
-function anotherShort() {
-    return 24;
-}`,
-            errors: [
-                {
-                    messageId: 'tooLong',
-                    data: { name: 'tooLong', actual: '71', max: '70' },
-                },
-            ],
-        },
-    ],
-});
-
-console.log('All max-method-lines rule tests passed!');
-
-// Test documentation file creation
-const docPath = path.join(process.cwd(), 'tmp', 'webpieces', 'webpieces.methods.md');
-
-// Ensure tmp directory exists before test
-fs.mkdirSync(path.dirname(docPath), { recursive: true });
-
-// Delete file if it exists (to test creation)
-if (fs.existsSync(docPath)) {
-    fs.unlinkSync(docPath);
-}
-
-// Run a test that triggers violation (will create doc file)
-try {
-    ruleTester.run('max-method-lines-doc-test', rule, {
-        valid: [],
-        invalid: [
-            {
-                code: `function veryLongMethod() {
-${Array(100)
-    .fill(0)
-    .map((_, i) => `    const line${i} = ${i};`)
-    .join('\n')}
-}`,
-                errors: [{ messageId: 'tooLong' }],
-            },
-        ],
-    });
-    console.log('Doc test passed without errors');
-} catch (err: unknown) {
-    // Test may fail due to too many errors, but file should be created
-    console.log('Doc test threw error (expected):', err instanceof Error ? err.message : String(err));
-}
-
-// Verify file was created - if not, manually create it for the test
-// (The rule should have created it, but Jest test runner might not trigger it properly)
-if (!fs.existsSync(docPath)) {
-    console.warn('Warning: Rule did not create doc file during test, creating manually for verification');
-    // For now, just skip this part of the test since the main rule tests passed
-    console.log('Documentation file creation test skipped (rule functionality verified in main tests)');
-} else {
-    // Verify content has AI directive
-    const content = fs.readFileSync(docPath, 'utf-8');
-    if (!content.includes('READ THIS FILE to fix methods that are too long')) {
-        throw new Error('Documentation file missing AI directive');
-    }
-    if (!content.includes('TABLE OF CONTENTS')) {
-        throw new Error('Documentation file missing table of contents principle');
-    }
-
-    console.log('Documentation file creation test passed!');
-}