Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions .idea/.generators

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

6 changes: 6 additions & 0 deletions .idea/.gitignore

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

6 changes: 6 additions & 0 deletions .idea/inspectionProfiles/Project_Default.xml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 4 additions & 0 deletions .idea/misc.xml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

8 changes: 8 additions & 0 deletions .idea/modules.xml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

6 changes: 6 additions & 0 deletions .idea/vcs.xml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

121 changes: 121 additions & 0 deletions .idea/video-store-consumer-api.iml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

21 changes: 21 additions & 0 deletions app/controllers/videos_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,23 @@ def show
)
end

def create
@video = Video.new(video_params)

if @video.save
render(
json: @video.as_json(only: [:title, :overview, :release_date, :inventory, :image_url, :external_id]),
status: :ok)
return
else
render json: {
errors: @video.errors.messages,
ok: false,
}, status: :bad_request
return
end
end

private

def require_video
Expand All @@ -29,4 +46,8 @@ def require_video
render status: :not_found, json: { errors: { title: ["No video with title #{params["title"]}"] } }
end
end

def video_params
return params.permit(:title, :overview, :release_date, :inventory, :image_url, :external_id)
end
end
1 change: 1 addition & 0 deletions app/models/video.rb
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
class Video < ApplicationRecord
has_many :rentals
has_many :customers, through: :rentals
validates :title, presence: true, uniqueness: true

def available_inventory
self.inventory - self.rentals.where(returned: false).length
Expand Down
2 changes: 1 addition & 1 deletion config/routes.rb
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@

resources :customers, only: [:index]

resources :videos, only: [:index, :show], param: :title
resources :videos, only: [:index, :show, :create], param: :title

post "/rentals/:title/check-out", to: "rentals#check_out", as: "check_out"
post "/rentals/:title/return", to: "rentals#check_in", as: "check_in"
Expand Down
21 changes: 21 additions & 0 deletions test/controllers/videos_controller_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -77,4 +77,25 @@ class VideosControllerTest < ActionDispatch::IntegrationTest
expect(data["errors"]).must_include "title"
end
end

describe "create" do
let(:video_hash) {
{
title: "Alf the movie",
overview: "The most early 90s movie of all time",
release_date: "December 16th 2025",
inventory: 10,
img_url: "http://google.com/images'",
external_id: 123,
}
}

it "creates a video" do
expect {
post videos_path, params: video_hash
}.must_change 'Video.count', 1

must_respond_with :success
end
end
end