Enable PWA on production (#1195)

Remove it locally as the SW code bails in that scenario anyway.

Author: microbit-robert

src/flags.ts

@@ -69,7 +69,7 @@ const allFlags: FlagMetadata[] = [
   { name: "noWelcome", defaultOnStages: ["local", "REVIEW"] },
   {
     name: "pwa",
-    defaultOnStages: ["local", "REVIEW", "STAGING"],
+    defaultOnStages: ["REVIEW", "STAGING", "PRODUCTION"],
   },
 ];
 

src/simulator/Simulator.tsx

@@ -46,7 +46,7 @@ const Simulator = ({
   const [brand500] = useToken("colors", ["brand.500"]);
   const url = useMemo(() => {
     const production =
-      "https://python-simulator.usermbit.org/v/0.1/simulator.html";
+      "https://python-simulator.usermbit.org/v/0.1/simulator.html?flag=sw";
     const staging =
       "https://python-simulator.usermbit.org/staging/simulator.html?flag=sw";
 

src/workbench/flags.test.ts

@@ -14,12 +14,15 @@ describe("flags", () => {
     expect(flags.dndDebug).toEqual(false);
   });
 
-  it("enables nothing in production", () => {
+  it("only enables PWA in production", () => {
     const params = new URLSearchParams([]);
 
     const flags = flagsForParams("PRODUCTION", params);
 
-    expect(Object.values(flags).every((x) => !x)).toEqual(true);
+    expect(flags.pwa).toBe(true);
+    const { pwa, ...filteredFlags } = flags;
+
+    expect(Object.values(filteredFlags).every((x) => !x)).toEqual(true);
   });
 
   it("enable specific flag", () => {
@@ -29,7 +32,7 @@ describe("flags", () => {
 
     expect(
       Object.entries(flags).every(
-        ([flag, status]) => (flag === "noWelcome") === status
+        ([flag, status]) => (flag === "noWelcome" || flag === "pwa") === status
       )
     ).toEqual(true);
   });