minimeteor icon indicating copy to clipboard operation
minimeteor copied to clipboard

DOCs: are there CI Tests? Badges?

Open 0o-de-lally opened this issue 7 years ago • 6 comments

It was mentioned there were CI tests. Can you post the status or badges of which node/meteor versions this is tested against?

0o-de-lally avatar Mar 08 '17 22:03 0o-de-lally

I have no badges unfortunately, since it's a completely custom test framework. It builds test images and runs them on a VPS, not on CircleCI or anything similar. If they run successfully, they are pushed to Docker Hub. Here are the images it tested so far: https://hub.docker.com/r/aedm/minimeteor-buildtest/tags/

I'm open to adding any badges to the readme. What kind of badges do you think would make sense? One I found was this, but it's not related to tests: https://img.shields.io/docker/pulls/aedm/minimeteor.svg

aedm avatar Mar 09 '17 14:03 aedm

I think it would be valuable for the community to see the builds on CircleCi or other. Specifically CircleCi allows you to use docker images now for building, so maybe it could streamline your build process. https://circleci.com/docs/1.0/docker/

0o-de-lally avatar Mar 09 '17 17:03 0o-de-lally

Unfortunately, CircleCI doesn't have the features I need to test it. When I push a new version, the test server fetches the list of Meteor releases from GitHub, and runs the test against all versions. This process takes several hours. If any of the tests fails, it's automatically retried forever until it succeeds, this protects agains issues like network outages. I understand that a CircleCI badge looks nice, but I can't do all that on Circle.

aedm avatar Mar 09 '17 19:03 aedm

I'm just interested in a process that the community can reproduce. I'm not sure everything you need, but circleci 2.0 will have more docker features. https://circleci.com/beta-access/

0o-de-lally avatar Mar 09 '17 20:03 0o-de-lally

I don't think CircleCI is necessary. But I do agree with the sentiment that it would be nice to make tests reproducible for the community.

ethanwillis avatar Mar 19 '17 01:03 ethanwillis

You're right, these tests need to be reproducible. The code is here: https://gitlab.com/aedm/meteor-crawler

It's really hackish at the moment, but I'll just leave it here to show it's not a secret. It's going to take some time to clean them up and write a manual.

aedm avatar Mar 20 '17 09:03 aedm