feat: add make verify infrastructure with expected outputs#13
Merged
Conversation
… examples - Add scripts/normalize_output.sh for reproducible output comparison - Add expected output files for 6 pycubrid and 6 SQLAlchemy examples - Add verify and demo targets to Makefile (Python-only) - Fix non-deterministic sort order in 04_relationships.py - Simplify GETTING_STARTED.md to under 20 lines with 3-line quick start Closes #8 (reimplemented from scratch after branch deletion) Partial #7 (verify infrastructure + fundamentals expected outputs)
…ples Add expected output files for 11 additional examples beyond the initial 12 fundamentals, bringing total coverage to 23/23 runnable examples passing make verify. Fixes: - migration/03_crud_orm.py: use :.2f in __repr__ for CUBRID float precision - migration/05_batch_operations.py: reorder main block so UPDATE/DELETE operate on correct dataset instead of post-reset empty table - quickstart/5min-sqlalchemy/app.py: add drop_all before create_all for deterministic auto_increment IDs - templates/batch-etl/seed_data.py: rename to 00_seed_data.py for correct sort-order execution in find|sort pipeline - scripts/normalize_output.sh: add bare Version: pattern normalization Ref: #7
- ci.yml: remove Node/Go lint jobs, run ruff on repo root instead of obsolete python/ directory - smoke-test.yml: replace old multi-language matrix with single make verify job against CUBRID service - Format 4 Python files to pass ruff format --check
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds
make verify/make demoinfrastructure and expected output files for all 12 fundamentals examples. Replaces #8 (branch was deleted, repo restructured since).Changes
scripts/normalize_output.sh— Normalizes dynamic values (VERSION, HOSTNAME, DATE, TIME, OID, BTREE, TIMESTAMP) and filters SQLAlchemy warnings for reproducible diff comparisonMakefile— Addedverify(find expected files → run script → normalize → diff) anddemo(up + verify) targets; simplified to Python-only per AGENTS.mdfundamentals/orm-basics/04_relationships.py— Sorted joined collection output for deterministic resultsGETTING_STARTED.md— Simplified from 313 lines to 14 lines with 3-line quick startVerification
Issue Progress
Closes #8 (reimplemented from scratch after branch deletion and repo restructure)
Partial progress on #7:
make verifytargetmake demotarget