feat(auth): enable emulator support on desktop (#1423)

* add auth emulator support * fix ios number parse * update the documentation part * reduce lint warning * more lint warnings * code format * use environment to decide using emulator or not * fix a typo * add readme entry for FirebaseApp.GetApps() * update for review comment * add missing ` * make json request always do restrict json * add readme info * prepare for review * some minor tweak * code format * update review comments * update the unit test after request json change --------- Co-authored-by: Cynthia Jiang <cynthiajiang@google.com>

Author: cynthiajoan

app/rest/request_json.h

@@ -41,6 +41,7 @@ class RequestJson : public Request {
  explicit RequestJson(const char* schema) : application_data_(new FbsTypeT()) {
  flatbuffers::IDLOptions fbs_options;
  fbs_options.skip_unexpected_fields_in_json = true;
+ fbs_options.strict_json = true;
  parser_.reset(new flatbuffers::Parser(fbs_options));
 
  bool parse_status = parser_->Parse(schema);

app/rest/tests/request_json_test.cc

@@ -64,8 +64,8 @@ TEST(RequestJsonTest, UpdatePostFields) {
  request.set_token("abc");
  EXPECT_EQ(
  "{\n"
- " token: \"abc\",\n"
- " number: 123\n"
+ " \"token\": \"abc\",\n"
+ " \"number\": 123\n"
  "}\n",
  request.options().post_fields);
 }

auth/src/android/auth_android.cc

@@ -195,10 +195,12 @@ void CheckEmulator(AuthData* auth_data) {
 
  // Use emulator as long as this env variable is set, regardless its value.
  if (std::getenv("USE_AUTH_EMULATOR") == nullptr) {
- LogDebug("Using Auth Prod for testing.");
+ LogInfo("Using Auth Prod for testing.");
  return;
  }
 
+ LogInfo("Using Auth Emulator for testing.");
+
  // Use AUTH_EMULATOR_PORT if it is set to non empty string,
  // otherwise use the default port.
  uint32_t port = std::stoi(kEmulatorPort);

auth/src/desktop/rpcs/auth_request.cc

@@ -88,22 +88,23 @@ std::string AuthRequest::GetUrl() {
  } else {
  std::string url(kHttp);
  url += emulator_url;
+ url += "/";
  url += kServerURL;
  return url;
  }
 }
 
 void AuthRequest::CheckEmulator() {
  if (!emulator_url.empty()) {
- LogDebug("Emulator Url already set: %s", emulator_url.c_str());
+ LogInfo("Emulator Url already set: %s", emulator_url.c_str());
  return;
  }
  // Use emulator as long as this env variable is set, regardless its value.
  if (std::getenv("USE_AUTH_EMULATOR") == nullptr) {
- LogDebug("Using Auth Prod for testing.");
+ LogInfo("Using Auth Prod for testing.");
  return;
  }
-
+ LogInfo("Using Auth Emulator.");
  emulator_url.append(kEmulatorLocalHost);
  emulator_url.append(":");
  // Use AUTH_EMULATOR_PORT if it is set to non empty string,

auth/src/ios/auth_ios.mm

@@ -160,10 +160,10 @@ void UpdateCurrentUser(AuthData *auth_data) {
 void CheckEmulator(AuthData *auth_data) {
  // Use emulator as long as this env variable is set, regardless its value.
  if (std::getenv("USE_AUTH_EMULATOR") == nullptr) {
- LogDebug("Using Auth Prod for testing.");
+ LogInfo("Using Auth Prod for testing.");
  return;
  }
-
+ LogInfo("Using Auth Emulator.");
  // Use AUTH_EMULATOR_PORT if it is set to non empty string,
  // otherwise use the default port.
  uint32_t port = std::stoi(kEmulatorPort);

auth/tests/desktop/rpcs/create_auth_uri_test.cc

@@ -36,8 +36,8 @@ TEST(CreateAuthUriTest, TestCreateAuthUriRequest) {
  request.options().url);
  EXPECT_EQ(
  "{\n"
- " identifier: \"email\",\n"
- " continueUri: \"http://localhost\"\n"
+ " \"identifier\": \"email\",\n"
+ " \"continueUri\": \"http://localhost\"\n"
  "}\n",
  request.options().post_fields);
 }

auth/tests/desktop/rpcs/delete_account_test.cc

@@ -37,7 +37,7 @@ TEST(DeleteAccountTest, TestDeleteAccountRequest) {
  request.options().url);
  EXPECT_EQ(
  "{\n"
- " idToken: \"token\"\n"
+ " \"idToken\": \"token\"\n"
  "}\n",
  request.options().post_fields);
 }

auth/tests/desktop/rpcs/get_account_info_test.cc

@@ -36,7 +36,7 @@ TEST(GetAccountInfoTest, TestGetAccountInfoRequest) {
  request.options().url);
  EXPECT_EQ(
  "{\n"
- " idToken: \"token\"\n"
+ " \"idToken\": \"token\"\n"
  "}\n",
  request.options().post_fields);
 }

auth/tests/desktop/rpcs/get_oob_confirmation_code_test.cc

@@ -40,8 +40,8 @@ TEST(GetOobConfirmationCodeTest, SendVerifyEmailRequest) {
  request->options().url);
  EXPECT_EQ(
  "{\n"
- " idToken: \"token\",\n"
- " requestType: \"VERIFY_EMAIL\"\n"
+ " \"idToken\": \"token\",\n"
+ " \"requestType\": \"VERIFY_EMAIL\"\n"
  "}\n",
  request->options().post_fields);
 }
@@ -56,8 +56,8 @@ TEST(GetOobConfirmationCodeTest, SendPasswordResetEmailRequest) {
  request->options().url);
  EXPECT_EQ(
  "{\n"
- " email: \"email\",\n"
- " requestType: \"PASSWORD_RESET\"\n"
+ " \"email\": \"email\",\n"
+ " \"requestType\": \"PASSWORD_RESET\"\n"
  "}\n",
  request->options().post_fields);
 }

auth/tests/desktop/rpcs/reset_password_test.cc

@@ -35,8 +35,8 @@ TEST(ResetPasswordTest, TestResetPasswordRequest) {
  request.options().url);
  EXPECT_EQ(
  "{\n"
- " oobCode: \"oob\",\n"
- " newPassword: \"password\"\n"
+ " \"oobCode\": \"oob\",\n"
+ " \"newPassword\": \"password\"\n"
  "}\n",
  request.options().post_fields);
 }