Rename updateItemInExpandedState -> updateExpandedItem and replaceItemInExpandedState -> replaceExpandedItem

Author: charisk

extensions/ql-vscode/src/databases/db-item-expansion.ts

@@ -31,7 +31,7 @@ export interface RemoteUserDefinedListExpandedDbItem {
   listName: string;
 }
 
-export function updateItemInExpandedState(
+export function updateExpandedItem(
   currentExpandedItems: ExpandedDbItem[],
   dbItem: DbItem,
   itemExpanded: boolean,
@@ -50,7 +50,7 @@ export function updateItemInExpandedState(
   }
 }
 
-export function replaceItemInExpandedState(
+export function replaceExpandedItem(
   currentExpandedItems: ExpandedDbItem[],
   currentDbItem: DbItem,
   newDbItem: DbItem,

extensions/ql-vscode/src/databases/db-manager.ts

@@ -11,8 +11,8 @@ import {
   RemoteUserDefinedListDbItem,
 } from "./db-item";
 import {
-  updateItemInExpandedState,
-  replaceItemInExpandedState,
+  updateExpandedItem,
+  replaceExpandedItem,
   ExpandedDbItem,
 } from "./db-item-expansion";
 import {
@@ -80,7 +80,7 @@ export class DbManager {
   ): Promise<void> {
     const currentExpandedItems = this.getExpandedItems();
 
-    const newExpandedItems = updateItemInExpandedState(
+    const newExpandedItems = updateExpandedItem(
       currentExpandedItems,
       dbItem,
       itemExpanded,
@@ -127,7 +127,7 @@ export class DbManager {
     }
 
     const newDbItem = { ...currentDbItem, listName: newName };
-    const newExpandedItems = replaceItemInExpandedState(
+    const newExpandedItems = replaceExpandedItem(
       this.getExpandedItems(),
       currentDbItem,
       newDbItem,

extensions/ql-vscode/test/unit-tests/databases/db-item-expansion.test.ts

@@ -3,18 +3,18 @@ import {
   RootRemoteDbItem,
 } from "../../../src/databases/db-item";
 import {
-  updateItemInExpandedState,
+  updateExpandedItem,
   ExpandedDbItem,
   ExpandedDbItemKind,
-  replaceItemInExpandedState,
+  replaceExpandedItem,
 } from "../../../src/databases/db-item-expansion";
 import {
   createRemoteUserDefinedListDbItem,
   createRootRemoteDbItem,
 } from "../../factories/db-item-factories";
 
 describe("db item expansion", () => {
-  describe("updateItemInExpandedState", () => {
+  describe("updateExpandedItem", () => {
     it("should add an expanded item to an existing list", () => {
       const currentExpandedItems: ExpandedDbItem[] = [
         {
@@ -31,7 +31,7 @@ describe("db item expansion", () => {
           listName: "list2",
         });
 
-      const newExpandedItems = updateItemInExpandedState(
+      const newExpandedItems = updateExpandedItem(
         currentExpandedItems,
         dbItem,
         true,
@@ -52,7 +52,7 @@ describe("db item expansion", () => {
           listName: "list2",
         });
 
-      const newExpandedItems = updateItemInExpandedState([], dbItem, true);
+      const newExpandedItems = updateExpandedItem([], dbItem, true);
 
       expect(newExpandedItems).toEqual([
         {
@@ -78,7 +78,7 @@ describe("db item expansion", () => {
           listName: "list1",
         });
 
-      const newExpandedItems = updateItemInExpandedState(
+      const newExpandedItems = updateExpandedItem(
         currentExpandedItems,
         dbItem,
         false,
@@ -100,7 +100,7 @@ describe("db item expansion", () => {
 
       const dbItem: RootRemoteDbItem = createRootRemoteDbItem();
 
-      const newExpandedItems = updateItemInExpandedState(
+      const newExpandedItems = updateExpandedItem(
         currentExpandedItems,
         dbItem,
         false,
@@ -110,7 +110,7 @@ describe("db item expansion", () => {
     });
   });
 
-  describe("replaceItemInExpandedState", () => {
+  describe("replaceExpandedItem", () => {
     it("should replace the db item", () => {
       const currentExpandedItems: ExpandedDbItem[] = [
         {
@@ -139,7 +139,7 @@ describe("db item expansion", () => {
         listName: "list1 (renamed)",
       };
 
-      const newExpandedItems = replaceItemInExpandedState(
+      const newExpandedItems = replaceExpandedItem(
         currentExpandedItems,
         currentDbItem,
         newDbItem,