Skip to content

Add CSV integration, DB factory, test harness, and sandbox docs (#488, #487, #486, #485) - #532

Merged
Tinna23 merged 1 commit into
StellarCommons:mainfrom
Maryermarh:test/csv-integration-db-factory-harness-docs
Jul 26, 2026
Merged

Add CSV integration, DB factory, test harness, and sandbox docs (#488, #487, #486, #485)#532
Tinna23 merged 1 commit into
StellarCommons:mainfrom
Maryermarh:test/csv-integration-db-factory-harness-docs

Conversation

@Maryermarh

@Maryermarh Maryermarh commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

Changes

Add CSV writer integration test (Closes #488)

  • Test CsvWriter creates valid CSV files with correct headers
  • Test CSV roundtrip with fee records containing timestamp, fee_stroops, sequence

Add DB factory integration test (Closes #487)

  • Test database connection string creation for SQLite
  • Test in-memory database connection string
  • Test multiple connections to same database path

Add test harness with common utilities (Closes #486)

  • TestContext: Create temporary directories and fixture files
  • TestDatabase: Create and clean up SQLite test databases
  • Common setup/teardown modules for integration tests

Add sandbox documentation to README (Closes #485)

  • Usage examples for sandbox environment
  • API documentation for sandbox components

- Add CSV writer integration test (StellarCommons#488): Test CsvWriter creates valid files and roundtrips
- Add DB factory integration test (StellarCommons#487): Test database connection string creation
- Add test harness with common utilities (StellarCommons#486): TestContext, TestDatabase, setup/teardown modules
- Add sandbox documentation to README (StellarCommons#485): Usage examples and API documentation
@drips-wave

drips-wave Bot commented Jul 26, 2026

Copy link
Copy Markdown

@Maryermarh Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Tinna23 Tinna23 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for the contribution — nice work! Approving and merging this in. 🎉

@Tinna23
Tinna23 merged commit 24f2209 into StellarCommons:main Jul 26, 2026
2 of 7 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

2 participants