630676 - Split up search.tests into several modules

Status: VERIFIED FIXED

(Input :: General, defect)

Description

[Fred Wenzel [:wenzel]]

Kind of a meta-bug: Look for long files that could benefit from being split up into smaller, reusable (?) pieces.

Comment 1

[Fred Wenzel [:wenzel]]

Our longest files:

     503 apps/search/tests.py
     398 apps/feedback/tests/test_views.py
     358 apps/input/tests.py
     346 settings.py
     340 apps/search/client.py
     295 apps/search/views.py
     279 apps/input/__init__.py
     274 apps/feedback/views.py
     223 docs/conf.py
     214 apps/website_issues/mapreduce/tasks.py
     194 apps/feedback/forms.py
     172 apps/feedback/models.py
     169 apps/dashboard/helpers.py
     161 apps/dashboard/tests.py
     155 apps/website_issues/views.py
     146 apps/input/helpers.py
     145 apps/website_issues/mapreduce/tests.py
     145 apps/input/urlresolvers.py
     138 apps/website_issues/forms.py
     137 apps/themes/views.py
     133 apps/website_issues/models.py
     122 apps/search/forms.py
     118 apps/themes/cron.py
     106 apps/api/cron.py
     100 apps/feedback/utils.py

Comment 2

[Fred Wenzel [:wenzel]]

Hmm, looks like the search tests could benefit from being split up, and feedback views tests.

I'll move input.tests into separate modules then assign to Dave Dash for some search.tests love.

Comment 3

[Fred Wenzel [:wenzel]]

Done splitting up input.tests:
http://github.com/fwenzel/reporter/commit/4bca85a

Split up feedback.tests.test_views into test_beta_views and test_release_views:
http://github.com/fwenzel/reporter/commit/c07d7fd


Over to Dave.

Comment 4

[Dave Dash [:davedash, :dd] (assign all bugs to mbrandt)]

http://github.com/fwenzel/reporter/commit/9313752

Comment 5

[Matt Brandt [:mbrandt]]

Closed as [qa-]