Add more API tests (and actually run them)

SciNim / nim-plotly

plotly wrapper for nim-lang

https://scinim.github.io/nim-plotly/

MIT License

179 stars 15 forks source link

Add more API tests (and actually run them) #47

Closed Vindaar closed 5 years ago

Vindaar commented 5 years ago

I decided to finally start writing a few more API tests. So far I only added tests for color and ErrorBar, but I'll see to adding more.

While writing the tests I realized how confusing the empty proc that returns a bool is. Since it's not exported (the proc is, but color.nim is not), I simply renamed it to isEmpty without any deprecation.

Vindaar commented 5 years ago

Merging this now. I'll add more tests in another PR.