diff --git a/.gitignore b/.gitignore new file mode 100644 index 000000000..31d97109a --- /dev/null +++ b/.gitignore @@ -0,0 +1,26 @@ +# See https://help.github.com/articles/ignoring-files for more about ignoring files. +# +# If you find yourself ignoring temporary files generated by your text editor +# or operating system, you probably want to add a global ignore instead: +# git config --global core.excludesfile '~/.gitignore_global' + +# Ignore bundler config. +/.bundle + +# Ignore all logfiles and tempfiles. +/log/* +/tmp/* +!/log/.keep +!/tmp/.keep + +# Ignore uploaded files in development +/storage/* +!/storage/.keep + +.byebug_history + +# Ignore master key for decrypting credentials and more. +/config/master.key + +#ignore coverage +/coverage/* diff --git a/.ruby-version b/.ruby-version new file mode 100644 index 000000000..160fe391c --- /dev/null +++ b/.ruby-version @@ -0,0 +1 @@ +2.5.5 \ No newline at end of file diff --git a/Gemfile b/Gemfile new file mode 100644 index 000000000..867f6676a --- /dev/null +++ b/Gemfile @@ -0,0 +1,62 @@ +source 'https://rubygems.org' +git_source(:github) { |repo| "https://github.com/#{repo}.git" } + +ruby '2.5.5' + +# Bundle edge Rails instead: gem 'rails', github: 'rails/rails' +gem 'rails', '~> 5.2.3' +# Use postgresql as the database for Active Record +gem 'pg', '>= 0.18', '< 2.0' +# Use Puma as the app server +gem 'puma', '~> 3.11' +# Build JSON APIs with ease. Read more: https://github.com/rails/jbuilder +# gem 'jbuilder', '~> 2.5' +# Use Redis adapter to run Action Cable in production +# gem 'redis', '~> 4.0' +# Use ActiveModel has_secure_password +# gem 'bcrypt', '~> 3.1.7' + +# Use ActiveStorage variant +# gem 'mini_magick', '~> 4.8' + +# Use Capistrano for deployment +# gem 'capistrano-rails', group: :development + +# Reduces boot times through caching; required in config/boot.rb +gem 'bootsnap', '>= 1.1.0', require: false + +# Use Rack CORS for handling Cross-Origin Resource Sharing (CORS), making cross-origin AJAX possible +# gem 'rack-cors' + +group :development, :test do + # Call 'byebug' anywhere in the code to stop execution and get a debugger console + gem 'byebug', platforms: [:mri, :mingw, :x64_mingw] +end + +group :development do + gem 'listen', '>= 3.0.5', '< 3.2' + # Spring speeds up development by keeping your application running in the background. Read more: https://github.com/rails/spring + gem 'spring' + gem 'spring-watcher-listen', '~> 2.0.0' +end + + +# Windows does not include zoneinfo files, so bundle the tzinfo-data gem +gem 'tzinfo-data', platforms: [:mingw, :mswin, :x64_mingw, :jruby] + +group :development, :test do + gem 'pry-rails' +end + +group :development do + gem 'debase', '>= 0.2.4.1' + gem 'ruby-debug-ide', '>= 0.7.0' +end + +group :test do + gem 'minitest-rails' + gem 'minitest-reporters' + gem 'simplecov', require: false +end + + diff --git a/Gemfile.lock b/Gemfile.lock new file mode 100644 index 000000000..cd0d0664d --- /dev/null +++ b/Gemfile.lock @@ -0,0 +1,186 @@ +GEM + remote: https://rubygems.org/ + specs: + actioncable (5.2.3) + actionpack (= 5.2.3) + nio4r (~> 2.0) + websocket-driver (>= 0.6.1) + actionmailer (5.2.3) + actionpack (= 5.2.3) + actionview (= 5.2.3) + activejob (= 5.2.3) + mail (~> 2.5, >= 2.5.4) + rails-dom-testing (~> 2.0) + actionpack (5.2.3) + actionview (= 5.2.3) + activesupport (= 5.2.3) + rack (~> 2.0) + rack-test (>= 0.6.3) + rails-dom-testing (~> 2.0) + rails-html-sanitizer (~> 1.0, >= 1.0.2) + actionview (5.2.3) + activesupport (= 5.2.3) + builder (~> 3.1) + erubi (~> 1.4) + rails-dom-testing (~> 2.0) + rails-html-sanitizer (~> 1.0, >= 1.0.3) + activejob (5.2.3) + activesupport (= 5.2.3) + globalid (>= 0.3.6) + activemodel (5.2.3) + activesupport (= 5.2.3) + activerecord (5.2.3) + activemodel (= 5.2.3) + activesupport (= 5.2.3) + arel (>= 9.0) + activestorage (5.2.3) + actionpack (= 5.2.3) + activerecord (= 5.2.3) + marcel (~> 0.3.1) + activesupport (5.2.3) + concurrent-ruby (~> 1.0, >= 1.0.2) + i18n (>= 0.7, < 2) + minitest (~> 5.1) + tzinfo (~> 1.1) + ansi (1.5.0) + arel (9.0.0) + bootsnap (1.4.5) + msgpack (~> 1.0) + builder (3.2.3) + byebug (11.0.1) + coderay (1.1.2) + concurrent-ruby (1.1.5) + crass (1.0.5) + debase (0.2.4.1) + debase-ruby_core_source (>= 0.10.2) + debase-ruby_core_source (0.10.6) + docile (1.3.2) + erubi (1.9.0) + ffi (1.11.1) + globalid (0.4.2) + activesupport (>= 4.2.0) + i18n (1.7.0) + concurrent-ruby (~> 1.0) + json (2.2.0) + listen (3.1.5) + rb-fsevent (~> 0.9, >= 0.9.4) + rb-inotify (~> 0.9, >= 0.9.7) + ruby_dep (~> 1.2) + loofah (2.3.1) + crass (~> 1.0.2) + nokogiri (>= 1.5.9) + mail (2.7.1) + mini_mime (>= 0.1.1) + marcel (0.3.3) + mimemagic (~> 0.3.2) + method_source (0.9.2) + mimemagic (0.3.3) + mini_mime (1.0.2) + mini_portile2 (2.4.0) + minitest (5.13.0) + minitest-rails (5.2.0) + minitest (~> 5.10) + railties (~> 5.2.0) + minitest-reporters (1.4.2) + ansi + builder + minitest (>= 5.0) + ruby-progressbar + minitest-skip (0.0.1) + minitest (~> 5.0) + msgpack (1.3.1) + nio4r (2.5.2) + nokogiri (1.10.5) + mini_portile2 (~> 2.4.0) + pg (1.1.4) + pry (0.12.2) + coderay (~> 1.1.0) + method_source (~> 0.9.0) + pry-rails (0.3.9) + pry (>= 0.10.4) + puma (3.12.1) + rack (2.0.7) + rack-test (1.1.0) + rack (>= 1.0, < 3) + rails (5.2.3) + actioncable (= 5.2.3) + actionmailer (= 5.2.3) + actionpack (= 5.2.3) + actionview (= 5.2.3) + activejob (= 5.2.3) + activemodel (= 5.2.3) + activerecord (= 5.2.3) + activestorage (= 5.2.3) + activesupport (= 5.2.3) + bundler (>= 1.3.0) + railties (= 5.2.3) + sprockets-rails (>= 2.0.0) + rails-dom-testing (2.0.3) + activesupport (>= 4.2.0) + nokogiri (>= 1.6) + rails-html-sanitizer (1.3.0) + loofah (~> 2.3) + railties (5.2.3) + actionpack (= 5.2.3) + activesupport (= 5.2.3) + method_source + rake (>= 0.8.7) + thor (>= 0.19.0, < 2.0) + rake (13.0.0) + rb-fsevent (0.10.3) + rb-inotify (0.10.0) + ffi (~> 1.0) + ruby-debug-ide (0.7.0) + rake (>= 0.8.1) + ruby-progressbar (1.10.1) + ruby_dep (1.5.0) + simplecov (0.17.1) + docile (~> 1.1) + json (>= 1.8, < 3) + simplecov-html (~> 0.10.0) + simplecov-html (0.10.2) + spring (2.1.0) + spring-watcher-listen (2.0.1) + listen (>= 2.7, < 4.0) + spring (>= 1.2, < 3.0) + sprockets (4.0.0) + concurrent-ruby (~> 1.0) + rack (> 1, < 3) + sprockets-rails (3.2.1) + actionpack (>= 4.0) + activesupport (>= 4.0) + sprockets (>= 3.0.0) + thor (0.20.3) + thread_safe (0.3.6) + tzinfo (1.2.5) + thread_safe (~> 0.1) + websocket-driver (0.7.1) + websocket-extensions (>= 0.1.0) + websocket-extensions (0.1.4) + +PLATFORMS + ruby + +DEPENDENCIES + bootsnap (>= 1.1.0) + byebug + debase (>= 0.2.4.1) + listen (>= 3.0.5, < 3.2) + minitest-rails + minitest-reporters + minitest-skip + pg (>= 0.18, < 2.0) + pry-rails + puma (~> 3.11) + rails (~> 5.2.3) + ruby-debug-ide (>= 0.7.0) + simplecov + spring + spring-watcher-listen (~> 2.0.0) + tzinfo-data + +RUBY VERSION + ruby 2.5.5p157 + +BUNDLED WITH + 2.0.2 diff --git a/Guardfile b/Guardfile new file mode 100644 index 000000000..fc88e3077 --- /dev/null +++ b/Guardfile @@ -0,0 +1,9 @@ +guard :minitest, autorun: false, spring: true do + watch(%r{^app/(.+).rb$}) { |m| "test/#{m[1]}_test.rb" } + watch(%r{^app/controllers/application_controller.rb$}) { 'test/controllers' } + watch(%r{^app/controllers/(.+)_controller.rb$}) { |m| "test/integration/#{m[1]}_test.rb" } + watch(%r{^app/views/(.+)_mailer/.+}) { |m| "test/mailers/#{m[1]}_mailer_test.rb" } + watch(%r{^lib/(.+).rb$}) { |m| "test/lib/#{m[1]}_test.rb" } + watch(%r{^test/.+_test.rb$}) + watch(%r{^test/test_helper.rb$}) { 'test' } +end diff --git a/README_TO_USE b/README_TO_USE new file mode 100644 index 000000000..32a44b003 --- /dev/null +++ b/README_TO_USE @@ -0,0 +1,30 @@ +ACTION : +METHOD URL : +PREFERRED HTTP METHOD : +ARGUMENTS: + ARGUMENT: + EXAMPLE: + REQUIRED?: + + + resources :customers, only: [:index] + http://localhost:3000/customers + + resources :movies, only: [:index, + http://localhost:3000/movies + :show, + http://localhost:3000/movie + + :create] + http://localhost:3000/movies + + post '/rentals/check-out', to: "rentals#checkout", as: "checkout" + post '/rentals/check-in', to: "rentals#checkin", as: "checkin" + +ACTION : +METHOD URL : +PREFERRED HTTP METHOD : +ARGUMENTS: + ARGUMENT: + EXAMPLE: + REQUIRED?: diff --git a/Rakefile b/Rakefile new file mode 100644 index 000000000..e85f91391 --- /dev/null +++ b/Rakefile @@ -0,0 +1,6 @@ +# Add your own tasks in files placed in lib/tasks ending in .rake, +# for example lib/tasks/capistrano.rake, and they will automatically be available to Rake. + +require_relative 'config/application' + +Rails.application.load_tasks diff --git a/app/channels/application_cable/channel.rb b/app/channels/application_cable/channel.rb new file mode 100644 index 000000000..d67269728 --- /dev/null +++ b/app/channels/application_cable/channel.rb @@ -0,0 +1,4 @@ +module ApplicationCable + class Channel < ActionCable::Channel::Base + end +end diff --git a/app/channels/application_cable/connection.rb b/app/channels/application_cable/connection.rb new file mode 100644 index 000000000..0ff5442f4 --- /dev/null +++ b/app/channels/application_cable/connection.rb @@ -0,0 +1,4 @@ +module ApplicationCable + class Connection < ActionCable::Connection::Base + end +end diff --git a/app/controllers/application_controller.rb b/app/controllers/application_controller.rb new file mode 100644 index 000000000..4ac8823b0 --- /dev/null +++ b/app/controllers/application_controller.rb @@ -0,0 +1,2 @@ +class ApplicationController < ActionController::API +end diff --git a/app/controllers/concerns/.keep b/app/controllers/concerns/.keep new file mode 100644 index 000000000..e69de29bb diff --git a/app/controllers/customers_controller.rb b/app/controllers/customers_controller.rb new file mode 100644 index 000000000..4f08a00d3 --- /dev/null +++ b/app/controllers/customers_controller.rb @@ -0,0 +1,12 @@ +class CustomersController < ApplicationController + CUSTOMER_FIELDS = ["id", "name", "registered_at", "postal_code", "phone", "movies_checked_out_count"] + + # def success + # render json: { statement: "It works!!" }, status: :ok + # end + + def index + customers = Customer.all.as_json(only: CUSTOMER_FIELDS) + render json: customers, status: :ok + end +end diff --git a/app/controllers/movies_controller.rb b/app/controllers/movies_controller.rb new file mode 100644 index 000000000..6d37d76bb --- /dev/null +++ b/app/controllers/movies_controller.rb @@ -0,0 +1,40 @@ +class MoviesController < ApplicationController + INDEX_MOVIE_FIELDS = ["id", "title", "release_date"] + SHOW_MOVIE_FIELDS = ["id", "title", "overview", "release_date", "inventory", "available_inventory"] + + def index + movies = Movie.all.as_json(only: INDEX_MOVIE_FIELDS) + render json: movies, status: :ok + end + + def show + movie = Movie.find_by(id: params[:id]) + + if movie + render json: movie.as_json(only: SHOW_MOVIE_FIELDS), status: :ok + return + else + render json: {errors: {id: "ID not found"}}, status: :bad_request + return + end + end + + def create + movie = Movie.new(movie_params) + + if movie.save + # status should be :created, but Postman wants :ok + render json: movie.as_json(only: [:id]), status: :ok + return + else + render json: {errors: movie.errors.messages}, status: :not_acceptable + return + end + end + + private + + def movie_params + params.permit(:title, :overview, :release_date, :inventory) + end +end diff --git a/app/controllers/rentals_controller.rb b/app/controllers/rentals_controller.rb new file mode 100644 index 000000000..b3053932a --- /dev/null +++ b/app/controllers/rentals_controller.rb @@ -0,0 +1,73 @@ +class RentalsController < ApplicationController + def checkout + rental = Rental.new(rental_params) + rental.checkout_date = Date.today + rental.due_date = Date.today + 7 + + movie = Movie.find_by(id: rental.movie_id) + customer = Customer.find_by(id: rental.customer_id) + + if customer.nil? + render json: {errors: {customer: "customer ID could not be found"}}, status: :bad_request + return + end + + if movie + if movie.available_inventory <=0 + render json: {errors: {movie: "insufficient available inventory"}},status: :bad_request + return + else + movie.available_inventory -= 1 + movie.save + + customer.movies_checked_out_count += 1 + customer.save + end + else + render json: {errors: {movie: "movie ID could not be found"}}, status: :bad_request + return + end + + if rental.save + render json: rental.as_json(only: [:due_date]), status: :ok + return + end + end + + def checkin + movie = Movie.find_by(id: rental_params[:movie_id]) + customer = Customer.find_by(id: rental_params[:customer_id]) + + if movie && customer + rental = Rental.where(movie_id: movie.id, customer_id: customer.id, checkin_date: nil).first + if rental.nil? + render json: {errors: {rental: "Rental is not for this location"}}, status: :bad_request + return + end + + rental.checkin_date = Date.today + rental.save + + movie.available_inventory += 1 + movie.save + + customer.movies_checked_out_count -= 1 + customer.save + + render json: rental.as_json(only: [:id]), status: :ok + return + elsif movie + render json: {errors: {customer: "customer ID could not be found"}}, status: :bad_request + return + else + render json: {errors: {movie: "movie ID could not be found"}}, status: :bad_request + return + end + end + + private + + def rental_params + params.permit(:customer_id, :movie_id) + end +end diff --git a/app/jobs/application_job.rb b/app/jobs/application_job.rb new file mode 100644 index 000000000..a009ace51 --- /dev/null +++ b/app/jobs/application_job.rb @@ -0,0 +1,2 @@ +class ApplicationJob < ActiveJob::Base +end diff --git a/app/mailers/application_mailer.rb b/app/mailers/application_mailer.rb new file mode 100644 index 000000000..286b2239d --- /dev/null +++ b/app/mailers/application_mailer.rb @@ -0,0 +1,4 @@ +class ApplicationMailer < ActionMailer::Base + default from: 'from@example.com' + layout 'mailer' +end diff --git a/app/models/application_record.rb b/app/models/application_record.rb new file mode 100644 index 000000000..10a4cba84 --- /dev/null +++ b/app/models/application_record.rb @@ -0,0 +1,3 @@ +class ApplicationRecord < ActiveRecord::Base + self.abstract_class = true +end diff --git a/app/models/concerns/.keep b/app/models/concerns/.keep new file mode 100644 index 000000000..e69de29bb diff --git a/app/models/customer.rb b/app/models/customer.rb new file mode 100644 index 000000000..1297f0b4f --- /dev/null +++ b/app/models/customer.rb @@ -0,0 +1,7 @@ +class Customer < ApplicationRecord + has_many :rentals + + validates :name, :registered_at, :postal_code, :phone, presence: true + + #built-in validations done by rails: registered_at will be parsed into a type of DateTime to be accepted into DB +end diff --git a/app/models/movie.rb b/app/models/movie.rb new file mode 100644 index 000000000..344c3565d --- /dev/null +++ b/app/models/movie.rb @@ -0,0 +1,12 @@ +class Movie < ApplicationRecord + + has_many :rentals + + + validates :title, :overview, :release_date, :inventory, presence: true + + validates :inventory, numericality: true + + validates_numericality_of :inventory, greater_than_or_equal_to: 0 + +end diff --git a/app/models/rental.rb b/app/models/rental.rb new file mode 100644 index 000000000..22be881b5 --- /dev/null +++ b/app/models/rental.rb @@ -0,0 +1,8 @@ +class Rental < ApplicationRecord + belongs_to :customer + belongs_to :movie + + validates :checkout_date, :due_date, presence: true + + #built-in validations done by rails: checkout_date and due_date will be parsed into a type of Date to be accepted into DB +end diff --git a/app/views/layouts/mailer.html.erb b/app/views/layouts/mailer.html.erb new file mode 100644 index 000000000..cbd34d2e9 --- /dev/null +++ b/app/views/layouts/mailer.html.erb @@ -0,0 +1,13 @@ + + + + + + + + + <%= yield %> + + diff --git a/app/views/layouts/mailer.text.erb b/app/views/layouts/mailer.text.erb new file mode 100644 index 000000000..37f0bddbd --- /dev/null +++ b/app/views/layouts/mailer.text.erb @@ -0,0 +1 @@ +<%= yield %> diff --git a/bin/bundle b/bin/bundle new file mode 100755 index 000000000..f19acf5b5 --- /dev/null +++ b/bin/bundle @@ -0,0 +1,3 @@ +#!/usr/bin/env ruby +ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../Gemfile', __dir__) +load Gem.bin_path('bundler', 'bundle') diff --git a/bin/rails b/bin/rails new file mode 100755 index 000000000..5badb2fde --- /dev/null +++ b/bin/rails @@ -0,0 +1,9 @@ +#!/usr/bin/env ruby +begin + load File.expand_path('../spring', __FILE__) +rescue LoadError => e + raise unless e.message.include?('spring') +end +APP_PATH = File.expand_path('../config/application', __dir__) +require_relative '../config/boot' +require 'rails/commands' diff --git a/bin/rake b/bin/rake new file mode 100755 index 000000000..d87d5f578 --- /dev/null +++ b/bin/rake @@ -0,0 +1,9 @@ +#!/usr/bin/env ruby +begin + load File.expand_path('../spring', __FILE__) +rescue LoadError => e + raise unless e.message.include?('spring') +end +require_relative '../config/boot' +require 'rake' +Rake.application.run diff --git a/bin/setup b/bin/setup new file mode 100755 index 000000000..a334d86a6 --- /dev/null +++ b/bin/setup @@ -0,0 +1,33 @@ +#!/usr/bin/env ruby +require 'fileutils' +include FileUtils + +# path to your application root. +APP_ROOT = File.expand_path('..', __dir__) + +def system!(*args) + system(*args) || abort("\n== Command #{args} failed ==") +end + +chdir APP_ROOT do + # This script is a starting point to setup your application. + # Add necessary setup steps to this file. + + puts '== Installing dependencies ==' + system! 'gem install bundler --conservative' + system('bundle check') || system!('bundle install') + + # puts "\n== Copying sample files ==" + # unless File.exist?('config/database.yml') + # cp 'config/database.yml.sample', 'config/database.yml' + # end + + puts "\n== Preparing database ==" + system! 'bin/rails db:setup' + + puts "\n== Removing old logs and tempfiles ==" + system! 'bin/rails log:clear tmp:clear' + + puts "\n== Restarting application server ==" + system! 'bin/rails restart' +end diff --git a/bin/spring b/bin/spring new file mode 100755 index 000000000..d89ee495f --- /dev/null +++ b/bin/spring @@ -0,0 +1,17 @@ +#!/usr/bin/env ruby + +# This file loads Spring without using Bundler, in order to be fast. +# It gets overwritten when you run the `spring binstub` command. + +unless defined?(Spring) + require 'rubygems' + require 'bundler' + + lockfile = Bundler::LockfileParser.new(Bundler.default_lockfile.read) + spring = lockfile.specs.detect { |spec| spec.name == 'spring' } + if spring + Gem.use_paths Gem.dir, Bundler.bundle_path.to_s, *Gem.path + gem 'spring', spring.version + require 'spring/binstub' + end +end diff --git a/bin/update b/bin/update new file mode 100755 index 000000000..67d0d4964 --- /dev/null +++ b/bin/update @@ -0,0 +1,28 @@ +#!/usr/bin/env ruby +require 'fileutils' +include FileUtils + +# path to your application root. +APP_ROOT = File.expand_path('..', __dir__) + +def system!(*args) + system(*args) || abort("\n== Command #{args} failed ==") +end + +chdir APP_ROOT do + # This script is a way to update your development environment automatically. + # Add necessary update steps to this file. + + puts '== Installing dependencies ==' + system! 'gem install bundler --conservative' + system('bundle check') || system!('bundle install') + + puts "\n== Updating database ==" + system! 'bin/rails db:migrate' + + puts "\n== Removing old logs and tempfiles ==" + system! 'bin/rails log:clear tmp:clear' + + puts "\n== Restarting application server ==" + system! 'bin/rails restart' +end diff --git a/config.ru b/config.ru new file mode 100644 index 000000000..f7ba0b527 --- /dev/null +++ b/config.ru @@ -0,0 +1,5 @@ +# This file is used by Rack-based servers to start the application. + +require_relative 'config/environment' + +run Rails.application diff --git a/config/application.rb b/config/application.rb new file mode 100644 index 000000000..c47885bb0 --- /dev/null +++ b/config/application.rb @@ -0,0 +1,41 @@ +require_relative 'boot' + +require "rails" +# Pick the frameworks you want: +require "active_model/railtie" +require "active_job/railtie" +require "active_record/railtie" +require "active_storage/engine" +require "action_controller/railtie" +require "action_mailer/railtie" +require "action_view/railtie" +require "action_cable/engine" +# require "sprockets/railtie" +require "rails/test_unit/railtie" + +# Require the gems listed in Gemfile, including any gems +# you've limited to :test, :development, or :production. +Bundler.require(*Rails.groups) + +module VideoStoreApi + class Application < Rails::Application + config.generators do |g| + # Force new test files to be generated in the minitest-spec style + g.test_framework :minitest, spec: true + # Always use .js files, never .coffee + g.javascript_engine :js + end + # Initialize configuration defaults for originally generated Rails version. + config.load_defaults 5.2 + + # Settings in config/environments/* take precedence over those specified here. + # Application configuration can go into files in config/initializers + # -- all .rb files in that directory are automatically loaded after loading + # the framework and any gems in your application. + + # Only loads a smaller set of middleware suitable for API only apps. + # Middleware like session, flash, cookies can be added back manually. + # Skip views, helpers and assets when generating a new resource. + config.api_only = true + end +end diff --git a/config/boot.rb b/config/boot.rb new file mode 100644 index 000000000..b9e460cef --- /dev/null +++ b/config/boot.rb @@ -0,0 +1,4 @@ +ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../Gemfile', __dir__) + +require 'bundler/setup' # Set up gems listed in the Gemfile. +require 'bootsnap/setup' # Speed up boot time by caching expensive operations. diff --git a/config/cable.yml b/config/cable.yml new file mode 100644 index 000000000..6cc616cd1 --- /dev/null +++ b/config/cable.yml @@ -0,0 +1,10 @@ +development: + adapter: async + +test: + adapter: async + +production: + adapter: redis + url: <%= ENV.fetch("REDIS_URL") { "redis://localhost:6379/1" } %> + channel_prefix: video-store-api_production diff --git a/config/credentials.yml.enc b/config/credentials.yml.enc new file mode 100644 index 000000000..f652c8a74 --- /dev/null +++ b/config/credentials.yml.enc @@ -0,0 +1 @@ +GhYuC0iPbu6EWSKeoIw/g1bYexhkC17wX7KA4FgNnX4yViVTlT7k0S5F/GtlPdva4ia9o7zTEq5/Zg+T25pLYyzzAPOYQr1wJyDhwKAT+T/TciczOq3i2X1GiX3GOszd9UgG+yfu18Fn1Ngp9BI5IvZfRqVANOjxaVynoTu9IgbO28xPCSPyJGtqzgOm/PW378I2FHXqFxK8LAaKHbVFmcpcmg+Mf1c/bZwByVQ2t4G646ZvjSSW544lp9t33swY73kbXMIG6NA5gNzbKMfd6SYCHsL/pYoB4AcrQ/tkwrHuM0G/PM+pt0h06boFckGaQ1IQDeiwfFQA6qr2z3bvXmnDFhNmfsV5F5KZtQOW9BANcplrcy1nzgbGcVIOwTM0l/eJLvWcQnd4ta3QzpQN21obd1IYrmjFXjP2--IZPON5kiy9IUvZTH--dlxCZn1mY6y5+ckVP65hYQ== \ No newline at end of file diff --git a/config/database.yml b/config/database.yml new file mode 100644 index 000000000..25f76c8e1 --- /dev/null +++ b/config/database.yml @@ -0,0 +1,85 @@ +# PostgreSQL. Versions 9.1 and up are supported. +# +# Install the pg driver: +# gem install pg +# On OS X with Homebrew: +# gem install pg -- --with-pg-config=/usr/local/bin/pg_config +# On OS X with MacPorts: +# gem install pg -- --with-pg-config=/opt/local/lib/postgresql84/bin/pg_config +# On Windows: +# gem install pg +# Choose the win32 build. +# Install PostgreSQL and put its /bin directory on your path. +# +# Configure Using Gemfile +# gem 'pg' +# +default: &default + adapter: postgresql + encoding: unicode + # For details on connection pooling, see Rails configuration guide + # http://guides.rubyonrails.org/configuring.html#database-pooling + pool: <%= ENV.fetch("RAILS_MAX_THREADS") { 5 } %> + +development: + <<: *default + database: video-store-api_development + + # The specified database role being used to connect to postgres. + # To create additional roles in postgres see `$ createuser --help`. + # When left blank, postgres will use the default role. This is + # the same name as the operating system user that initialized the database. + #username: video-store-api + + # The password associated with the postgres role (username). + #password: + + # Connect on a TCP socket. Omitted by default since the client uses a + # domain socket that doesn't need configuration. Windows does not have + # domain sockets, so uncomment these lines. + #host: localhost + + # The TCP port the server listens on. Defaults to 5432. + # If your server runs on a different port number, change accordingly. + #port: 5432 + + # Schema search path. The server defaults to $user,public + #schema_search_path: myapp,sharedapp,public + + # Minimum log levels, in increasing order: + # debug5, debug4, debug3, debug2, debug1, + # log, notice, warning, error, fatal, and panic + # Defaults to warning. + #min_messages: notice + +# Warning: The database defined as "test" will be erased and +# re-generated from your development database when you run "rake". +# Do not set this db to the same as development or production. +test: + <<: *default + database: video-store-api_test + +# As with config/secrets.yml, you never want to store sensitive information, +# like your database password, in your source code. If your source code is +# ever seen by anyone, they now have access to your database. +# +# Instead, provide the password as a unix environment variable when you boot +# the app. Read http://guides.rubyonrails.org/configuring.html#configuring-a-database +# for a full rundown on how to provide these environment variables in a +# production deployment. +# +# On Heroku and other platform providers, you may have a full connection URL +# available as an environment variable. For example: +# +# DATABASE_URL="postgres://myuser:mypass@localhost/somedatabase" +# +# You can use this database configuration with: +# +# production: +# url: <%= ENV['DATABASE_URL'] %> +# +production: + <<: *default + database: video-store-api_production + username: video-store-api + password: <%= ENV['VIDEO-STORE-API_DATABASE_PASSWORD'] %> diff --git a/config/environment.rb b/config/environment.rb new file mode 100644 index 000000000..426333bb4 --- /dev/null +++ b/config/environment.rb @@ -0,0 +1,5 @@ +# Load the Rails application. +require_relative 'application' + +# Initialize the Rails application. +Rails.application.initialize! diff --git a/config/environments/development.rb b/config/environments/development.rb new file mode 100644 index 000000000..d52ec9efb --- /dev/null +++ b/config/environments/development.rb @@ -0,0 +1,54 @@ +Rails.application.configure do + # Settings specified here will take precedence over those in config/application.rb. + + # In the development environment your application's code is reloaded on + # every request. This slows down response time but is perfect for development + # since you don't have to restart the web server when you make code changes. + config.cache_classes = false + + # Do not eager load code on boot. + config.eager_load = false + + # Show full error reports. + config.consider_all_requests_local = true + + # Enable/disable caching. By default caching is disabled. + # Run rails dev:cache to toggle caching. + if Rails.root.join('tmp', 'caching-dev.txt').exist? + config.action_controller.perform_caching = true + + config.cache_store = :memory_store + config.public_file_server.headers = { + 'Cache-Control' => "public, max-age=#{2.days.to_i}" + } + else + config.action_controller.perform_caching = false + + config.cache_store = :null_store + end + + # Store uploaded files on the local file system (see config/storage.yml for options) + config.active_storage.service = :local + + # Don't care if the mailer can't send. + config.action_mailer.raise_delivery_errors = false + + config.action_mailer.perform_caching = false + + # Print deprecation notices to the Rails logger. + config.active_support.deprecation = :log + + # Raise an error on page load if there are pending migrations. + config.active_record.migration_error = :page_load + + # Highlight code that triggered database queries in logs. + config.active_record.verbose_query_logs = true + + + # Raises error for missing translations + # config.action_view.raise_on_missing_translations = true + + # Use an evented file watcher to asynchronously detect changes in source code, + # routes, locales, etc. This feature depends on the listen gem. + config.file_watcher = ActiveSupport::EventedFileUpdateChecker +end diff --git a/config/environments/production.rb b/config/environments/production.rb new file mode 100644 index 000000000..5f023a796 --- /dev/null +++ b/config/environments/production.rb @@ -0,0 +1,85 @@ +Rails.application.configure do + # Settings specified here will take precedence over those in config/application.rb. + + # Code is not reloaded between requests. + config.cache_classes = true + + # Eager load code on boot. This eager loads most of Rails and + # your application in memory, allowing both threaded web servers + # and those relying on copy on write to perform better. + # Rake tasks automatically ignore this option for performance. + config.eager_load = true + + # Full error reports are disabled and caching is turned on. + config.consider_all_requests_local = false + config.action_controller.perform_caching = true + + # Ensures that a master key has been made available in either ENV["RAILS_MASTER_KEY"] + # or in config/master.key. This key is used to decrypt credentials (and other encrypted files). + # config.require_master_key = true + + # Disable serving static files from the `/public` folder by default since + # Apache or NGINX already handles this. + config.public_file_server.enabled = ENV['RAILS_SERVE_STATIC_FILES'].present? + + # Enable serving of images, stylesheets, and JavaScripts from an asset server. + # config.action_controller.asset_host = 'http://assets.example.com' + + # Specifies the header that your server uses for sending files. + # config.action_dispatch.x_sendfile_header = 'X-Sendfile' # for Apache + # config.action_dispatch.x_sendfile_header = 'X-Accel-Redirect' # for NGINX + + # Store uploaded files on the local file system (see config/storage.yml for options) + config.active_storage.service = :local + + # Mount Action Cable outside main process or domain + # config.action_cable.mount_path = nil + # config.action_cable.url = 'wss://example.com/cable' + # config.action_cable.allowed_request_origins = [ 'http://example.com', /http:\/\/example.*/ ] + + # Force all access to the app over SSL, use Strict-Transport-Security, and use secure cookies. + # config.force_ssl = true + + # Use the lowest log level to ensure availability of diagnostic information + # when problems arise. + config.log_level = :debug + + # Prepend all log lines with the following tags. + config.log_tags = [ :request_id ] + + # Use a different cache store in production. + # config.cache_store = :mem_cache_store + + # Use a real queuing backend for Active Job (and separate queues per environment) + # config.active_job.queue_adapter = :resque + # config.active_job.queue_name_prefix = "video-store-api_#{Rails.env}" + + config.action_mailer.perform_caching = false + + # Ignore bad email addresses and do not raise email delivery errors. + # Set this to true and configure the email server for immediate delivery to raise delivery errors. + # config.action_mailer.raise_delivery_errors = false + + # Enable locale fallbacks for I18n (makes lookups for any locale fall back to + # the I18n.default_locale when a translation cannot be found). + config.i18n.fallbacks = true + + # Send deprecation notices to registered listeners. + config.active_support.deprecation = :notify + + # Use default logging formatter so that PID and timestamp are not suppressed. + config.log_formatter = ::Logger::Formatter.new + + # Use a different logger for distributed setups. + # require 'syslog/logger' + # config.logger = ActiveSupport::TaggedLogging.new(Syslog::Logger.new 'app-name') + + if ENV["RAILS_LOG_TO_STDOUT"].present? + logger = ActiveSupport::Logger.new(STDOUT) + logger.formatter = config.log_formatter + config.logger = ActiveSupport::TaggedLogging.new(logger) + end + + # Do not dump schema after migrations. + config.active_record.dump_schema_after_migration = false +end diff --git a/config/environments/test.rb b/config/environments/test.rb new file mode 100644 index 000000000..0a38fd3ce --- /dev/null +++ b/config/environments/test.rb @@ -0,0 +1,46 @@ +Rails.application.configure do + # Settings specified here will take precedence over those in config/application.rb. + + # The test environment is used exclusively to run your application's + # test suite. You never need to work with it otherwise. Remember that + # your test database is "scratch space" for the test suite and is wiped + # and recreated between test runs. Don't rely on the data there! + config.cache_classes = true + + # Do not eager load code on boot. This avoids loading your whole application + # just for the purpose of running a single test. If you are using a tool that + # preloads Rails for running tests, you may have to set it to true. + config.eager_load = false + + # Configure public file server for tests with Cache-Control for performance. + config.public_file_server.enabled = true + config.public_file_server.headers = { + 'Cache-Control' => "public, max-age=#{1.hour.to_i}" + } + + # Show full error reports and disable caching. + config.consider_all_requests_local = true + config.action_controller.perform_caching = false + + # Raise exceptions instead of rendering exception templates. + config.action_dispatch.show_exceptions = false + + # Disable request forgery protection in test environment. + config.action_controller.allow_forgery_protection = false + + # Store uploaded files on the local file system in a temporary directory + config.active_storage.service = :test + + config.action_mailer.perform_caching = false + + # Tell Action Mailer not to deliver emails to the real world. + # The :test delivery method accumulates sent emails in the + # ActionMailer::Base.deliveries array. + config.action_mailer.delivery_method = :test + + # Print deprecation notices to the stderr. + config.active_support.deprecation = :stderr + + # Raises error for missing translations + # config.action_view.raise_on_missing_translations = true +end diff --git a/config/initializers/action_view.rb b/config/initializers/action_view.rb new file mode 100644 index 000000000..142d382f8 --- /dev/null +++ b/config/initializers/action_view.rb @@ -0,0 +1 @@ +Rails.application.config.action_view.form_with_generates_remote_forms = false diff --git a/config/initializers/application_controller_renderer.rb b/config/initializers/application_controller_renderer.rb new file mode 100644 index 000000000..89d2efab2 --- /dev/null +++ b/config/initializers/application_controller_renderer.rb @@ -0,0 +1,8 @@ +# Be sure to restart your server when you modify this file. + +# ActiveSupport::Reloader.to_prepare do +# ApplicationController.renderer.defaults.merge!( +# http_host: 'example.org', +# https: false +# ) +# end diff --git a/config/initializers/backtrace_silencers.rb b/config/initializers/backtrace_silencers.rb new file mode 100644 index 000000000..59385cdf3 --- /dev/null +++ b/config/initializers/backtrace_silencers.rb @@ -0,0 +1,7 @@ +# Be sure to restart your server when you modify this file. + +# You can add backtrace silencers for libraries that you're using but don't wish to see in your backtraces. +# Rails.backtrace_cleaner.add_silencer { |line| line =~ /my_noisy_library/ } + +# You can also remove all the silencers if you're trying to debug a problem that might stem from framework code. +# Rails.backtrace_cleaner.remove_silencers! diff --git a/config/initializers/cors.rb b/config/initializers/cors.rb new file mode 100644 index 000000000..3b1c1b5ed --- /dev/null +++ b/config/initializers/cors.rb @@ -0,0 +1,16 @@ +# Be sure to restart your server when you modify this file. + +# Avoid CORS issues when API is called from the frontend app. +# Handle Cross-Origin Resource Sharing (CORS) in order to accept cross-origin AJAX requests. + +# Read more: https://github.com/cyu/rack-cors + +# Rails.application.config.middleware.insert_before 0, Rack::Cors do +# allow do +# origins 'example.com' +# +# resource '*', +# headers: :any, +# methods: [:get, :post, :put, :patch, :delete, :options, :head] +# end +# end diff --git a/config/initializers/filter_parameter_logging.rb b/config/initializers/filter_parameter_logging.rb new file mode 100644 index 000000000..4a994e1e7 --- /dev/null +++ b/config/initializers/filter_parameter_logging.rb @@ -0,0 +1,4 @@ +# Be sure to restart your server when you modify this file. + +# Configure sensitive parameters which will be filtered from the log file. +Rails.application.config.filter_parameters += [:password] diff --git a/config/initializers/inflections.rb b/config/initializers/inflections.rb new file mode 100644 index 000000000..ac033bf9d --- /dev/null +++ b/config/initializers/inflections.rb @@ -0,0 +1,16 @@ +# Be sure to restart your server when you modify this file. + +# Add new inflection rules using the following format. Inflections +# are locale specific, and you may define rules for as many different +# locales as you wish. All of these examples are active by default: +# ActiveSupport::Inflector.inflections(:en) do |inflect| +# inflect.plural /^(ox)$/i, '\1en' +# inflect.singular /^(ox)en/i, '\1' +# inflect.irregular 'person', 'people' +# inflect.uncountable %w( fish sheep ) +# end + +# These inflection rules are supported but not enabled by default: +# ActiveSupport::Inflector.inflections(:en) do |inflect| +# inflect.acronym 'RESTful' +# end diff --git a/config/initializers/mime_types.rb b/config/initializers/mime_types.rb new file mode 100644 index 000000000..dc1899682 --- /dev/null +++ b/config/initializers/mime_types.rb @@ -0,0 +1,4 @@ +# Be sure to restart your server when you modify this file. + +# Add new mime types for use in respond_to blocks: +# Mime::Type.register "text/richtext", :rtf diff --git a/config/initializers/wrap_parameters.rb b/config/initializers/wrap_parameters.rb new file mode 100644 index 000000000..bbfc3961b --- /dev/null +++ b/config/initializers/wrap_parameters.rb @@ -0,0 +1,14 @@ +# Be sure to restart your server when you modify this file. + +# This file contains settings for ActionController::ParamsWrapper which +# is enabled by default. + +# Enable parameter wrapping for JSON. You can disable this by setting :format to an empty array. +ActiveSupport.on_load(:action_controller) do + wrap_parameters format: [:json] +end + +# To enable root element in JSON for ActiveRecord objects. +# ActiveSupport.on_load(:active_record) do +# self.include_root_in_json = true +# end diff --git a/config/locales/en.yml b/config/locales/en.yml new file mode 100644 index 000000000..decc5a857 --- /dev/null +++ b/config/locales/en.yml @@ -0,0 +1,33 @@ +# Files in the config/locales directory are used for internationalization +# and are automatically loaded by Rails. If you want to use locales other +# than English, add the necessary files in this directory. +# +# To use the locales, use `I18n.t`: +# +# I18n.t 'hello' +# +# In views, this is aliased to just `t`: +# +# <%= t('hello') %> +# +# To use a different locale, set it with `I18n.locale`: +# +# I18n.locale = :es +# +# This would use the information in config/locales/es.yml. +# +# The following keys must be escaped otherwise they will not be retrieved by +# the default I18n backend: +# +# true, false, on, off, yes, no +# +# Instead, surround them with single quotes. +# +# en: +# 'true': 'foo' +# +# To learn more, please read the Rails Internationalization guide +# available at http://guides.rubyonrails.org/i18n.html. + +en: + hello: "Hello world" diff --git a/config/puma.rb b/config/puma.rb new file mode 100644 index 000000000..a5eccf816 --- /dev/null +++ b/config/puma.rb @@ -0,0 +1,34 @@ +# Puma can serve each request in a thread from an internal thread pool. +# The `threads` method setting takes two numbers: a minimum and maximum. +# Any libraries that use thread pools should be configured to match +# the maximum value specified for Puma. Default is set to 5 threads for minimum +# and maximum; this matches the default thread size of Active Record. +# +threads_count = ENV.fetch("RAILS_MAX_THREADS") { 5 } +threads threads_count, threads_count + +# Specifies the `port` that Puma will listen on to receive requests; default is 3000. +# +port ENV.fetch("PORT") { 3000 } + +# Specifies the `environment` that Puma will run in. +# +environment ENV.fetch("RAILS_ENV") { "development" } + +# Specifies the number of `workers` to boot in clustered mode. +# Workers are forked webserver processes. If using threads and workers together +# the concurrency of the application would be max `threads` * `workers`. +# Workers do not work on JRuby or Windows (both of which do not support +# processes). +# +# workers ENV.fetch("WEB_CONCURRENCY") { 2 } + +# Use the `preload_app!` method when specifying a `workers` number. +# This directive tells Puma to first boot the application and load code +# before forking the application. This takes advantage of Copy On Write +# process behavior so workers use less memory. +# +# preload_app! + +# Allow puma to be restarted by `rails restart` command. +plugin :tmp_restart diff --git a/config/routes.rb b/config/routes.rb new file mode 100644 index 000000000..25446e85b --- /dev/null +++ b/config/routes.rb @@ -0,0 +1,10 @@ +Rails.application.routes.draw do + # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html + + get "/zomg", to: "customers#success", as: "test" + resources :customers, only: [:index] + resources :movies, only: [:index, :show, :create] + + post '/rentals/check-out', to: "rentals#checkout", as: "checkout" + post '/rentals/check-in', to: "rentals#checkin", as: "checkin" +end \ No newline at end of file diff --git a/config/spring.rb b/config/spring.rb new file mode 100644 index 000000000..9fa7863f9 --- /dev/null +++ b/config/spring.rb @@ -0,0 +1,6 @@ +%w[ + .ruby-version + .rbenv-vars + tmp/restart.txt + tmp/caching-dev.txt +].each { |path| Spring.watch(path) } diff --git a/config/storage.yml b/config/storage.yml new file mode 100644 index 000000000..d32f76e8f --- /dev/null +++ b/config/storage.yml @@ -0,0 +1,34 @@ +test: + service: Disk + root: <%= Rails.root.join("tmp/storage") %> + +local: + service: Disk + root: <%= Rails.root.join("storage") %> + +# Use rails credentials:edit to set the AWS secrets (as aws:access_key_id|secret_access_key) +# amazon: +# service: S3 +# access_key_id: <%= Rails.application.credentials.dig(:aws, :access_key_id) %> +# secret_access_key: <%= Rails.application.credentials.dig(:aws, :secret_access_key) %> +# region: us-east-1 +# bucket: your_own_bucket + +# Remember not to checkin your GCS keyfile to a repository +# google: +# service: GCS +# project: your_project +# credentials: <%= Rails.root.join("path/to/gcs.keyfile") %> +# bucket: your_own_bucket + +# Use rails credentials:edit to set the Azure Storage secret (as azure_storage:storage_access_key) +# microsoft: +# service: AzureStorage +# storage_account_name: your_account_name +# storage_access_key: <%= Rails.application.credentials.dig(:azure_storage, :storage_access_key) %> +# container: your_container_name + +# mirror: +# service: Mirror +# primary: local +# mirrors: [ amazon, google, microsoft ] diff --git a/db/migrate/20191105213301_create_customers.rb b/db/migrate/20191105213301_create_customers.rb new file mode 100644 index 000000000..a1658c620 --- /dev/null +++ b/db/migrate/20191105213301_create_customers.rb @@ -0,0 +1,16 @@ +class CreateCustomers < ActiveRecord::Migration[5.2] + def change + create_table :customers do |t| + t.string :name + t.datetime :registered_at + t.string :postal_code + t.string :phone + t.integer :movies_checked_out_count + t.string :address + t.string :city + t.string :state + + t.timestamps + end + end +end diff --git a/db/migrate/20191105213358_create_movies.rb b/db/migrate/20191105213358_create_movies.rb new file mode 100644 index 000000000..701b5b545 --- /dev/null +++ b/db/migrate/20191105213358_create_movies.rb @@ -0,0 +1,13 @@ +class CreateMovies < ActiveRecord::Migration[5.2] + def change + create_table :movies do |t| + t.string :title + t.string :overview + t.date :release_date + t.integer :inventory + t.integer :available_inventory + + t.timestamps + end + end +end diff --git a/db/migrate/20191105213440_create_rentals.rb b/db/migrate/20191105213440_create_rentals.rb new file mode 100644 index 000000000..c98328095 --- /dev/null +++ b/db/migrate/20191105213440_create_rentals.rb @@ -0,0 +1,10 @@ +class CreateRentals < ActiveRecord::Migration[5.2] + def change + create_table :rentals do |t| + t.date :checkout_date + t.date :due_date + + t.timestamps + end + end +end diff --git a/db/migrate/20191105213615_add_customer_and_movie_ids_to_rentals.rb b/db/migrate/20191105213615_add_customer_and_movie_ids_to_rentals.rb new file mode 100644 index 000000000..84551d402 --- /dev/null +++ b/db/migrate/20191105213615_add_customer_and_movie_ids_to_rentals.rb @@ -0,0 +1,6 @@ +class AddCustomerAndMovieIdsToRentals < ActiveRecord::Migration[5.2] + def change + add_reference :rentals, :customer, foreign_key: true + add_reference :rentals, :movie, foreign_key: true + end +end diff --git a/db/migrate/20191108051010_check_in_date_for_rentals.rb b/db/migrate/20191108051010_check_in_date_for_rentals.rb new file mode 100644 index 000000000..66d7f0988 --- /dev/null +++ b/db/migrate/20191108051010_check_in_date_for_rentals.rb @@ -0,0 +1,5 @@ +class CheckInDateForRentals < ActiveRecord::Migration[5.2] + def change + add_column :rentals, :checkin_date, :date + end +end diff --git a/db/schema.rb b/db/schema.rb new file mode 100644 index 000000000..e0fb838ae --- /dev/null +++ b/db/schema.rb @@ -0,0 +1,55 @@ +# This file is auto-generated from the current state of the database. Instead +# of editing this file, please use the migrations feature of Active Record to +# incrementally modify your database, and then regenerate this schema definition. +# +# Note that this schema.rb definition is the authoritative source for your +# database schema. If you need to create the application database on another +# system, you should be using db:schema:load, not running all the migrations +# from scratch. The latter is a flawed and unsustainable approach (the more migrations +# you'll amass, the slower it'll run and the greater likelihood for issues). +# +# It's strongly recommended that you check this file into your version control system. + +ActiveRecord::Schema.define(version: 2019_11_08_051010) do + + # These are extensions that must be enabled in order to support this database + enable_extension "plpgsql" + + create_table "customers", force: :cascade do |t| + t.string "name" + t.datetime "registered_at" + t.string "postal_code" + t.string "phone" + t.integer "movies_checked_out_count" + t.string "address" + t.string "city" + t.string "state" + t.datetime "created_at", null: false + t.datetime "updated_at", null: false + end + + create_table "movies", force: :cascade do |t| + t.string "title" + t.string "overview" + t.date "release_date" + t.integer "inventory" + t.integer "available_inventory" + t.datetime "created_at", null: false + t.datetime "updated_at", null: false + end + + create_table "rentals", force: :cascade do |t| + t.date "checkout_date" + t.date "due_date" + t.datetime "created_at", null: false + t.datetime "updated_at", null: false + t.bigint "customer_id" + t.bigint "movie_id" + t.date "checkin_date" + t.index ["customer_id"], name: "index_rentals_on_customer_id" + t.index ["movie_id"], name: "index_rentals_on_movie_id" + end + + add_foreign_key "rentals", "customers" + add_foreign_key "rentals", "movies" +end diff --git a/db/seeds.rb b/db/seeds.rb index 5322340ba..ada23637f 100644 --- a/db/seeds.rb +++ b/db/seeds.rb @@ -1,7 +1,15 @@ JSON.parse(File.read('db/seeds/customers.json')).each do |customer| - Customer.create!(customer) + customer = Customer.new(customer) + customer.movies_checked_out_count = 0 + customer.save! end JSON.parse(File.read('db/seeds/movies.json')).each do |movie| - Movie.create!(movie) + movie = Movie.new(movie) + movie.available_inventory = movie.inventory + movie.save! +end + +JSON.parse(File.read('db/seeds/rentals.json')).each do |rental| + Rental.create!(rental) end diff --git a/db/seeds/rentals.json b/db/seeds/rentals.json new file mode 100644 index 000000000..1345591c5 --- /dev/null +++ b/db/seeds/rentals.json @@ -0,0 +1,170 @@ +[ + { + "checkout_date": "2019-01-02", + "due_date": "2019-01-09", + "customer_id": 199, + "movie_id": 1, + "checkin_date": "2019-01-10" + }, + { + "checkout_date": "2019-01-03", + "due_date": "2019-01-10", + "customer_id": 125, + "movie_id": 31, + "checkin_date": "2019-01-07" + }, + { + "checkout_date": "2019-01-02", + "due_date": "2019-01-09", + "customer_id": 113, + "movie_id": 46, + "checkin_date": "2019-01-08" + }, + { + "checkout_date": "2019-01-09", + "due_date": "2019-01-16", + "customer_id": 85, + "movie_id": 9, + "checkin_date": "nil" + }, + { + "checkout_date": "2019-01-02", + "due_date": "2019-01-09", + "customer_id": 188, + "movie_id": 21, + "checkin_date": "2019-01-03" + }, + { + "checkout_date": "2019-01-02", + "due_date": "2019-01-09", + "customer_id": 160, + "movie_id": 82, + "checkin_date": "2019-01-03" + }, + { + "checkout_date": "2019-01-09", + "due_date": "2019-01-16", + "customer_id": 4, + "movie_id": 89, + "checkin_date": "2019-01-13" + }, + { + "checkout_date": "2019-01-02", + "due_date": "2019-01-09", + "customer_id": 40, + "movie_id": 76, + "checkin_date": "2019-01-05" + }, + { + "checkout_date": "2019-01-02", + "due_date": "2019-01-09", + "customer_id": 52, + "movie_id": 80, + "checkin_date": "2019-01-06" + }, + { + "checkout_date": "2019-01-03", + "due_date": "2019-01-10", + "customer_id": 141, + "movie_id": 35, + "checkin_date": "2019-01-05" + }, + { + "checkout_date": "2019-01-04", + "due_date": "2019-01-11", + "customer_id": 15, + "movie_id": 51, + "checkin_date": "nil" + }, + { + "checkout_date": "2019-01-05", + "due_date": "2019-01-12", + "customer_id": 119, + "movie_id": 26, + "checkin_date": "2019-01-08" + }, + { + "checkout_date": "2019-01-09", + "due_date": "2019-01-16", + "customer_id": 112, + "movie_id": 51, + "checkin_date": "2019-01-13" + }, + { + "checkout_date": "2019-01-02", + "due_date": "2019-01-09", + "customer_id": 91, + "movie_id": 25, + "checkin_date": "2019-01-03" + }, + { + "checkout_date": "2019-01-02", + "due_date": "2019-01-09", + "customer_id": 64, + "movie_id": 62, + "checkin_date": "2019-01-06" + }, + { + "checkout_date": "2019-01-09", + "due_date": "2019-01-16", + "customer_id": 125, + "movie_id": 65, + "checkin_date": "2019-01-12" + }, + { + "checkout_date": "2019-01-02", + "due_date": "2019-01-09", + "customer_id": 171, + "movie_id": 53, + "checkin_date": "2019-01-07" + }, + { + "checkout_date": "2019-01-03", + "due_date": "2019-01-10", + "customer_id": 130, + "movie_id": 68, + "checkin_date": "nil" + }, + { + "checkout_date": "2019-01-09", + "due_date": "2019-01-16", + "customer_id": 58, + "movie_id": 10, + "checkin_date": "nil" + }, + { + "checkout_date": "2019-01-02", + "due_date": "2019-01-09", + "customer_id": 158, + "movie_id": 27, + "checkin_date": "2019-01-03" + }, + { + "checkout_date": "2019-01-02", + "due_date": "2019-01-09", + "customer_id": 188, + "movie_id": 30, + "checkin_date": "2019-01-07" + }, + { + "checkout_date": "2019-01-07", + "due_date": "2019-01-14", + "customer_id": 134, + "movie_id": 67, + "checkin_date": "2019-01-12" + }, + { + "checkout_date": "2019-01-08", + "due_date": "2019-01-15", + "customer_id": 176, + "movie_id": 85, + "checkin_date": "2019-01-15" + }, + { + "checkout_date": "2019-01-09", + "due_date": "2019-01-16", + "customer_id": 21, + "movie_id": 64, + "checkin_date": "nil" + } +] \ No newline at end of file diff --git a/feedback.md b/feedback.md index b6f69f1eb..203a4c658 100644 --- a/feedback.md +++ b/feedback.md @@ -1,4 +1,4 @@ -# Video Store + # Video Store ## What We're Looking For Feature | Feedback diff --git a/lib/tasks/.keep b/lib/tasks/.keep new file mode 100644 index 000000000..e69de29bb diff --git a/log/.keep b/log/.keep new file mode 100644 index 000000000..e69de29bb diff --git a/public/robots.txt b/public/robots.txt new file mode 100644 index 000000000..37b576a4a --- /dev/null +++ b/public/robots.txt @@ -0,0 +1 @@ +# See http://www.robotstxt.org/robotstxt.html for documentation on how to use the robots.txt file diff --git a/storage/.keep b/storage/.keep new file mode 100644 index 000000000..e69de29bb diff --git a/test/controllers/.keep b/test/controllers/.keep new file mode 100644 index 000000000..e69de29bb diff --git a/test/controllers/customers_controller_test.rb b/test/controllers/customers_controller_test.rb new file mode 100644 index 000000000..ae0cc89ef --- /dev/null +++ b/test/controllers/customers_controller_test.rb @@ -0,0 +1,63 @@ +require "test_helper" + +describe CustomersController do + # constant for keys + CUSTOMER_FIELDS = ["id", "name", "registered_at", "postal_code", "phone", "movies_checked_out_count"].sort + + # describe "zomg" do + # it "checks to see if zomg produces zombies? :D" do + # success_statement = "It works!!" + # get test_path + # must_respond_with :ok + # expect(response.header['Content-Type']).must_include 'json' + # body = JSON.parse(response.body) + # expect(body).must_be_instance_of Hash + # expect(body["statement"]).must_equal success_statement + # end + # end + + describe "index" do + it "returns JSON, success, and a list of customers" do + # the route + get customers_path + + # check that the content type of the header is JSON + # check the response code + # check that the body is a list + body = check_response(expected_type: Array, expected_status: :ok) + # check that the list contains hashes + # check that the customer hash has the right keys + body.each do |customer| + expect(customer).must_be_instance_of Hash + expect(customer.keys.sort).must_equal CUSTOMER_FIELDS + end + # check length of list against Customer.all.count + customer_count = Customer.all.count + expect(body.length).must_equal customer_count + end + + it "returns JSON, success, and an empty list if no customers" do + Rental.destroy_all + # delete all customers + Customer.destroy_all + + # the route + get customers_path + + # check that the content type of the header is JSON + # check the response code + # check that the body is a list + body = check_response(expected_type: Array, expected_status: :ok) + # check that the list contains hashes + # check that the customer hash is empty + body.each do |customer| + expect(customer).must_be_instance_of Hash + expect(customer.empty?).must_equal true + end + # check length of list against Customer.all.count + customer_count = Customer.all.count + expect(body.length).must_equal customer_count + end + end + +end diff --git a/test/controllers/movies_controller_test.rb b/test/controllers/movies_controller_test.rb new file mode 100644 index 000000000..00196bdbb --- /dev/null +++ b/test/controllers/movies_controller_test.rb @@ -0,0 +1,137 @@ +require "test_helper" + +describe MoviesController do + INDEX_MOVIE_FIELDS = ["id", "title", "release_date"].sort + SHOW_MOVIE_FIELDS = ["id", "title", "overview", "release_date", "inventory", "available_inventory"].sort + + describe "index" do + it "returns JSON, success, and a list of movies" do + # the route + get movies_path + + # check that the content type of the header is JSON + # check the response code + # check that the body is a list + body = check_response(expected_type: Array, expected_status: :ok) + # check that the list contains hashes + # check that the movie hash has the right keys + body.each do |movie| + expect(movie).must_be_instance_of Hash + expect(movie.keys.sort).must_equal INDEX_MOVIE_FIELDS + end + # check length of list against Movie.all.count + movie_count = Movie.all.count + expect(body.length).must_equal movie_count + end + + it "returns JSON, success, and an empty list if no movies" do + Rental.destroy_all + Movie.destroy_all + + # the route + get movies_path + + # check that the content type of the header is JSON + # check the response code + # check that the body is a list + body = check_response(expected_type: Array, expected_status: :ok) + # check that the list contains hashes + # check that the movie hash is empty + body.each do |movie| + expect(movie).must_be_instance_of Hash + expect(movie.empty?).must_equal true + end + # check length of list against Movie.all.count + movies_count = Movie.all.count + expect(body.length).must_equal movies_count + end + end + + describe "show" do + it "returns JSON, success, and movie data for valid movie" do + # get a movie + movie = movies(:m_1) + + # the route + get movie_path(movie.id) + + # check the content type and response code + # expected_status should be :found, but Postman wants :ok + body = check_response(expected_type: Hash, expected_status: :ok) + # binding.pry + # check that keys are correct + expect(body.keys.sort).must_equal SHOW_MOVIE_FIELDS + # check that each value matches the fixture value + expect(body["id"]).must_equal movie.id + expect(body["title"]).must_equal movie.title + expect(body["overview"]).must_equal movie.overview + expect(body["release_date"]).must_equal movie.release_date.to_s + expect(body["inventory"]).must_equal movie.inventory + expect(body["available_inventory"]).must_equal movie.inventory + end + + it "returns JSON, not_found, and error message for invalid movie" do + # the route + get movie_path(-9) + + # content type and response code not_found + body = check_response(expected_type: Hash, expected_status: :bad_request) + + # check the message that's returned + expect(body.keys).must_include 'errors' + expect(body['errors'].keys).must_include "id" + end + end + + describe "create" do + it "creates a movie with valid parameters, returns new movie id" do + # parameters + movie_params = { + title: "The Martian Elf", + overview: "Elf brings holiday cheer to the people stranded on Mars", + release_date: "2018-12-25", + inventory: 2 + } + + # the route + # check that database was updated by one + expect { + post movies_path, params: movie_params + }.must_differ "Movie.count", 1 + + # check the content type and response code + # expected_status should be :created, but Postman wants :ok + body = check_response(expected_type: Hash, expected_status: :ok) + # check that the body contains the id + expect(body.keys).must_equal ["id"] + # check that we can find the movie by the id + new_movie = Movie.find_by(id: body["id"]) + # and its variables match the parameters + expect(new_movie.title).must_equal movie_params[:title] + expect(new_movie.overview).must_equal movie_params[:overview] + expect(new_movie.release_date).must_equal Date.parse(movie_params[:release_date]) + expect(new_movie.inventory).must_equal movie_params[:inventory] + end + + it "gives appropriate response if attempting to create invalid movie" do + # invalid parameters + movie_params = { + overview: "Elf brings holiday cheer to the people stranded on Mars", + release_date: "2018-12-25", + inventory: 2 + } + + # the route + # check that the database does not change count + expect { + post movies_path, params: movie_params + }.wont_differ "Movie.count" + + # check the content type and response code, :not_acceptable + body = check_response(expected_type: Hash, expected_status: :not_acceptable) + # check the message that's returned + expect(body.keys).must_include 'errors' + expect(body['errors'].keys).must_include "title" + end + end +end diff --git a/test/controllers/rentals_controller_test.rb b/test/controllers/rentals_controller_test.rb new file mode 100644 index 000000000..f3809b7b7 --- /dev/null +++ b/test/controllers/rentals_controller_test.rb @@ -0,0 +1,246 @@ +require "test_helper" + +describe RentalsController do + describe "check-out" do + it "creates a rental given valid movie id and valid customer id and sufficient inventory" do + # params + initial_inventory = 3 + available_inventory = 3 + test_movie = movies(:m_1) + test_movie.inventory = initial_inventory + test_movie.available_inventory = available_inventory + test_movie.save + + test_customer = customers(:c_3) + test_customer.movies_checked_out_count = 0 + + rental_params = { + customer_id: test_customer.id, + movie_id: test_movie.id + } + + # need some comparison dates + checkout_date = Date.today + due_date = Date.today + 7 + + # the route + # it should update the rental database count + expect { + post checkout_path, params: rental_params + }.must_differ "Rental.count", 1 + + # returns JSON, status ok, and the new rental's due date + body = check_response(expected_type: Hash, expected_status: :ok) + expect(body["due_date"]).must_equal due_date.to_s + + # find the rental by new parameters customer id and movie id + new_rental = Rental.where(customer_id: rental_params[:customer_id], movie_id: rental_params[:movie_id]).first + # the dates we assign the new rental should be correct + expect(new_rental.checkout_date).must_equal checkout_date + expect(new_rental.due_date).must_equal due_date + + # find the movie and customer by new rental's id + movie = Movie.find_by(id: new_rental.movie_id) + customer = Customer.find_by(id: new_rental.customer_id) + # we should see the new rental in the customer.rentals list + # and movie.rentals list of the customer and movie assigned + expect(movie.rentals).must_include new_rental + expect(customer.rentals).must_include new_rental + + #verifies that movie inventory is unchanged + expect(movie.inventory).must_equal initial_inventory + + #verifies that movie available inventory decreased by 1 + expect(movie.available_inventory).must_equal initial_inventory - 1 + + #verifies that customer's movies_checked_out_count must increase by 1 + expect(customer.movies_checked_out_count).must_equal 1 + + end + + it "returns an error with valid movie id, valid customer id, but insufficient inventory" do + + # params + initial_inventory = 3 + available_inventory = 0 + test_movie = movies(:m_1) + test_movie.inventory = initial_inventory + test_movie.available_inventory = available_inventory + test_movie.save + + rental_params = { + customer_id: customers(:c_3).id, + movie_id: test_movie.id + } + + # the route + # it should update the rental database count + expect { + post checkout_path, params: rental_params + }.wont_differ "Rental.count" + + #returns JSON, status not_modified, and an error message + body = check_response(expected_type: Hash, expected_status: :bad_request) + expect(body.keys).must_include 'errors' + expect(body['errors'].keys).must_include 'movie' + + #verifies that movie inventory is unchanged + movie = Movie.find_by(id: test_movie.id) + expect(movie.inventory).must_equal initial_inventory + + #verifies that movie available inventory decreased by 1 + expect(movie.available_inventory).must_equal available_inventory + + end + + it "returns an error if given invalid movie id" do + # params + rental_params = { + customer_id: customers(:c_3).id, + movie_id: -1 + } + + # route + # check that the database rental count did not change + expect { + post checkout_path, params: rental_params + }.wont_differ "Rental.count" + + # returns JSON, status bad_request, and an error + body = check_response(expected_type: Hash, expected_status: :bad_request) + expect(body.keys).must_include 'errors' + expect(body['errors'].keys).must_include 'movie' + end + + it "returns an error if given invalid customer id" do + # params + rental_params = { + customer_id: -1, + movie_id: movies(:m_1).id + } + + # route + # check that the database rental count did not change + expect { + post checkout_path, params: rental_params + }.wont_differ "Rental.count" + + # returns JSON, status bad_request, and an error + body = check_response(expected_type: Hash, expected_status: :bad_request) + expect(body.keys).must_include 'errors' + expect(body['errors'].keys).must_include 'customer' + end + end + + describe "check-in" do + it "renders JSON and success when given valid movie id, customer id WITH that rental being checked out" do + # params + test_movie = movies(:m_2) + test_customer = customers(:c_1) + + test_customer.movies_checked_out_count = 1 + test_customer.save + + + rental_params = { + customer_id: test_customer.id, + movie_id: test_movie.id + } + + initial_available_inventory = test_movie.available_inventory + + # the route + # expect that it won't change rental count + expect { + post checkin_path, params: rental_params + }.wont_differ "Rental.count" + + # returns JSON, status bad_request, and ok + body = check_response(expected_type: Hash, expected_status: :ok) + + expect(Movie.find_by(id: test_movie.id).available_inventory).must_equal initial_available_inventory + 1 + + # checks that checkin date must be today + rental = Rental.find_by(id: body["id"]) + customer = Customer.find_by(id: rental.customer_id) + + expect(rental.checkin_date).must_equal Date.today + + expect(customer.movies_checked_out_count).must_equal 0 + + end + + it "renders JSON and success when given valid movie id, customer id WITHOUT that rental being checked out aka person returns a movie from another store" do + #r_1 is being rereturned + # params + test_movie = movies(:m_1) + test_customer = customers(:c_1) + rental_params = { + customer_id: test_customer.id, + movie_id: test_movie.id + } + + initial_available_inventory = test_movie.available_inventory + + initial_rentals = Rental.where(movie_id: test_movie.id, customer_id: test_customer.id).sort + + # the route + # expect that it won't change rental count + expect { + post checkin_path, params: rental_params + }.wont_differ "Rental.count" + + # returns JSON, status bad_request, and ok + body = check_response(expected_type: Hash, expected_status: :bad_request) + expect(body.keys).must_include 'errors' + expect(body['errors'].keys).must_include 'rental' + + # checks rental from movie perspective + expect(Movie.find_by(id: test_movie.id).available_inventory).must_equal initial_available_inventory + + + rentals = Rental.where(movie_id: test_movie.id, customer_id: test_customer.id).sort + + expect(rentals).must_equal initial_rentals + + end + + it "renders JSON and bad_request when given invalid movie id" do + # params + rental_params = { + customer_id: customers(:c_3).id, + movie_id: -1 + } + + # the route + # expect that it won't change rental count + expect { + post checkin_path, params: rental_params + }.wont_differ "Rental.count" + + # returns JSON, status bad_request, and ok + body = check_response(expected_type: Hash, expected_status: :bad_request) + expect(body.keys).must_include 'errors' + expect(body['errors'].keys).must_include 'movie' + end + + it "renders JSON and bad_request when given invalid customer id" do + # params + rental_params = { + customer_id: -1, + movie_id: movies(:m_1).id + } + + # the route + # expect that it won't change rental count + expect { + post checkin_path, params: rental_params + }.wont_differ "Rental.count" + + # returns JSON, status bad_request, and ok + body = check_response(expected_type: Hash, expected_status: :bad_request) + expect(body.keys).must_include 'errors' + expect(body['errors'].keys).must_include 'customer' + end + end +end diff --git a/test/fixtures/.keep b/test/fixtures/.keep new file mode 100644 index 000000000..e69de29bb diff --git a/test/fixtures/customers.yml b/test/fixtures/customers.yml new file mode 100644 index 000000000..05a79c245 --- /dev/null +++ b/test/fixtures/customers.yml @@ -0,0 +1,39 @@ +c_1: + name: Shelley Rocha + registered_at: Wed 29 Apr 2015 07:54:14 -0700 + address: Ap #292-5216 Ipsum Rd. + city: Hillsboro + state: OR + postal_code: 24309 + phone: (322) 510-8695 + movies_checked_out_count: 0 + +c_2: + name: Curran Stout + registered_at: Wed 16 Apr 2014 21:40:20 -0700 + address: Ap #658-1540 Erat Rd. + city: San Francisco + state: California + postal_code: 94267 + phone: (908) 949-6758 + movies_checked_out_count: 0 + +c_3: + name: Roanna Robinson + registered_at: Fri 28 Nov 2014 13:14:08 -0800 + address: Ap #561-4214 Eget St. + city: Harrisburg + state: PA + postal_code: 15867 + phone: (323) 336-1841 + movies_checked_out_count: 0 + +c_4: + name: Carolyn Chandler + registered_at: Fri 04 Jul 2014 11:05:11 -0700 + address: 133-8707 Arcu. Avenue + city: Fort Wayne + state: IN + postal_code: 73684 + phone: (234) 837-2886 + movies_checked_out_count: 0 \ No newline at end of file diff --git a/test/fixtures/files/.keep b/test/fixtures/files/.keep new file mode 100644 index 000000000..e69de29bb diff --git a/test/fixtures/movies.yml b/test/fixtures/movies.yml new file mode 100644 index 000000000..f4a34a49e --- /dev/null +++ b/test/fixtures/movies.yml @@ -0,0 +1,35 @@ + +m_1: + title: Blacksmith Of The Banished + overview: The unexciting life of a boy will be permanently altered as a strange woman enters his life. + release_date: 1979-01-18 + inventory: 10 + available_inventory: 10 + +m_2: + title: Savior Of The Curse + overview: The strange woman claims the boy has a DNA strain which very likely offers the key to cure several crippling diseases. Tests will have to be done but the key to save millions of lives is within the grasp of science through this DNA strain. Unsure what to think of all this and of this strange woman the boy hesitantly agrees to the proposal there's something exciting about this whole situation surely the right choice was made. + release_date: 2010-11-05 + inventory: 1 + available_inventory: 1 + +m_3: + title: Women Of Destruction + overview: But what if this strange woman is a con artist. Or what if everything told is completely true. How could an ordinary boy be relied upon in something this big. Time will tell. + release_date: 2006-10-01 + inventory: 1 + available_inventory: 1 + +m_4: + title: Robots Of Eternity + overview: The laid-back life of a woman is going in a different direction as a childhood friend enters her life. + release_date: 2007-10-10 + inventory: 7 + available_inventory: 7 + +m_5: + title: Rats And Strangers + overview: The childhood friend claims the woman is heir to an incredible fortune all paperwork has been taken care of already all there's left to do is sign them. Distrustful of both this situation and of this childhood friend the woman somewhat gladly agrees to the proposal but there's no time to waste a decision had to be made quickly. + release_date: 2012-09-25 + inventory: 2 + available_inventory: 2 \ No newline at end of file diff --git a/test/fixtures/rentals.yml b/test/fixtures/rentals.yml new file mode 100644 index 000000000..5a66fadb4 --- /dev/null +++ b/test/fixtures/rentals.yml @@ -0,0 +1,27 @@ +r_1: + checkout_date: 2010-11-05 + due_date: 2010-11-12 + customer: c_1 + movie: m_1 + checkin_date: 2019-11-11 + +r_2: + checkout_date: 2010-11-05 + due_date: 2010-11-12 + customer: c_1 + movie: m_2 + checkin_date: nil + +r_3: + checkout_date: 2010-11-05 + due_date: 2010-11-12 + customer: c_2 + movie: m_3 + checkin_date: 2010-11-15 + +r_4: + checkout_date: 2010-11-05 + due_date: 2010-11-12 + customer: c_3 + movie: m_3 + checkin_date: 2010-11-06 \ No newline at end of file diff --git a/test/integration/.keep b/test/integration/.keep new file mode 100644 index 000000000..e69de29bb diff --git a/test/mailers/.keep b/test/mailers/.keep new file mode 100644 index 000000000..e69de29bb diff --git a/test/models/.keep b/test/models/.keep new file mode 100644 index 000000000..e69de29bb diff --git a/test/models/customer_test.rb b/test/models/customer_test.rb new file mode 100644 index 000000000..c193cdc2e --- /dev/null +++ b/test/models/customer_test.rb @@ -0,0 +1,62 @@ +require "test_helper" + +describe Customer do + describe "validations" do + + before do + @valid_customer = Customer.first + end + + it "checks to see if fixtures are sticking" do + #also checks peripherally (sp) that the customers are valid + num = Customer.all.count + expect(num).must_equal 4 + end + + it "customer is invalid if name is missing" do + @valid_customer.name = "" + refute(@valid_customer.valid?) + end + + it "customer is invalid if registered_at (time of creation) is missing" do + @valid_customer.registered_at = "" + refute(@valid_customer.valid?) + end + + it "customer's registered_at must be instance of dateime" do + @valid_customer.registered_at = "bananas" + refute(@valid_customer.valid?) + end + + it "customer's registered_at must be instance of dateime" do + valid_date_time = DateTime.now + @valid_customer.registered_at = valid_date_time + assert(@valid_customer.valid?) + end + + it "customer is invalid if postal code is missing" do + @valid_customer.postal_code = "" + refute(@valid_customer.valid?) + end + + it "customer is invalid if phone number is missing" do + @valid_customer.phone = "" + refute(@valid_customer.valid?) + end + + end + + describe "relationships" do + it "customer rentals are verifiable" do + expected_rental = 2 #checked by hand based on yml + customer = customers(:c_1) + expect(customer.rentals.count).must_equal expected_rental + end + + it "customer without rentals does not have rentals" do + expected_rental = 0 + customer = customers(:c_4) + expect(customer.rentals.count).must_equal expected_rental + end + end +end diff --git a/test/models/movie_test.rb b/test/models/movie_test.rb new file mode 100644 index 000000000..996dd2966 --- /dev/null +++ b/test/models/movie_test.rb @@ -0,0 +1,120 @@ +require "test_helper" + +describe Movie do + describe "validations" do + + before do + @valid_movie = Movie.first + end + + it "checks to see if fixtures are sticking" do + num = Movie.all.count + expect(num).must_equal 5 + end + + it "movies are invalid if title is missing" do + @valid_movie.title = "" + refute(@valid_movie.valid?) + end + + it "movies are invalid if overview is missing" do + @valid_movie.overview = "" + refute(@valid_movie.valid?) + end + + it "movies are invalid if release_date is missing" do + @valid_movie.release_date = "" + refute(@valid_movie.valid?) + end + + it "movies are invalid if release_date is not a type of date" do + @valid_movie.release_date = "bananas" + refute(@valid_movie.valid?) + end + + it "movies are invalid if release_date is not a type of date" do + valid_date = Date.today + @valid_movie.release_date = valid_date + assert(@valid_movie.valid?) + end + + it "movies are invalid if inventory is missing" do + @valid_movie.inventory = nil + refute(@valid_movie.valid?) + end + + it "movies are invalid if inventory is not an integer" do + @valid_movie.inventory = "bananas" + refute(@valid_movie.valid?) + end + + it "movies are invalid if inventory is a negative integer" do + @valid_movie.inventory = -2 + refute(@valid_movie.valid?) + end + + it "movies are valid if inventory is a positive integer" do + @valid_movie.inventory = 1 + # binding.pry + assert(@valid_movie.valid?) + end + + it "movies are valid if inventory is zero" do + @valid_movie.inventory = 0 + assert(@valid_movie.valid?) + end + + # it "movies are invalid if available_inventory is missing" do + # @valid_movie.available_inventory = nil + # refute(@valid_movie.valid?) + # end + + # it "movies are invalid if available_inventory is not an integer" do + # @valid_movie.available_inventory = "bananas" + # refute(@valid_movie.valid?) + # end + + # it "movies are invalid if available_inventory is a negative integer" do + # @valid_movie.available_inventory = -2 + # refute(@valid_movie.valid?) + # end + + # it "movies are valid if available_inventory is a positive integer" do + # @valid_movie.available_inventory = 1 + # assert(@valid_movie.valid?) + # end + + # it "movies are valid if available_inventory is zero" do + # @valid_movie.available_inventory = 0 + # assert(@valid_movie.valid?) + # end + end + + describe "relationships" do + it "movie rentals are verifiable" do + expected_rental = 1 + movie = movies(:m_2) + expect(movie.rentals.count).must_equal expected_rental + end + + it "movie without rentals does not have rentals" do + expected_rental = 0 + movie = movies(:m_5) + expect(movie.rentals.count).must_equal expected_rental + end + end + + # describe "adjust_available_inventory" do + # it "set available inventory to match inventory" do + # # get a movie + # movie = movies(:m_2) + # # set available inventory to nil + # movie.available_inventory = nil + # # call method on the movie + # movie.adjust_available_inventory() + # # check that the inventory and available inventory match + # expect(movie.available_inventory).must_equal movie.inventory + # end + # end + +end diff --git a/test/models/rental_test.rb b/test/models/rental_test.rb new file mode 100644 index 000000000..aafa583e3 --- /dev/null +++ b/test/models/rental_test.rb @@ -0,0 +1,90 @@ +require "test_helper" + +describe Rental do + describe "validations" do + + before do + @valid_rental = Rental.first + end + + it "checks that fixtures are sticking" do + num = Rental.all.count + expect(num).must_equal 4 + end + + it "rental is invalid if due date is missing" do + @valid_rental.due_date = "" + refute(@valid_rental.valid?) + end + + it "rental is invalid if due date is not a type of date" do + @valid_rental.due_date = "bananas" + refute(@valid_rental.valid?) + end + + it "rental is valid if due date is type of date" do + valid_date = Date.today + @valid_rental.due_date = valid_date + assert(@valid_rental.valid?) + end + + it "rental is invalid if checkout date is missing" do + @valid_rental.checkout_date = "" + refute(@valid_rental.valid?) + end + + it "rental is invalid if checkout date is not a type of date" do + @valid_rental.checkout_date = "bananas" + refute(@valid_rental.valid?) + end + + it "rental is valid if checkout date is type of date" do + valid_date = Date.today + @valid_rental.checkout_date = valid_date + assert(@valid_rental.valid?) + end + + end + describe "relationship" do + before do + @valid_rental = Rental.first + end + + it "rental is invalid if customer is not associated" do + @valid_rental.customer_id = nil + refute(@valid_rental.valid?) + end + + it "rental is invalid if associated customer id is invalid" do + @valid_rental.customer_id = -5 + refute(@valid_rental.valid?) + end + + it "rental is invalid if movie is not associated" do + @valid_rental.movie_id = nil + refute(@valid_rental.valid?) + end + + it "rental is invalid if associated movie id is invalid" do + @valid_rental.movie_id = -1 + refute(@valid_rental.valid?) + end + + it "valid customer and valid movie are associated with a rental" do + + rental = rentals(:r_1) + + movie = Movie.find_by(id: rental.movie_id) + + customer = Customer.find_by(id: rental.customer_id) + + expect(movie).wont_be_nil + expect(customer).wont_be_nil + + end + + end + + #at some point, check that due date is in the future from checkout date + +end diff --git a/test/test_helper.rb b/test/test_helper.rb new file mode 100644 index 000000000..08ff032cb --- /dev/null +++ b/test/test_helper.rb @@ -0,0 +1,26 @@ +ENV['RAILS_ENV'] ||= 'test' +require_relative '../config/environment' + +require 'simplecov' +SimpleCov.start do + add_filter '/test/' +end + +require 'rails/test_help' +require 'minitest/rails' +require 'minitest/reporters' + +class ActiveSupport::TestCase + # Setup all fixtures in test/fixtures/*.yml for all tests in alphabetical order. + fixtures :all + + # Add more helper methods to be used by all tests here... + def check_response(expected_type:, expected_status: :ok) + must_respond_with expected_status + expect(response.header['Content-Type']).must_include 'json' + + body = JSON.parse(response.body) + expect(body).must_be_kind_of expected_type + return body + end +end diff --git a/tmp/.keep b/tmp/.keep new file mode 100644 index 000000000..e69de29bb diff --git a/vendor/.keep b/vendor/.keep new file mode 100644 index 000000000..e69de29bb