Add test module

[burdges]

There is maybe no reason for this in a macro only crate, but minor benefits exist according to http://stackoverflow.com/a/40271926/667457

I tested that adding #[cfg(test)] before each #[test] worked too, if that seemed preferable, but probably not

[droundy]

Looks like a reasonable cleanup.

[burdges]

Appears #[test] actually does imply #[cfg(test)], so my original reasons for suggesting this are vapor. I suppose moving the quickcheck dependency to just the test module improves matters though, which I'd completely missed before. :)

[droundy]

Yes, I assumed that was the main reason. And to accommodate any helper
functions for new tests introduced in the future.

On Mon, Oct 31, 2016, 1:57 PM Jeff Burdges notifications@github.com wrote:

Appears #[test] actually does imply #[cfg(test)], so my original reasons
for suggesting this are vapor. I suppose moving the quickcheck dependency
to just the test module improves matters though, which I'd completely
missed before. :)

—
You are receiving this because you modified the open/close state.

Reply to this email directly, view it on GitHub
#6 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAIZKXiDSmKza3_W6lkXW8vkxds2-tYQks5q5lYtgaJpZM4KhstV
.