Skip to content

test(build): raise the standalone size guard to 550 KiB #1047

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Chemaclass wants to merge 1 commit into
base: main
Choose a base branch
from
+5 −2
Open

test(build): raise the standalone size guard to 550 KiB #1047

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 5 additions & 2 deletions tests/unit/project/build_test.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -321,7 +321,10 @@ function test_built_binary_contains_no_source_lines() {
assert_equals "0" "$(grep -c '^source ' "$build_dir/bashunit")"
}

function test_built_binary_stays_below_500_kib() {
# A ceiling on accidental bloat (a stray embed, a disabled comment strip), not a
# cap on the project: features legitimately grow the distributable. Re-baseline
# deliberately when planned growth reaches it, keeping ~10% headroom.
function test_built_binary_stays_below_550_kib() {
if ! build_optimizer_is_available; then
bashunit::skip "shfmt and jq are required for standalone optimization"
return
Expand All @@ -334,7 +337,7 @@ function test_built_binary_stays_below_500_kib() {

local bytes
bytes=$(wc -c <"$build_dir/bashunit" | tr -d ' ')
assert_less_or_equal_than 512000 "$bytes"
assert_less_or_equal_than 563200 "$bytes"
}

function test_build_assert_valid_syntax_rejects_broken_file() {
Expand Down