Skip to content

test(utils): cover TomlReader reads/read and decode errors#10315

Draft
Bartok9 wants to merge 1 commit into
marimo-team:mainfrom
Bartok9:top10/test-utils-toml-reader
Draft

test(utils): cover TomlReader reads/read and decode errors#10315
Bartok9 wants to merge 1 commit into
marimo-team:mainfrom
Bartok9:top10/test-utils-toml-reader

Conversation

@Bartok9

@Bartok9 Bartok9 commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

This pull request was authored by a coding agent.

AI-assisted; human-reviewed by the campaign operator before submit.

Summary

  • Unit tests for marimo._utils.toml.TomlReader (reads, read, decode_error)
  • Covers simple tables, nested tool.marimo tables, invalid input, and file-vs-string parity

Motivation

Config/frontmatter TOML parsing is reader-critical. Adds missing offline unit lock without touching production reader code.

Verification

  • Offline importlib runner — 4 passed
  • Exercises tomllib path on the host Python

Checklist notes

  • Draft + agent disclosure
  • DCO -s
  • Human review prior to push

Agent-Owner: sera · Platform: hermes · Claim-TTL: 24h

Exercise tomllib/tomlkit-backed reader for simple tables, nesting, and invalid input.

Signed-off-by: Bartok9 <danielrpike9@gmail.com>
@vercel

vercel Bot commented Jul 23, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
marimo-docs Ready Ready Preview, Comment Jul 23, 2026 10:58pm

Request Review

@Bartok9

Bartok9 commented Jul 23, 2026

Copy link
Copy Markdown
Contributor Author

I have read the CLA Document and I hereby sign the CLA

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.

1 participant