Skip to content

test(redirect): add append redirect spec tests#1137

Merged
chaliy merged 1 commit intomainfrom
fix/issue-1119-append-dup
Apr 7, 2026
Merged

test(redirect): add append redirect spec tests#1137
chaliy merged 1 commit intomainfrom
fix/issue-1119-append-dup

Conversation

@chaliy
Copy link
Copy Markdown
Contributor

@chaliy chaliy commented Apr 7, 2026

Summary

  • Added comprehensive spec tests for >> append behavior in compound commands
  • Tests cover: simple append, brace groups, existing files, rebuild cycles, if/else
  • All tests pass — the basic append mechanism works correctly
  • The issue may require a more specific bashblog-style reproduction to trigger

Test plan

  • append_simple — basic >> append
  • append_brace_group{ echo a; echo b; } >> file
  • append_brace_group_existing_file — append to existing file
  • append_no_duplicate — repeated appends
  • append_if_else — if/else with >> redirect
  • append_rebuild_cycle — double brace group append (simulating rebuild)

Closes #1119

Adds comprehensive tests for >> append behavior in compound commands,
brace groups, if/else, and rebuild cycle patterns. All tests pass —
the basic append mechanism is correct. If the issue reappears with a
specific reproduction, these tests provide a baseline.

Closes #1119
@chaliy chaliy merged commit 9fddfdc into main Apr 7, 2026
18 checks passed
@chaliy chaliy deleted the fix/issue-1119-append-dup branch April 7, 2026 02:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(redirect): >> append to file duplicates content in compound commands

1 participant