Emit OK HTTP status in import callback instead of handler

functions/src/handlers.ts

@@ -16,7 +16,7 @@
 
 import cors from 'cors';
 import { DecodedIdToken } from 'firebase-admin/auth';
-import { onRequest, Request } from 'firebase-functions/v2/https';
+import { Request, onRequest } from 'firebase-functions/v2/https';
 import type { Response } from 'express';
 import { StatusCodes } from 'http-status-codes';
 import { getDecodedIdToken } from './common/auth';
@@ -123,7 +123,6 @@ export async function invokeCallbackAsync(
       res,
       user,
       () => {
-        res.status(StatusCodes.OK).end();
         resolve(undefined);
       },
       (errorCode: number, message: string) => {

functions/src/import-geojson.ts

@@ -118,7 +118,7 @@ export function importGeoJsonCallback(
       await Promise.all(inserts);
       const count = inserts.length;
       console.debug(`${count} LOIs imported`);
-      res.send(JSON.stringify({ count }));
+      res.status(StatusCodes.OK).json({ count });
       done();
     } catch (err) {
       console.debug(err);

functions/src/testing/http-test-helpers.ts

@@ -44,6 +44,7 @@ export function createResponseSpy(chunks?: string[]): Response<any> {
   const res = jasmine.createSpyObj<Response<any>>('response', [
     'send',
     'status',
+    'json',
     'end',
     'write',
     'type',
@@ -54,6 +55,7 @@ export function createResponseSpy(chunks?: string[]): Response<any> {
     'removeListener',
   ]);
   res.status.and.callThrough().and.returnValue(res);
+  res.json.and.callThrough().and.returnValue(res);
   res.end.and.callThrough().and.returnValue(res);
   res.write.and.callFake((chunk: any): boolean => {
     chunks?.push(chunk.toString());