Skip to content

Routing specs and cleanup

Matthias Käppler requested to merge github/fork/rspeicher/route_specs into master

Created by: rspeicher

First, this adds routing specs. Why? Because I was going to modify the routes and wanted to make sure the current ones still worked after my changes.

Then I made a few changes:

  • Adds an AdminController base controller for the other Admin controllers to subclass. It currently handles setting the layout and authorizing an admin and just removes some duplication from those controllers.
  • Style changes in routes.rb, mostly 1.9 Hash syntax and alignment and spacing.
  • Removes a few actions that are no longer used from various controllers.
  • Removes unused actions from various routes ("logs" had all the default actions but only used "show", for instance.)
  • Removes the 'team' action from the Projects controller and instead uses the 'index' action in TeamMembers.

Merge request reports