Closed Bug 904358 Opened 13 years ago Closed 11 years ago

[cleanup][fireplace] Re-organize hearth/templates/ and smokealarm/tests/ directories

Categories

(Marketplace Graveyard :: Code Quality, defect, P4)

Avenir
x86_64
Windows 7
defect

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: basta, Unassigned)

Details

(Whiteboard: [good first bug][lang=html])

The templates directory is a real crapshoot right now. Most of the stuff in there was built well before we'd even developed the builder. Stuff that needs doing: - Change files to have names that match their views (search/main.html -> search.html) - Move utility templates to a directory (deprecated.html, obsolete.html) - Group similar templates in directories (cat dropdown templates to their own directory) - The "user" directory doesn't make sense; figure out how to combine it with "settings" - "detail" -> "app"? The smokealarm tests also need to be re-organized: - Classify into user flow/UI directories (tests/flow/, tests/ui/). E.g.: cat dropdown tests to tests/ui/cat_dropdown/ - Organize flow tests by view (tests/flow/search/, tests/flow/detail_page/, etc.) - Organize UI tests by component (tests/ui/cat_dropdown/, tests/ui/tabs/, etc.) - Rename flow test files and directories to match view names where possible - Make test names less redundant (tests/app/app_from_abuse.js -> tests/app/from_abuse.js) https://github.com/mozilla/fireplace
The app structure has changed significantly. This doesn't seem relevant anymore.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.