test: verify post-increment fix under set -e (#618)

Add tests that run get_executable_lines, report_lcov, and str::rpad
in subshells with set -e to ensure ((++var)) does not cause silent
early exit when the counter starts at zero.

Author: Chemaclass

tests/unit/coverage_core_test.sh

@@ -174,6 +174,29 @@ EOF
   rm -f "$temp_file"
 }
 
+function test_coverage_get_executable_lines_does_not_exit_under_set_e() {
+  local temp_file
+  temp_file=$(mktemp)
+
+  cat >"$temp_file" <<'EOF'
+#!/usr/bin/env bash
+echo "line 1"
+echo "line 2"
+EOF
+
+  # ((var++)) when var=0 evaluates to 0 (falsy) causing exit code 1;
+  # under set -e this silently terminates the function (#618)
+  local result
+  result=$(
+    set -e
+    bashunit::coverage::get_executable_lines "$temp_file"
+  )
+
+  assert_equals "2" "$result"
+
+  rm -f "$temp_file"
+}
+
 function test_coverage_record_line_writes_to_file() {
   BASHUNIT_COVERAGE="true"
   BASHUNIT_COVERAGE_PATHS="/"

tests/unit/coverage_reporting_test.sh

@@ -146,6 +146,40 @@ EOF
   rm -f "$temp_file" "$report_file"
 }
 
+function test_coverage_report_lcov_completes_under_set_e() {
+  BASHUNIT_COVERAGE="true"
+  bashunit::coverage::init
+
+  local temp_file
+  temp_file=$(mktemp)
+  cat >"$temp_file" <<'EOF'
+#!/usr/bin/env bash
+echo "line 1"
+echo "line 2"
+EOF
+
+  echo "$temp_file" >"$_BASHUNIT_COVERAGE_TRACKED_FILES"
+
+  local report_file
+  report_file=$(mktemp)
+
+  # ((lineno++)) when lineno=0 returns exit code 1 under set -e
+  # causing incomplete LCOV output (#618)
+  (
+    set -e
+    bashunit::coverage::report_lcov "$report_file"
+  )
+
+  local content
+  content=$(cat "$report_file")
+
+  assert_contains "end_of_record" "$content"
+  assert_contains "DA:2," "$content"
+  assert_contains "DA:3," "$content"
+
+  rm -f "$temp_file" "$report_file"
+}
+
 function test_coverage_report_text_shows_no_files_message() {
   BASHUNIT_COVERAGE="true"
   bashunit::coverage::init

tests/unit/str_test.sh

@@ -51,6 +51,18 @@ function test_rpad_custom_width_padding_text_too_long_and_special_chars() {
     "$actual"
 }
 
+function test_rpad_does_not_exit_under_set_e() {
+  # ((i++)) when i=0 evaluates to 0 (falsy) causing exit code 1;
+  # under set -e this silently terminates the function (#618)
+  local actual
+  actual=$(
+    set -e
+    bashunit::str::rpad "input" "1" 20
+  )
+
+  assert_same "input              1" "$actual"
+}
+
 function test_rpad_width_smaller_than_right_word() {
   local actual=$(bashunit::str::rpad "foo" "verylongword" 5)