terrymanu opened a new issue #7522: URL: https://github.com/apache/shardingsphere/issues/7522
The class TextProtocolBackendHandlerFactory has already split into multiple Factory classes. It is better to split test cases into suitable classes. - [ ] Move TCL related test cases into `TransactionBackendHandlerFactoryTest` - [ ] Keep one TCL SQL test case in `TextProtocolBackendHandlerFactoryTest` and assert return TransactionBackendHandler - [ ] Move DAL related test cases into `DALBackendHandlerFactoryTest` - [ ] Keep one DAL SQL test case in `TextProtocolBackendHandlerFactoryTest` and assert return UnicastBackendHandler - [ ] Add test cases for RDL (3 branches) in `TextProtocolBackendHandlerFactoryTest` - [ ] Add test case for MySQLShowTablesStatement in `DALBackendHandlerFactoryTest` - [ ] Add test case for CommitStatement in `TransactionBackendHandlerFactoryTest` - [ ] Add test case for RollbackStatement in `TransactionBackendHandlerFactoryTest` - [ ] Add test case for return BroadcastBackendHandler in `TransactionBackendHandlerFactoryTest` ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
