fix(ssr): throw compile-time error on @wire-decorated getter/setter/method

Author: wjhsf

packages/@lwc/engine-server/src/__tests__/fixtures.spec.ts

@@ -11,6 +11,7 @@ import { rollup } from 'rollup';
 import lwcRollupPlugin from '@lwc/rollup-plugin';
 import { testFixtureDir, formatHTML } from '@lwc/test-utils-lwc-internals';
 import { setFeatureFlagForTest } from '../index';
+import type { FeatureFlagName } from '@lwc/features/dist/types';
 import type { RollupLwcOptions } from '@lwc/rollup-plugin';
 import type * as lwc from '../index';
 
@@ -128,16 +129,17 @@ function testFixtures(options?: RollupLwcOptions) {
             // the LightningElement. Therefor the compiled module should also be evaluated in the
             // same sandbox registry as the engine.
             const lwcEngineServer = await import('../index');
-            const module = (await import(compiledFixturePath)) as FixtureModule;
-
-            const features = module!.features ?? [];
-            features.forEach((flag) => {
-                lwcEngineServer!.setFeatureFlagForTest(flag, true);
-            });
 
             let result;
             let err;
+            let features: FeatureFlagName[] = [];
             try {
+                const module = (await import(compiledFixturePath)) as FixtureModule;
+
+                features = module!.features ?? [];
+                features.forEach((flag) => {
+                    lwcEngineServer!.setFeatureFlagForTest(flag, true);
+                });
                 result = formatHTML(
                     lwcEngineServer!.renderComponent(
                         module!.tagName,
@@ -146,10 +148,10 @@ function testFixtures(options?: RollupLwcOptions) {
                     )
                 );
             } catch (_err: any) {
-                if (_err.name === 'AssertionError') {
+                if (_err?.name === 'AssertionError') {
                     throw _err;
                 }
-                err = _err.message;
+                err = _err?.message || 'An empty error occurred?!';
             }
 
             features.forEach((flag) => {

packages/@lwc/engine-server/src/__tests__/fixtures/wire/errors/throws-on-computed-getter/config.json

@@ -0,0 +1,6 @@
+{
+    "ssrFiles": {
+        "error": "error-ssr.txt",
+        "expected": "expected-ssr.html"
+    }
+}

packages/@lwc/engine-server/src/__tests__/fixtures/wire/errors/throws-on-computed-getter/error-ssr.txt

@@ -0,0 +1 @@
+@wire cannot be used on computed properties in SSR context.

…/errors/throws-on-computed-key/error.txt …rors/throws-on-computed-getter/error.txtpackages/@lwc/engine-server/src/__tests__/fixtures/wire/errors/throws-on-computed-key/error.txt renamed to packages/@lwc/engine-server/src/__tests__/fixtures/wire/errors/throws-on-computed-getter/error.txt packages/@lwc/engine-server/src/__tests__/fixtures/wire/errors/throws-on-computed-key/error.txt renamed to packages/@lwc/engine-server/src/__tests__/fixtures/wire/errors/throws-on-computed-getter/error.txt

@@ -8,11 +8,6 @@ export default class Wire extends LightningElement {
         throw new Error('getter should not be called');
     }
 
-    @wire(adapter, { name: 'symbol' })
-    set [sym](v) {
-        throw new Error('setter should not be called');
-    }
-
     get exposedSymbol() {
         return this[sym];
     }