Skip to content

ci:check llgo test's stderr to exit#364

Merged
MeteorsLiu merged 3 commits intogoplus:mainfrom
luoliwoshang:ci/test
May 22, 2025
Merged

ci:check llgo test's stderr to exit#364
MeteorsLiu merged 3 commits intogoplus:mainfrom
luoliwoshang:ci/test

Conversation

@luoliwoshang
Copy link
Member

@luoliwoshang luoliwoshang commented May 22, 2025

can exit expect when llgo test

https://github.com/goplus/llcppg/actions/runs/15177637874/job/42680771642?pr=364
https://github.com/goplus/llcppg/actions/runs/15177637874/job/42680866353?pr=364

Run # llgo test ./_xtool/internal/parser/...
todo: unknown builtin type: Ibm128
--- FAIL: TestBuiltinType (0.02s)
    ???:1: Void Kind mismatch:got 0 want 1, 
FAIL
/home/runner/go/bin/parser.test: exit code 1
llgo test fail
Error: Process completed with exit code 1.

@codecov
Copy link

codecov bot commented May 22, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.43%. Comparing base (6194a92) to head (a071b69).
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #364   +/-   ##
=======================================
  Coverage   80.43%   80.43%           
=======================================
  Files          28       28           
  Lines        3496     3496           
=======================================
  Hits         2812     2812           
  Misses        645      645           
  Partials       39       39           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@luoliwoshang luoliwoshang requested a review from MeteorsLiu May 22, 2025 06:57
@MeteorsLiu MeteorsLiu merged commit 63a0fe0 into goplus:main May 22, 2025
6 checks passed
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.

2 participants