Skip to content

Add test for node-version *#491

Merged
dmitry-shibanov merged 2 commits intoactions:mainfrom
jablko:patch-3
May 18, 2022
Merged

Add test for node-version *#491
dmitry-shibanov merged 2 commits intoactions:mainfrom
jablko:patch-3

Conversation

@jablko
Copy link
Copy Markdown
Contributor

@jablko jablko commented May 13, 2022

Description:
This PR only adds test coverage for existing behavior: It adds a unit and e2e test for node-version *.

What I did was:

  • Combine the following three tests with it.each(table) --- but I'm happy to leave them unrolled if you prefer?
  • Added a new row for testing *:
    it.each([
      ['12.16.2', 'darwin', '12.16.2', 'Erbium'],
      ['12', 'linux', '12.16.2', 'Erbium'],
      ['10', 'win32', '10.20.1', 'Dubnium'],
      ['*', 'linux', '14.0.0', 'Fermium'] // <- Added a new row
    ])(
      'can find %s from manifest on %s',
      async (versionSpec, platform, expectedVersion, expectedLts) => {
  • Added an e2e test for node-version *.

Check list:

  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.

@jablko jablko requested a review from a team May 13, 2022 23:06
@dmitry-shibanov
Copy link
Copy Markdown
Contributor

Hello @jablko. Thank you for your pull request. I think only unit tests will be enough.

@jablko
Copy link
Copy Markdown
Contributor Author

jablko commented May 16, 2022

@dmitry-shibanov dmitry-shibanov merged commit 4d62faf into actions:main May 18, 2022
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