test: Reduce output noise (#316)

* test: Prevent invalid React HTML attribute error

This context property was changed in the component itself, but
overlooked in the test as it did not create a failure, just an error
log.

```
    console.error
      Warning: React does not recognize the `projectPath` prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase `projectpath` instead. If you accidentally passed it from a parent component, remove it from the DOM element.
          at code
          at CodeBlock (/Users/davidcalhoun/Sites/a8c/studio/src/components/assistant-code-block.tsx:22:32)

      15 |
      16 |      it( 'should render inline styles for language-generic code', () => {
    > 17 |              render( <CodeBlock children="example-code" /> );
         |                    ^
      18 |
      19 |              expect( screen.getByText( 'example-code' ) ).toBeVisible();
      20 |              expect( screen.queryByText( 'Copy' ) ).not.toBeInTheDocument();
```

* test: Exclude expected console error and warning message in test output

Minimize unnecessary noise in test output.

```
    console.warn
      PHP.run() output was: #!/usr/bin/env php

      at _NodePHP.run (node_modules/@php-wasm/node/index.cjs:72997:17)

    console.error
      PHPExecutionFailureError2: PHP.run() failed with exit code 1 and the following output:
          at _NodePHP.run (/Users/davidcalhoun/Sites/a8c/studio/node_modules/@php-wasm/node/index.cjs:72998:23) {
        response: _PHPResponse {
          httpStatusCode: 500,
          headers: { 'x-powered-by': [Array], 'content-type': [Array] },
          bytes: Uint8Array(19) [
             35,  33,  47, 117, 115, 114,
             47,  98, 105, 110,  47, 101,
            110, 118,  32, 112, 104, 112,
             10
          ],
          exitCode: 1,
          errors: ''
        },
        source: 'request'
      }

      at _NodePHP.run (node_modules/@php-wasm/node/index.cjs:73003:17)
```

* test: Replace use of deprecated `fs.rmdir` with `fs.rm`

Addresses warning logged in test output:

```
(node:82794) [DEP0147] DeprecationWarning: In future versions of Node.js, fs.rmdir(path, { recursive: true }) will be removed. Use fs.rm(path, { recursive: true }) instead
(Use `node --trace-deprecation ...` to show where the warning was created)
```

* test: Filter verbose console messages

Reduce the noise in test results caused by various usages of `console`
within the source.

Author: dcalhoun

jest-setup.ts

@@ -29,6 +29,39 @@ if ( typeof window !== 'undefined' ) {
 nock.disableNetConnect();
 nock.enableNetConnect( 'raw.githubusercontent.com' );
 
+/**
+ * TODO: Replace this manual filter with a more robust logger that can be
+ * disabled for the testing environment. Consider enabling a lint rule that
+ * discourages direct use of `console` methods.
+ */
+function filteredConsole( level: ( ...args: any[] ) => void ) {
+	const ignoredMessages = [
+		'Loaded user data from ',
+		'Saved user data to ',
+		'Server start',
+		'Would have bumped stat: ',
+		'Starting new session',
+		'App version: ',
+		'Built from commit: mock-hash',
+		'Local timezone: UTC',
+		'App locale: ',
+		'System locale: ',
+		'Used language: ',
+	];
+
+	return ( ...args: any[] ) => {
+		if ( ignoredMessages.some( ( ignoredMessage ) => args[ 0 ].includes( ignoredMessage ) ) ) {
+			return;
+		}
+
+		level( ...args );
+	};
+}
+console.log = filteredConsole( console.log );
+console.error = filteredConsole( console.error );
+console.warn = filteredConsole( console.warn );
+console.info = filteredConsole( console.info );
+
 // We consider the app to be online by default.
 jest.mock( './src/hooks/use-offline', () => ( {
 	useOffline: jest.fn().mockReturnValue( false ),

src/components/tests/assistant-code-block.test.tsx

@@ -8,7 +8,7 @@ describe( 'createCodeComponent', () => {
 	const contextProps = {
 		blocks: [],
 		updateMessage: jest.fn(),
-		projectPath: '/path/to/project',
+		siteId: '1',
 		messageId: 1,
 	};
 	const CodeBlock = createCodeComponent( contextProps );
@@ -152,7 +152,7 @@ describe( 'createCodeComponent', () => {
 					},
 				],
 				updateMessage: jest.fn(),
-				projectPath: '/path/to/project',
+				siteId: '1',
 				messageId: 1,
 			};
 			const CodeBlock = createCodeComponent( contextProps );

src/tests/execute-wp-cli.test.ts

@@ -18,7 +18,7 @@ describe( 'executeWPCli', () => {
 		fs.writeFileSync( path.join( tmpPath, 'index.php' ), '' );
 	} );
 	afterAll( () => {
-		fs.rmdirSync( tmpPath, { recursive: true } );
+		fs.rmSync( tmpPath, { recursive: true } );
 	} );
 
 	it( 'should execute wp-cli version command and return stdout and stderr', async () => {
@@ -31,6 +31,10 @@ describe( 'executeWPCli', () => {
 	} );
 
 	it( 'should return error if wp-cli command does not exist', async () => {
+		const originalConsoleError = console.error;
+		const originalConsoleWarn = console.warn;
+		console.error = jest.fn();
+		console.warn = jest.fn();
 		const args = [ 'yoda' ];
 
 		const result = await executeWPCli( tmpPath, args );
@@ -39,6 +43,9 @@ describe( 'executeWPCli', () => {
 		expect( result.stderr ).toContain(
 			"'yoda' is not a registered wp command. See 'wp help' for available commands."
 		);
+
+		console.error = originalConsoleError;
+		console.warn = originalConsoleWarn;
 	} );
 
 	it( 'should return the correct version of WP-CLI', async () => {