Validate references consider aliased resources too #43

Merged
merged 6 commits into from Jan 2, 2017

Projects

None yet

1 participant

@kpaulisse
Contributor

The new validate references feature did not consider the alias parameter, and as such returned an error in case of a valid pattern such as the following:

exec { 'some resource':
  notify => Exec['the alias'],
}

exec { 'the target':
  alias => 'the alias',
}
kpaulisse added some commits Jan 2, 2017
@kpaulisse kpaulisse Merge branch 'kpaulisse-arbitrary-command-line-pass' into kpaulisse-v…
…alidate-references-alias
401a07a
@kpaulisse kpaulisse Add fixtures for validation of resources identified by aliases a725639
@kpaulisse kpaulisse Add test for alias resources 04019a9
@kpaulisse kpaulisse Add alias to resource hash if an alias is defined c9b36ef
@kpaulisse kpaulisse Add unit test for resource hash builder handling aliases
abf23a2
@kpaulisse kpaulisse Merge remote-tracking branch 'origin/master' into kpaulisse-validate-…
…references-alias
9185848
@kpaulisse kpaulisse merged commit 15049e3 into master Jan 2, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@kpaulisse kpaulisse deleted the kpaulisse-validate-references-alias branch Jan 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment