[tests] Fix breakage on Windows

The Windows equivalent of $HOME is $LocalAppData, we need to give checks
access to that var as well or else `go` commands fail because they can't
evaluate a cache directory, which has broken the tests in the Windows
GitHub workflow.

Also be more consistent about not using trailing commas after textproto
fields, since they're unnecessary.

Change-Id: Ia3c396bd78f2d72eff18ebb2b1e1394eb878fefe
Reviewed-on: https://fuchsia-review.googlesource.com/c/shac-project/shac/+/930374
Fuchsia-Auto-Submit: Oliver Newman <olivernewman@google.com>
Reviewed-by: Ina Huh <ihuh@google.com>
Commit-Queue: Auto-Submit <auto-submit@fuchsia-infra.iam.gserviceaccount.com>

internal/engine/run_test.go

@@ -751,14 +751,19 @@ func TestRun_PassthroughEnv(t *testing.T) {
 				IsPath:    true,
 				Writeable: true,
 			},
+			// Additionally give access to HOME (or LocalAppData on Windows),
+			// which contains the Go cache, so checks that run `go run` can use
+			// cached artifacts.
 			{
-				// Additionally give access to HOME, which contains the
-				// Go cache, so checks that run `go run` can use cached
-				// artifacts.
 				Name:      "HOME",
 				IsPath:    true,
 				Writeable: true,
 			},
+			{
+				Name:      "LocalAppData",
+				IsPath:    true,
+				Writeable: true,
+			},
 		},
 		Vars: []*Var{{Name: "VAR_PREFIX"}},
 	}

internal/engine/testdata/fail_or_throw/shac.textproto

@@ -12,17 +12,23 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-# Give access to $GOCACHE (and $HOME so $GOCACHE can be inferred if unset), so
-# checks that run `go run` can use cached artifacts.
+# Give access to $GOCACHE, as well as $HOME and $LocalAppData (for Windows) so
+# $GOCACHE can be inferred if unset, so checks that run `go run` can use cached
+# artifacts.
 passthrough_env: [
     {
-        name: "HOME",
-        is_path: true,
+        name: "HOME"
+        is_path: true
         writeable: true
     },
     {
-        name: "GOCACHE",
-        is_path: true,
+       name: "LocalAppData"
+       is_path: true
+       writeable: true
+    },
+    {
+        name: "GOCACHE"
+        is_path: true
         writeable: true
     }
-]
+]

internal/engine/testdata/print/shac.textproto

@@ -12,17 +12,23 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-# Give access to $GOCACHE (and $HOME so $GOCACHE can be inferred if unset), so
-# checks that run `go run` can use cached artifacts.
+# Give access to $GOCACHE, as well as $HOME and $LocalAppData (for Windows) so
+# $GOCACHE can be inferred if unset, so checks that run `go run` can use cached
+# artifacts.
 passthrough_env: [
     {
-        name: "HOME",
-        is_path: true,
+        name: "HOME"
+        is_path: true
         writeable: true
     },
     {
-        name: "GOCACHE",
-        is_path: true,
+       name: "LocalAppData"
+       is_path: true
+       writeable: true
+    },
+    {
+        name: "GOCACHE"
+        is_path: true
         writeable: true
     }
-]
+]

shac.textproto

@@ -35,17 +35,22 @@ vars: [
 passthrough_env: [
     {
         # Provide Go commands access to the Go cache to speed up compilation.
-        name: "GOCACHE",
+        name: "GOCACHE"
         is_path: true
         writeable: true
     },
+    # The Go cache directory is computed based on $HOME (or $LocalAppData) on
+    # Windows in the absence of $GOCACHE.
+    # TODO(olivernewman): Implement support for constructing pass-throughs using
+    # Starlark, and pass through the value returned by `go env GOCACHE` instead
+    # of all of $HOME/$LocalAppData.
     {
-        # The Go cache directory is computed based on $HOME in the absence of
-        # $GOCACHE.
-        # TODO(olivernewman): Implement support for constructing pass-throughs
-        # using Starlark, and pass through the value returned by `go env
-        # GOCACHE` instead of all of $HOME.
-        name: "HOME",
+        name: "HOME"
+        is_path: true
+        writeable: true
+    },
+    {
+        name: "LocalAppData"
         is_path: true
         writeable: true
     }