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

Update testing-unit-ts dependencies #1644

Merged
merged 1 commit into from
Jun 8, 2024
Merged

Conversation

justinvp
Copy link
Member

@justinvp justinvp commented Jun 8, 2024

I tried running the example locally with npx mocha -r ts-node/register ec2tests.ts and ran into an error. However, after upgrading the dependencies to the latest versions, everything works as expected.

Updated the CI workflows to use the same npx mocha ... command we mention in https://github.com/pulumi/examples/blob/master/testing-unit-ts/mocha/README.md, and updated those to run on the current active LTS version of Node (20).

Fixes #1645

I tried running the example locally with `npx mocha -r ts-node/register ec2tests.ts` and ran into an error. However, after upgrading the dependencies to the latest versions, everything works as expected.

Updated the CI workflows to use the same `npx mocha ...` command we mention in https://github.com/pulumi/examples/blob/master/testing-unit-ts/mocha/README.md, and updated those to run on the current active LTS version of Node (20).
Copy link

pulumi-staging bot commented Jun 8, 2024

🍹 The Update for pulumi/k8s-ci-cluster/542ffa85ec18eaf5e81de8196717ecb8b42905cc-1778 was successful.

Resource Changes

    Name                                                          Type                                        Operation
+   multicloud                                                    pulumi-kubernetes:ci:GkeCluster             create
+   password                                                      random:index/randomPassword:RandomPassword  create
+   ephemeral-ci-cluster                                          gcp:container/cluster:Cluster               create
+   primary-node-pool                                             gcp:container/nodePool:NodePool             create
+   gke                                                           pulumi:providers:kubernetes                 create
+   k8s-ci-cluster-542ffa85ec18eaf5e81de8196717ecb8b42905cc-1778  pulumi:pulumi:Stack                         create

Copy link

pulumi-staging bot commented Jun 8, 2024

🍹 The Destroy for pulumi/k8s-ci-cluster/542ffa85ec18eaf5e81de8196717ecb8b42905cc-1778 was successful.

Resource Changes

    Name                                                          Type                                        Operation
-   ephemeral-ci-cluster                                          gcp:container/cluster:Cluster               delete
-   multicloud                                                    pulumi-kubernetes:ci:GkeCluster             delete
-   password                                                      random:index/randomPassword:RandomPassword  delete
-   k8s-ci-cluster-542ffa85ec18eaf5e81de8196717ecb8b42905cc-1778  pulumi:pulumi:Stack                         delete
-   gke                                                           pulumi:providers:kubernetes                 delete
-   primary-node-pool                                             gcp:container/nodePool:NodePool             delete

Copy link
Contributor

@cnunciato cnunciato left a comment

Choose a reason for hiding this comment

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

Thanks @justinvp!

@justinvp justinvp merged commit bb6beb4 into master Jun 8, 2024
49 of 51 checks passed
@justinvp justinvp deleted the justin/testing-unit-ts branch June 8, 2024 22:52
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.

testing-unit-ts doesn't work with current package.json dependencies
2 participants