mm_sql 1.4.1 → 1.4.3

package/detailed_error_test.js

@@ -0,0 +1,171 @@
+const { mysqlAdmin } = require('../mm_mysql');
+
+/**
+ * 测试MySQL语法错误
+ * @param {Object} mysql - MySQL实例
+ */
+async function testMysqlSyntax(mysql) {
+  console.log('\n测试1: 语法错误的SQL...');
+  try {
+    await mysql.run('SELECT * FROM'); // 语法错误
+    console.log('❌ MySQL应该对语法错误SQL抛出错误，但没有抛出');
+  } catch (error) {
+    console.log('✅ MySQL正确抛出语法错误:', error.message);
+  }
+}
+
+/**
+ * 测试MySQL不存在的表
+ * @param {Object} mysql - MySQL实例
+ */
+async function testMysqlTable(mysql) {
+  console.log('\n测试2: 不存在的表...');
+  try {
+    const result = await mysql.run('SELECT * FROM non_existent_table_xyz');
+    console.log('MySQL返回结果:', result);
+    console.log('结果类型:', typeof result);
+    console.log('结果长度:', Array.isArray(result) ? result.length : 'N/A');
+    
+    if (Array.isArray(result) && result.length === 0) {
+      console.log('⚠️ MySQL对不存在的表返回空数组而不是抛出错误');
+    } else {
+      console.log('❌ MySQL应该对不存在的表抛出错误，但没有抛出');
+    }
+  } catch (error) {
+    console.log('✅ MySQL正确抛出表不存在错误:', error.message);
+  }
+}
+
+/**
+ * 测试MySQL不存在的数据库
+ * @param {Object} mysql - MySQL实例
+ */
+async function testMysqlDb(mysql) {
+  console.log('\n测试3: 不存在的数据库...');
+  try {
+    await mysql.run('SELECT * FROM non_existent_db.non_existent_table');
+    console.log('❌ MySQL应该对不存在的数据库抛出错误，但没有抛出');
+  } catch (error) {
+    console.log('✅ MySQL正确抛出数据库不存在错误:', error.message);
+  }
+}
+
+/**
+ * 测试MySQL权限错误
+ * @param {Object} mysql - MySQL实例
+ */
+async function testMysqlPerm(mysql) {
+  console.log('\n测试4: 权限错误...');
+  try {
+    await mysql.run('DROP TABLE IF EXISTS important_system_table');
+    console.log('❌ MySQL应该对权限错误抛出错误，但没有抛出');
+  } catch (error) {
+    console.log('✅ MySQL正确抛出权限错误:', error.message);
+  }
+}
+
+/**
+ * 详细测试MySQL错误处理
+ */
+async function mysqlErrorTest() {
+  console.log('=== 详细测试MySQL错误处理 ===');
+  
+  try {
+    // 创建mysqlAdmin实例
+    const mysql = mysqlAdmin('detailed_test', {
+      host: 'localhost',
+      port: 3306,
+      user: 'root',
+      password: 'Asd159357',
+      database: 'mm'
+    });
+    
+    console.log('✅ MySQL实例创建成功');
+    
+    await testMysqlSyntax(mysql);
+    await testMysqlTable(mysql);
+    await testMysqlDb(mysql);
+    await testMysqlPerm(mysql);
+    
+  } catch (error) {
+    console.log('❌ MySQL实例创建失败:', error.message);
+  }
+}
+
+/**
+ * 测试SQLite语法错误
+ * @param {Object} sqlite - SQLite实例
+ */
+async function testSqliteSyntax(sqlite) {
+  console.log('\n测试1: 语法错误的SQL...');
+  try {
+    await sqlite.run('SELECT * FROM'); // 语法错误
+    console.log('❌ SQLite应该对语法错误SQL抛出错误，但没有抛出');
+  } catch (error) {
+    console.log('✅ SQLite正确抛出语法错误:', error.message);
+  }
+}
+
+/**
+ * 测试SQLite不存在的表
+ * @param {Object} sqlite - SQLite实例
+ */
+async function testSqliteTable(sqlite) {
+  console.log('\n测试2: 不存在的表...');
+  try {
+    const result = await sqlite.run('SELECT * FROM non_existent_table_xyz');
+    console.log('SQLite返回结果:', result);
+    console.log('结果类型:', typeof result);
+    console.log('结果长度:', Array.isArray(result) ? result.length : 'N/A');
+    
+    if (Array.isArray(result) && result.length === 0) {
+      console.log('⚠️ SQLite对不存在的表返回空数组而不是抛出错误');
+    } else {
+      console.log('❌ SQLite应该对不存在的表抛出错误，但没有抛出');
+    }
+  } catch (error) {
+    console.log('✅ SQLite正确抛出表不存在错误:', error.message);
+  }
+}
+
+/**
+ * 测试SQLite错误处理
+ */
+async function sqliteErrorTest() {
+  console.log('\n=== 详细测试SQLite错误处理 ===');
+  
+  try {
+    const { sqliteAdmin } = require('../mm_sqlite');
+    
+    // 创建sqliteAdmin实例
+    const sqlite = sqliteAdmin('detailed_test', {
+      dir: './db',
+      database: 'mm'
+    });
+    
+    console.log('✅ SQLite实例创建成功');
+    
+    await testSqliteSyntax(sqlite);
+    await testSqliteTable(sqlite);
+    
+  } catch (error) {
+    console.log('❌ SQLite实例创建失败:', error.message);
+  }
+}
+
+/**
+ * 主测试函数
+ */
+async function main() {
+  console.log('开始详细测试数据库错误处理...\n');
+  
+  await detailedMysqlErrorTest();
+  await detailedSqliteErrorTest();
+  
+  console.log('\n详细测试完成');
+}
+
+// 运行测试
+if (require.main === module) {
+  main().catch(console.error);
+}

package/eslint.config.js

@@ -0,0 +1,235 @@
+'use strict';
+
+const namingConventionPlugin = require('mm_eslint');
+
+/**
+ * 基于个人规则的ESLint配置
+ * 符合最新ESLint Flat Config格式
+ */
+
+module.exports = [
+  {
+    files: ['**/*.js'],
+    plugins: {
+      'naming-convention': namingConventionPlugin,
+      jsdoc: require('eslint-plugin-jsdoc'),
+    },
+    languageOptions: {
+      ecmaVersion: 'latest',
+      sourceType: 'module',
+      parserOptions: {
+        ecmaVersion: 'latest',
+        sourceType: 'module',
+      },
+    },
+
+    // 命名规范规则 - 由自定义插件处理
+    rules: {
+      // 自定义命名规范插件规则（优先使用）
+      'naming-convention/class-name': 'error',
+      'naming-convention/function-name': 'error',
+      'naming-convention/method-name': 'error',
+      'naming-convention/variable-name': 'warn',
+      'naming-convention/constant-name': 'error',
+      'naming-convention/private-method-naming': 'warn',
+      'naming-convention/private-variable-naming': 'warn',
+      'naming-convention/param-name': 'warn',
+      'naming-convention/property-name': 'warn',
+      'naming-convention/instance-property': 'warn',
+      'naming-convention/prototype-method': 'error',
+
+      // 禁用与命名规范插件冲突的默认规则
+      camelcase: 'off',
+      'id-match': 'off',
+      'new-cap': 'off',
+      'id-length': 'off',
+      'id-denylist': 'off',
+      'id-blacklist': 'off',
+    },
+  },
+
+  {
+    // 代码风格规则
+    rules: {
+      // 最大行长度100字符
+      'max-len': [
+        'error',
+        {
+          code: 100,
+          ignoreComments: true,
+          ignoreUrls: true,
+          ignoreStrings: true,
+          ignoreTemplateLiterals: true,
+        },
+      ],
+
+      // 缩进2空格
+      indent: [
+        'error',
+        2,
+        {
+          SwitchCase: 1,
+          VariableDeclarator: 1,
+          outerIIFEBody: 1,
+          MemberExpression: 1,
+          FunctionDeclaration: { parameters: 1, body: 1 },
+          FunctionExpression: { parameters: 1, body: 1 },
+          CallExpression: { arguments: 1 },
+          ArrayExpression: 1,
+          ObjectExpression: 1,
+          ImportDeclaration: 1,
+          flatTernaryExpressions: false,
+          ignoreComments: false,
+        },
+      ],
+
+      // 单引号
+      quotes: [
+        'error',
+        'single',
+        {
+          avoidEscape: true,
+          allowTemplateLiterals: true,
+        },
+      ],
+
+      // 禁止制表符
+      'no-tabs': 'error',
+
+      // 行尾分号
+      semi: ['error', 'always'],
+
+      // 逗号风格
+      'comma-style': ['error', 'last'],
+
+      // 逗号间距
+      'comma-spacing': [
+        'error',
+        {
+          before: false,
+          after: true,
+        },
+      ],
+    },
+  },
+
+  {
+    // 错误处理规则
+    rules: {
+      // 必须进行参数校验
+      'no-unused-vars': [
+        'error',
+        {
+          args: 'all',
+          caughtErrors: 'all',
+          ignoreRestSiblings: true,
+        },
+      ],
+
+      // 建议使用try-catch
+      'no-unsafe-finally': 'warn',
+
+      // 禁止直接抛出字符串
+      'no-throw-literal': 'error',
+    },
+  },
+
+  {
+    // 最佳实践规则
+    rules: {
+      // 优先使用async/await
+      'prefer-promise-reject-errors': 'error',
+
+      // 避免副作用
+      'no-param-reassign': [
+        'error',
+        {
+          props: true,
+          ignorePropertyModificationsFor: [
+            'acc', // for reduce accumulators
+            'accumulator', // for reduce accumulators
+            'e', // for e.returnvalue
+            'ctx', // for Koa routing
+            'context', // for Koa routing
+            'req', // for Express requests
+            'request', // for Express requests
+            'res', // for Express responses
+            'response', // for Express responses
+            '$scope', // for Angular 1 scopes
+            'staticContext', // for ReactRouter context
+          ],
+        },
+      ],
+
+      // 优先链式编程
+      'prefer-object-spread': 'error',
+
+      // 优先函数式编程
+      'prefer-arrow-callback': 'error',
+
+      // 方法长度限制
+      'max-lines-per-function': [
+        'warn',
+        {
+          max: 40,
+          skipBlankLines: true,
+          skipComments: true,
+        },
+      ],
+
+      // 复杂度限制
+      complexity: ['warn', 10],
+    },
+  },
+
+  {
+    // JSDoc规则
+    rules: {
+      // 要求公开方法有JSDoc注释
+      'jsdoc/require-jsdoc': [
+        'warn',
+        {
+          publicOnly: true,
+          require: {
+            FunctionDeclaration: true,
+            MethodDefinition: true,
+            ClassDeclaration: true,
+            ArrowFunctionExpression: false,
+            FunctionExpression: false,
+          },
+        },
+      ],
+
+      // 检查JSDoc语法
+      'jsdoc/check-alignment': 'warn',
+      'jsdoc/check-indentation': 'warn',
+      'jsdoc/check-param-names': 'warn',
+      'jsdoc/check-syntax': 'warn',
+      'jsdoc/check-tag-names': 'warn',
+      'jsdoc/check-types': 'warn',
+      'jsdoc/no-undefined-types': 'warn',
+      'jsdoc/require-description': 'warn',
+      'jsdoc/require-param': 'warn',
+      'jsdoc/require-param-description': 'warn',
+      'jsdoc/require-param-name': 'warn',
+      'jsdoc/require-param-type': 'warn',
+      'jsdoc/require-returns': 'warn',
+      'jsdoc/require-returns-check': 'warn',
+      'jsdoc/require-returns-description': 'warn',
+      'jsdoc/require-returns-type': 'warn',
+      'jsdoc/valid-types': 'warn',
+    },
+  },
+
+  {
+    // 忽略规则
+    ignores: [
+      'node_modules/**',
+      'dist/**',
+      'build/**',
+      'coverage/**',
+      '*.min.js',
+      'eslint.config.js'
+    ],
+  },
+];