Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: initial version of e2e tests #41

Closed
wants to merge 37 commits into from
Closed

test: initial version of e2e tests #41

wants to merge 37 commits into from

Conversation

jeromevdl
Copy link
Contributor

Issue #, if available:

Description of changes:

Initiate the e2e tests:

  • I've created the role with basic policy (will need additional permissions for the real tests).
  • I have created the secret associated to it.
  • I have created the workflow using what they did on powertools typescript.
  • I have issues with projen/lerna to specify groups to run (e2e should not be run part of the standard build).

TODO:

  • fix the projen/lerna groups and make sure the build is passing
  • try the workflow (manual execution)

Checklist

Breaking change checklist

RFC issue #:

  • Migration process documented
  • Implement warnings (if it can live side by side)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@jeromevdl jeromevdl requested a review from a user August 25, 2023 10:03
@jeromevdl jeromevdl marked this pull request as ready for review August 29, 2023 10:29
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A couple of comments, see below.

.projenrc.ts Outdated Show resolved Hide resolved
.projenrc.ts Outdated Show resolved Hide resolved
.projenrc.ts Outdated Show resolved Hide resolved
.projenrc.ts Outdated Show resolved Hide resolved
framework/test/e2e/test-stack.ts Show resolved Hide resolved
.github/workflows/run-e2e-tests.yml Outdated Show resolved Hide resolved
.github/workflows/run-e2e-tests.yml Show resolved Hide resolved
Dmitry Balabanov and others added 26 commits August 29, 2023 14:22
Bumps [eslint-plugin-import](https://github.com/import-js/eslint-plugin-import) from 2.28.0 to 2.28.1.
- [Release notes](https://github.com/import-js/eslint-plugin-import/releases)
- [Changelog](https://github.com/import-js/eslint-plugin-import/blob/main/CHANGELOG.md)
- [Commits](import-js/eslint-plugin-import@v2.28.0...v2.28.1)

---
updated-dependencies:
- dependency-name: eslint-plugin-import
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [@types/jest](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/jest) from 29.5.3 to 29.5.4.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/jest)

---
updated-dependencies:
- dependency-name: "@types/jest"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [cdk-nag](https://github.com/cdklabs/cdk-nag) from 2.27.108 to 2.27.111.
- [Release notes](https://github.com/cdklabs/cdk-nag/releases)
- [Commits](cdklabs/cdk-nag@v2.27.108...v2.27.111)

---
updated-dependencies:
- dependency-name: cdk-nag
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
@jeromevdl
Copy link
Contributor Author

closed, replaced by #56 (messed up with git)

@jeromevdl jeromevdl closed this Aug 29, 2023
@ghost ghost mentioned this pull request Aug 29, 2023
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant