Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Write a test suite #25

Open
6 tasks
Libbum opened this issue May 29, 2017 · 6 comments
Open
6 tasks

Write a test suite #25

Libbum opened this issue May 29, 2017 · 6 comments
Assignees
Labels

Comments

@Libbum
Copy link
Owner

Libbum commented May 29, 2017

This hasn't been done at all yet, but is required.

  • main.rs
  • config.rs
  • input.rs
  • output.rs
  • grid.rs
  • potential.rs
@Libbum Libbum added the TODO label May 29, 2017
@Libbum Libbum self-assigned this May 29, 2017
@Libbum
Copy link
Owner Author

Libbum commented Jun 1, 2017

7b284a3 links codecov, so we can see how we're doing with this.

@Libbum
Copy link
Owner Author

Libbum commented Jun 6, 2017

There have been a few issues with coverage rankings that are now fixed. We have a low coverage amount mainly because all of my functions pass in config et al. which are quite bulky to build for a test. I wonder what the best method of fixing that is?

Also, many things I do are large computations or file io, which I need to read up on how to build tests for these things. For the moment, without coverage here there's going to be little use of the testing suite.

@Libbum Libbum modified the milestones: Version 0.2, Initial release Aug 20, 2017
@Libbum
Copy link
Owner Author

Libbum commented Aug 22, 2017

The git version of tarpaulin now supports codecov. So we can update to that soon.

@Libbum
Copy link
Owner Author

Libbum commented Aug 24, 2017

There's an issue in the compiler that is making tarpaulin fail when trying to compile serde_derive with -no-pie. Monitoring the status, but this is out of my league to fix right now, so will park this move.

@Libbum
Copy link
Owner Author

Libbum commented Oct 22, 2017

Check tarpaulin now, there have been a number of version bumps since last looking into this. Nothing on the open issue, but it may be resolved through other means...

Edit: nope. The author is attempting to patch the rust core. Seems deep.

@Libbum
Copy link
Owner Author

Libbum commented Jan 30, 2018

The current code coverage map has some pretty heavy issues, so don't trust it too much.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant