diff options
| author | Markus Roberts <Markus@reality.com> | 2010-05-01 10:13:19 -0700 |
|---|---|---|
| committer | test branch <puppet-dev@googlegroups.com> | 2010-02-17 06:50:53 -0800 |
| commit | 11189fb1fcf8fb410df3c833d20decb2ec7aa8c2 (patch) | |
| tree | 4f87620fd0b3aaaf0a9a6685e1f69f0e73aa3601 /test | |
| parent | 913b63cddfaed68605aa3341010b0aa53c9870a5 (diff) | |
| download | puppet-11189fb1fcf8fb410df3c833d20decb2ec7aa8c2.tar.gz puppet-11189fb1fcf8fb410df3c833d20decb2ec7aa8c2.tar.xz puppet-11189fb1fcf8fb410df3c833d20decb2ec7aa8c2.zip | |
Fix for #2910 -- Tidy/matches is too tricky to use
The semantic interaction of tidy/matches and tidy/recurse is tricky to get
right; it only makes sense to use matches with recursion (a fixed path will
either statically match or it won't, no need for a run-time check) but there
was nothing to warn users of this fact. To compound matters, the example
in the matches parameter doc string even made this mistake.
This patch: 1) fixes the doc string; 2) prohibits the use of match without a
value of recurse capable of generating files to match, 3) fixes tests that
were passing for the wrong reason and adds tests on the prohibition added
in (2).
Diffstat (limited to 'test')
0 files changed, 0 insertions, 0 deletions
