-
Notifications
You must be signed in to change notification settings - Fork 26
Testing help needed? #13
Comments
thanks. i want to get some browser tests and js tests running before we On Tue, May 19, 2015 at 4:05 PM, Orlando [email protected] wrote:
|
Would be happy to help. Word of caution: Often to get js tests going, code may need to be refactored. So keep that in mind when estimating. |
@higs4281 where's the front end for this project? is it possible to get a localhost instance going on my mac? |
it's a self-contained one-page app; page is in retirement_api/templates. |
ok, one thing that would be extremely helpful is to use unique ID's for elements that the user will interact with,, similar to what you did with the retirement age selector in step 3: {% trans "Choose age" %} 62 63 64 65 66 67 68 69 70 |
@Ooblioob @higs4281 @mistergone We should really not deploy without testing. The bare essentials for front end (unit) testing are:
|
The front-end's logic has grown complex. A small amount of refactoring would allow unit tests to be written for it. Shoot me an email if you'd like some help with this. |
I'm all ears. I'm also booked solid for a week or so. That said, I'm pretty good with writing unit-testable code, but I need some guidance on what our current tools are and how we're working them. I ain't afraid of no tests. |
@mistergone Great! Schedule a time most convenient to you. In the mean time, @contolini and I can do a code review and make refactoring suggestions. To start, can we change the file structure, so that javascript code is not buried inside api directory and not add more code into one giant file? Let me know what you think |
I see lots of good stuff going on in here,, feel free to ask me or the software delivery team if you need help setting up browser, API, performance tests, we're more than glad to help you get started :)
We'll start setting up BDD security testing for this project soon.
The text was updated successfully, but these errors were encountered: