Add E2E test

Author: Mrtenz

packages/snaps-cli/src/commands/watch/__test__/invalid/input.mjs

@@ -0,0 +1,6 @@
+/* eslint-disable */
+// This file is used for the E2E watch test.
+
+export const onRpcRequest = () => {
+  // no-op
+}

packages/snaps-cli/src/commands/watch/__test__/invalid/package.json

@@ -0,0 +1,4 @@
+{
+  "name": "watch-e2e-test",
+  "version": "0.0.0"
+}

packages/snaps-cli/src/commands/watch/__test__/invalid/snap.config.ts

@@ -0,0 +1,11 @@
+import type { SnapConfig } from '@metamask/snaps-cli';
+
+const config: SnapConfig = {
+  input: 'input.mjs',
+  output: {
+    path: './dist',
+    filename: 'bundle.js',
+  },
+};
+
+export default config;

packages/snaps-cli/src/commands/watch/__test__/invalid/snap.manifest.json

@@ -0,0 +1,18 @@
+{
+  "version": "0.0.0",
+  "description": "E2E test for Snap `watch` command with invalid manifest",
+  "proposedName": "E2E Test",
+  "source": {
+    "shasum": "9r/u7rEURvdwOi8MZc9nccNVNdIFgcMXjQCGIn9Mx/s=",
+    "location": {
+      "npm": {
+        "filePath": "bundle.js",
+        "packageName": "watch-e2e-test",
+        "registry": "https://registry.npmjs.org/"
+      }
+    }
+  },
+  "initialPermissions": {},
+  "platformVersion": "9.3.0",
+  "manifestVersion": "0.1"
+}

packages/snaps-cli/src/commands/watch/watch.e2e.test.ts

@@ -64,4 +64,46 @@ describe('mm-snap watch', () => {
       );
     },
   );
+
+  it('evaluates a bundle and reports manifest warnings', async () => {
+    runner = getCommandRunner(
+      'watch',
+      ['--port', '0'],
+      resolve(__dirname, '__test__', 'invalid'),
+    );
+
+    await runner.waitForStderr(
+      /Compiled \d+ files? in \d+ms with \d+ warnings?\./u,
+    );
+
+    expect(runner.stdout).toContainEqual(
+      expect.stringMatching(/Checking the input file\./u),
+    );
+    expect(runner.stdout).toContainEqual(
+      expect.stringMatching(/Starting the development server\./u),
+    );
+    expect(runner.stdout).toContainEqual(
+      expect.stringMatching(
+        /The server is listening on http:\/\/localhost:\d+\./u,
+      ),
+    );
+    expect(runner.stdout).toContainEqual(
+      expect.stringMatching(/Building the Snap bundle\./u),
+    );
+    expect(runner.stderr).toContainEqual(
+      expect.stringMatching(
+        /Compiled \d+ files? in \d+ms with \d+ warnings?\./u,
+      ),
+    );
+    expect(runner.stderr).toContainEqual(
+      expect.stringContaining(
+        'No icon found in the Snap manifest. It is recommended to include an icon for the Snap. See https://docs.metamask.io/snaps/how-to/design-a-snap/#guidelines-at-a-glance for more information.',
+      ),
+    );
+    expect(runner.stderr).toContainEqual(
+      expect.stringContaining(
+        'The Snap exports the following handlers, but does not request permission for them: onRpcRequest (endowment:rpc).',
+      ),
+    );
+  });
 });