-
Notifications
You must be signed in to change notification settings - Fork 0
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
end time error #53
Comments
This needs to be separated into at least 2 separate tickets. 1 for the bug here with the end time, and 1 for unit tests in general. We need unit tests for the entire process up to the point that the job is submitted to ANUGA. These tests need to validate that security is working properly, and that values passed through from the UI wind up in the json properly. Im not sure how best to test that what is put into the UI gets passed through to the api properly. Andreas, any ideas there? |
Regarding the initial issue, I cannot reproduce it. The outgoing json always has the correct value for the end_time. Regarding the unit tests: there are several ways to write automated UI tests, but all are very time consuming. My estimate for an initial test setup with TestAnotherWay (the test framework used for OpenLayers, GeoExt and gxp) would be 12 hours. A test case for the simulation parameters would be another 4 hours. For the other forms even more, because of their higher complexity. |
I will give it a test, but for now, Im mostly just reading through all Jeff On Tue, Jun 28, 2011 at 12:50 PM, ahocevar
|
The thing is the users are still using it as we speak. So the big problem is the STOP status, but saving is also high priority but at least they can run it. N On 29/06/2011, at 5:54 AM, jj0hns0n wrote:
|
Hi Andreas,
Another new bug in the time stuff! The end time which is default 3600s, when changed returns a value of 2 to the back end. This happens all the time.
Can we put some unit tests in for passing the correct values from UI to the backend json file that goes to anuga.
Cheers
Nick
The text was updated successfully, but these errors were encountered: