kcl-lang / kcl-go

KCL Go SDK
https://pkg.go.dev/kcl-lang.io/kcl-go@main
Apache License 2.0
49 stars 25 forks source link

Improve test for no logmessage test. #362

Closed liangyuanpeng closed 1 month ago

liangyuanpeng commented 1 month ago

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

Add a no log message test.

The PR https://github.com/kcl-lang/kcl-go/pull/361 delete the no logMessage test and this PR added it back.

2. What is the scope of this PR (e.g. component or file name):

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

Peefy commented 1 month ago

LGTM! Thank you!

coveralls commented 1 month ago

Pull Request Test Coverage Report for Build 10162793048

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details


Totals Coverage Status
Change from base Build 10162415420: 0.006%
Covered Lines: 5604
Relevant Lines: 17342

đŸ’› - Coveralls