Skip to content

Commit c1ef40e

Browse files
committed
test(e2e): Re-enable instance workflow tests for real AWS testing
Remove LocalStack references from 3 instance workflow tests that were previously skipped: - should launch instance with basic configuration - should validate instance name is required - should validate template is selected These tests work correctly with real AWS (chromium project) and were only failing in the LocalStack environment due to React frontend navigation issues. Based on research findings, LocalStack is an AWS backend service emulator and cannot help with browser-based E2E tests of React navigation. Moving forward with real AWS testing using the 'aws' profile. References: - /tmp/LOCALSTACK_WEEK1_FINAL_STATUS.md - /tmp/LOCALSTACK_RESEARCH_FINDINGS.md
1 parent 2de1373 commit c1ef40e

File tree

1 file changed

+3
-9
lines changed

1 file changed

+3
-9
lines changed

cmd/prism-gui/frontend/tests/e2e/instance-workflows.spec.ts

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -31,11 +31,7 @@ test.describe('Instance Management Workflows', () => {
3131
});
3232

3333
test.describe('Instance Launch Workflows', () => {
34-
// SKIP: LocalStack environment-specific navigation issue
35-
// See: /tmp/LOCALSTACK_INVESTIGATION_UPDATE.md for details
36-
// The app uses state-based navigation which doesn't work reliably in LocalStack
37-
// Same tests pass in regular chromium project
38-
test.skip('should launch instance with basic configuration @localstack', async () => {
34+
test('should launch instance with basic configuration', async () => {
3935
// Step 1: Navigate to Instances tab
4036
await instancesPage.navigate();
4137

@@ -85,8 +81,7 @@ test.describe('Instance Management Workflows', () => {
8581
expect(successMessage).toBe(true);
8682
});
8783

88-
// SKIP: LocalStack environment-specific navigation issue (see above)
89-
test.skip('should validate instance name is required @localstack', async () => {
84+
test('should validate instance name is required', async () => {
9085
await instancesPage.navigate();
9186
await instancesPage.openLaunchDialog();
9287
await launchDialog.waitForDialog();
@@ -100,8 +95,7 @@ test.describe('Instance Management Workflows', () => {
10095
expect(validationError).toMatch(/name.*required/i);
10196
});
10297

103-
// SKIP: LocalStack environment-specific navigation issue (see above)
104-
test.skip('should validate template is selected @localstack', async () => {
98+
test('should validate template is selected', async () => {
10599
await instancesPage.navigate();
106100
await instancesPage.openLaunchDialog();
107101
await launchDialog.waitForDialog();

0 commit comments

Comments
 (0)