fix lint

Author: Varun Rathore

src/commands/remoteconfig-rollouts-delete.ts

@@ -2,19 +2,20 @@ import { Command } from "../command";
 import { Options } from "../options";
 import { requireAuth } from "../requireAuth";
 import { requirePermissions } from "../requirePermissions";
-import * as clc from "colorette";
 import { logger } from "../logger";
 import { needProjectNumber } from "../projectUtils";
 import { NAMESPACE_FIREBASE } from "../remoteconfig/interfaces";
 import * as rcRollout from "../remoteconfig/deleteRollout";
 import { getRollout, parseRolloutIntoTable } from "../remoteconfig/getRollout";
-import { FirebaseError } from "../error";
 import { confirm } from "../prompt";
 
 export const command = new Command("remoteconfig:rollouts:delete [rolloutId]")
   .description("delete a Remote Config rollout.")
   .before(requireAuth)
-  .before(requirePermissions, ["cloudconfigs.config.update", "firebaseanalytics.resources.googleAnalyticsEdit"])
+  .before(requirePermissions, [
+    "cloud.configs.update",
+    "firebaseanalytics.resources.googleAnalyticsEdit",
+  ])
   .action(async (rolloutId: string, options: Options) => {
     const projectNumber: string = await needProjectNumber(options);
     const rollout = await getRollout(projectNumber, NAMESPACE_FIREBASE, rolloutId);

src/commands/remoteconfig-rollouts-get.ts

@@ -6,7 +6,6 @@ import { logger } from "../logger";
 import { needProjectNumber } from "../projectUtils";
 import { RemoteConfigRollout, NAMESPACE_FIREBASE } from "../remoteconfig/interfaces";
 import * as rcRollout from "../remoteconfig/getRollout";
-import { FirebaseError } from "../error";
 
 export const command = new Command("remoteconfig:rollouts:get [rolloutId]")
   .description("get a Remote Config rollout")

src/commands/remoteconfig-rollouts-list.ts

@@ -11,7 +11,6 @@ import {
   DEFAULT_PAGE_SIZE,
 } from "../remoteconfig/interfaces";
 import * as rcRollout from "../remoteconfig/listRollouts";
-import { FirebaseError } from "../error";
 
 export const command = new Command("remoteconfig:rollouts:list")
   .description("get a list of Remote Config rollouts.")

src/remoteconfig/deleteRollout.spec.ts

@@ -26,22 +26,21 @@ describe("Remote Config Rollout Delete", () => {
   });
 
   it("should throw FirebaseError if rollout is running", async () => {
-        const errorMessage = `Rollout ${ROLLOUT_ID} is currently running and cannot be deleted. If you want to delete this rollout, stop it at https://console.firebase.google.com/project/${PROJECT_ID}/config/env/firebase/rollout/${ROLLOUT_ID}`;
-        nock(remoteConfigApiOrigin())
-          .delete(
-            `/v1/projects/${PROJECT_ID}/namespaces/${NAMESPACE_FIREBASE}/rollouts/${ROLLOUT_ID}`,
-          )
-          .reply(400, {
-            error: {
-              message: errorMessage,
-            },
-          });
-    
-        await expect(
-          deleteRollout(PROJECT_ID, NAMESPACE_FIREBASE, ROLLOUT_ID),
-        ).to.be.rejectedWith(FirebaseError, errorMessage);
+    const errorMessage = `Rollout ${ROLLOUT_ID} is currently running and cannot be deleted. If you want to delete this rollout, stop it at https://console.firebase.google.com/project/${PROJECT_ID}/config/env/firebase/rollout/${ROLLOUT_ID}`;
+    nock(remoteConfigApiOrigin())
+      .delete(`/v1/projects/${PROJECT_ID}/namespaces/${NAMESPACE_FIREBASE}/rollouts/${ROLLOUT_ID}`)
+      .reply(400, {
+        error: {
+          message: errorMessage,
+        },
       });
 
+    await expect(deleteRollout(PROJECT_ID, NAMESPACE_FIREBASE, ROLLOUT_ID)).to.be.rejectedWith(
+      FirebaseError,
+      errorMessage,
+    );
+  });
+
   it("should throw FirebaseError if an internal error occurred", async () => {
     nock(remoteConfigApiOrigin())
       .delete(`/v1/projects/${PROJECT_ID}/namespaces/${NAMESPACE_FIREBASE}/rollouts/${ROLLOUT_ID}`)