aws-amplify / amplify-backend

Home to all tools related to Amplify's code-first DX (Gen 2) for building fullstack apps on AWS
Apache License 2.0
166 stars 55 forks source link

Increase timeout while waiting for bucked delete. #2061

Closed sobolk closed 2 days ago

sobolk commented 2 days ago

Problem

This times out rather frequently. https://github.com/aws-amplify/amplify-backend/actions/runs/11046537942/job/30686126851

test at packages/integration-tests/lib/test-e2e/deployment.test.js:42:22
✖ [data-storage-auth] deploys fully (514201.551588ms)
  AssertionError [ERR_ASSERTION]: Timed out waiting for amplify-d7y3glu8p86zn-testb-testnamebucketb4152ad5-3omuomryy4hn to be deleted
      at DataStorageAuthWithTriggerTestProject.waitForBucketDeletion (/home/runner/work/amplify-backend/amplify-backend/packages/integration-tests/src/test-project-setup/data_storage_auth_with_triggers.ts:382:12)
      at async DataStorageAuthWithTriggerTestProject.assertExpectedCleanup (/home/runner/work/amplify-backend/amplify-backend/packages/integration-tests/src/test-project-setup/data_storage_auth_with_triggers.ts:361:5)
      at async DataStorageAuthWithTriggerTestProject.tearDown (/home/runner/work/amplify-backend/amplify-backend/packages/integration-tests/src/test-project-setup/data_storage_auth_with_triggers.ts:168:5)
      at async TestContext.<anonymous> (/home/runner/work/amplify-backend/amplify-backend/packages/integration-tests/src/test-e2e/deployment.test.ts:54:11)
      at async TestHook.run (node:internal/test_runner/test:790:9)
      at async node:internal/test_runner/test:716:9
      at async Suite.runHook (node:internal/test_runner/test:714:7)
      at async node:internal/test_runner/test:753:9
      at async Test.run (node:internal/test_runner/test:799:7)
      at async Promise.all (index 0) {
    generatedMessage: false,
    code: 'ERR_ASSERTION',
    actual: undefined,
    expected: undefined,
    operator: 'fail'
  }

Changes

Increase timeout to see if this improves stability.

Validation

PR checks.

Checklist

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

changeset-bot[bot] commented 2 days ago

🦋 Changeset detected

Latest commit: e68aac18d9f57c727154bb7f0687ea58beef3bb5

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 0 packages When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR