Apply suggestions from code review

Author: mafredri

agent/agent_test.go

@@ -300,9 +300,7 @@ func TestAgent_Session_EnvironmentVariables(t *testing.T) {
 		},
 	}
 	banner := codersdk.ServiceBannerConfig{}
-	fs := afero.NewMemMapFs()
 	session := setupSSHSession(t, manifest, banner, nil, func(_ *agenttest.Client, opts *agent.Options) {
-		opts.Filesystem = fs
 		opts.ScriptDataDir = tmpdir
 		opts.EnvironmentVariables["MY_OVERRIDE"] = "true"
 	})

cli/agent.go

@@ -345,7 +345,7 @@ func (r *RootCmd) workspaceAgent() *clibase.Cmd {
 			Flag:        "script-data-dir",
 			Default:     os.TempDir(),
 			Description: "Specify the location for storing script data.",
-			Env:         "CODER_AGENT_SCRIPT_DIR",
+			Env:         "CODER_AGENT_SCRIPT_DATA_DIR",
 			Value:       clibase.StringOf(&scriptDataDir),
 		},
 		{

cli/testdata/coder_agent_--help.golden

@@ -33,7 +33,7 @@ OPTIONS:
       --prometheus-address string, $CODER_AGENT_PROMETHEUS_ADDRESS (default: 127.0.0.1:2112)
           The bind address to serve Prometheus metrics.
 
-      --script-data-dir string, $CODER_AGENT_SCRIPT_DIR (default: /tmp)
+      --script-data-dir string, $CODER_AGENT_SCRIPT_DATA_DIR (default: /tmp)
           Specify the location for storing script data.
 
       --ssh-max-timeout duration, $CODER_AGENT_SSH_MAX_TIMEOUT (default: 72h)