[Tests] migrate installation_iojs test suite to GitHub Actions
#3476
Conversation
Remove equivalent TravisCI entries Signed-off-by: Jessica Wagantall <jwagantall@linuxfoundation.org>
|
@ljharb , looks like this test suit is passing without issues, except for the v0.40.0 tests that have been failing for a while in production. Is there anyone looking into those? |
|
Those are expected to fail forever, since v0.40.0 won't ever have the bug fixed. |
Thank you! On regards to the Travis failure: https://app.travis-ci.com/github/nvm-sh/nvm/jobs/628361855 Looks like this failure is the same than the one I just started seeing in the slow test suits. It just started happening and is not an issue with the node version since the same stable version was passing a week before.. I'm gonna continue looking and see if this is something related to anything else.. :( |
|
I think it's a legit failure in both suites, though - these tests talk to the network and the registry and can have their results change over time, legitimately. |
jwagantall
force-pushed
the
enable-installation_iojs
branch
from
22496f8 to
0027153
Compare
Actually, I was looking into them a little more and these are indeed legitimate failures I started seeing today in the slow test suit. Even if this change is meant for "installation_iojs", so we need to make sure the slow tests are passing in TravisCI before we could merge this change? |
|
It would be ideal for maximum confidence to have them passing in travis as well, yes, if only to figure out the cause of the failure conclusively. |
So,,, looking more into this, I noticed that we are installing the latest versions of node and npm but when I look at the errors we get in the slow test suit, I noticed it is reported in an older version of node: Sorry I am not very familiar with node, but is this test expected to not use the latest version of these packages? |
|
I'm seeing the same failure on master, fwiw. that particular tests is indeed supposed to be installing node v5.10.1. The new failure seems to be from running There's probably no reason the test needs to use any given node version, so let me try to update it in master. |
ljharb
force-pushed
the
enable-installation_iojs
branch
from
0027153 to
0236703
Compare
ljharb
changed the title
installation_iojs test suite to GitHub Actions
ljharb
force-pushed
the
enable-installation_iojs
branch
from
0236703 to
14507f8
Compare
New Stuff - `install.sh`: add `$ZDOTDIR` to zsh search (#3458) Fixes - `reinstall-packages`: do not reinstall corepack (#3544) - avoid bash-specific syntax (#3499) - `install-latest-npm`: npm v11 is out - `nvm_install_latest_npm`: avoid unbound variable (#3447) - give a more helpful message when `lts` alias is mistakenly used (#3441) - `nvm ls`, `nvm alias`, `nvm install`: error when an LTS name is invalid - `nvm_normalize_lts`: error when an LTS name is not lowercase (#3417) Documentation - [readme] update link - [readme] fix `--no-use` example (#3479) - [readme] update copyright notice (#3507) - [readme] note zsh-nvm's AUTO_USE option (#2670) - [readme] add note about reloading zshrc after editing (#3052) - [readme] Update shell profile file install notes (#2241) - [readme] add docker tips (#2318) - [readme] remove `avn` from readme (#3469) - [readme] fnm -> nvm.fish (#2541) Refactors - prefer `case` over if/else chains - combine `sed -e` invocations/arguments Tests - `nvm exec`/`nvm run`: add `--silent` tests (#1259) - [actions] release test needs git tags - migrate `installation_iojs` test suite to GitHub Actions (#3476) - Migrate slow test suite from Travis CI (#3470) - temporarily skip this failing travis test to unblock progress - [actions] TOC: use latest LTS node - `install.sh`: clean up `nvm_detect_profile` tests - `nvm_detect_profile`: refactor (#3467) - run urchin tests on pull requests (#3466) - update mocks - ensure that unit tests use only mocked LTS names - [actions] use `node/install` instead of `node/run` Meta - disable blank issues - update issue template - add DCO (#3456) - Rename .github/ISSUE_TEMPLATE.md to .github/ISSUE_TEMPLATE/ISSUE_TEMPLATE.md (#3454)
No description provided.