Fail integration tests on missing env vars instead of skipping

Restore the original fail-loudly behavior that existed before this PR
(via env.MustGet). Silently skipping can hide CI misconfigurations,
making tests appear green when they aren't actually running.

Author: alxgsv

test/file.go

@@ -81,7 +81,7 @@ func fileLocalCopy(t *testing.T, r *testenv.Runner) {
 
 func fileRemoteCopy(t *testing.T, r *testenv.Runner) {
 	if r.Artifacts.CustomStorage == "" {
-		t.Skip("skipping: CUSTOM_STORAGE_BUCKET env var required")
+		t.Fatal("CUSTOM_STORAGE_BUCKET env var is required")
 	}
 	ctx := context.Background()
 	_, err := r.File.RemoteCopy(

test/integration_test.go

@@ -46,7 +46,7 @@ func TestIntegration(t *testing.T) {
 	secretKey := os.Getenv("SECRET_KEY")
 	publicKey := os.Getenv("PUBLIC_KEY")
 	if secretKey == "" || publicKey == "" {
-		t.Skip("skipping integration test: SECRET_KEY and PUBLIC_KEY env vars required")
+		t.Fatal("SECRET_KEY and PUBLIC_KEY env vars are required")
 	}
 
 	creds := ucare.APICreds{