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

chore(mojaloop/#3438): nodejs upgrade #309

Merged
merged 24 commits into from
Sep 6, 2023

Conversation

oderayi
Copy link
Contributor

@oderayi oderayi commented Sep 4, 2023

chore(mojaloop/#3438): nodejs upgrade - mojaloop/project#3438

  • Upgraded NodeJS from v16 to v18 LTS
  • Updated CI config to use NodeJS latest LTS version
  • Updated CI config to display runtime versions for each job
  • Upgraded NPM dependencies
  • Fixed CI bugs caused by the NodeJS upgrade
  • Updated CI pipeline use NVM to select node version from .nvmrc file
  • Upgraded circleci orbs
  • Migrated master branch to main
  • Optimized ci workflow to use cached image for some jobs
  • Updated unit test scripts to run "in band" to avoid failures by flaky tests. (Further investigation is needed to ascertain why test/unit/server.tests.js and test/unit/serverStart.test.js fail when run in parallel)

@oderayi oderayi marked this pull request as ready for review September 5, 2023 10:56
Comment on lines +58 to +61
"test:unit": "jest --runInBand --testMatch '**/test/unit/**/*.test.js'",
"test:coverage": "jest --runInBand --coverage --coverageThreshold='{}' --testMatch '**/test/unit/**/*.test.js'",
"test:coverage-check": "jest --runInBand --coverage --testMatch '**/test/unit/**/*.test.js'",
"test:junit": "jest --runInBand --reporters=default --reporters=jest-junit --testMatch '**/test/unit/**/*.test.js'",
Copy link
Contributor Author

@oderayi oderayi Sep 5, 2023

Choose a reason for hiding this comment

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

test/unit/server.test.js and test/unit/serverStart.test.js fail when tests are run in parallel.

@oderayi oderayi self-assigned this Sep 5, 2023
Copy link
Member

@mdebarros mdebarros left a comment

Choose a reason for hiding this comment

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

Looks good....I did find a single minor change

.circleci/config.yml Outdated Show resolved Hide resolved
Copy link
Member

@mdebarros mdebarros left a comment

Choose a reason for hiding this comment

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

+1

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.

2 participants