fix CI scripts referencing nonexistent app/ subdirectory
All six test/build scripts assumed a nested app/ package that doesn't exist — the Flutter package lives at the repo root. Removed every `cd app` and phantom path, consolidated the redundant dart/flutter analyze passes in test.sh, and scoped test_core.sh to core subsystem dirs so it no longer overlaps with the full flutter test run. Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
+1
-1
@@ -1,5 +1,5 @@
|
||||
{
|
||||
"exported_at": "2026-04-26T11:09:45Z",
|
||||
"exported_at": "2026-04-26T11:27:24Z",
|
||||
"decisions": [
|
||||
{
|
||||
"id": "D-1",
|
||||
|
||||
+2
-2
@@ -10,10 +10,10 @@ set -euo pipefail
|
||||
|
||||
cd "$(dirname "$0")/.."
|
||||
|
||||
BUNDLE="app/build/linux/x64/release/bundle/clide_app"
|
||||
BUNDLE="build/linux/x64/release/bundle/clide"
|
||||
|
||||
echo "==> build linux release bundle"
|
||||
(cd app && flutter build linux --release)
|
||||
flutter build linux --release
|
||||
|
||||
if [[ ! -x "$BUNDLE" ]]; then
|
||||
echo "smoke: bundle not found at $BUNDLE" >&2
|
||||
|
||||
+7
-12
@@ -1,20 +1,15 @@
|
||||
#!/usr/bin/env bash
|
||||
# Fast test layer — unit + widget + golden. Runs in <60s on a warm
|
||||
# cache. Called from `make test` and the pre-push hook.
|
||||
# Fast test layer — analyze + format + unit + widget + golden.
|
||||
# Runs in <60s on a warm cache. Called from `make test` and the
|
||||
# pre-push hook.
|
||||
set -euo pipefail
|
||||
cd "$(dirname "$0")/.."
|
||||
|
||||
echo "==> dart analyze (root package)"
|
||||
dart analyze
|
||||
echo "==> flutter analyze"
|
||||
flutter analyze
|
||||
|
||||
echo "==> dart format (whole tree)"
|
||||
dart format --set-exit-if-changed .
|
||||
|
||||
echo "==> dart test (root package)"
|
||||
dart test
|
||||
|
||||
echo "==> flutter analyze (app)"
|
||||
(cd app && flutter analyze)
|
||||
|
||||
echo "==> flutter test (app unit + widget + golden)"
|
||||
(cd app && flutter test)
|
||||
echo "==> flutter test (unit + widget + golden)"
|
||||
flutter test
|
||||
|
||||
+1
-1
@@ -5,4 +5,4 @@ set -euo pipefail
|
||||
cd "$(dirname "$0")/.."
|
||||
|
||||
echo "==> a11y suite"
|
||||
(cd app && flutter test test/a11y/)
|
||||
flutter test test/a11y/
|
||||
|
||||
+4
-2
@@ -33,9 +33,11 @@ TIMEOUT_SECONDS=${TIMEOUT_SECONDS:-120}
|
||||
# Run dart test in its own process group so we can kill descendants on
|
||||
# timeout. `setsid` starts a new session; `timeout --kill-after` SIGKILLs
|
||||
# after SIGTERM if the test ignores it.
|
||||
echo "test-core: dart test test/ (timeout ${TIMEOUT_SECONDS}s)"
|
||||
CORE_DIRS="test/ipc test/pty test/daemon test/git test/panes test/files test/editor test/cli test/pql"
|
||||
|
||||
echo "test-core: dart test ${CORE_DIRS} (timeout ${TIMEOUT_SECONDS}s)"
|
||||
if ! timeout --kill-after=5s "${TIMEOUT_SECONDS}s" \
|
||||
setsid --wait dart test test/ ; then
|
||||
setsid --wait dart test $CORE_DIRS ; then
|
||||
rc=$?
|
||||
if [[ $rc -eq 124 ]]; then
|
||||
echo "test-core: TIMEOUT — killing descendants" >&2
|
||||
|
||||
+2
-5
@@ -6,12 +6,9 @@ set -euo pipefail
|
||||
cd "$(dirname "$0")/.."
|
||||
|
||||
echo "==> flutter test --coverage"
|
||||
(cd app && flutter test --coverage)
|
||||
|
||||
echo "==> root dart coverage (skipped until dart_test + coverage integrate)"
|
||||
# dart test doesn't emit lcov natively; wire package:coverage later.
|
||||
flutter test --coverage
|
||||
|
||||
if command -v lcov >/dev/null 2>&1; then
|
||||
echo "==> lcov summary"
|
||||
lcov --summary app/coverage/lcov.info
|
||||
lcov --summary coverage/lcov.info
|
||||
fi
|
||||
|
||||
@@ -6,7 +6,6 @@
|
||||
set -euo pipefail
|
||||
cd "$(dirname "$0")/.."
|
||||
|
||||
cd app
|
||||
for f in integration_test/*_test.dart; do
|
||||
echo "==> integration_test: $f"
|
||||
flutter test "$f"
|
||||
|
||||
Reference in New Issue
Block a user