dev: update provider integ tests

Author: PierreBeucher

test/integ/stable/core/providers/azure/lifecycle.spec.ts

@@ -78,8 +78,10 @@ describe('Azure lifecycle', () => {
                 notificationEmail: "test@test.com"
             }
         }, {
-            wolf: {
-                enable: true
+            sunshine: {
+                enable: true,
+                username: "sunshine",
+                passwordBase64: Buffer.from("S3un$h1ne!").toString('base64'),
             }, 
         });
     })
@@ -105,6 +107,10 @@ describe('Azure lifecycle', () => {
         assert.strictEqual(instance.hardwareProfile?.vmSize, vmSize);
     }).timeout(30*60*1000); // 30 minutes timeout as deployment with image creation may be long
 
+    it('should wait for instance readiness after deployment', async () => {
+        await waitForInstanceReadiness('deployment');
+    }).timeout(2*60*1000);
+
     it('should have resources matching state output after deployment', async () => {
         const azureClient = await getAzureClient();
         const state = await getCurrentTestState();

test/integ/stable/core/providers/linode/lifecycle.spec.ts

@@ -61,6 +61,7 @@ describe('Linode lifecycle', () => {
                 baseImageSnapshot: {
                     enable: true,
                 },
+                additionalLabels: ['test-label-1', 'test-label-2'],
             }, {
                 sunshine: {
                     enable: true,
@@ -115,17 +116,63 @@ describe('Linode lifecycle', () => {
         }
     }).timeout(10000)
 
-    it('should have a valid instance server output with existing server', async () => {
+    it('should have a valid instance server output with existing server and additional labels applied to instance and data disk', async () => {
         const state = await getCurrentTestState()
+        const linodeClient = getLinodeClient()
 
         assert.ok(state.provision.output?.instanceServerId)
         const currentInstanceServerId = state.provision.output.instanceServerId
 
-        const linodeClient = getLinodeClient()
-        
+        // Check instance server status
         const serverStatus = await linodeClient.getInstanceStatus(currentInstanceServerId)
-        // Instance should be in a valid state (running, stopped, etc.)
-        assert.equal(serverStatus, 'running')
+        assert.equal(serverStatus, 'running', 'Instance should be running')
+
+        // Verify instance has correct labels
+        const instance = await linodeClient.getLinode(currentInstanceServerId)
+        assert.ok(instance, 'Instance should exist')
+        
+        const expectedAdditionalLabels = ['test-label-1', 'test-label-2']
+        const actualTags = instance.tags || []
+
+        // Check that all additional labels are present
+        for (const expectedLabel of expectedAdditionalLabels) {
+            assert.ok(
+                actualTags.includes(expectedLabel),
+                `Instance should have additional label '${expectedLabel}'. Actual tags: ${JSON.stringify(actualTags)}`
+            )
+        }
+
+        // Ensure there is at least one instance-name tag generated by Pulumi/linodeLabel
+        assert.ok(
+            actualTags.some(tag => tag.startsWith('instance-')),
+            `Instance should have an 'instance-' tag derived from instance name. Actual tags: ${JSON.stringify(actualTags)}`
+        )
+
+        // Verify data disk has correct labels
+        assert.ok(state.provision.output?.dataDiskId, 'Data disk ID should be in output')
+        // dataDiskId is a filesystem path basename like "scsi-0Linode_Volume_my-instance-vol"
+        // Extract the volume label by removing the "scsi-0Linode_Volume_" prefix
+        const volumeLabel = state.provision.output.dataDiskId.replace(/^scsi-0Linode_Volume_/, '')
+        const volume = await linodeClient.getVolumeByLabel(volumeLabel)
+        assert.ok(volume, 'Data volume should exist in Linode')
+        
+        // Expected additional labels from state
+        const volumeExpectedTags = ['test-label-1', 'test-label-2']
+        const volumeActualTags = volume.tags || []
+
+        // Check that all additional labels are present
+        for (const expectedTag of volumeExpectedTags) {
+            assert.ok(
+                volumeActualTags.includes(expectedTag),
+                `Data disk should have additional label '${expectedTag}'. Actual tags: ${JSON.stringify(volumeActualTags)}`
+            )
+        }
+
+        // Ensure there is at least one instance-name tag generated by Pulumi/linodeLabel
+        assert.ok(
+            volumeActualTags.some(tag => tag.startsWith('instance-')),
+            `Data disk should have an 'instance-' tag derived from instance name. Actual tags: ${JSON.stringify(volumeActualTags)}`
+        )
     }).timeout(10000)
 
     it('should verify instance configuration after deployment', async () => {
@@ -152,7 +199,7 @@ describe('Linode lifecycle', () => {
         assert.ok(dataDisk);
         assert.strictEqual(dataDisk.size, (dataDiskSizeGb+2) * 1024 * 1024 * 1024); // 2GB in bytes
 
-    }).timeout(10000)
+    }).timeout(30*60*1000) // 30 minutes timeout, might be long
 
     // run twice for idempotency
     for (let i = 0; i < 2; i++) {