coolprop icon indicating copy to clipboard operation
coolprop copied to clipboard

Make the Travis build succeed with no errors so that we can do proper unit testing

Open ibell opened this issue 11 years ago • 5 comments

It is hard to tell whether we have screwed things up with the build since there are always errors in the Travis CI build. Need to resolve/remove the failing tests.

ibell avatar Jan 23 '14 11:01 ibell

So I was thinking about this! The idea was:

  • When the project builds, send a list of test results somewhere such that,
  • When it comes to the next build a success is determined by that test not changing state (failing)

I was looking though the travisci build system, and we can also have it automatically zip up and push binaries to pypi, sourceforge and wherever else.

Rod-Persky avatar Jan 25 '14 06:01 Rod-Persky

One thing they have is a passing or failing png you can embed in a website, a la https://github.com/philsquared/Catch (see in the readme.md)

On Sat, Jan 25, 2014 at 7:38 AM, Rod Persky [email protected]:

So I was thinking about this! The idea was:

  • When the project builds, send a list of test results somewhere such that,
  • When it comes to the next build a success is determined by that test not changing state

I was looking though the travisci build system, and we can also have it automatically zip up and push binaries to pypi, sourceforge and wherever else.

— Reply to this email directly or view it on GitHubhttps://github.com/ibell/coolprop/issues/108#issuecomment-33282542 .

ibell avatar Jan 26 '14 17:01 ibell

If we get it to stop failing the specific tests that it does, then that png should be pretty good - otherwise i'll be deceiving that the program is completely broken.

Rod-Persky avatar Feb 03 '14 08:02 Rod-Persky

Right, which is why I haven't implemented that yet. I've been adding a lot of tests recently at the C++ level which we can also call from python. Should make for a pretty solid testing system.

On Mon, Feb 3, 2014 at 9:30 AM, Rod Persky [email protected] wrote:

If we get it to stop failing the specific tests that it does, then that png should be pretty good - otherwise i'll be deceiving that the program is completely broken.

Reply to this email directly or view it on GitHubhttps://github.com/ibell/coolprop/issues/108#issuecomment-33931996 .

ibell avatar Feb 03 '14 08:02 ibell

@Rod-Persky Can you show me where to set up the hooks for the pypi uploading, etc?

ibell avatar Feb 22 '14 20:02 ibell