-
Notifications
You must be signed in to change notification settings - Fork 300
Run the js-ipfs-api
tests again in go-ipfs CI
#486
Comments
That is strange @Kubuxu , o you experience the same in your local machine or just team city? |
Haven't tested it on local machine but I will give it a try. |
The tests fail locally for me in many weird places but not the same failure mode as the TheamCity and I don't want to try debugging the node env, if you could try reproducing it in your env it would be great. It doesn't fail 100% but I think it is more than 50%-70% of failures. |
@Kubuxu Unfortunately I'm not being able to reproduce it. Which Node.js and npm versions are you using? Totally agree that this is undesirable as you will just start skipping the tests, thank you for reporting. |
Just had a failure on node with: npm: 3.10.8 and node: 6.9.1 |
npm-debug.log:
|
@Kubuxu can I get an update from you on this one? |
The teamcity is for now down, when we have Jenkins running I think about running this test in a loop to check if it still crashes. |
js-ipfs-api
tests again in go-ipfs CI
@victorbjelkholm we need to bring this back as soon as possible. Is it possible to have it with Jenkins? |
@victorbjelkholm any updates on this one? |
Ping :) |
@victorbjelkholm is this in progress with the new CI? |
ANy update here? |
@victorbjelkholm any update? |
I did a one time run locally with 0.14rc1 and a lot of tests failed. I think it's important to have the tests run before the final release. If that's hard to do automatically on CI, I think a short term solution would be to open a PR on js-ipfs-api which always uses the most recent RC. That would at least give some indication of broken things. |
@vmx can you share the test failures. We don't want to release something that breaks js-ipfs-api. |
@Kubuxu It's a lot of tests failing (31): https://friendpaste.com/72TbssZSoSxmdOpKxT67je What I did to run it locally. Checkout js-ipfs-api and after an |
As a reference, EDIT: Jakins stays |
👌 |
@victorbjelkholm is tracking this also in ipfs-inactive/dev-team-enablement#15 |
getting 404. |
Track at ipfs-inactive/dev-team-enablement#15 |
We use js-ipfs-api tests in go-ipfs to check if we didn't break those parts of API. Currently the CI is almost always red and it means that we will get used to that and just start ignoring the status as it isn't reliable.
See: http://ci.ipfs.team:8111/viewLog.html?buildId=10087&buildTypeId=GoIpfs_JsIpfsApiTests&tab=buildLog for error message
The test is started with
npm run test:node
.The text was updated successfully, but these errors were encountered: