Skip to content

Exclude generated columns from fixture snapshots - #101

Closed
nertzy wants to merge 3 commits into
rdy:masterfrom
nertzy:exclude-generated-columns
Closed

Exclude generated columns from fixture snapshots#101
nertzy wants to merge 3 commits into
rdy:masterfrom
nertzy:exclude-generated-columns

Conversation

@nertzy

@nertzy nertzy commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

Generated fixture YAML included database-generated values that Rails cannot write back when loading fixtures.

This filters only adapter-reported generated columns after both model-backed and raw-query extraction, retaining custom query fields and ordinary persisted attributes. It adds real SQLite generated-column coverage for both paths, including custom model-table and custom SQL-alias regressions.

The README documents the behavior and the changelog records the 0.6 fix.

@nertzy
nertzy force-pushed the exclude-generated-columns branch 2 times, most recently from 711113e to c61537e Compare September 2, 2026 22:22
The Test::Unit tests shared one top-level create_and_blow_away_old_db
helper defined in test_helper.rb, which mixed connection configuration
and schema definition into the global namespace.

Move it into a test/support/test_database.rb module that the four test
classes using it include. The module exposes the same
create_and_blow_away_old_db entry point and keeps the file-backed
test.db configuration unchanged, so behavior is identical; it also
separates connection setup from schema creation so the connection
configuration can be changed in isolation later.

The module deliberately defines no setup/teardown instance methods:
including classes define their own, so any future lifecycle hook must
be registered through Test::Unit's class-level setup/teardown
callbacks.
@nertzy
nertzy force-pushed the exclude-generated-columns branch from c61537e to 54ca3fc Compare September 3, 2026 16:58
@nertzy nertzy closed this Sep 3, 2026
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