add else

scotthart · scotthart · commit 63acd82f2634 · 2026-01-18T13:26:55.000-05:00
diff --git a/google/cloud/testing_util/credentials.h b/google/cloud/testing_util/credentials.h
@@ -53,8 +53,11 @@ struct TestCredentialsVisitor : public internal::CredentialsVisitor {
   }
   void visit(internal::ServiceAccountConfig const& cfg) override {
     name = "ServiceAccountConfig";
-    if (cfg.json_object()) json_object = *cfg.json_object();
-    json_object = "cfg.file_path testing not implemented";
+    if (cfg.json_object()) {
+      json_object = *cfg.json_object();
+    } else {
+      json_object = "cfg.file_path testing not implemented";
+    }
   }
   void visit(internal::ExternalAccountConfig const& cfg) override {
     name = "ExternalAccountConfig";

Original file line number	Diff line number	Diff line change
`@@ -53,8 +53,11 @@ struct TestCredentialsVisitor : public internal::CredentialsVisitor {`
`53`	`53`	`}`
`54`	`54`	`void visit(internal::ServiceAccountConfig const& cfg) override {`
`55`	`55`	`name = "ServiceAccountConfig";`
`56`		`- if (cfg.json_object()) json_object = *cfg.json_object();`
`57`		`- json_object = "cfg.file_path testing not implemented";`
	`56`	`+ if (cfg.json_object()) {`
	`57`	`+ json_object = *cfg.json_object();`
	`58`	`+ } else {`
	`59`	`+ json_object = "cfg.file_path testing not implemented";`
	`60`	`+ }`
`58`	`61`	`}`
`59`	`62`	`void visit(internal::ExternalAccountConfig const& cfg) override {`
`60`	`63`	`name = "ExternalAccountConfig";`