refactor(appBuilder): move WalkthroughNode

Author: jonife

packages/core/src/awsService/appBuilder/explorer/nodes/rootNode.ts

@@ -13,6 +13,7 @@ import { Commands } from '../../../../shared/vscode/commands2'
 import { AppNode } from './appNode'
 import { detectSamProjects } from '../detectSamProjects'
 import globals from '../../../../shared/extensionGlobals'
+import { WalkthroughNode } from './walkthroughNode'
 
 export async function getAppNodes(): Promise<TreeNode[]> {
     // no active workspace, show buttons in welcomeview
@@ -37,29 +38,6 @@ export async function getAppNodes(): Promise<TreeNode[]> {
     return nodesToReturn
 }
 
-/**
- * Create Open Walkthrough Node in App builder sidebar
- *
- */
-export class WalkthroughNode implements TreeNode {
-    public readonly id = 'walkthrough'
-    public readonly resource = this
-    public constructor() {}
-
-    public getTreeItem() {
-        const itemLabel = localize('AWS.appBuilder.openWalkthroughTitle', 'Walkthrough of Application Builder')
-
-        const item = new vscode.TreeItem(itemLabel)
-        item.contextValue = 'awsWalkthroughNode'
-        item.command = {
-            title: localize('AWS.appBuilder.openWalkthroughTitle', 'Walkthrough of Application Builder'),
-            command: 'aws.toolkit.lambda.openWalkthrough',
-        }
-
-        return item
-    }
-}
-
 export class AppBuilderRootNode implements TreeNode {
     public readonly id = 'appBuilder'
     public readonly resource = this

packages/core/src/awsService/appBuilder/explorer/nodes/walkthroughNode.ts

@@ -0,0 +1,37 @@
+/*!
+ * Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+import * as vscode from 'vscode'
+import { TreeNode } from '../../../../shared/treeview/resourceTreeDataProvider'
+import { localize } from '../../../../shared/utilities/vsCodeUtils'
+
+/**
+ * Create Open Walkthrough Node in App builder sidebar
+ *
+ */
+export class WalkthroughNode implements TreeNode {
+    public readonly id = 'walkthrough'
+    public readonly resource: WalkthroughNode = this
+
+    // Constructor left empty intentionally for future extensibility
+    public constructor() {}
+
+    /**
+     * Generates the TreeItem for the Walkthrough Node.
+     * This item will appear in the sidebar with a label and command to open the walkthrough.
+     */
+    public getTreeItem() {
+        const itemLabel = localize('AWS.appBuilder.openWalkthroughTitle', 'Walkthrough of Application Builder')
+
+        const item = new vscode.TreeItem(itemLabel)
+        item.contextValue = 'awsWalkthroughNode'
+        item.command = {
+            title: localize('AWS.appBuilder.openWalkthroughTitle', 'Walkthrough of Application Builder'),
+            command: 'aws.toolkit.lambda.openWalkthrough',
+        }
+
+        return item
+    }
+}

packages/core/src/test/shared/applicationBuilder/explorer/nodes/rootNode.test.ts

@@ -11,11 +11,7 @@ import * as VsCodeUtils from '../../../../../shared/utilities/vsCodeUtils'
 import * as DetectSamProjects from '../../../../../../src/awsService/appBuilder/explorer/detectSamProjects'
 import globals from '../../../../../shared/extensionGlobals'
 
-import {
-    AppBuilderRootNode,
-    getAppNodes,
-    WalkthroughNode,
-} from '../../../../../awsService/appBuilder/explorer/nodes/rootNode'
+import { AppBuilderRootNode, getAppNodes } from '../../../../../awsService/appBuilder/explorer/nodes/rootNode'
 
 import { AppNode } from '../../../../../awsService/appBuilder/explorer/nodes/appNode'
 
@@ -50,28 +46,6 @@ describe('getAppNodes', async () => {
     })
 })
 
-describe('WalkthroughNode', () => {
-    const walkthroughNode = new WalkthroughNode()
-    describe('constructor', () => {
-        it('should create a WalkthroughNode with correct properties', () => {
-            assert.strictEqual(walkthroughNode.id, 'walkthrough')
-            assert.strictEqual(walkthroughNode.resource, walkthroughNode)
-        })
-    })
-    describe('getTreeItem', () => {
-        it('should generate correct TreeItem', () => {
-            const treeItem = walkthroughNode.getTreeItem()
-
-            assert.strictEqual(treeItem.label, 'Walkthrough of Application Builder')
-            assert.strictEqual(treeItem.contextValue, 'awsWalkthroughNode')
-            assert.deepStrictEqual(treeItem.command, {
-                title: 'Walkthrough of Application Builder',
-                command: 'aws.toolkit.lambda.openWalkthrough',
-            })
-        })
-    })
-})
-
 describe('AppBuilderRootNode', () => {
     let commandRegisterStub: sinon.SinonStub
     let openUrlStub: sinon.SinonStub

packages/core/src/test/shared/applicationBuilder/explorer/nodes/walkthrough.test.ts

@@ -0,0 +1,33 @@
+/*!
+ * Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
+ * SPDX-License-Identifier: Apache-2.0
+ */
+import assert from 'assert'
+import { WalkthroughNode } from '../../../../../awsService/appBuilder/explorer/nodes/walkthroughNode'
+
+describe('WalkthroughNode', () => {
+    let walkthroughNode: WalkthroughNode
+
+    // Initialize a new instance before each test to ensure test isolation
+    beforeEach(() => {
+        walkthroughNode = new WalkthroughNode()
+    })
+    describe('constructor', () => {
+        it('should create a WalkthroughNode with correct properties', () => {
+            assert.strictEqual(walkthroughNode.id, 'walkthrough')
+            assert.strictEqual(walkthroughNode.resource, walkthroughNode)
+        })
+    })
+    describe('getTreeItem', () => {
+        it('should generate correct TreeItem', () => {
+            const treeItem = walkthroughNode.getTreeItem()
+
+            assert.strictEqual(treeItem.label, 'Walkthrough of Application Builder')
+            assert.strictEqual(treeItem.contextValue, 'awsWalkthroughNode')
+            assert.deepStrictEqual(treeItem.command, {
+                title: 'Walkthrough of Application Builder',
+                command: 'aws.toolkit.lambda.openWalkthrough',
+            })
+        })
+    })
+})