fix(runtime): adds a testing check to the forceUpdate method (#4682)

This commit adds a check for `isTesting` in addition to `isBrowser` so that the forceUpdate logic can run in test environments without needing to change the testing build conditional values. These changes were causing tests to fail in Framework.

Author: tanner-reits

src/runtime/test/globals.spec.tsx

@@ -33,10 +33,10 @@ describe('globals', () => {
   });
 
   it('build values', () => {
-    expect(Build.isBrowser).toBe(true);
+    expect(Build.isBrowser).toBe(false);
     expect(Build.isDev).toBe(true);
     expect(Build.isTesting).toBe(true);
-    expect(Build.isServer).toBe(false);
+    expect(Build.isServer).toBe(true);
   });
 
   it('Env is defined', () => {

src/runtime/update-component.ts

@@ -362,7 +362,7 @@ export const postUpdateComponent = (hostRef: d.HostRef) => {
 };
 
 export const forceUpdate = (ref: any) => {
-  if (BUILD.updatable && Build.isBrowser) {
+  if (BUILD.updatable && (Build.isBrowser || Build.isTesting)) {
     const hostRef = getHostRef(ref);
     const isConnected = hostRef.$hostElement$.isConnected;
     if (

src/testing/platform/testing-build.ts

@@ -2,7 +2,7 @@ import type * as d from '@stencil/core/internal';
 
 export const Build: d.UserBuildConditionals = {
   isDev: true,
-  isBrowser: true,
-  isServer: false,
+  isBrowser: false,
+  isServer: true,
   isTesting: true,
 };

test/end-to-end/src/build-data/build-data.spec.ts

@@ -11,7 +11,7 @@ describe('build-data', () => {
     expect(root).toEqualHtml(`
       <build-data>
         <p>isDev: true</p>
-        <p>isBrowser: true</p>
+        <p>isBrowser: false</p>
         <p>isTesting: true</p>
       </build-data>
     `);