Use slugs in URLs

app/controllers/projects_controller.rb

@@ -2,7 +2,4 @@ class ProjectsController < ApplicationController
   def index
     @projects = Project.all
   end
-  def show
-    @project = Project.find(params[:id])
-  end
 end

app/controllers/runs_controller.rb

@@ -2,8 +2,9 @@ class RunsController < ApplicationController
   skip_before_action :verify_authenticity_token
 
   def show
-    @suite = Suite.find(params[:suite_id])
-    @run = @suite.runs.find_by_sequential_id(params[:id])
+    @project = Project.find_by_slug(params[:project_slug])
+    @suite = @project.suites.find_by_slug(params[:suite_slug])
+    @run = @suite.runs.find_by_sequential_id(params[:sequential_id])
     @tests = TestFilters.new(@run.tests, params)
   end
 
@@ -14,12 +15,12 @@ def new
   def create
     @project = Project.find_by_name(params[:project])
     if @project.nil?
-      @project = Project.create(name: params[:project])
+      @project = Project.create(name: params[:project], slug: params[:project].parameterize)
     end
 
     @suite = @project.suites.find_by_name(params[:suite])
     if @suite.nil?
-      @suite = @project.suites.create(name: params[:suite])
+      @suite = @project.suites.create(name: params[:suite], slug: params[:suite].parameterize)
     end
 
     @run = @suite.runs.create

app/controllers/suites_controller.rb

@@ -1,6 +1,7 @@
 class SuitesController < ApplicationController
   def show
-    @suite = Suite.find(params[:id])
+    @project = Project.find_by_slug(params[:project_slug])
+    @suite = @project.suites.find_by_slug(params[:slug])
     @baselines = TestFilters.new(@suite.baselines, params)
   end
 end

app/models/project.rb

@@ -1,3 +1,7 @@
 class Project < ActiveRecord::Base
   has_many :suites
+
+  def to_param
+    slug
+  end
 end

app/models/run.rb

@@ -7,9 +7,9 @@ def self.reversed
     order('created_at DESC')
   end
 
-  # def to_param
-  #   self.sequential_id.to_s
-  # end
+  def to_param
+    sequential_id.to_s
+  end
 
   def passing_tests
     self.tests.where(pass: true).count

app/models/suite.rb

@@ -10,4 +10,8 @@ def latest_run
   def baselines
     tests.where(baseline: true)
   end
+
+  def to_param
+    slug
+  end
 end

config/routes.rb

@@ -3,9 +3,9 @@
 
   get '/' => redirect('/projects')
 
-  resources :projects, only: [:index] do
-    resources :suites, only: [:show] do
-      resources :runs, only: [:show]
+  resources :projects, param: :slug, only: [:index] do
+    resources :suites, param: :slug, only: [:show] do
+      resources :runs, param: :sequential_id, only: [:show]
     end
   end
 

db/migrate/20160106085402_add_slugs.rb

@@ -0,0 +1,6 @@
+class AddSlugs < ActiveRecord::Migration
+  def change
+    add_column :projects, :slug, :string
+    add_column :suites, :slug, :string
+  end
+end

db/schema.rb

@@ -11,7 +11,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20160105085402) do
+ActiveRecord::Schema.define(version: 20160106085402) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -20,6 +20,7 @@
     t.string   "name"
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
+    t.string   "slug"
   end
 
   create_table "runs", force: :cascade do |t|
@@ -36,6 +37,7 @@
     t.integer  "project_id"
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
+    t.string   "slug"
   end
 
   create_table "tests", force: :cascade do |t|