Define apache_name via a let block
Define apache_name via a let block

Defining variables in rspec is dangerous since they end up being defined
on classes. That means that in a for loop it may end up being defined as
the last running version. A let block provides proper variable scoping.

This wasn't a problem until fb0d0782d8221741474f8170dacf522f6ce32b69
where it ran spec tests on all operating systems. Then it became
dependent on the order of the supported operating systems.

Fixes: fb0d0782d8221741474f8170dacf522f6ce32b69

puppetlabs/puppetlabs-apache,traylenator/puppetlabs-apache,puppetlabs/puppetlabs-apache,traylenator/puppetlabs-apache