#2126 add SQLLoggerTest

[betterjava]

Fixes #2126 .

Changes proposed in this pull request:
-add SQLLoggerTest to test SQLLogger

[coveralls]

Pull Request Test Coverage Report for Build 7421

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 116 unchanged lines in 12 files lost coverage.
• Overall coverage increased (+0.1%) to 54.374%

---

Files with Coverage Reduction	New Missed Lines	%
sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/antlr/sql/segment/InsertSegment.java	1	0.0%
sharding-core/sharding-core-merge/src/main/java/org/apache/shardingsphere/core/merge/dql/DQLMergeEngine.java	1	88.37%
sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/antlr/sql/segment/column/ColumnSegment.java	4	0.0%
sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/antlr/extractor/impl/TableNameExtractor.java	6	0.0%
sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/antlr/extractor/impl/dml/InsertExtractor.java	6	0.0%
sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/parser/token/TableToken.java	7	0.0%
sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/antlr/filler/encrypt/segment/impl/EncryptInsertFiller.java	7	0.0%
sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/antlr/sql/segment/table/TableSegment.java	8	0.0%
sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/antlr/filler/common/segment/impl/dql/OrderItemBuilder.java	9	0.0%
sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/antlr/filler/sharding/statement/impl/dml/InsertFiller.java	15	0.0%

Totals
Change from base Build 7399:	0.1%
Covered Lines:	8484
Relevant Lines:	15603

---

💛 - Coveralls

[betterjava]

@terrymanu can you help me to review my pull ?

[terrymanu]

The idea of mock test is good. Cloud you use checkstyle to check your code and fix waring of your code?

[terrymanu]

FYI: https://shardingsphere.apache.org/community/en/contribute/code-conduct/

[betterjava]

OK,I have used checkstyle,but seems not work fo Test class.I will check.

[betterjava]

I am sorry to set the wrong property for checkstyle. I have checked the code with the correct checkstyle config.

[terrymanu]

That's prefect, I will merge them