check if ensure present before including alias module

[maneeshmp]

Even when default_ssl_vhosts is false, the value of scriptalias gets defined and vhost resource includes the alias module. To avoid that we need to check the condition if ensure => present before including the alias module.

[maneeshmp]

Fixed the test failure.

[igalic]

could you squash those two commits please?

[maneeshmp]

I have squashed the two changes into one commit

[igalic]

o_O why is it failing Travis now :(

[maneeshmp]

that failure doesn't have to do anything with my change :(

On Thu, Apr 16, 2015 at 3:11 PM, Igor Galić notifications@github.com
wrote:

o_O why is it failing Travis now :(

—
Reply to this email directly or view it on GitHub
#1102 (comment)
.

Regards
Maneesh

[maneeshmp]

rather i mean it doesn't have anything to do with my change. what to do now
?

Thanks,
Maneesh

On Thu, Apr 16, 2015 at 4:59 PM, Maneesh maneeshmp@gmail.com wrote:

that failure doesn't have to do anything with my change :(

On Thu, Apr 16, 2015 at 3:11 PM, Igor Galić notifications@github.com
wrote:

o_O why is it failing Travis now :(

—
Reply to this email directly or view it on GitHub
#1102 (comment)
.

Regards
Maneesh

Regards
Maneesh

[underscorgan]

@maneeshmp thanks for the contribution! I reran the tests and they worked, so merging!