fix(expect-expect): make assert work well with expect-expect (#508)

src/rules/expect-expect.ts

@@ -60,7 +60,7 @@ export default createEslintRule<Options, MESSAGE_ID>({
       noAssertions: 'Test has no assertions'
     }
   },
-  defaultOptions: [{ assertFunctionNames: ['expect'], additionalTestBlockFunctions: [] }],
+  defaultOptions: [{ assertFunctionNames: ['expect', 'assert'], additionalTestBlockFunctions: [] }],
   create(context, [{ assertFunctionNames = ['expect'], additionalTestBlockFunctions = [] }]) {
     const unchecked: TSESTree.CallExpression[] = []
     const settings = parsePluginSettings(context.settings)
@@ -108,16 +108,12 @@ export default createEslintRule<Options, MESSAGE_ID>({
         }
       },
       'Program:exit'() {
-        const isActuallyNotValid = unchecked.filter((node) => node.arguments.some((arg) =>
-          (arg.type === AST_NODE_TYPES.Literal && arg.value === "assert")))
-
-        if (!isActuallyNotValid.length)
-          unchecked.forEach((node) => {
-            context.report({
-              node: node.callee,
-              messageId: 'noAssertions'
-            })
+        unchecked.forEach((node) => {
+          context.report({
+            node: node.callee,
+            messageId: 'noAssertions'
           })
+        })
       }
     }
   }

tests/expect-expect.test.ts

@@ -15,6 +15,15 @@ ruleTester.run(RULE_NAME, rule, {
     `test('assert', () => {
   assert('foo' !== 'bar', 'foo should not be equal to bar')
 })`,
+    `test('cleanPrCommitTitle', () => {
+	const clean = 'Something done';
+	assert.equal(cleanPrCommitTitle('Something done (#123)', 123), clean);
+	assert.equal(cleanPrCommitTitle('  Something done  (#123)  ', 123), clean);
+	assert.equal(cleanPrCommitTitle(' Something done ', 123), clean);
+
+	assert.notEqual(cleanPrCommitTitle('Something done (fixes #123)', 123), clean);
+	assert.notEqual(cleanPrCommitTitle('Something done (#23454)', 123), clean);
+});`
   ],
   invalid: [
     {