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

Add Java verifier #3

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Add Java verifier #3

wants to merge 11 commits into from

Conversation

dinoboff
Copy link
Contributor

Compile solution and test in memory and run the test case:

  • the solution should implement a class name "SingPath"
  • the solution needs to implement the body of a TestCase method and its local includes: "static org.junit.Assert.", "junit.framework.", "import com.singpath.SolutionRunner";

Note that the verifier only support JSON POST requests.

@dinoboff dinoboff changed the title Add Java (Beanshell) verifier Add Java verifier Oct 26, 2015
@dinoboff
Copy link
Contributor Author

@SingaporeClouds

Unlike the Beanshell solution I cannot show a table of all tests calls; I can only show the failed calls.

I also do not know how to capture the standard streams when using org.junit.runner.JUnitCore.runClasses. I am assuming multiple threads (like multiple verification request thread) are using the same System.out and I cannot set my own printer to capture it.

If it's an issue, I could compile the solution memory using the jdk and run the tests with Beanshell. The tests would have to be tweaked to work in Beanshell compare to normal Java code. For example, the test code would have to not use varargs: Arrays.asList("a","b","c") should be written in the tests as Arrays.asList([]String{"a","b","c"}).

- removed Left over debug echo
- fixed issue with the test to check the image builder is ready. (I should use instance metadata instead or stating a server to give the status).
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.

1 participant