Skip to content
This repository was archived by the owner on Jun 5, 2024. It is now read-only.

ci: updated smoke tests to run npm install to get latest and console log the version so we know what is being tested in CI #191

Merged
merged 2 commits into from
Jun 1, 2023

Conversation

bizob2828
Copy link
Member

Details

Realized our smoke tests pin the aws version so we can't test newer versions. Also logging it out.

…log the version so we know what is being tested in CI
@bizob2828 bizob2828 added the dev:automation Indicates CI automation label May 31, 2023
@jmartin4563 jmartin4563 self-assigned this May 31, 2023
@codecov
Copy link

codecov bot commented May 31, 2023

Codecov Report

Merging #191 (8358663) into main (3a92ab3) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #191   +/-   ##
=======================================
  Coverage   95.66%   95.66%           
=======================================
  Files          15       15           
  Lines         946      946           
=======================================
  Hits          905      905           
  Misses         41       41           
Flag Coverage Δ
unit-tests-14.x 3.91% <ø> (ø)
unit-tests-16.x 3.91% <ø> (ø)
unit-tests-18.x 3.91% <ø> (ø)
versioned-tests-14.x 95.66% <ø> (ø)
versioned-tests-16.x 95.66% <ø> (ø)
versioned-tests-18.x 95.66% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Contributor

@jmartin4563 jmartin4563 left a comment

Choose a reason for hiding this comment

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

Confirmed locally that console.log is output when running the smoke tests with this PR checked out, just had two small thoughts

@bizob2828 bizob2828 requested a review from jmartin4563 June 1, 2023 15:23
@bizob2828 bizob2828 merged commit 242aae3 into newrelic:main Jun 1, 2023
This was referenced Jun 30, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dev:automation Indicates CI automation
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants