thanos / terminusdb-client-elixir / 02beff45ac1b0684450e9646dfc90eb109109990-PR-16

97%

main: 93%

Pull #16

github

Summary of all fixes
High
- F1 — Flaky telemetry test fixed: uses a unique path ("telemetry/noop-<unique>") and filters refute_receive by %{path: ^unique} so it can't catch leaked events from concurrent tests. Verified: 0 failures in 50 runs with different seeds.
- F2 — unfold/minimized/compress_ids/as_list now use Params.bool_param/2 (sends false explicitly) instead of maybe_param (which dropped false). Added a regression test asserting unfold=false, minimized=false, compress_ids=false, as_list=false all appear in the query string.
Medium
- F3 — Document.stream/2 now uses a case to handle {:error, error} and raise error (raising TerminusDB.Error), instead of {:ok, resp} = ... which raised MatchError. Added two tests (404 error + transport error).
- F4 — Removed the false "JSON array" claim from the Streaming moduledoc and document_stream/1 doc. Now correctly documents that only concatenated JSON is supported and warns not to use as_list with streaming.
- F5 — Branch.create/3 now builds the origin body from the resolved org/repo (honoring :organization/:repo overrides) instead of config.* defaults. Added a test asserting the origin body reflects the overrides.
- F6 — Replaced all 15 occurrences of :ok = ...delete(...) with {:ok, _} = ...delete(...) across README, all 3 guides, Livebook, Database/Branch moduledocs, and the integration test.
- F7/F8 — Added integration tests for query/3 (inserts two docs with different ages, queries by template, asserts only matching docs returned), stream/2 (inserts 3 docs, streams them, asserts all labels), and Schema.frame/3 (retrieves a specific class frame after inserting a schema).
Low
- F9 — Extracted TerminusDB.Client.Params module with flag_param/2 (omit false) and bool_param/2 (always send non-nil). Replaced the 3 duplicated maybe_param definitions in Document, Schema, and Database.
- F10 — Added a receive ... after timeout to next_document so the stream halts if no chunk arrives within the timeout window (def... (continued)

108 of 114 new or added lines in 6 files covered. (94.74%)

249 of 256 relevant lines covered (97.27%)

105.39 hits per line