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

Write comprehensive tests for recurring todos #1006

Closed
dnrce opened this issue Jun 25, 2014 · 4 comments
Closed

Write comprehensive tests for recurring todos #1006

dnrce opened this issue Jun 25, 2014 · 4 comments
Labels
Enhancement ⚡️ New features, big or small. testing
Milestone

Comments

@dnrce
Copy link
Member

dnrce commented Jun 25, 2014

Migrated from the original issue at https://www.assembla.com/spaces/tracks-tickets/tickets/1006

Recurring todos have a huge amount of frontend and model code. This needs a comprehensive test suite.

I'm working on cleaning up the model and views in a branch at http://github.com/epall/tracks/tree/recurring-cleanup, but I can't really ensure things are working without comprehensive tests. I already broke some functionality in the jQuery re-write, so this has all ready been a problem for us.

Originally reported by epall on March 3, 2010 at 16:42:14 (-0500) against version git-devel

@dnrce dnrce added this to the 2.4 milestone Jun 25, 2014
@dnrce
Copy link
Member Author

dnrce commented Jun 25, 2014

On March 3, 2010 at 16:48:14 (-0500), epall commented:

Related association with ticket #967 was added

@dnrce
Copy link
Member Author

dnrce commented Jun 25, 2014

On March 3, 2010 at 04:10:46 (-0500), lrbalt commented:

I made lots of tests for the model. The front-end needs tests. I needed to get the cucumber working first to tackle the front-end.

@dnrce
Copy link
Member Author

dnrce commented Jul 3, 2014

@lrbalt What do you think about moving this to 2.3? The recurring todos haven't been exercised much since the refactor. I would feel better if we had a good test suite prior to release.

@lrbalt
Copy link
Member

lrbalt commented Jul 3, 2014

I think we can close this one. With the last refactor, I have added a lot of tests. This should cover most use cases I think

@lrbalt lrbalt closed this as completed Jul 3, 2014
@dnrce dnrce modified the milestones: 2.3, 2.4 Jul 8, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement ⚡️ New features, big or small. testing
Projects
None yet
Development

No branches or pull requests

2 participants