Remove #[cfg(notest)] and use #[cfg(not(test))]

Makes it more consistent, imo.