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

Duda I/O: adding Duda I/O framework tests. #946

Merged
merged 1 commit into from
Aug 9, 2014
Merged

Duda I/O: adding Duda I/O framework tests. #946

merged 1 commit into from
Aug 9, 2014

Conversation

edsiper
Copy link
Contributor

@edsiper edsiper commented Jul 27, 2014

The following patch adds a web service based on Duda I/O framework,
at the moment only tests #1 and #6 are implemented. For the next round
we will cover Database tests.

For more details about the stack and components please refer to:

http://duda.io
http://monkey-project.com

note: this is the second pull request upon requirement to drop the stack
from the patch. This PR replaces #905.

Signed-off-by: Eduardo Silva [email protected]

The following patch adds a web service based on Duda I/O framework,
at the moment only tests #1 and #6 are implemented. For the next round
we will cover Database tests.

For more details about the stack and components please refer to:

http://duda.io
http://monkey-project.com

note: this is the second pull request upon requirement to drop the stack
from the patch. This PR replaces #905.

Signed-off-by: Eduardo Silva <[email protected]>
@aschneider-techempower
Copy link
Contributor

This is a much more sensible PR. There's a few issues with the PR though:

When I try to run the test, I get the following error:

/bin/sh: 1: ./run_application: not found

I suspect it is because you're missing the install file that #900 requries. The frameworks should require no manual setup beyond running the installer command.

Are you also including the entire framework? The files in duda/dudac seem extraordinarily long for python files. The framework ideally should be setup in the aforementioned install.sh. Here is some more information on writing one.

@hamiltont hamiltont mentioned this pull request Aug 9, 2014
@hamiltont hamiltont merged commit da554f7 into TechEmpower:master Aug 9, 2014
@hamiltont
Copy link
Contributor

closed with #981

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.

3 participants