Move some stuff around, add Admin event show view (#54)

* Move some stuff around, add Admin event show view
* Add test for show view render

app/views/admin/events/index.html.erb

@@ -5,11 +5,16 @@
     <% @events.each do |event| %>
       <li class="flex gap-2 justify-between mb-2 w-72">
         <span class="flex-grow">
-          <%= event.name %>
+          <%= link_to event.name, admin_event_path(event) %>
         </span>
         <%= link_to "Edit", edit_admin_event_path(event), class: "text-underline" %>
         <%= button_to "Delete", admin_event_path(event), method: :delete, class: "rounded bg-red-600 px-2 py-1 text-xs font-semibold text-white shadow-sm hover:bg-red-500 focus-visible:outline focus-visible:outline-2 focus-visible:outline-offset-2 focus-visible:outline-red-600", data: {turbo_confirm: "Are you sure?"} %>
       </li>
     <% end %>
   </ul>
 </div>
+
+<% content_for :page_title do %>
+Admin Home
+<% end %>
+

app/views/admin/events/show.html.erb

@@ -0,0 +1,6 @@
+<%= content_for :event_content do %>
+  <%= link_to "Edit this event", edit_admin_event_path(@event) %>
+  <%= render partial: "events/event", locals: { event: @event } %>
+<% end %>
+
+<%= render partial: "layouts/event_layout" %>

app/views/events/_event.html.erb

@@ -45,7 +45,7 @@
     <h3>Sponsor</h3>
     <div class="flex justify-center items-center">
       <%= link_to event.sponsor_link do %>
-        <% if event.sponsor_logo %>
+        <% if event.sponsor_logo.present? %>
           <%= image_tag "sponsors/#{event.sponsor_logo}", class: 'max-h-20' %>
         <% else %>
           <%= event.sponsor.titleize %>

app/views/events/index.html.erb

@@ -15,4 +15,4 @@ Home
 Toronto Ruby is about building a community for Ruby users in the GTA and providing a positive space for everyone to connect, learn and grow. We're looking forward to seeing you!
 <% end %>
 
-<%= render partial: "events/layout" %>
+<%= render partial: "layouts/event_layout" %>

app/views/events/show.html.erb

@@ -1,5 +1,10 @@
+<% content_for :page_title do %>
+ <%= @event.name %>
+<% end %>
+
+
 <%= content_for :event_content do %>
-  <%= render partial: "event", locals: { event: @event } %>
+  <%= render partial: "events/event", locals: { event: @event } %>
 <% end %>
 
-<%= render partial: "events/layout" %>
+<%= render partial: "layouts/event_layout" %>

app/views/events/_layout.html.erb → app/views/layouts/_event_layout.html.erb

@@ -11,8 +11,8 @@
       <%= yield(:event_content) %>
     </div>
     <div class="mx-auto w-4/5 md:w-2/3">
-      <%= render 'newsletter_form' %>
+      <%= render 'layouts/newsletter_form' %>
     </div>
-    <%= render 'supporting_companies' %>
+    <%= render 'layouts/supporting_companies' %>
   </div>
 </main>

test/controllers/admin/events_controller_test.rb

@@ -31,7 +31,7 @@ def setup
     assert_response :success
   end
 
-  test 'should show draft events' do
+  test 'index should show draft events' do
     Event.update_all(status: :draft)
 
     get admin_events_path,
@@ -47,6 +47,19 @@ def setup
     assert response.body.include?(Event.last.name)
   end
 
+  test 'should show a single event' do
+    get admin_event_path(Event.first.slug),
+        headers: {
+          Authorization:
+            ActionController::HttpAuthentication::Basic.encode_credentials(
+              'admin', 'admin'
+            )
+        }
+
+    assert_response :ok
+    assert response.body.include?(Event.first.name)
+  end
+
   test 'should redirect to events page if event is not found' do
     get admin_event_path('nonexistent-event'),
         headers: {