puppetlabs/puppetlabs-apt — pull request #80 by crayfishx (2012-07-16 22:36:44+00:00)
Title: Fixed handling of apt update refreshonly setting.

* Parameterized the apt::update class to allow the base class to pass in the value for refreshonly based on the setting of always_apt_update

The base class had an interesting way of trying to override the refresh only parameter of the exec for apt update...

```
if $always_apt_update == true {
Exec <| title=='apt_update' |> {
refreshonly => false,
}
}
```

I'm pretty sure that has never worked - it caused it to fail many tests with;

```
1) apt when specifying class parameters
Failure/Error: it { should include_class("apt::params") }
Puppet::Error:
Puppet::Parser::Compiler failed with error ArgumentError: wrong number of arguments (1 for 0)
```

This patch makes all spec tests pass.