Skip to content

Expand SFW conversion fixtures and edge-case tests - #5

Merged
CrushOnAI merged 1 commit into
mainfrom
feat/comprehensive-fixtures
Aug 20, 2026
Merged

CrushOnAI merged 1 commit into
mainfrom
feat/comprehensive-fixtures

Conversation

@CrushOnAI

Copy link
Copy Markdown
Owner

Summary

  • add a comprehensive original SFW SillyTavern fixture
  • cover Character, Location, Organization, Event, Rule and Item mappings
  • verify Always On behavior for entries without keywords
  • cover disabled entries, empty content, duplicate keywords and duplicate titles
  • add an original SFW World Card fixture for namespaced optional age-group and gender fields
  • document that optional fields are preserved in extensions without claiming native third-party compatibility

Privacy and sanitization

  • all names, settings and text are original fictional SFW content
  • no real user data, account identifiers, character IDs, conversations or internal URLs
  • no credentials, tokens, cookies, email addresses or storage URLs
  • numeric uid values are local fixture sequence numbers only

Verification

  • PYTHONPATH=src python3 -m unittest discover -s tests -v (7 passed)
  • comprehensive Static Converter smoke test (7 entries across 6 note types)
  • git diff --check (passed)

@CrushOnAI
CrushOnAI marked this pull request as ready for review August 20, 2026 10:19
@CrushOnAI
CrushOnAI merged commit 67a31a6 into main Aug 20, 2026
3 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.

1 participant