umputun / spot / 28690570935

84%

Build Type

push

github

Committed by web-flow

Commit Message

Fix multiline wait swallowing errors, wire delete exclude through (#352)

* Fix multiline wait swallowing errors, wire delete exclude through

Previously, a multiline wait command that timed out was reported as
successful when the remote temp-dir cleanup succeeded, because the
teardown defer overwrote the returned error with nil.

Previously, the delete command ignored its documented exclude patterns
and removed files the playbook asked to keep. Wiring it through:

- isExcludedSubPath now protects any ancestor of an excluded path, not
  just the immediate parent, so exclusions nested deeper than one level
  survive.
- Remote recursive delete removes the whole tree when no exclude pattern
  matched, matching the local executor, and logs a WARN in that case so
  a mistyped pattern is not silently destructive.
- The dir-glob rule (a 'dir/*' pattern also protecting the directory it
  names) is now gated on the entry being a directory, threaded through
  all isExcluded call sites, so a pattern like 'dir*/*' no longer
  excludes a plain file such as 'data.txt'.
- Exclusion matching uses forward-slash semantics on all platforms, so
  documented patterns work with windows local paths too.
- exclude requires recur and is rejected without it, symmetric between
  sudo and non-sudo; sudo+exclude is still rejected. Both errors name the
  offending location, and mdelete validates all entries before deleting
  any of them.

The wait teardown fix mirrors Script: the teardown error is only
surfaced when there is no primary error.

* Address round-2 review: remote warn gating, glob dir excludes, site docs

- Gate the remote no-match delete WARN on a non-empty exclude list, so an
  exclude-free recursive delete (e.g. temp-dir cleanup after every
  multiline script or wait) no longer logs a spurious warning; local was
  already guarded by its len(excl)==0 early return.
- Remote Upload and findMatchedFiles now stat each glob match and pass the
  real isDir to isExcluded, ... (continued)

Coverage Stats

97 of 106 new or added lines in 4 files covered. (91.51%)

1 existing line in 1 file now uncovered.

3190 of 3798 relevant lines covered (83.99%)

58.93 hits per line