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.

Sign up for GitHub

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

Jump to bottom

Test coverage for src/vector_tile_load_tile.hpp #192

Open
springmeyer opened this issue Apr 4, 2016 · 1 comment
Open

Test coverage for src/vector_tile_load_tile.hpp #192

springmeyer opened this issue Apr 4, 2016 · 1 comment

Comments

@springmeyer
Copy link
Contributor

We have 0% test coverage for https://coveralls.io/builds/5659740/source?filename=src%2Fvector_tile_load_tile.hpp. This is a pretty important file (heavy usage via node-mapnik) so we should prioritize adding some tests of it.

Note: coveralls has an epic bug (or is it the way we generate coverage data?) whereby 0% shows as 100%. So not funny it's almost funny.
springmeyer referenced this issue Apr 4, 2016
for best perf prefer unique_ptr over shared_ptr
f8e60ab
@jakepruitt
Copy link

whereby 0% shows as 100%.

birthday

springmeyer pushed a commit that referenced this issue Apr 10, 2016
Start on test coverage of tile loading - refs #192
3148804
springmeyer pushed a commit that referenced this issue Apr 18, 2016
Add test coverage of loading raster data - refs #192 (indirectly #202)
df00aea
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@springmeyer @jakepruitt