e2e-test: rename back to areas (#5765)

I know, I'm a flip-flopper, but I think this is the right move as our
verbiage is `areas` everywhere else.

.vscode/settings.json

@@ -50,6 +50,7 @@
 		"out-vscode-reh/**": true,
 		"extensions/**/dist/**": true,
 		"extensions/**/out/**": true,
+		"test/e2e/out/**": true,
 		"test/smoke/out/**": true,
 		"test/automation/out/**": true,
 		"test/integration/browser/out/**": true

build/secrets/.secrets.baseline

@@ -94,6 +94,10 @@
       "path": "detect_secrets.filters.common.is_baseline_file",
       "filename": "build/secrets/.secrets.baseline"
     },
+    {
+      "path": "detect_secrets.filters.common.is_ignored_due_to_verification_policies",
+      "min_level": 2
+    },
     {
       "path": "detect_secrets.filters.heuristic.is_indirect_reference"
     },
@@ -1940,5 +1944,5 @@
       }
     ]
   },
-  "generated_at": "2024-12-16T16:39:50Z"
+  "generated_at": "2024-12-16T22:03:58Z"
 }

eslint.config.js

@@ -1269,10 +1269,10 @@ export default tseslint.config(
 			'local/code-import-patterns': [
 				'warn',
 				{
-					'target': 'test/smoke/**',
+					'target': 'test/e2e/**',
 					'restrictions': [
 						'test/automation',
-						'test/smoke/**',
+						'test/e2e/**',
 						'@vscode/*',
 						'@parcel/*',
 						'@playwright/*',

playwright.config.ts

@@ -4,15 +4,15 @@
  *--------------------------------------------------------------------------------------------*/
 
 import { defineConfig } from '@playwright/test';
-import { CustomTestOptions } from './test/e2e/features/_test.setup';
+import { CustomTestOptions } from './test/e2e/areas/_test.setup';
 import type { GitHubActionOptions } from '@midleman/github-actions-reporter';
 import { currentsReporter } from '@currents/playwright';
 
 /**
  * See https://playwright.dev/docs/test-configuration.
  */
 export default defineConfig<CustomTestOptions>({
-	globalSetup: './test/e2e/features/_global.setup.ts',
+	globalSetup: './test/e2e/areas/_global.setup.ts',
 	testDir: './test/e2e',
 	testIgnore: '**/example.test.ts',
 	testMatch: '*.test.ts',

scripts/slack-skipped-tests.js

@@ -8,7 +8,7 @@ const { execSync } = require('child_process');
 const slackSkippedTests = (slackWebhookUrl) => {
 	try {
 		const skippedTests = execSync(
-			`grep -r --include \\*.test.ts -E "describe\\.skip|test\\.skip" test/e2e/features | sed 's/\\.test\\.ts.*$/.test.ts/'`
+			`grep -r --include \\*.test.ts -E "describe\\.skip|test\\.skip" test/e2e/areas | sed 's/\\.test\\.ts.*$/.test.ts/'`
 		).toString();
 
 		const slackMessage = {

test/e2e/README.md

@@ -20,16 +20,16 @@ This document provides guidelines and setup instructions for effectively running
 
 ### Test Code Location
 
-- `test/e2e/features`
+- `test/e2e/areas`
 
-For instance, the e2e tests for the help pane are at `test/e2e/features/help/help.test.ts`
+For instance, the e2e tests for the help pane are at `test/e2e/areas/help/help.test.ts`
 
 ### Test Helpers Location
 
 - General helpers dir: `test/automation/src`
 - Positron helpers dir: `test/automation/src/positron`
 
-For each area under test, there is typically a companion class that assists with locating and interacting with elements (similar to POM pattern). For instance, the e2e tests for the help pane are at `test/e2e/features/help/help.test.ts`
+For each area under test, there is typically a companion class that assists with locating and interacting with elements (similar to POM pattern). For instance, the e2e tests for the help pane are at `test/e2e/areas/help/help.test.ts`
 
 ### Test Template
 

test/e2e/features/_test.setup.ts → test/e2e/areas/_test.setup.ts

@@ -11,26 +11,26 @@ const { test: base, expect: playwrightExpect } = playwright;
 import { join } from 'path';
 import * as os from 'os';
 import * as fs from 'fs';
-
 import path = require('path');
-// eslint-disable-next-line local/code-import-patterns
 import { rename, rm, access, mkdir } from 'fs/promises';
 import { constants } from 'fs';
+import { randomUUID } from 'crypto';
 
 // Third-party packages
-import { randomUUID } from 'crypto';
 import archiver from 'archiver';
 
 // Local imports
 import { createLogger, createApp, TestTags } from '../helpers';
 import { Application, Logger, PositronPythonFixtures, PositronRFixtures, PositronUserSettingsFixtures, UserSetting } from '../../automation';
 
+// Constants
 const TEMP_DIR = `temp-${randomUUID()}`;
 const ROOT_PATH = process.cwd();
 const LOGS_ROOT_PATH = join(ROOT_PATH, 'test-logs');
 let SPEC_NAME = '';
 let fixtureScreenshot: Buffer;
 
+// Test fixtures
 export const test = base.extend<TestFixtures, WorkerFixtures>({
 	suiteId: ['', { scope: 'worker', option: true }],