Skip to content

Comments

[Transaction] Add transaction integration test#8594

Merged
codelipenghui merged 7 commits intoapache:masterfrom
gaoran10:txn-integration-test
Nov 19, 2020
Merged

[Transaction] Add transaction integration test#8594
codelipenghui merged 7 commits intoapache:masterfrom
gaoran10:txn-integration-test

Conversation

@gaoran10
Copy link
Contributor

Motivation

Currently, the transaction is a lack of integration test for validation.

Modifications

Add transaction integration test, the test based on the mock transfer business scene.

Verifying this change

This change added tests and can be verified as follows:

  • org.apache.pulsar.tests.integration.transaction.TransactionTest

Does this pull request potentially affect one of the following parts:

If yes was chosen, please highlight the changes

  • Dependencies (does it add or upgrade a dependency): (no)
  • The public API: (no)
  • The schema: (no)
  • The default values of configurations: (no)
  • The wire protocol: (no)
  • The rest endpoints: (no)
  • The admin cli options: (no)
  • Anything that affects deployment: (no)

@gaoran10
Copy link
Contributor Author

/pulsarbot run-failure-checks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants