Remove CasesController (#35)

- Got rid of unnecessary redirect when case not found
- Remove CasesController
- Change superclass of PassportCasesController to ApplicationController
- Add index.html.erb for PassportCasesController

Eliminate the concept of reusable controllers and instead focus on reusable views. Controllers are often tied to application specific middleware logic such as authentication.

Author: lorenyu

template/{{app_name}}/engines/flex/app/controllers/flex/cases_controller.rb

@@ -1,2 +1,32 @@
-class PassportCasesController < Flex::CasesController
+class PassportCasesController < ApplicationController
+  def index
+    @cases = PassportCase.order(created_at: :desc)
+                         .all
+  end
+
+  def closed
+    @cases = PassportCase.where(status: "closed")
+                         .order(created_at: :desc)
+    render :index
+  end
+
+  def new
+  end
+
+  def create
+  end
+
+  def show
+    @case = PassportCase.find(params[:id])
+  end
+
+  def edit
+  end
+
+  def update
+  end
+
+  def model_class
+    controller_path.classify.constantize
+  end
 end

template/{{app_name}}/engines/flex/spec/dummy/app/controllers/passport_cases_controller.rb

@@ -0,0 +1 @@
+<%= render template: 'flex/cases/index', locals: { model_class: PassportCase } %>

template/{{app_name}}/engines/flex/spec/dummy/app/views/passport_cases/index.html.erb

@@ -8,4 +8,4 @@
   <button class="usa-button usa-button--outline">Reject</button>
 <% end %>
 
-<%= render template: 'flex/cases/show' %>
+<%= render template: 'flex/cases/show', locals: { model_class: PassportCase } %>

template/{{app_name}}/engines/flex/spec/dummy/app/views/passport_cases/show.html.erb

@@ -1,6 +1,13 @@
 require 'rails_helper'
 
 RSpec.describe "PassportCases", type: :request do
+  describe "GET /index" do
+    it "returns http success" do
+      get "/passport_cases"
+      expect(response).to have_http_status(:success)
+    end
+  end
+
   describe "GET /show" do
     let!(:passport_case) { PassportCase.create }
 
@@ -11,7 +18,7 @@
 
     it "returns redirects if case not found" do
       get "/passport_cases/00000000"
-      expect(response).to have_http_status(:redirect)
+      expect(response).to have_http_status(:not_found)
     end
   end
 end