Skip to content

Move integration tests to cohesive test files#110

Merged
SeanTAllen merged 1 commit intomainfrom
move-integration-tests-to-cohesive-files
Feb 12, 2026
Merged

Move integration tests to cohesive test files#110
SeanTAllen merged 1 commit intomainfrom
move-integration-tests-to-cohesive-files

Conversation

@SeanTAllen
Copy link
Member

Moves cancel and transaction integration tests from _test_query.pony to their feature-specific test files:

  • _TestCancelPgSleep and _TestCancelSSLPgSleep_test_cancel.pony
  • _TestTransactionCommit and _TestTransactionRollbackAfterFailure_test_transaction_status.pony

These tests were in _test_query.pony because it was the original home for all query-related integration tests, but as features got their own test files the integration tests weren't moved with them. Each feature's tests now live together regardless of whether they use a mock server or a real PostgreSQL instance.

Closes #109

Cancel integration tests (_TestCancelPgSleep, _TestCancelSSLPgSleep) now
live in _test_cancel.pony alongside the mock-server cancel unit tests.
Transaction integration tests (_TestTransactionCommit,
_TestTransactionRollbackAfterFailure) now live in
_test_transaction_status.pony alongside the mock-server transaction
status unit tests.

Previously these lived in _test_query.pony because it was the original
home for all query-related integration tests, but as cancel and
transaction status got their own test files, the integration tests
weren't moved with them.

Closes #109
@ponylang-main ponylang-main added the discuss during sync Should be discussed during an upcoming sync label Feb 12, 2026
@SeanTAllen SeanTAllen merged commit 4768b00 into main Feb 12, 2026
10 checks passed
@SeanTAllen SeanTAllen deleted the move-integration-tests-to-cohesive-files branch February 12, 2026 20:56
@ponylang-main ponylang-main removed the discuss during sync Should be discussed during an upcoming sync label Feb 12, 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.

Move integration tests to cohesive test files

2 participants