+
+
+
+
+
+
+
+ -
+ 1
+
+
class ApplicationController < ActionController::Base
+
+
+ -
+ 1
+
+
protect_from_forgery with: :exception
+
+
+ -
+ 1
+
+
before_action :find_user
+
+
+ -
+ 1
+
+
before_action :find_cart
+
+
+ -
+ 1
+
+
before_action :set_session_to_cart
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
#
+
+
+ -
+
+
+
# def render_404
+
+
+ -
+
+
+
# raise ActionController::RoutingError.new('Not Found')
+
+
+ -
+
+
+
# end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
private
+
+
+ -
+ 1
+
+
def find_user
+
+
+ -
+ 119
+
+
if session[:user_id]
+
+
+ -
+ 36
+
+
@user = User.find_by(id: session[:user_id])
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def find_cart
+
+
+ -
+ 121
+
+
if @user #find auth_user cart
+
+
+ -
+ 38
+
+
@pending_order = @user.find_pending_order
+
+
+ -
+ 38
+
+
session[:pending_order_id] = @pending_order.id if @pending_order
+
+
+ -
+ 38
+
+
if !(@pending_order) #make them one if they don't have
+
+
+ -
+
+
+
# @pending_order = Order.find_by(session_id: session[:session_id])
+
+
+ -
+ 5
+
+
@pending_order = Order.new(session_id: session[:session_id])
+
+
+ -
+ 5
+
+
@pending_order.user_id = @user.id
+
+
+ -
+ 5
+
+
@pending_order.save
+
+
+ -
+ 5
+
+
session[:pending_order_id] = @pending_order.id
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
else
+
+
+ -
+ 83
+
+
if session[:pending_order_id] #find guest's current cart
+
+
+ -
+ 4
+
+
@pending_order = Order.find(session[:pending_order_id])
+
+
+ -
+
+
+
else #make them one if they don't have
+
+
+ -
+
+
+
# @pending_order = new_pending_order(session[:session_id], nil)
+
+
+ -
+ 79
+
+
@pending_order = Order.new
+
+
+ -
+ 79
+
+
@pending_order.session_id = "guest_user"
+
+
+ -
+
+
+
# binding.pry
+
+
+ -
+ 79
+
+
@pending_order.save
+
+
+ -
+ 79
+
+
session[:pending_order_id] = @pending_order.id
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
end
+
+
+ -
+ 1
+
+
def set_session_to_cart
+
+
+ -
+ 119
+
+
@pending_order.session_id = session[:session_id]
+
+
+ -
+ 119
+
+
@pending_order.save
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
# def not_found
+
+
+ -
+
+
+
# raise ActionController::RoutingError.new('Not Found')
+
+
+ -
+
+
+
# redirect_to root_path
+
+
+ -
+
+
+
# end
+
+
+ -
+
+
+
# this one is harder to test
+
+
+ -
+
+
+
# def render_404
+
+
+ -
+
+
+
# # DPR: supposedly this will actually render a 404 page in production
+
+
+ -
+
+
+
# raise ActionController::RoutingError.new('Not Found')
+
+
+ -
+
+
+
# end
+
+
+ -
+
+
+
# this one just gives 404 status, easier to test
+
+
+ -
+ 1
+
+
def render_404
+
+
+ -
+ 18
+
+
render file: "/public/404.html", status: 404
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def new_pending_order(input_session_id, input_user_id)
+
+
+ -
+
+
+
new_order = Order.new
+
+
+ -
+
+
+
new_order.session_id = input_session_id
+
+
+ -
+
+
+
new_order.user_id = input_user_id
+
+
+ -
+
+
+
return new_order
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
class BillingDataController < ApplicationController
+
+
+ -
+ 1
+
+
before_action :find_billing, only: [:permission, :edit, :update, :show]
+
+
+ -
+ 1
+
+
before_action :permission, only: [:edit, :update, :show]
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def index
+
+
+ -
+ 2
+
+
render render_404
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def show
+
+
+ -
+
+
+
unless @billing_data
+
+
+ -
+
+
+
flash[:error] = "Billing data not found"
+
+
+ -
+
+
+
redirect_to root_path
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def edit
+
+
+ -
+
+
+
unless @billing_data
+
+
+ -
+
+
+
redirect_to root_path
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def update
+
+
+ -
+
+
+
if @billing_data.update_attributes billing_data_params
+
+
+ -
+
+
+
redirect_to billing_datum_path(@billing_data.id)
+
+
+ -
+
+
+
@billing_data.user = @user
+
+
+ -
+
+
+
else
+
+
+ -
+
+
+
flash[:result_text] = "Didn't save. Make sure your fields are complete!"
+
+
+ -
+
+
+
render :edit
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def new
+
+
+ -
+ 1
+
+
@user = session[:user_id]
+
+
+ -
+ 1
+
+
@billing_data = BillingDatum.new
+
+
+ -
+
+
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def create
+
+
+ -
+ 2
+
+
@billing_data = BillingDatum.new billing_data_params
+
+
+ -
+ 1
+
+
@billing_data.user = @user
+
+
+ -
+ 1
+
+
if @billing_data.save
+
+
+ -
+ 1
+
+
flash[:result_text] = "Does this look right?"
+
+
+ -
+ 1
+
+
redirect_to billing_datum_path(@billing_data.id)
+
+
+ -
+ 1
+
+
save_billing(@billing_data.id)
+
+
+ -
+
+
+
else
+
+
+ -
+
+
+
flash.now[:error] = "Could not create Billing Data"
+
+
+ -
+
+
+
render :new
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
private
+
+
+ -
+ 1
+
+
def find_billing
+
+
+ -
+ 3
+
+
@billing_data = BillingDatum.find_by(id: params[:id].to_i)
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def save_billing(id)
+
+
+ -
+ 1
+
+
@user = session[:user_id]
+
+
+ -
+ 1
+
+
if @user
+
+
+ -
+
+
+
user = User.find_by(id: @user)
+
+
+ -
+
+
+
user.billing_datum_id = id
+
+
+ -
+
+
+
user.save
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
@pending_order.billing_datum_id = id
+
+
+ -
+ 1
+
+
@pending_order.save
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def billing_data_params
+
+
+ -
+ 2
+
+
return params.require(:billing_datum).permit(:email, :mailing_address, :credit_card_name, :credit_card_number, :credit_card_cvv, :billing_zip_code, :expiration_date)
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def permission
+
+
+ -
+ 3
+
+
unless session[:user_id] == @billing_data.user_id || @pending_order.billing_datum_id == @billing_data.id
+
+
+ -
+ 2
+
+
render render_404
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
class CategoriesController < ApplicationController
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def index
+
+
+ -
+ 1
+
+
@categories = Category.order(:id)
+
+
+ -
+ 1
+
+
@user = session[:user_id]
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
#show all products in a specific category
+
+
+ -
+ 1
+
+
def show
+
+
+ -
+ 2
+
+
@category = Category.find_by(id: params[:id])
+
+
+ -
+ 2
+
+
unless @category
+
+
+ -
+ 1
+
+
flash[:error] = "Category does not exist"
+
+
+ -
+ 1
+
+
redirect_to root_path
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
#makes changes to a category that's already created
+
+
+ -
+ 1
+
+
def edit
+
+
+ -
+
+
+
@category = Category.find_by(id: params[:id])
+
+
+ -
+
+
+
unless @category
+
+
+ -
+
+
+
redirect_to categories_path
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def update
+
+
+ -
+ 1
+
+
@category = Category.find_by(id: params[:id])
+
+
+ -
+ 1
+
+
redirect_to categories_path unless @category
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
if @category.update_attributes category_params
+
+
+ -
+ 1
+
+
flash[:success] = "Category edited successfully"
+
+
+ -
+ 1
+
+
redirect_to categories_path
+
+
+ -
+
+
+
else
+
+
+ -
+
+
+
flash.now[:error] = "Category not edited successfully"
+
+
+ -
+
+
+
render :edit
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
#sign-in user can create a new category
+
+
+ -
+ 1
+
+
def new
+
+
+ -
+ 1
+
+
@category = Category.new
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def create
+
+
+ -
+ 2
+
+
@category = Category.new category_params
+
+
+ -
+ 2
+
+
if @category.save
+
+
+ -
+ 1
+
+
puts "SUCCESS"
+
+
+ -
+ 1
+
+
flash[:success] = "Category created!"
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
redirect_to categories_path
+
+
+ -
+
+
+
+
+ -
+
+
+
else
+
+
+ -
+ 1
+
+
puts "FAIL"
+
+
+ -
+ 1
+
+
flash.now[:error] = "Could not create category"
+
+
+ -
+ 1
+
+
render :new
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
#signed-in user can delete a category
+
+
+ -
+ 1
+
+
def destroy
+
+
+ -
+ 1
+
+
@category = Category.find_by(id: params[:id])
+
+
+ -
+ 1
+
+
@category.destroy
+
+
+ -
+ 1
+
+
flash[:status] = :success
+
+
+ -
+ 1
+
+
flash[:result_text] = "Successfully destroyed category"
+
+
+ -
+ 1
+
+
redirect_to categories_path
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
private
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def category_params
+
+
+ -
+ 3
+
+
return params.require(:category).permit(:name)
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
class ItemsController < ApplicationController
+
+
+ -
+ 1
+
+
def index
+
+
+ -
+ 1
+
+
@item = Item.order(:id)
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def show
+
+
+ -
+ 2
+
+
@item = Item.find_by(id: params[:id])
+
+
+ -
+ 2
+
+
puts params[:id]
+
+
+ -
+
+
+
+
+ -
+
+
+
# render_404 unless @item
+
+
+ -
+
+
+
+
+ -
+ 2
+
+
unless @item
+
+
+ -
+ 1
+
+
flash[:error] = "Item not found"
+
+
+ -
+ 1
+
+
redirect_to root_path
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def new
+
+
+ -
+ 1
+
+
@item = Item.new
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def create
+
+
+ -
+
+
+
# if !(Product.find_by(id: params[:id]))
+
+
+ -
+
+
+
# render_404
+
+
+ -
+
+
+
# return
+
+
+ -
+
+
+
# end
+
+
+ -
+ 2
+
+
@item = Item.new(product_id: params[:id])
+
+
+ -
+
+
+
# binding.pry
+
+
+ -
+
+
+
+
+ -
+ 2
+
+
if @item.save
+
+
+ -
+
+
+
# puts "success"
+
+
+ -
+ 1
+
+
flash[:success] = "#{Product.find(params[:id]).name} item added successfully"
+
+
+ -
+ 1
+
+
redirect_back(fallback_location: root_path)
+
+
+ -
+
+
+
else
+
+
+ -
+
+
+
# puts "fail"
+
+
+ -
+
+
+
# puts @item.errors.messages
+
+
+ -
+ 1
+
+
flash.now[:error] = "Item not added to Product # #{params[:id]}"
+
+
+ -
+ 1
+
+
redirect_to profile_path, status: 400
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def destroy
+
+
+ -
+
+
+
+
+ -
+ 2
+
+
if Item.where(product_id: params[:id]).length > 0
+
+
+ -
+ 1
+
+
@item = Item.where(product_id: params[:id])
+
+
+ -
+ 1
+
+
@item.last.destroy
+
+
+ -
+ 1
+
+
redirect_back(fallback_location: root_path)
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
flash[:result_text] = "1 #{Product.find(params[:id]).name} removed from inventory!"
+
+
+ -
+
+
+
+
+ -
+
+
+
else
+
+
+ -
+ 1
+
+
flash[:result_text] = "There are no more items to remove"
+
+
+ -
+ 1
+
+
redirect_to profile_path, status: 400
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
private
+
+
+ -
+
+
+
# GM: This never gets called in this controller
+
+
+ -
+
+
+
# def item_params
+
+
+ -
+
+
+
# return params.require(:items).permit(:shipping_status, :purchase_status, :product_id, :order_id)
+
+
+ -
+
+
+
# end
+
+
+ -
+
+
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
class MainPageController < ApplicationController
+
+
+ -
+ 1
+
+
def index
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
class ProductsController < ApplicationController
+
+
+ -
+ 1
+
+
before_action :find_product, except: [:index, :new, :create]
+
+
+ -
+ 1
+
+
before_action :permission, except: [:show, :index, :add_to_order]
+
+
+ -
+ 1
+
+
before_action :owns, only: [:edit, :update]
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def index
+
+
+ -
+ 2
+
+
@products = Product.show_available
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def show
+
+
+ -
+ 4
+
+
unless @product
+
+
+ -
+ 1
+
+
render_404
+
+
+ -
+
+
+
end
+
+
+ -
+ 4
+
+
@current_user_is_not_product_owner = session[:user_id] != @product.user_id
+
+
+ -
+
+
+
+
+ -
+ 3
+
+
@user = @product.user
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def edit
+
+
+ -
+ 1
+
+
if @product.id.nil?
+
+
+ -
+
+
+
render render_404
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def update
+
+
+ -
+ 1
+
+
unless @product
+
+
+ -
+
+
+
render render_404
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
@product.update_attributes product_params
+
+
+ -
+ 1
+
+
if @product.save
+
+
+ -
+ 1
+
+
flash[:status] = :success
+
+
+ -
+ 1
+
+
flash[:result_text] = "Successfully updated #{@product.name}"
+
+
+ -
+ 1
+
+
redirect_to profile_path(@user)
+
+
+ -
+
+
+
else
+
+
+ -
+
+
+
flash[:status] = :failure
+
+
+ -
+
+
+
flash[:result_text] = "Could not update #{@product.name}"
+
+
+ -
+
+
+
flash.now[:messages] = @product.errors.message
+
+
+ -
+
+
+
render :edit, status: :not_found
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def new
+
+
+ -
+ 1
+
+
@product = Product.new
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def create
+
+
+ -
+
+
+
# as long as this exists: session[:user_id]
+
+
+ -
+
+
+
# product_params.user_id = session[:user_id]
+
+
+ -
+
+
+
+
+ -
+ 5
+
+
@product = Product.new(product_params)
+
+
+ -
+ 4
+
+
if @product.save
+
+
+ -
+ 1
+
+
flash[:status] = :success
+
+
+ -
+ 1
+
+
flash[:result_text] = "Successfully created product #{@product.id} #{@product.name}"
+
+
+ -
+ 1
+
+
redirect_to profile_path(@user)
+
+
+ -
+
+
+
else
+
+
+ -
+ 3
+
+
flash[:status] = :failure
+
+
+ -
+ 3
+
+
flash[:result_text] = "Unable to create a product"
+
+
+ -
+ 3
+
+
flash[:messages] = @product.errors.messages
+
+
+ -
+ 3
+
+
render :new, status: :bad_request
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def add_to_order
+
+
+ -
+
+
+
# binding.pry
+
+
+ -
+ 1
+
+
@cart_entry = @pending_order.check_for_duplicates(@product.id)
+
+
+ -
+ 1
+
+
if @cart_entry
+
+
+ -
+ 1
+
+
@cart_entry.quantity += params[:quantity].to_i
+
+
+ -
+
+
+
else
+
+
+ -
+
+
+
@cart_entry = create_entry(params[:quantity])
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
if @cart_entry.save
+
+
+ -
+
+
+
flash[:status] = :success
+
+
+ -
+
+
+
flash[:result_text] = "Successfully added to your cart!"
+
+
+ -
+
+
+
# redirect_to order_path(@pending_order)
+
+
+ -
+
+
+
redirect_to product_path(@product.id)
+
+
+ -
+
+
+
else
+
+
+ -
+ 1
+
+
flash[:result_text] = "Could not add that product to your cart"
+
+
+ -
+ 1
+
+
flash[:messages] = @cart_entry.errors
+
+
+ -
+ 1
+
+
redirect_to product_path(@product.id)
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def change_visibility
+
+
+ -
+ 1
+
+
if @product.visibility == false
+
+
+ -
+
+
+
@product.visibility = true
+
+
+ -
+
+
+
@product.save
+
+
+ -
+
+
+
flash[:result_text] = "Your product is now visible in browsing"
+
+
+ -
+
+
+
redirect_back(fallback_location: products_path)
+
+
+ -
+
+
+
+
+ -
+
+
+
else
+
+
+ -
+
+
+
# binding.pry
+
+
+ -
+ 1
+
+
@product.visibility = false
+
+
+ -
+ 1
+
+
@product.save
+
+
+ -
+ 1
+
+
flash[:result_text] = "Your product is no longer visibile in browsing"
+
+
+ -
+ 1
+
+
redirect_back(fallback_location: products_path)
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
private
+
+
+ -
+ 1
+
+
def product_params
+
+
+ -
+ 6
+
+
params.require(:product).permit(:name, :description, :product_photo, :user_id, :price, category_ids:[])
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def entry_params
+
+
+ -
+
+
+
# params.require(:entry).permit(:quantity)
+
+
+ -
+
+
+
# return
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def find_product
+
+
+ -
+ 13
+
+
@product = Product.find_by(id: params[:id].to_i)
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def permission
+
+
+ -
+ 16
+
+
unless @user
+
+
+ -
+ 4
+
+
render render_404
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def owns
+
+
+ -
+ 5
+
+
unless @product && @user.id == @product.user_id
+
+
+ -
+ 3
+
+
render render_404
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def create_entry(input_quantity)
+
+
+ -
+
+
+
entry = OrderProduct.new
+
+
+ -
+
+
+
entry.product_id = @product.id
+
+
+ -
+
+
+
entry.order_id = @pending_order.id
+
+
+ -
+
+
+
entry.quantity = input_quantity.to_i
+
+
+ -
+
+
+
return entry
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
class ReviewsController < ApplicationController
+
+
+ -
+
+
+
# def index
+
+
+ -
+
+
+
# @reviews = Review.order(:id)
+
+
+ -
+
+
+
# end
+
+
+ -
+
+
+
+
+ -
+
+
+
# def show
+
+
+ -
+
+
+
# @review = Review.find_by(id: params[:id].to_i)
+
+
+ -
+
+
+
#
+
+
+ -
+
+
+
# unless @review
+
+
+ -
+
+
+
# flash[:error] = "Review not found"
+
+
+ -
+
+
+
# redirect_to root_path
+
+
+ -
+
+
+
# end
+
+
+ -
+
+
+
# end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
# def edit
+
+
+ -
+
+
+
# @review = Review.find_by(id: params[:id].to_i)
+
+
+ -
+
+
+
# unless @review
+
+
+ -
+
+
+
# redirect_to root_path
+
+
+ -
+
+
+
# end
+
+
+ -
+
+
+
# end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
# def update
+
+
+ -
+
+
+
# @review = Review.find_by(id: params[:id].to_i)
+
+
+ -
+
+
+
#
+
+
+ -
+
+
+
# if @review.update_attributes review_params
+
+
+ -
+
+
+
# redirect_to root_path
+
+
+ -
+
+
+
# else
+
+
+ -
+
+
+
# render :edit
+
+
+ -
+
+
+
# end
+
+
+ -
+
+
+
# end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def new
+
+
+ -
+ 1
+
+
@review = Review.new
+
+
+ -
+ 1
+
+
@product = Product.find_by(id: params[:product_id].to_i)
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def create
+
+
+ -
+ 1
+
+
@review = Review.new(review_params)
+
+
+ -
+
+
+
# puts "*********"
+
+
+ -
+
+
+
# puts "review = #{@review}"
+
+
+ -
+
+
+
# puts "review valid? = #{@review.valid?}"
+
+
+ -
+
+
+
#
+
+
+ -
+
+
+
# puts "*********"
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
if @review.save
+
+
+ -
+ 1
+
+
redirect_to product_path(review_params[:product_id])
+
+
+ -
+
+
+
else
+
+
+ -
+
+
+
flash.now[:error] = "Could not create Review"
+
+
+ -
+
+
+
render :new
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
# def destroy
+
+
+ -
+
+
+
# @review = Review.find_by(id: params[:id].to_i)
+
+
+ -
+
+
+
# @review.destroy
+
+
+ -
+
+
+
# redirect_to root_path
+
+
+ -
+
+
+
# end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
private
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def review_params
+
+
+ -
+ 2
+
+
return params.require(:review).permit(:user_id, :product_id, :text, :rating)
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
class SessionsController < ApplicationController
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def login_form
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def not_logged_in
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def login
+
+
+ -
+
+
+
username = params[:username]
+
+
+ -
+
+
+
if username && user = User.find_by(user: username)
+
+
+ -
+
+
+
session[:user_id] = user.id
+
+
+ -
+
+
+
flash[:status] = :success
+
+
+ -
+
+
+
flash[:result_text] = "Successfully logged in as #{@user.username}"
+
+
+ -
+
+
+
redirect_to root_path
+
+
+ -
+
+
+
else
+
+
+ -
+
+
+
user = User.new(username: username)
+
+
+ -
+
+
+
if user.save
+
+
+ -
+
+
+
session[:user_id] = @user.id
+
+
+ -
+
+
+
flash[:success] = "Successfully created user: #{@user.username}"
+
+
+ -
+
+
+
redirect_to root_path
+
+
+ -
+
+
+
else
+
+
+ -
+
+
+
flash.now[:status] = :failure
+
+
+ -
+
+
+
flash.now[:result_text] = "Could not log in"
+
+
+ -
+
+
+
flash.now[:messages] = users.errors.messages
+
+
+ -
+
+
+
render "login_form", status: bad_request
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def create
+
+
+ -
+ 30
+
+
@auth_hash = request.env['omniauth.auth']
+
+
+ -
+
+
+
# ap @auth_hash
+
+
+ -
+
+
+
+
+ -
+ 30
+
+
@user = User.find_by(uid: @auth_hash['uid'], provider: @auth_hash['provider'])
+
+
+ -
+
+
+
+
+ -
+ 30
+
+
if @user
+
+
+ -
+ 29
+
+
session[:user_id] = @user.id
+
+
+ -
+ 29
+
+
flash[:result_text] = "#{@user.username} has been logged in!"
+
+
+ -
+
+
+
+
+ -
+
+
+
else
+
+
+ -
+ 1
+
+
@user = User.new(uid: @auth_hash['uid'], provider: @auth_hash['provider'], email: @auth_hash['info']['email'], username: @auth_hash['info']['nickname'])
+
+
+ -
+ 1
+
+
@user.save
+
+
+ -
+ 1
+
+
if @user.save
+
+
+ -
+ 1
+
+
session[:user_id] = @user.id
+
+
+ -
+ 1
+
+
flash[:result_text] = "#{@user.username} has been logged in!"
+
+
+ -
+
+
+
else
+
+
+ -
+
+
+
flash[:result_text] = "Unable to save user!"
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+ 30
+
+
redirect_to root_path
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def logout
+
+
+ -
+ 1
+
+
session[:user_id] = nil
+
+
+ -
+ 1
+
+
flash[:result_text] = "Successfully logged out!"
+
+
+ -
+ 1
+
+
session[:pending_order_id] = nil
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
redirect_to root_path
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
module ApplicationHelper
+
+
+ -
+ 1
+
+
def readable_date(date)
+
+
+ -
+
+
+
("<span class='date'>" + date.strftime("%A, %b %d") + "</span>").html_safe
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def readable_money(money)
+
+
+ -
+
+
+
#TODO: helper method that will format cost and revenue appropriate
+
+
+ -
+
+
+
#10 => $10.00
+
+
+ -
+
+
+
#10.0 => $10.00
+
+
+ -
+
+
+
#10.00 => $10.00
+
+
+ -
+
+
+
#10.3333 => $10.33
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
module BillingDataHelper
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
module CategoriesHelper
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
module ItemsHelper
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
module MainPageHelper
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
module OrderProductsHelper
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
module OrdersHelper
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
module ProductsHelper
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
module ReviewsHelper
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
module SessionsHelper
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
module UsersHelper
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
class ApplicationRecord < ActiveRecord::Base
+
+
+ -
+ 1
+
+
self.abstract_class = true
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
class BillingDatum < ApplicationRecord
+
+
+ -
+ 1
+
+
has_many :orders
+
+
+ -
+ 1
+
+
belongs_to :user, optional: true
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
validates :email, presence: true
+
+
+ -
+ 1
+
+
validates :mailing_address, presence: true
+
+
+ -
+ 1
+
+
validates :credit_card_name, presence: true
+
+
+ -
+ 1
+
+
validates :credit_card_number, presence: true
+
+
+ -
+ 1
+
+
validates :credit_card_cvv, presence: true
+
+
+ -
+ 1
+
+
validates :billing_zip_code, presence: true
+
+
+ -
+ 1
+
+
validates :expiration_date, presence: true
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
class Category < ApplicationRecord
+
+
+ -
+
+
+
# add through: to associate join table
+
+
+ -
+ 1
+
+
has_and_belongs_to_many :products, through: :categories_products
+
+
+ -
+ 1
+
+
validates :name, presence: true
+
+
+ -
+ 1
+
+
validates :name, uniqueness: true
+
+
+ -
+
+
+
+
+ -
+
+
+
# before_validation :fix_category
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
# private
+
+
+ -
+
+
+
# def fix_category
+
+
+ -
+
+
+
# if self.category
+
+
+ -
+
+
+
# self.category = self.category.downcase.singularize
+
+
+ -
+
+
+
# end
+
+
+ -
+
+
+
# end
+
+
+ -
+
+
+
+
+ -
+
+
+
#TODO Julia removed the below method because it is a duplicate of a method in the product class (where it belongs)
+
+
+ -
+
+
+
# def show_available
+
+
+ -
+
+
+
# Product.where(visibility: true)
+
+
+ -
+
+
+
# end
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
class Item < ApplicationRecord
+
+
+ -
+ 1
+
+
belongs_to :product
+
+
+ -
+ 1
+
+
belongs_to :order, optional: true
+
+
+ -
+ 1
+
+
validates :shipping_status, inclusion: {in: [true,false]}
+
+
+ -
+ 1
+
+
validates :purchase_status, inclusion: {in: [true,false]}
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def purchase(input_order_id)
+
+
+ -
+ 2
+
+
if self.purchase_status != false
+
+
+ -
+
+
+
self.errors[:order_id] << "does not match item. This items is tied to order #{self.order_id}, you were trying to ship it for #{input_order_id}."
+
+
+ -
+
+
+
return false
+
+
+ -
+
+
+
else
+
+
+ -
+ 2
+
+
self.order_id = input_order_id
+
+
+ -
+ 2
+
+
self.purchase_status = true
+
+
+ -
+ 2
+
+
return true
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def ship(input_order_id)
+
+
+ -
+
+
+
if self.purchase_status != true
+
+
+ -
+
+
+
self.errors[:purchase_status] << "Cannot ship an unpurchased item. Please make sure item is purchased before shipping."
+
+
+ -
+
+
+
return false
+
+
+ -
+
+
+
elsif input_order_id != self.order_id
+
+
+ -
+
+
+
self.errors[:order_id] << "does not match item. This items is tied to order #{self.order_id}, you were trying to ship it for #{input_order_id}."
+
+
+ -
+
+
+
return false
+
+
+ -
+
+
+
else
+
+
+ -
+
+
+
self.purchase_status = true
+
+
+ -
+
+
+
return true
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
class Order < ApplicationRecord
+
+
+ -
+ 1
+
+
belongs_to :billing_datum, optional: true
+
+
+ -
+ 1
+
+
has_many :order_products
+
+
+ -
+ 1
+
+
has_many :items
+
+
+ -
+ 1
+
+
belongs_to :user, optional: true
+
+
+ -
+ 1
+
+
validates :session_id, presence: true
+
+
+ -
+ 1
+
+
validates :order_status, presence: true
+
+
+ -
+ 1
+
+
validate :has_billing_datum?
+
+
+ -
+ 1
+
+
validate :has_valid_order_status?
+
+
+ -
+
+
+
# validate :has_valid_order_products?
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def sub_total
+
+
+ -
+ 12
+
+
entries = self.order_products
+
+
+ -
+ 12
+
+
sub_total = 0
+
+
+ -
+ 12
+
+
entries.each do |entry|
+
+
+ -
+ 12
+
+
unit_price = entry.product.price
+
+
+ -
+ 12
+
+
entry_cost = unit_price * entry.quantity
+
+
+ -
+ 12
+
+
sub_total += entry_cost
+
+
+ -
+
+
+
end
+
+
+ -
+ 12
+
+
return sub_total
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def tax_amount
+
+
+ -
+ 6
+
+
sub_total = self.sub_total
+
+
+ -
+ 6
+
+
tax = sub_total * 0.0887
+
+
+ -
+ 6
+
+
return tax
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def taxed_total
+
+
+ -
+ 4
+
+
tax = self.tax_amount
+
+
+ -
+ 4
+
+
sub_total = self.sub_total
+
+
+ -
+ 4
+
+
return sub_total + tax
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def has_billing_datum?
+
+
+ -
+ 206
+
+
if self.order_status != "pending"
+
+
+ -
+ 2
+
+
self.errors[:billing_datum_id] << "is missing. Please entire valid billing information." if !(self.billing_datum_id)
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def has_valid_order_status?
+
+
+ -
+ 206
+
+
valid_statuses = ["pending", "paid", "shipped", "canceled"]
+
+
+ -
+ 206
+
+
if !(valid_statuses.include? self.order_status)
+
+
+ -
+
+
+
self.errors[:order_status] << "is invalid."
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def has_invalid_entries?
+
+
+ -
+
+
+
invalid_entries = []
+
+
+ -
+
+
+
order_products = self.order_products
+
+
+ -
+
+
+
order_products.each do |entry|
+
+
+ -
+
+
+
invalid_entries << entry if !(entry.valid?)
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
if invalid_entries.first != nil
+
+
+ -
+
+
+
self.errors[:entries] << "are invalid."
+
+
+ -
+
+
+
#TODO: Julia took out below line. Does not seem needed. Also, sytnax is incorrect- does not use variable entry in ternary.
+
+
+ -
+
+
+
+
+ -
+
+
+
invalid_entries.each {|entry| self.errors[:entries] << entry.errors}
+
+
+ -
+
+
+
+
+ -
+
+
+
return true #is NOT a valid order
+
+
+ -
+
+
+
else
+
+
+ -
+
+
+
return false #is a valid order
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def entries_by_product
+
+
+ -
+ 2
+
+
entries_by_product = self.order_products.group_by {|entry| entry.product_id}
+
+
+ -
+ 1
+
+
return entries_by_product
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
#used in the controller method add_to_order.
+
+
+ -
+
+
+
#If there is already a productorder in the cart that has the same product_id, this method will return the product_order object. If there is no duplicate, it will return false.
+
+
+ -
+ 1
+
+
def check_for_duplicates(input_product_id)
+
+
+ -
+ 1
+
+
entries = self.entries_by_product
+
+
+ -
+ 1
+
+
if entries[input_product_id]
+
+
+ -
+ 1
+
+
return entries[input_product_id][0]
+
+
+ -
+
+
+
else
+
+
+ -
+
+
+
return false
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
#gets called if check_for_duplicates returns true
+
+
+ -
+
+
+
#TODO: Julia removed below method. Does not appear to be applied in any part of the project.
+
+
+ -
+
+
+
# def combine_entries(entry_id)
+
+
+ -
+
+
+
# entries = self.entries_by_product
+
+
+ -
+
+
+
# product = entry.product
+
+
+ -
+
+
+
# end
+
+
+ -
+
+
+
+
+ -
+
+
+
#TODO: Julia removed below method. Does not appear to be applied in any part of the project. Also repeats the logic of the above has_invalid_entries? method above
+
+
+ -
+
+
+
# def has_valid_order_products
+
+
+ -
+
+
+
# entries = self.order_products
+
+
+ -
+
+
+
# entries.each { |entry| return false if !(entry.valid?)}
+
+
+ -
+
+
+
# return true
+
+
+ -
+
+
+
# end
+
+
+ -
+
+
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
class OrderProduct < ApplicationRecord
+
+
+ -
+ 1
+
+
belongs_to :order
+
+
+ -
+ 1
+
+
belongs_to :product
+
+
+ -
+ 1
+
+
validates :quantity, presence: true
+
+
+ -
+ 1
+
+
validates :quantity, numericality: { only_integer: true }
+
+
+ -
+ 1
+
+
validates :quantity, numericality: { greater_than: 0 }
+
+
+ -
+ 1
+
+
validate :items_available?
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def items_available?
+
+
+ -
+ 8
+
+
product = self.product
+
+
+ -
+ 8
+
+
quantity = self.quantity
+
+
+ -
+ 8
+
+
return false if !(product.num_available)
+
+
+ -
+ 8
+
+
if product.num_available >= quantity
+
+
+ -
+ 5
+
+
return true
+
+
+ -
+
+
+
else
+
+
+ -
+ 3
+
+
self.errors[:quantity] << "is invalid. There are currently only #{product.num_available} of #{product.name}. Please update your order."
+
+
+ -
+
+
+
end
+
+
+ -
+ 3
+
+
return false
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def pending?
+
+
+ -
+ 27
+
+
order = self.order
+
+
+ -
+
+
+
# binding.pry
+
+
+ -
+ 27
+
+
if order.order_status == "pending"
+
+
+ -
+ 18
+
+
return true
+
+
+ -
+
+
+
else
+
+
+ -
+ 9
+
+
return false
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def paid?
+
+
+ -
+ 3
+
+
order = self.order
+
+
+ -
+ 3
+
+
if order.order_status == "paid"
+
+
+ -
+ 3
+
+
return true
+
+
+ -
+
+
+
else
+
+
+ -
+
+
+
return false
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def canceled?
+
+
+ -
+ 2
+
+
order = self.order
+
+
+ -
+ 2
+
+
if order.order_status == "canceled"
+
+
+ -
+
+
+
return true
+
+
+ -
+
+
+
else
+
+
+ -
+ 2
+
+
return false
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def shipped?
+
+
+ -
+ 3
+
+
order = self.order
+
+
+ -
+ 3
+
+
if order.order_status == "shipped"
+
+
+ -
+
+
+
return true
+
+
+ -
+
+
+
else
+
+
+ -
+ 3
+
+
return false
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def subtotal
+
+
+ -
+ 3
+
+
return (self.quantity) * (self.product.price)
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def items_shipped?
+
+
+ -
+ 4
+
+
order = self.order
+
+
+ -
+ 4
+
+
entry_items = []
+
+
+ -
+ 4
+
+
if order
+
+
+ -
+ 4
+
+
order.items.each do |item|
+
+
+ -
+ 2
+
+
entry_items << item if item.product_id == product.id
+
+
+ -
+
+
+
end
+
+
+ -
+ 4
+
+
if entry_items
+
+
+ -
+ 4
+
+
entry_items.each do |item|
+
+
+ -
+ 2
+
+
return true if item.shipping_status #if item has been shipped, return true
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+ 4
+
+
return false
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
class Product < ApplicationRecord
+
+
+ -
+ 1
+
+
has_many :reviews
+
+
+ -
+ 1
+
+
has_many :items
+
+
+ -
+ 1
+
+
belongs_to :user
+
+
+ -
+
+
+
# add through: to associate join table
+
+
+ -
+ 1
+
+
has_and_belongs_to_many :categories, through: :categories_products
+
+
+ -
+ 1
+
+
validates :user_id, presence: true
+
+
+ -
+ 1
+
+
validates :name, uniqueness: true
+
+
+ -
+ 1
+
+
validates :name, presence: true
+
+
+ -
+ 1
+
+
validates :price, presence: true
+
+
+ -
+
+
+
# validates :category_id, presence: true
+
+
+ -
+ 1
+
+
validates :price, numericality: {greater_than: 0}
+
+
+ -
+ 1
+
+
validates :visibility, inclusion: {in: [true,false]}
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def num_available
+
+
+ -
+ 21
+
+
available = available_items
+
+
+ -
+ 21
+
+
if available
+
+
+ -
+ 14
+
+
return available.length
+
+
+ -
+
+
+
else
+
+
+ -
+ 7
+
+
return 0
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def available_items
+
+
+ -
+ 23
+
+
available = []
+
+
+ -
+ 23
+
+
self.items.each do |item|
+
+
+ -
+ 69
+
+
available << item if !(item.shipping_status) && !(item.purchase_status)
+
+
+ -
+
+
+
end
+
+
+ -
+ 23
+
+
return available if available.first != nil
+
+
+ -
+ 7
+
+
return false
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
def self.show_available
+
+
+ -
+ 4
+
+
Product.where(visibility: true)
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
class Review < ApplicationRecord
+
+
+ -
+ 1
+
+
belongs_to :user, optional: true
+
+
+ -
+ 1
+
+
belongs_to :product
+
+
+ -
+ 1
+
+
validates :rating, numericality: { only_integer: true, greater_than: 0, less_than: 6 }
+
+
+ -
+ 1
+
+
validates :rating, presence: true
+
+
+ -
+
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
require 'test_helper'
+
+
+ -
+
+
+
+
+ -
+
+
+
#IF IT'S COMMENTED OUT, IT'S GOT ERRORS
+
+
+ -
+ 1
+
+
describe BillingDataController do
+
+
+ -
+ 1
+
+
describe "Index" do
+
+
+ -
+ 1
+
+
it "should not show a list of all billing data" do
+
+
+ -
+ 1
+
+
get billing_data_path
+
+
+ -
+ 1
+
+
must_respond_with :not_found
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe "creating new billing information" do
+
+
+ -
+ 1
+
+
it "it should produce a form for entering payment information" do
+
+
+ -
+ 1
+
+
get new_billing_datum_path
+
+
+ -
+ 1
+
+
must_respond_with :success
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "should create new billing information with valid information" do
+
+
+ -
+ 1
+
+
billing_data = {
+
+
+ -
+
+
+
billing_datum: {
+
+
+ -
+
+
+
email: "test@testdata.com",
+
+
+ -
+
+
+
mailing_address: "2200 New Street, Seattle, WA, 09020",
+
+
+ -
+
+
+
credit_card_name: "test card",
+
+
+ -
+
+
+
credit_card_number: "4332 2321 1213 1241",
+
+
+ -
+
+
+
credit_card_cvv: "333",
+
+
+ -
+
+
+
billing_zip_code: "09020",
+
+
+ -
+
+
+
expiration_date: "121"
+
+
+ -
+
+
+
}
+
+
+ -
+
+
+
}
+
+
+ -
+ 1
+
+
proc {post billing_data_path, params: billing_data}.must_change 'BillingDatum.count', 1
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
must_respond_with :redirect
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "should not create new billing information when valid data isn't present" do
+
+
+ -
+ 1
+
+
billing_data = {
+
+
+ -
+
+
+
billing_datum: {
+
+
+ -
+
+
+
email: "test@testdata.com",
+
+
+ -
+
+
+
mailing_address: "2200 New Street, Seattle, WA, 09020",
+
+
+ -
+
+
+
credit_card_name: "test card",
+
+
+ -
+
+
+
credit_card_number: "4332 2321 1213 1241",
+
+
+ -
+
+
+
credit_card_cvv: "333",
+
+
+ -
+
+
+
billing_zip_code: "09020",
+
+
+ -
+
+
+
expiration_date: "211"
+
+
+ -
+
+
+
}
+
+
+ -
+
+
+
}
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
billing_data.each do |key, value|
+
+
+ -
+ 1
+
+
billing_data[key] = nil
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
proc {post billing_data_path, params: billing_data}.must_change 'BillingDatum.count', 0
+
+
+ -
+ 1
+
+
must_respond_with :bad_request
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "!!!should produce an edit form for billing data" do
+
+
+ -
+
+
+
#issue connecting user to data
+
+
+ -
+ 1
+
+
log_in(users(:carl), :github)
+
+
+ -
+ 1
+
+
session[:user_id].must_equal users(:carl).id
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
get edit_billing_datum_path(billing_data(:carl_billing_datum))
+
+
+ -
+ 1
+
+
must_respond_with :success
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "should update billing information" do
+
+
+ -
+ 1
+
+
person_data = billing_data(:carl_billing_datum)
+
+
+ -
+ 1
+
+
user = User.find_by(id: person_data.user_id)
+
+
+ -
+ 1
+
+
log_in( user, :github)
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
billing_data = {
+
+
+ -
+
+
+
billing_datum: {
+
+
+ -
+
+
+
email: "test@testdata.com",
+
+
+ -
+
+
+
mailing_address: "2200 New Street, Seattle, WA, 09020",
+
+
+ -
+
+
+
credit_card_name: "test card",
+
+
+ -
+
+
+
credit_card_number: "4332 2321 1213 1241",
+
+
+ -
+
+
+
credit_card_cvv: "333",
+
+
+ -
+
+
+
billing_zip_code: "09020",
+
+
+ -
+
+
+
expiration_date: "121"
+
+
+ -
+
+
+
}
+
+
+ -
+
+
+
}
+
+
+ -
+
+
+
+
+ -
+
+
+
session[:user_id].must_equal User.users(:carl).id
+
+
+ -
+
+
+
session[:user_id].must_equal BillingDatum.find_by(id: billing_data(:carl_billing_datum)).user_id
+
+
+ -
+
+
+
+
+ -
+
+
+
put billing_datum_path(person_data), params: billing_data
+
+
+ -
+
+
+
must_respond_with :redirect
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
must_redirect_to billing_info_path(billing_data(:carl_billing_datum))
+
+
+ -
+
+
+
# should redirect to the "show billing info/confirm purchase"
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe "#SHOW" do
+
+
+ -
+ 1
+
+
it "shows a logged in user her billing information" do
+
+
+ -
+ 1
+
+
log_in(users(:carl), :github)
+
+
+ -
+
+
+
#there is a problem with the connection between user and user id
+
+
+ -
+ 1
+
+
get billing_datum_path(billing_data(:carl_billing_datum).id)
+
+
+ -
+ 1
+
+
must_respond_with :success
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "does not show billing information to anyone not tied to it through a product" do
+
+
+ -
+ 1
+
+
get billing_data_path(billing_data(:carl_billing_datum).id)
+
+
+ -
+ 1
+
+
must_respond_with :not_found
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "doesn't show billing data that doesn't exist" do
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
get billing_datum_path(BillingDatum.last.id+1)
+
+
+ -
+ 1
+
+
must_redirect_to root_path
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
require "test_helper"
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe CategoriesController do
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe "index" do
+
+
+ -
+
+
+
# it "succeeds when there are categories" do
+
+
+ -
+
+
+
# get categories_path
+
+
+ -
+
+
+
# must_respond_with :success
+
+
+ -
+
+
+
# end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "succeeds when there are no categories" do
+
+
+ -
+ 1
+
+
Category.all.each do |category|
+
+
+ -
+ 3
+
+
category.destroy
+
+
+ -
+
+
+
end
+
+
+ -
+ 1
+
+
get categories_path
+
+
+ -
+ 1
+
+
must_respond_with :success
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe "show" do
+
+
+ -
+ 1
+
+
it "can show one category" do
+
+
+ -
+ 1
+
+
get category_path(categories(:music).id)
+
+
+ -
+ 1
+
+
must_respond_with :success
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "cannot show a category that does not exist" do
+
+
+ -
+ 1
+
+
category_id = Category.last.id + 1
+
+
+ -
+ 1
+
+
get category_path(category_id)
+
+
+ -
+
+
+
#add test for flash message here
+
+
+ -
+ 1
+
+
must_respond_with :redirect
+
+
+ -
+ 1
+
+
must_redirect_to root_path
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe "new" do
+
+
+ -
+ 1
+
+
it "can reach a new category" do
+
+
+ -
+ 1
+
+
get new_category_path
+
+
+ -
+ 1
+
+
must_respond_with :success
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe "create" do
+
+
+ -
+ 1
+
+
it "Can create a new category" do
+
+
+ -
+ 1
+
+
proc {
+
+
+ -
+ 1
+
+
post categories_path, params: {category: {name: "socks"} }
+
+
+ -
+
+
+
}.must_change 'Category.count', 1
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
must_respond_with :redirect
+
+
+ -
+ 1
+
+
must_redirect_to categories_path
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "does not create a new category if a duplicate category is entered" do
+
+
+ -
+ 1
+
+
proc {
+
+
+ -
+ 1
+
+
post categories_path, params: {category: {name: "shoes"} }
+
+
+ -
+
+
+
}.must_change 'Category.count', 0
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
must_respond_with :success
+
+
+ -
+
+
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe "edit" do
+
+
+ -
+ 1
+
+
it "should be able to successfully update a category name" do
+
+
+ -
+ 1
+
+
put category_path( categories(:shoes) ), params: { category: { name: "footwear" } }
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
updated_category = Category.find( categories(:shoes).id )
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
updated_category.name.must_equal "footwear"
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
must_redirect_to categories_path
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe "destroy" do
+
+
+ -
+ 1
+
+
it "succeeds for an extant category ID" do
+
+
+ -
+ 1
+
+
category_id = Category.first.id
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
delete category_path(category_id)
+
+
+ -
+ 1
+
+
must_redirect_to categories_path
+
+
+ -
+
+
+
+
+ -
+
+
+
# The category should really be gone
+
+
+ -
+ 1
+
+
Category.find_by(id: category_id).must_be_nil
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
require "test_helper"
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe ItemsController do
+
+
+ -
+ 1
+
+
describe "Index" do
+
+
+ -
+ 1
+
+
it "should get index" do
+
+
+ -
+ 1
+
+
get items_path
+
+
+ -
+ 1
+
+
must_respond_with :success
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe "show" do
+
+
+ -
+ 1
+
+
it "should be able to show an item" do
+
+
+ -
+ 1
+
+
get item_path(items(:converse_no_order).id)
+
+
+ -
+ 1
+
+
must_respond_with :success
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "Cannot show an Item that does not exist" do
+
+
+ -
+ 1
+
+
item_id = Item.last.id + 1
+
+
+ -
+ 1
+
+
get item_path(item_id)
+
+
+ -
+
+
+
+
+ -
+
+
+
# must_respond_with :not_found
+
+
+ -
+ 1
+
+
must_respond_with :redirect
+
+
+ -
+ 1
+
+
must_redirect_to root_path
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe "New" do
+
+
+ -
+ 1
+
+
it "should get new" do
+
+
+ -
+ 1
+
+
get new_item_path
+
+
+ -
+ 1
+
+
must_respond_with :success
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
# TODO
+
+
+ -
+ 1
+
+
describe "Create" do
+
+
+ -
+ 1
+
+
it "Should be able to create a new item" do
+
+
+ -
+ 1
+
+
proc {
+
+
+ -
+ 1
+
+
post create_item_path, params: {item: {shipping_status: false, purchase_status: false, product_id: products[:converse].id, order_id: nil }}}.must_change 'Item.count', 1
+
+
+ -
+
+
+
+
+ -
+
+
+
must_respond_with :redirect
+
+
+ -
+
+
+
must_redirect_to root_path
+
+
+ -
+
+
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe "Create" do
+
+
+ -
+ 1
+
+
it "Should be able to create a new item" do
+
+
+ -
+
+
+
+
+ -
+
+
+
# user = users(:carl)
+
+
+ -
+
+
+
# log_in(user, :github)
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
proc {
+
+
+ -
+ 1
+
+
post create_item_path(products(:converse).id) }.must_change 'Item.count', 1
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
must_respond_with :redirect
+
+
+ -
+ 1
+
+
must_redirect_to root_path
+
+
+ -
+
+
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "should respond with bad request if you try to make an item with bad data" do
+
+
+ -
+ 1
+
+
post create_item_path("not good")
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
must_respond_with :bad_request
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe "Destroy" do
+
+
+ -
+ 1
+
+
it "Should be able to delete an item" do
+
+
+ -
+ 1
+
+
proc {
+
+
+ -
+ 1
+
+
delete item_path(products(:converse).id)}.must_change 'Item.count', -1
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
must_respond_with :redirect
+
+
+ -
+ 1
+
+
must_redirect_to root_path
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "should respond with bad request if there are no items to destroy for that item" do
+
+
+ -
+ 1
+
+
product = products(:out_of_stock)
+
+
+ -
+ 1
+
+
start_count = Item.count
+
+
+ -
+ 1
+
+
delete item_path(product.id)
+
+
+ -
+ 1
+
+
must_respond_with :bad_request
+
+
+ -
+ 1
+
+
Item.count.must_equal start_count
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
require "test_helper"
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe MainPageController do
+
+
+ -
+ 1
+
+
it "should get index" do
+
+
+ -
+ 1
+
+
get main_page_path
+
+
+ -
+
+
+
must_respond_with :success
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
require "test_helper"
+
+
+ -
+
+
+
#IF IT'S COMMENTED OUT, IT'S FAILING!
+
+
+ -
+ 1
+
+
describe ProductsController do
+
+
+ -
+ 1
+
+
describe "index" do
+
+
+ -
+ 1
+
+
it "can display a list of all products available on the website" do
+
+
+ -
+ 1
+
+
get products_path
+
+
+ -
+ 1
+
+
must_respond_with :success
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "can display index page with no products" do
+
+
+ -
+
+
+
#not destroying products!
+
+
+ -
+ 1
+
+
OrderProduct.destroy_all
+
+
+ -
+ 1
+
+
OrderProduct.count.must_equal 0
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
Item.destroy_all
+
+
+ -
+ 1
+
+
Item.count.must_equal 0
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
Product.destroy_all
+
+
+ -
+ 1
+
+
Product.count.must_equal 0
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
get products_path
+
+
+ -
+ 1
+
+
must_respond_with :success
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
#
+
+
+ -
+ 1
+
+
describe "Show" do
+
+
+ -
+ 1
+
+
it "shows an existing product" do
+
+
+ -
+ 1
+
+
get product_path(products(:converse))
+
+
+ -
+ 1
+
+
must_respond_with :success
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
#
+
+
+ -
+ 1
+
+
it "responds with 404 for non-existing product" do
+
+
+ -
+ 1
+
+
get product_path(Product.last.id + 1)
+
+
+ -
+ 1
+
+
must_respond_with :not_found
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "users cannot leave reviews for their products" do
+
+
+ -
+ 1
+
+
user = users(:carl)
+
+
+ -
+ 1
+
+
log_in(user, :github)
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
get product_path(products(:converse).id)
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
controller.instance_variable_get("@current_user_is_not_product_owner").must_equal false
+
+
+ -
+
+
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe "edit" do
+
+
+ -
+ 1
+
+
it "logged in owner can see an edit form for their product " do
+
+
+ -
+ 1
+
+
user = users(:carl)
+
+
+ -
+ 1
+
+
log_in(user, :github)
+
+
+ -
+ 1
+
+
session[:user_id].must_equal user.id
+
+
+ -
+ 1
+
+
get edit_product_path(products(:converse))
+
+
+ -
+ 1
+
+
must_respond_with :success
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
# it "responds with 404 for non-existing product" do
+
+
+ -
+
+
+
# get product_path(Product.last.id + 1)
+
+
+ -
+
+
+
# must_respond_with :bad_request
+
+
+ -
+
+
+
# end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "must be the creator of a product to edit it" do
+
+
+ -
+ 1
+
+
get edit_product_path(products(:converse))
+
+
+ -
+ 1
+
+
must_respond_with :not_found
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "does not produce an edit form for a product a user doesn't own" do
+
+
+ -
+ 1
+
+
user = users(:mia)
+
+
+ -
+ 1
+
+
log_in(user, :github)
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
get edit_product_path(products(:converse))
+
+
+ -
+ 1
+
+
must_respond_with :not_found
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "does not produce an edit form for bogus data" do
+
+
+ -
+ 1
+
+
user = users(:carl)
+
+
+ -
+ 1
+
+
log_in(user, :github)
+
+
+ -
+ 1
+
+
nonexistant_data = Product.last.id + 1
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
get edit_product_path(nonexistant_data)
+
+
+ -
+ 1
+
+
must_respond_with :not_found
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe "update" do
+
+
+ -
+ 1
+
+
it "a product cannot be updated, except by the owner" do
+
+
+ -
+ 1
+
+
product = products(:converse)
+
+
+ -
+ 1
+
+
product_datum = {
+
+
+ -
+
+
+
product: {
+
+
+ -
+
+
+
name: product.name + " updated"
+
+
+ -
+
+
+
}
+
+
+ -
+
+
+
}
+
+
+ -
+ 1
+
+
patch product_path(product), params: product_datum
+
+
+ -
+ 1
+
+
must_respond_with :not_found
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "can update an existing work with valid data" do
+
+
+ -
+ 1
+
+
user = users(:carl)
+
+
+ -
+ 1
+
+
log_in(user, :github)
+
+
+ -
+ 1
+
+
product = products(:converse)
+
+
+ -
+ 1
+
+
product_datum = {
+
+
+ -
+
+
+
product: {
+
+
+ -
+
+
+
name: product.name + " updated"
+
+
+ -
+
+
+
}
+
+
+ -
+
+
+
}
+
+
+ -
+ 1
+
+
patch product_path(product), params: product_datum
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "cannot update a work that does not exist" do
+
+
+ -
+ 1
+
+
user = users(:carl)
+
+
+ -
+ 1
+
+
log_in(user, :github)
+
+
+ -
+ 1
+
+
product = (Product.last.id + 1)
+
+
+ -
+ 1
+
+
product_datum = {
+
+
+ -
+
+
+
product: {
+
+
+ -
+
+
+
name: "Name updated"
+
+
+ -
+
+
+
}
+
+
+ -
+
+
+
}
+
+
+ -
+ 1
+
+
patch product_path(product), params: product_datum
+
+
+ -
+ 1
+
+
must_respond_with :not_found
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe "new" do
+
+
+ -
+ 1
+
+
it "it should get a new form when a user is logged in" do
+
+
+ -
+ 1
+
+
log_in(users(:carl), :github)
+
+
+ -
+ 1
+
+
get new_product_path
+
+
+ -
+ 1
+
+
must_respond_with :success
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "shouldn't get a new product form when no user is logged in" do
+
+
+ -
+ 1
+
+
get new_product_path
+
+
+ -
+ 1
+
+
must_respond_with 404
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe "create" do
+
+
+ -
+ 1
+
+
it "should create a new product when given valid data" do
+
+
+ -
+ 1
+
+
user = users(:carl)
+
+
+ -
+ 1
+
+
log_in(user, :github)
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
product_data = {
+
+
+ -
+
+
+
product: {
+
+
+ -
+
+
+
name: "God Ugly Converse",
+
+
+ -
+
+
+
price: 99,
+
+
+ -
+
+
+
description: "Great pink shoes!",
+
+
+ -
+
+
+
user_id: user.id
+
+
+ -
+
+
+
}
+
+
+ -
+
+
+
}
+
+
+ -
+ 1
+
+
post products_path, params: product_data
+
+
+ -
+ 1
+
+
must_respond_with :redirect
+
+
+ -
+ 1
+
+
must_redirect_to profile_path(user)
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "should throw an error when not given valid data" do
+
+
+ -
+ 1
+
+
user = users(:carl)
+
+
+ -
+ 1
+
+
log_in(user, :github)
+
+
+ -
+ 1
+
+
product_data = {
+
+
+ -
+
+
+
product: {
+
+
+ -
+
+
+
name: "God Ugly Converse",
+
+
+ -
+
+
+
price: 99,
+
+
+ -
+
+
+
user_id: user.id
+
+
+ -
+
+
+
}
+
+
+ -
+
+
+
}
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
nil_product = product_data.deep_dup
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
product_data[:product].each do |key, value|
+
+
+ -
+ 3
+
+
nil_product = product_data.deep_dup
+
+
+ -
+ 3
+
+
nil_product[:product][key] = nil
+
+
+ -
+ 3
+
+
post products_path, params: nil_product
+
+
+ -
+ 3
+
+
must_respond_with :bad_request
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "should throw 404 when no user is logged in" do
+
+
+ -
+ 1
+
+
user = users(:carl)
+
+
+ -
+ 1
+
+
product_data = {
+
+
+ -
+
+
+
product: {
+
+
+ -
+
+
+
name: "God Ugly Converse",
+
+
+ -
+
+
+
price: 99,
+
+
+ -
+
+
+
description: "Great pink shoes!",
+
+
+ -
+
+
+
user_id: user.id
+
+
+ -
+
+
+
}
+
+
+ -
+
+
+
}
+
+
+ -
+ 1
+
+
post products_path, params: product_data
+
+
+ -
+ 1
+
+
must_respond_with :not_found
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe "change_visibility" do
+
+
+ -
+ 1
+
+
it "allows a user to change the visibility of a product they own" do
+
+
+ -
+ 1
+
+
user = users(:carl)
+
+
+ -
+ 1
+
+
log_in(user, :github)
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
product = products(:flats)
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
product.visibility.must_equal true
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
post change_visibility_path(product.id)
+
+
+ -
+ 1
+
+
product = Product.find_by(id: product.id)
+
+
+ -
+
+
+
#fixtures is weird and needs scaffolding. hence line 190
+
+
+ -
+ 1
+
+
product.visibility.must_equal false
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe "add_to_order" do
+
+
+ -
+ 1
+
+
it "should change number of order products" do
+
+
+ -
+ 1
+
+
user = users(:mia)
+
+
+ -
+ 1
+
+
product = products(:converse)
+
+
+ -
+ 1
+
+
log_in(user, :github)
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
proc {
+
+
+ -
+ 1
+
+
post products_path(product.id), params: {quantity: "3"}
+
+
+ -
+
+
+
}.must_change 'OrderProduct.count', 1
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "should add an orderproduct to current pending order for auth user" do
+
+
+ -
+ 1
+
+
user = users(:mia)
+
+
+ -
+ 1
+
+
product = products(:converse)
+
+
+ -
+ 1
+
+
quantity_data = {
+
+
+ -
+
+
+
quantity: "5"
+
+
+ -
+
+
+
}
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
start_count = OrderProduct.count
+
+
+ -
+
+
+
# pending_order = orders(:mias_pending_order)
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
log_in(user, :github)
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
get product_path(product.id)
+
+
+ -
+ 1
+
+
pending_order = Order.find(session[:pending_order_id])
+
+
+ -
+
+
+
# session[:pending_order_id].must_equal pending_order.id
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
post product_path(product.id), params: quantity_data
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
OrderProduct.count.must_equal (start_count + 1)
+
+
+ -
+
+
+
pending_order.order_products.length.must_equal 2
+
+
+ -
+
+
+
+
+ -
+
+
+
pending_order.order_products.each do |entry|
+
+
+ -
+
+
+
entry.product_id.must_equal product.id
+
+
+ -
+
+
+
entry.order_id.must_equal pending_order.id
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
require "test_helper"
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe ReviewsController do
+
+
+ -
+ 1
+
+
describe "Guest (Unauthenticated)" do
+
+
+ -
+ 1
+
+
describe "NEW" do
+
+
+ -
+ 2
+
+
let(:product) { products(:converse) }
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "Should be able to get to the new review form" do
+
+
+ -
+ 1
+
+
get new_review_path(product)
+
+
+ -
+ 1
+
+
must_respond_with :success
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe "CREATE" do
+
+
+ -
+ 1
+
+
it "Guests can leave reviews for any product" do
+
+
+ -
+ 1
+
+
proc {
+
+
+ -
+ 1
+
+
post reviews_path, params: {review: {text: "sample review", rating: 5, user_id: nil, product_id: products(:converse).id } }
+
+
+ -
+
+
+
}.must_change 'Review.count', 1
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
must_respond_with :redirect
+
+
+ -
+ 1
+
+
must_redirect_to product_path(products(:converse).id)
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "If correct info is not provided, render new" do
+
+
+ -
+ 1
+
+
r = reviews(:one)
+
+
+ -
+ 1
+
+
r.valid?
+
+
+ -
+ 1
+
+
r.rating = 10
+
+
+ -
+ 1
+
+
r.valid?.must_equal false
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
require "test_helper"
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe SessionsController do
+
+
+ -
+ 1
+
+
describe "auth_callback" do
+
+
+ -
+ 1
+
+
it "should log in an existing user and redirects them back to the homepage" do
+
+
+ -
+ 1
+
+
start_count = User.count
+
+
+ -
+ 1
+
+
user = users(:carl)
+
+
+ -
+
+
+
+
+ -
+
+
+
#Action
+
+
+ -
+ 1
+
+
log_in(user, :github)
+
+
+ -
+
+
+
#Assert
+
+
+ -
+ 1
+
+
must_respond_with :redirect
+
+
+ -
+ 1
+
+
must_redirect_to root_path
+
+
+ -
+ 1
+
+
User.count.must_equal start_count
+
+
+ -
+ 1
+
+
session[:user_id].must_equal user.id
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "can log in a new user" do
+
+
+ -
+ 1
+
+
start_count = User.count
+
+
+ -
+ 1
+
+
user = User.new username: "Greg", provider: 'github', email: 'RapMonster@naver.com'
+
+
+ -
+ 1
+
+
user.uid = 901
+
+
+ -
+ 1
+
+
log_in(user, :github)
+
+
+ -
+
+
+
+
+ -
+
+
+
#Assert
+
+
+ -
+ 1
+
+
must_respond_with :redirect
+
+
+ -
+ 1
+
+
must_redirect_to root_path
+
+
+ -
+ 1
+
+
User.count.must_equal (start_count + 1)
+
+
+ -
+ 1
+
+
saved_user = User.find_by(username: "Greg")
+
+
+ -
+ 1
+
+
session[:user_id].must_equal saved_user.id
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "can log a user out" do
+
+
+ -
+ 1
+
+
log_in(users(:carl), :github)
+
+
+ -
+ 1
+
+
session[:user_id].must_equal users(:carl).id
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
get logout_path
+
+
+ -
+ 1
+
+
must_respond_with :redirect
+
+
+ -
+ 1
+
+
must_redirect_to root_path
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+
+
+
+
+
+ -
+ 1
+
+
require "test_helper"
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe UsersController do
+
+
+ -
+ 1
+
+
describe "order_fulfillment" do
+
+
+ -
+ 1
+
+
it "should get order_fulfillment page for logged in user" do
+
+
+ -
+ 1
+
+
user = users(:mia)
+
+
+ -
+ 1
+
+
log_in(user, :github)
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
get order_fulfillment_path
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
must_respond_with :success
+
+
+ -
+
+
+
+
+ -
+
+
+
end
+
+
+ -
+ 1
+
+
it "should respond with an error if not logged in " do
+
+
+ -
+ 1
+
+
get order_fulfillment_path
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
must_respond_with :not_found
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
#GM how to test instance variables )-':
+
+
+ -
+ 1
+
+
it "should populate with entries with user has sold products" do
+
+
+ -
+ 1
+
+
user = users(:sophie_no_pending_order)
+
+
+ -
+ 1
+
+
log_in(user, :github)
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
get order_fulfillment_path
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
must_respond_with :success
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+ 1
+
+
describe "INDEX" do
+
+
+ -
+ 1
+
+
it "should get index" do
+
+
+ -
+ 1
+
+
get users_path
+
+
+ -
+ 1
+
+
must_respond_with :success
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe "SHOW" do
+
+
+ -
+ 1
+
+
it "should get show" do
+
+
+ -
+ 1
+
+
get user_path(users(:carl).id)
+
+
+ -
+ 1
+
+
must_respond_with :success
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
it "Cannot show a user that doesnt exist" do
+
+
+ -
+
+
+
# User.last gives the user with the highest ID
+
+
+ -
+ 1
+
+
user_id = User.last.id + 1
+
+
+ -
+ 1
+
+
get user_path(user_id)
+
+
+ -
+
+
+
# must_respond_with :not_found
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
must_respond_with :redirect
+
+
+ -
+ 1
+
+
must_redirect_to root_path
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+ 1
+
+
describe "profile" do
+
+
+ -
+ 1
+
+
it "auth users should successfully get profile_path" do
+
+
+ -
+ 1
+
+
user = users(:mia)
+
+
+ -
+ 1
+
+
log_in(user, :github)
+
+
+ -
+ 1
+
+
get profile_path
+
+
+ -
+ 1
+
+
must_respond_with :success
+
+
+ -
+
+
+
end
+
+
+ -
+ 1
+
+
it "should respond with 404 for guests" do
+
+
+ -
+ 1
+
+
get profile_path
+
+
+ -
+ 1
+
+
must_respond_with :not_found
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
# it "should successfully create a new user" do
+
+
+ -
+
+
+
# #GM: Should users be successfully created through user controller?
+
+
+ -
+
+
+
# #Or should this be limited to sessions#login?
+
+
+ -
+
+
+
# proc {
+
+
+ -
+
+
+
# post users_path, params: {users: {username: "Test", email: "test@test.com"}}}.must_change 'User.count', 1
+
+
+ -
+
+
+
#
+
+
+ -
+
+
+
# must_respond_with :redirect
+
+
+ -
+
+
+
# must_redirect_to root_path
+
+
+ -
+
+
+
# end
+
+
+ -
+
+
+
#
+
+
+ -
+
+
+
# it "should be able to update a user" do
+
+
+ -
+
+
+
# log_in(users(:carl), :github)
+
+
+ -
+
+
+
# put user_path(users(:carl)), params: {user: {username: "Karl"} }
+
+
+ -
+
+
+
# user = User.find(users(:carl).id)
+
+
+ -
+
+
+
# user.username = "Karl"
+
+
+ -
+
+
+
# user.username.must_equal "Karl"
+
+
+ -
+
+
+
#
+
+
+ -
+
+
+
# must_respond_with :redirect
+
+
+ -
+
+
+
# must_redirect_to root_path
+
+
+ -
+
+
+
# end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
# it "Should be able to delete a user" do
+
+
+ -
+
+
+
# proc {
+
+
+ -
+
+
+
# delete user_path(users(:carl).id)}.must_change 'User.count', - 1
+
+
+ -
+
+
+
#
+
+
+ -
+
+
+
# must_respond_with :redirect
+
+
+ -
+
+
+
# must_redirect_to root_path
+
+
+ -
+
+
+
# end
+
+
+ -
+
+
+
+
+ -
+
+
+
+
+ -
+
+
+
# it "should get edit" do
+
+
+ -
+
+
+
# end
+
+
+ -
+
+
+
end
+
+
+
+
+
+
+
+
+