reorder the child adding

Author: Hweinstock

packages/core/src/awsService/ec2/explorer/ec2InstanceNode.ts

@@ -29,9 +29,9 @@ export class Ec2InstanceNode extends AWSTreeNodeBase implements AWSResourceNode
         public readonly instance: SafeEc2Instance
     ) {
         super('')
+        this.parent.addChild(this)
         this.updateInstance(instance)
         this.id = this.InstanceId
-        this.parent.addChild(this)
     }
 
     public updateInstance(newInstance: SafeEc2Instance) {

packages/core/src/test/awsService/ec2/explorer/ec2ParentNode.test.ts

@@ -134,7 +134,6 @@ describe('ec2ParentNode', function () {
         ]
 
         getInstanceStub.resolves(mapToInstanceCollection(instances))
-
         await testNode.updateChildren()
         assert.strictEqual(testNode.pollingSet.size, 1)
         getInstanceStub.restore()
@@ -188,4 +187,23 @@ describe('ec2ParentNode', function () {
         assert.throws(() => testNode.getInstanceNode('node2'))
         getInstanceStub.restore()
     })
+
+    it('adds node to polling set when asked to track it', async function () {
+        const instances = [{ Name: 'firstOne', InstanceId: 'node1', LastSeenStatus: 'pending' }]
+
+        getInstanceStub.resolves(mapToInstanceCollection(instances))
+        await testNode.updateChildren()
+        testNode.trackPendingNode('node1')
+        assert.strictEqual(testNode.pollingSet.size, 1)
+        getInstanceStub.restore()
+    })
+
+    it('throws error when asked to track non-child node', async function () {
+        const instances = [{ Name: 'firstOne', InstanceId: 'node1', LastSeenStatus: 'pending' }]
+
+        getInstanceStub.resolves(mapToInstanceCollection(instances))
+        await testNode.updateChildren()
+        assert.throws(() => testNode.trackPendingNode('node2'))
+        getInstanceStub.restore()
+    })
 })