Ugbot / Ugbot/Agentic-Streaming
Test integrity: tests that pass without testing
Nobody has claimed this yet.
- Dominant language
- Java
- Stars
- 37
- Forks
- 2
- Avg merge
- 10h 46m
- Merged PRs (30d)
- 28
Description
The suite is large and mostly green (770 tests pass in the main module), but three mechanisms let it report green on untested code.
- Fixture-missing skips hide the flagship parity proof. The cross-runtime parity tests resolve
examples/pipelines/banking.yamlvia a CWD-relative path guarded byassumeTrue. Under the documented build invocation the path misses and they silently skip — measured: 3/3 and 5/6 skipped. Re-run with the correct working directory, all 21 parity/replay/CEP tests pass — so the parity claim is real but unverified by default. The same fail-open pattern exists in the Pekko and Clojure parity tests. - H2 stands in for Postgres, which is precisely why the invalid-SQL bug in #5 shipped. Needs Testcontainers.
*IT.javafiles are unrunnable in every profile — no failsafe plugin, and surefires filename filter excludes them before the tag filter applies. Net effect: all Redis/Fluss integration tests have zero executed coverage, and no IT exists at all for Postgres, pgvector, Qdrant or Milvus (~2,300 LOC of untested vector-store drivers).
Also worth a sweep: a set of tests assert nothing meaningful — one re-declares the parser regexes locally instead of importing the class under test, one constructs a function and never invokes it, and the config test never sets an env var despite the class existing to resolve them.
Roadmap epic. Detailed findings, file:line citations and sub-tasks live in the AI-grind tracker (project AGS), which is the source of truth for this work. Local dev — build, test, tracking, profiling — is managed with AI-grind.
Tracked as AGS-8 in the devtools-mcp tracker.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the AI-grind tracker entry for AGS-8 and the documented build invocation, then inspect the CWD-relative examples/pipelines/banking.yaml lookup and the *IT.java test naming. Verify which parity, replay, CEP, Redis and Fluss tests actually execute, and audit the cited no-op tests. Done means the intended tests run by default, database coverage uses the required services, and assertions exercise the production code.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java, postgres, redis
- Domain
- ci-cd, databases, testing
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Quiet
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100