Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improved existing tests and implemented new tests #123

Open
wants to merge 24 commits into
base: dev
Choose a base branch
from

Conversation

nanoandrew4
Copy link
Contributor

@nanoandrew4 nanoandrew4 commented Aug 10, 2019

This PR depends on the ezcli backport (#124), so that PR should be merged before this one.

This PR adds more tests to test code from the backport, as well as existing code, to raise code coverage. The initial target was >50% but I ended up leaving it at 39% for now since the rest of the code is very hard or impossible to test. Test libraries are also replace, JUnit Jupiter was used before, which has been replaced with JUnit, and Mockito was added since it is used in new tests.

All tests were verified as passing in my local environment.

Sergix and others added 24 commits November 24, 2017 14:03
View changelog entry Sergix#116 and the associated documentation for details.
TODO: Fix broken links
Removed unnecessary throws in initWindows function Sergix#120
…ds to the underlying system (currently only Unix/bash commands are supported), and reading/writing a history file similar to bash_history. The ported code also includes more documentation, which will be expanded on
…ed custom command running to enable custom commands previously implemented by JTerm
…y did previously, though more in depth tests are pending
# Conflicts:
#	src/main/java/jterm/JTerm.java
#	src/main/java/jterm/command/Dir.java
#	src/main/java/jterm/command/Exec.java
#	src/main/java/jterm/command/Files.java
#	src/main/java/jterm/command/Ping.java
#	src/main/java/jterm/io/input/InputHandler.java
#	src/main/java/jterm/io/input/Keys.java
#	src/main/java/jterm/io/output/CollectorPrinter.java
#	src/main/java/jterm/io/output/GuiPrinter.java
#	src/main/java/jterm/io/output/HeadlessPrinter.java
#	src/main/java/jterm/io/output/Printer.java
#	src/test/java/jterm/io/InputHandlerTest.java
#	src/test/java/jterm/util/UtilUnitTest.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants