BewareMyPower commented on code in PR #984:
URL: https://github.com/apache/pulsar-client-go/pull/984#discussion_r1146120366
##########
pulsar/transaction_test.go:
##########
@@ -79,6 +81,138 @@ func TestTCClient(t *testing.T) {
defer client.Close()
}
+/**
+* Test points:
+* 1. Abort and commit txn.
+* 1. Do nothing, just open a transaction and then commit
it or abort it.
+* The operations of committing/aborting txn should
success at the first time and fail at the second time.
+* 2. The internal API, registerSendOrAckOp and endSendOrAckOp
+* 1. Register 4 operation but only end 3 operations, the
transaction can not be committed or aborted.
+* 2. Register 3 operation and end 3 operation the
transaction can be committed and aborted.
+* 3. Register an operation and end the operation with an
error,
+* and then the state of the transaction should be
replaced to errored.
+* 3. The internal API, registerAckTopic and registerProducerTopic
+* 1. Register ack topic and send topic, and call http
request to get the stats of the transaction
+* to do verification.
+ */
+/** TestTxnImplCommitOrAbort Test abort and commit txn */
Review Comment:
Please don't use such type of comments. Use:
```go
// xxx
// yyy
// zzz
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
