You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The bug fixed in #1631 showed that our current CLI tests are merely smoke tests and not sufficient to detect if arguments are passed properly and have the intended effect. We should add thorough tests for all command line arguments.
The text was updated successfully, but these errors were encountered:
Would it be possible to run a select few unit tests from org.lflang.tests using the existing CliToolTestFixture? Or do you think we should just write new tests specifically for the CLI?
I think so, yes. When writing the issue, I forgot that @oowekyala already made a start at this. I think it only makes sense to build on this and add tests for all the command line arguments. Most important is, that the tests indeed verify the outcome of the argument. Are interested in taking a shot at this @patilatharva?
The bug fixed in #1631 showed that our current CLI tests are merely smoke tests and not sufficient to detect if arguments are passed properly and have the intended effect. We should add thorough tests for all command line arguments.
The text was updated successfully, but these errors were encountered: