cf-tool icon indicating copy to clipboard operation
cf-tool copied to clipboard

Add cf customtest command

Open user202729 opened this issue 4 years ago • 7 comments

user202729 avatar Feb 28 '20 13:02 user202729

Great works! But I have changed a lot in the local repo. Maybe there would be a lot of conflicts.

xalanq avatar Feb 28 '20 15:02 xalanq

Pushed. You can try to solve the conflicts

xalanq avatar Feb 29 '20 11:02 xalanq

I think language id is not user-friendly. Any thought?

xalanq avatar Feb 29 '20 13:02 xalanq

Maybe you can ask what user wants, just like template configuration.

xalanq avatar Feb 29 '20 13:02 xalanq

Comment:

  • Docopt doesn't show any error messages when the format of the usage string is wrong. It just silently exits.
  • When the user provide an invalid language ID, the program will hang forever. It's possible to check whether the ID is valid before submitting, but it will require the language list to be up-to-date (otherwise custom test of newly added language is not possible).

user202729 avatar Feb 29 '20 15:02 user202729

I think the custom test should be independent of templates. I mean you can pop out an available language list if the template detection failed instead of showing the error message.

xalanq avatar Mar 01 '20 10:03 xalanq

BTW the command customtest could be ctest XD

xalanq avatar Mar 01 '20 10:03 xalanq