brigadier icon indicating copy to clipboard operation
brigadier copied to clipboard

Add unit tests for StringReader.isAllowedInUnquotedString

Open diffblue-assistant opened this issue 7 years ago • 6 comments

The test cases contained in this PR were written by Diffblue's Diffblue Cover software. These tests are intended to help you detect regressions caused by future code changes.

One of our missions is to provide useful tests to open source projects. In return, we would appreciate your feedback on our tests to help us further improve our tool.

Results for various open source projects can be viewed at:

http://diff.blue/demo

diffblue-assistant avatar Oct 24 '18 08:10 diffblue-assistant

CLA assistant check
All CLA requirements met.

msftclas avatar Oct 24 '18 08:10 msftclas

In my opinion the checks are too verbose, they could easily be written in one line, for example:

assertTrue(StringReader.isAllowedInUnquotedString('2'));

Additionally I doubt that the copyright header "Copyright (c) Diffblue Limited. All rights reserved." would work well for this project.

Marcono1234 avatar Oct 26 '18 12:10 Marcono1234

@Marcono1234 Many thanks for your feedback, we've merged the tests into your existing source file.

diffblue-assistant avatar Nov 06 '18 14:11 diffblue-assistant

@Marcono1234 We've also now modified the tests to take into account your feedback regarding the verbosity.

diffblue-assistant avatar Nov 06 '18 15:11 diffblue-assistant

@diffblue-assistant

into your existing source file

My comment only described my personal opinion, I am neither a member nor contributor of this project. Though I think your proposed changes are better now.

Marcono1234 avatar Nov 09 '18 13:11 Marcono1234

@Dinnerbone Can I request a review please?

diffblue-assistant avatar Nov 09 '18 14:11 diffblue-assistant