Nitpick, ```assert_not_empty puppetclasses``` or ```assert_not puppetclasses.empty?``` look easier to read for me.