Week10

app/controllers/artists_controller.rb

@@ -6,4 +6,23 @@ def index
   def show
     @artist = Artist.find(params[:id])
   end
+
+  def new
+    @artist = Artist.new
+  end
+
+  def create
+    @artist = Artist.new(artist_params)
+    if @artist.save
+      redirect_to @artist, notice: "Artist Created"
+    else
+      render :new, status: :unprocessable_entity
+    end
+  end
+
+  private
+
+    def artist_params
+      params.require(:artist).permit(:name)
+    end
 end

app/controllers/tracks_controller.rb

@@ -6,4 +6,42 @@ def index
   def show
     @track = Track.find(params[:id])
   end
+
+  def new
+    @track = Track.new
+  end
+
+  def create
+    @track = Track.new(track_params)
+    @track.user = User.find_by(username: "Twhite")
+    if @track.save
+      redirect_to @track, notice: "Track Created!"
+    else
+      render :new, status: :unprocessable_entity
+    end
+  end
+
+  def edit
+    @track = Track.find(params[:id])
+  end
+
+  def update
+    @track = Track.find(params[:id])
+    @track.user = User.find_by(username: "Twhite")
+    if @track.update(track_params)
+      redirect_to @track, notice: "Track Updated"
+    else
+      render :edit, status: :unprocessable_entity
+    end
+  end
+
+  def destroy
+    @track = Track.find(params[:id])
+    @track.destroy
+    redirect_to dashboard_path, notice: "Track Deleted"
+  end
+
+  def track_params
+    params.require(:track).permit(:title, :artist_id, :year, :bpm, :key, :notes)
+  end
 end

app/helpers/tracks_helper.rb

@@ -0,0 +1,2 @@
+module TracksHelper
+end

app/models/track.rb

@@ -1,5 +1,5 @@
 class Track < ApplicationRecord
-  belongs_to :user
+  belongs_to :user, optional: true
   belongs_to :artist
 
   has_many :comments, dependent: :destroy

app/views/artists/_form.html.erb

@@ -0,0 +1,21 @@
+<%= form_with model: @artist do |form| %>
+    <% if @artist.errors.any? %>
+        <div class="error-messages">
+            <ul>
+                <% @artist.errors.full_messgaes.each do |msg| %>
+                    <li><%= msg %></li>
+                <% end %>
+            </ul>
+        </div>
+    <% end %>
+
+    <div>
+    <%= form.label :name %><br>
+    <%= form.text_field :name %>
+    </div>
+
+    <div>
+    <%= form.submit %>
+    </div>
+
+    <% end %>

app/views/artists/new.html.erb

@@ -0,0 +1,2 @@
+<h1> New Artist </h1>
+<%= render "form" %>

app/views/artists/show.html.erb

@@ -1,4 +1,4 @@
-<h1><%= @artist.name %></h1>
+<h1>Artist: <%= @artist.name %></h1>
 
 <h2>Tracks</h2>
 <ul>

app/views/dashboard/index.html.erb

@@ -20,4 +20,8 @@
       "<%= comment.body %>"
     </li>
   <% end %>
-</ul>
+</ul>
+
+<p>
+ <%= link_to "Add New Track", new_track_path %>
+</p>

app/views/tracks/_form.html.erb

@@ -0,0 +1,48 @@
+<%= form_with model: @track do |form| %>
+  <% if @track.errors.any? %>
+    <div class="error-messages">
+      <h3><%= pluralize(@track.errors.count, "error") %> prohibited this track from being saved:</h3>
+      <ul>
+        <% @track.errors.full_messages.each do |msg| %>
+          <li><%= msg %></li>
+        <% end %>
+      </ul>
+    </div>
+  <% end %>
+
+  <div>
+    <%= form.label :title %><br>
+    <%= form.text_field :title %>
+  </div>
+
+  <div>
+    <%= form.label :artist_id, "Artist" %><br>
+    <%= form.collection_select :artist_id, Artist.all, :id, :name, prompt: "Choose an artist" %>
+  </div>
+  <p>
+    <%= link_to "Add New Artist", new_artist_path %>
+  </p>
+  <div>
+    <%= form.label :year %><br>
+    <%= form.number_field :year %>
+  </div>
+
+  <div>
+    <%= form.label :bpm %><br>
+    <%= form.number_field :bpm %>
+  </div>
+
+  <div>
+    <%= form.label :key %><br>
+    <%= form.text_field :key %>
+  </div>
+
+  <div>
+    <%= form.label :notes %><br>
+    <%= form.text_area :notes %>
+  </div>
+
+  <div>
+    <%= form.submit %>
+  </div>
+<% end %>

app/views/tracks/edit.html.erb

@@ -0,0 +1,2 @@
+<h1> Edit Track </h1>
+<%= render "form" %>

app/views/tracks/new.html.erb

@@ -0,0 +1,2 @@
+<h1> New Track </h1>
+<%= render "form" %>

app/views/tracks/show.html.erb

@@ -1,27 +1,45 @@
-<h1><%= @track.title %></h1>
+<h1>Track: <%= @track.title %></h1>
 <p>
   Artist: <%= @track.artist.name %><br>
   Year: <%= @track.year %><br>
   BPM: <%= @track.bpm %>
 </p>
 
 <h2>Samples Used</h2>
-<ul>
-  <% @track.sampled_tracks.each do |source| %>
-    <li><%= link_to source.title, track_path(source) %> - <%= link_to source.artist.name, artist_path(source) %></li>
-  <% end %>
-</ul>
+
+<% if @track.sampled_tracks.any? %>
+  <ul>
+    <% @track.sampled_tracks.each do |source| %>
+      <li><%= link_to source.title, track_path(source) %> - 
+      <%= link_to source.artist.name, artist_path(source) %></li>
+    <% end %>
+  </ul>
+<% else %>
+  <p>None</p>
+<% end %>   
 
 <h2>Sampled By</h2>
-<ul>
-  <% @track.sampled_by_tracks.each do |derived| %>
-    <li><%= link_to derived.title, track_path(derived) %> - <%= link_to derived.artist.name, artist_path(derived) %></li>
-  <% end %>
-</ul>
+
+<% if @track.sampled_by_tracks.any? %>
+  <ul>
+    <% @track.sampled_by_tracks.each do |derived| %>
+      <li><%= link_to derived.title, track_path(derived) %> - 
+      <%= link_to derived.artist.name, artist_path(derived) %></li>
+    <% end %>
+  </ul>
+<% else %>
+  <p>None</p>
+<% end %>    
 
 <h2>Comments</h2>
 <ul>
   <% @track.comments.each do |comment| %>
     <li><%= comment.user.username %>: <%= comment.body %></li>
   <% end %>
-</ul>
+</ul>
+
+<p>
+  <%= link_to "Edit Track", edit_track_path(@track) %> |
+  <%= link_to "Delete Track", track_path(@track),
+              data: { turbo_method: :delete, turbo_confirm: "Are you sure you want to delete this track?" } %>
+</p>

config/routes.rb

@@ -1,7 +1,7 @@
 Rails.application.routes.draw do
   # Define your application routes per the DSL in https://guides.rubyonrails.org/routing.html
-  resources :artists, only: [ :index, :show ]
-  resources :tracks,  only: [ :index, :show ]
+  resources :artists, only: [ :index, :show, :new, :create ]
+  resources :tracks,  only: [ :index, :show, :new, :create, :edit, :update, :destroy ]
 
   get "/dashboard", to: "dashboard#index"
 

spec/helpers/tracks_helper_spec.rb

@@ -0,0 +1,15 @@
+require 'rails_helper'
+
+# Specs in this file have access to a helper object that includes
+# the TracksHelper. For example:
+#
+# describe TracksHelper do
+#   describe "string concat" do
+#     it "concats two strings with spaces" do
+#       expect(helper.concat_strings("this","that")).to eq("this that")
+#     end
+#   end
+# end
+RSpec.describe TracksHelper do
+  pending "add some examples to (or delete) #{__FILE__}"
+end

spec/requests/tracks_spec.rb

@@ -0,0 +1,44 @@
+require 'rails_helper'
+
+RSpec.describe "Tracks" do
+  let!(:user)   { User.create!(email: "twhite@example.com", username: "Twhite") }
+  let!(:artist) { Artist.create!(name: "Test Artist") }
+
+  describe "POST /tracks" do
+   it "creates a new track" do
+     expect {
+       post tracks_path, params: { track: { title: "New Song", year: 2024, artist_id: artist.id } }
+     }.to change(Track, :count).by(1)
+   end
+
+   it "redirects to the new track page" do
+     post tracks_path, params: { track: { title: "New Song", year: 2024, artist_id: artist.id } }
+     expect(response).to redirect_to(track_path(Track.last))
+   end
+
+   it "saves the correct title" do
+     post tracks_path, params: { track: { title: "New Song", year: 2024, artist_id: artist.id } }
+     expect(Track.last.title).to eq("New Song")
+   end
+
+   it "saves the correct artist" do
+     post tracks_path, params: { track: { title: "New Song", year: 2024, artist_id: artist.id } }
+     expect(Track.last.artist).to eq(artist)
+   end
+ end
+
+  describe "PATCH /tracks/:id" do
+     let!(:track) { Track.create!(title: "Old Song", year: 2020, user: user, artist: artist) }
+
+     it "redirects after update" do
+       patch track_path(track), params: { track: { key: "C minor" } }
+       expect(response).to redirect_to(track_path(track))
+     end
+
+     it "updates the key" do
+       patch track_path(track), params: { track: { key: "C minor" } }
+       track.reload
+       expect(track.key).to eq("C minor")
+     end
+   end
+end