Fix issue where accounts:lookup is case-sensitive for emails (#8351)

Author: aalej

CHANGELOG.md

@@ -1 +1,2 @@
 - Fix bug where functions:artifacts:setpolicy command's --none option didn't work as expected (#8330)
+- Fix bug in Auth emulator where accounts:lookup is case-sensitive for emails (#8344)

src/emulator/auth/misc.spec.ts

@@ -339,6 +339,54 @@ describeAuthEmulator("createSessionCookie", ({ authApi }) => {
 });
 
 describeAuthEmulator("accounts:lookup", ({ authApi }) => {
+  it("should return user by email when privileged", async () => {
+    const { email } = await registerUser(authApi(), {
+      email: "[email protected]",
+      password: "password",
+    });
+
+    await authApi()
+      .post(`/identitytoolkit.googleapis.com/v1/projects/${PROJECT_ID}/accounts:lookup`)
+      .set("Authorization", "Bearer owner")
+      .send({ email: [email] })
+      .then((res) => {
+        expectStatusCode(200, res);
+        expect(res.body.users).to.have.length(1);
+        expect(res.body.users[0].email).to.equal(email);
+      });
+  });
+
+  it("should return user by email even when uppercased", async () => {
+    const { email } = await registerUser(authApi(), {
+      email: "[email protected]",
+      password: "password",
+    });
+    const caplitalizedEmail = email.toUpperCase();
+
+    await authApi()
+      .post(`/identitytoolkit.googleapis.com/v1/projects/${PROJECT_ID}/accounts:lookup`)
+      .set("Authorization", "Bearer owner")
+      .send({ email: [caplitalizedEmail] })
+      .then((res) => {
+        console.log(res.body.users);
+        expectStatusCode(200, res);
+        expect(res.body.users).to.have.length(1);
+        expect(res.body.users[0].email).to.equal(email);
+      });
+  });
+
+  it("should return empty result when email is not found", async () => {
+    await authApi()
+      .post(`/identitytoolkit.googleapis.com/v1/projects/${PROJECT_ID}/accounts:lookup`)
+      .set("Authorization", "Bearer owner")
+      .send({ email: ["[email protected]"] })
+      .then((res) => {
+        console.log(res.body.users);
+        expectStatusCode(200, res);
+        expect(res.body).not.to.have.property("users");
+      });
+  });
+
   it("should return user by localId when privileged", async () => {
     const { localId } = await registerAnonUser(authApi());
 

src/emulator/auth/operations.ts

@@ -300,7 +300,8 @@ function lookup(
       tryAddUser(state.getUserByLocalId(localId));
     }
     for (const email of reqBody.email ?? []) {
-      tryAddUser(state.getUserByEmail(email));
+      const canonicalizedEmail = canonicalizeEmailAddress(email);
+      tryAddUser(state.getUserByEmail(canonicalizedEmail));
     }
     for (const phoneNumber of reqBody.phoneNumber ?? []) {
       tryAddUser(state.getUserByPhoneNumber(phoneNumber));