Skip to content

emrg: add config module tests - #27

Merged
argszero merged 1 commit into
masterfrom
test/add-config-tests
Jul 18, 2026
Merged

emrg: add config module tests#27
argszero merged 1 commit into
masterfrom
test/add-config-tests

Conversation

@argszero

Copy link
Copy Markdown
Owner

Adds 10 unit tests for emrg/config.py covering:

  • config_dir() / config_path() path construction
  • LlmConfig and EmrgConfig default values
  • load_config() valid file + defaults
  • load_config()${VAR} env-var resolution (set + unset)
  • load_config() FileNotFoundError
  • ensure_config() creates default + noop when exists

@argszeroargszero left a comment

Copy link
Copy Markdown
OwnerAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM — cycle #28

@argszeroargszero left a comment

Copy link
Copy Markdown
OwnerAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM — cycle #29

@argszero
argszeroforce-pushed the test/add-config-tests branch from 2545a41 to ebd538bCompareJuly 18, 2026 07:23

@argszeroargszero left a comment

Copy link
Copy Markdown
OwnerAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM — cycle #30

Sign up for freeto 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

@argszero