Skip to content

Commit

Permalink
Creating orders.html, presenting orders history
Browse files Browse the repository at this point in the history
A page on Hotails, where the user can watch the history of all of his
orders, cancel the orders which havn't started yet, and approve orders
as a daycare.

Signed-off-by: Ofir Matasas <[email protected]>
  • Loading branch information
OfirMatasas committed May 4, 2022
1 parent 8f1996c commit 7bb5e24
Show file tree
Hide file tree
Showing 9 changed files with 201 additions and 3 deletions.
1 change: 1 addition & 0 deletions Hotails/urls.py
Original file line number Diff line number Diff line change
Expand Up @@ -26,4 +26,5 @@
path('homepage/', views.homepage, name='homepage'),
path('logout/', views.logout_view, name='logout'),
path('about/', views.about, name='about'),
path('orders/', views.orders_view, name='orders'),
]
2 changes: 1 addition & 1 deletion main/templates/main/base_template.html
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@
<ul class="navbar-nav mr-auto">
{% if request.user.is_authenticated %}
<li class="nav-item"><a class="nav-link text-white" href="/profile">Profile</a></li>
<li class="nav-item"><a class="nav-link text-white" href="">Orders</a></li>
<li class="nav-item"><a class="nav-link text-white" href="/orders">Orders</a></li>
<li class="nav-item"><a class="nav-link text-white" href="">Search</a></li>
<li class="nav-item"><a class="nav-link text-white" href="">Chats</a></li>
{% endif %}
Expand Down
13 changes: 13 additions & 0 deletions main/tests.py
Original file line number Diff line number Diff line change
Expand Up @@ -69,3 +69,16 @@ def test_unlogged_user_access_to_homepage(self, client):
response = client.get("/homepage/")
assert response.status_code == 302
assert response['Location'] == '/login/?next=/homepage/'


@pytest.mark.django_db
class TestOrdersView:
def test_orders_page_is_visible_for_dog_owner_user(self, client, create_dog_owner_user):
client.force_login(user=create_dog_owner_user.user)
response = client.get("/orders/")
assert response.status_code == 200

def test_orders_page_is_visible_for_daycare_user(self, client, create_daycare_user):
client.force_login(user=create_daycare_user.user)
response = client.get("/orders/")
assert response.status_code == 200
5 changes: 5 additions & 0 deletions main/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
from daycare.models import DayCare
from dogowner.models import DogOwner
from daycare.views import daycare_home
import orders.views


def index(request):
Expand All @@ -28,3 +29,7 @@ def about(request):
def logout_view(request):
logout(request)
return index(request)


def orders_view(request):
return orders.views.orders(request)
49 changes: 49 additions & 0 deletions orders/models.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
from django.utils import timezone
from daycare.models import DayCare
from dogowner.models import DogOwner
import datetime


class StatusOptions(models.TextChoices):
Expand Down Expand Up @@ -71,3 +72,51 @@ def cancel_order(self):

def get_order_total_price(self):
return self.price_per_day * (self.end_date - self.start_date).days

def get_order_status(self):
if self.status == StatusOptions.Pending:
return "Pending"
elif self.status == StatusOptions.Approved:
return "Approved"
elif self.status == StatusOptions.Canceled:
return "Canceled"
elif self.status == StatusOptions.OnGoing:
return "Ongoing"
return "Finished"

@staticmethod
def get_capacity_of_daycare_in_dates_range(daycare_id, start_datetime, end_datetime):
number_of_days = (end_datetime - start_datetime).days
capacity_per_day: list[int] = [0] * number_of_days
start_date = datetime.date(year=start_datetime.year, month=start_datetime.month, day=start_datetime.day)
end_date = datetime.date(year=end_datetime.year, month=end_datetime.month, day=end_datetime.day)
relevant_orders = Order.objects.filter(daycare_id=daycare_id,
status__in=['A', 'O'],
end_date__gte=start_date,
start_date__lte=end_date)

for order in relevant_orders:
for day in range(number_of_days):
current_date = order.start_date + datetime.timedelta(days=day)
if current_date < start_date:
continue
elif current_date > end_date:
break
else:
capacity_per_day[day] = capacity_per_day[day] + 1

return capacity_per_day

def are_order_dates_available(self):
capacity_per_day: list[int] = self.get_capacity_of_daycare_in_dates_range(self.daycare_id,
self.start_date,
self.end_date)

daycare_capacity = DayCare.objects.get(name=self.daycare_id.name).capacity
return all(current_capacity < daycare_capacity for current_capacity in capacity_per_day)

def is_the_order_cancelable(self):
return self.status == StatusOptions.Pending or self.status == StatusOptions.Approved

def is_the_order_approvable(self):
return self.status == StatusOptions.Pending and self.are_order_dates_available()
54 changes: 54 additions & 0 deletions orders/templates/orders/orders.html
Original file line number Diff line number Diff line change
@@ -0,0 +1,54 @@
{% extends "main/base_template.html" %}
{% load static %}

{% block stylesheets %}
<link href="https://cdn.jsdelivr.net/npm/[email protected]/dist/css/bootstrap.min.css" rel="stylesheet" integrity="sha384-EVSTQN3/azprG1Anm3QDgpJLIm9Nao0Yz1ztcQTwFspd3yD65VohhpuuCOmLASjC" crossorigin="anonymous">
<link rel="stylesheet" type="text/css" href="{% static 'css/orders.css' %}">
{% endblock %}

{% block content %}
<div class="container">
<div class="row">
<div class="col-12">
<table class="table table-bordered">
<thead>
<tr>
<th scope="col">Name</th>
<th scope="col">Book Date</th>
<th scope="col">Start Date</th>
<th scope="col">End Date</th>
<th scope="col">Status</th>
<th scope="col">Actions</th>
</tr>
</thead>

<tbody>
{% for order in orders %}
<tr>
<th scope="row">
{% if user == 'daycare' %}
{{order.dog_owner_id.name }}
{% else %}
{{order.daycare_id.name }}
{% endif %}
</th>
<td>{{order.book_date}}</td>
<td>{{order.start_date}}</td>
<td>{{order.end_date}}</td>
<td>{{order.get_order_status}}</td>
<td>
{% if order.is_the_order_approvable and user == 'daycare' %}
<button type="button" class="btn btn-success">Approve</button>
{% endif %}
{% if order.is_the_order_cancelable %}
<button type="button" class="btn btn-danger">Cancel</button>
{% endif %}
</td>
</tr>
{% endfor %}
</tbody>
</table>
</div>
</div>
</div>
{% endblock %}
52 changes: 52 additions & 0 deletions orders/tests.py
Original file line number Diff line number Diff line change
Expand Up @@ -52,3 +52,55 @@ def test_dog_owner_id_is_deleted_when_dog_owner_is_deleted(self, create_order):
def test_daycare_id_is_deleted_when_daycare_is_deleted(self, create_order):
DayCare.objects.get(id=1).delete()
assert Order.objects.get(id=create_order.id).daycare_id is None

@pytest.mark.parametrize("dog_owner_1_id, dog_owner_2_id, delta_1, delta_2, capacity, expected_result",
[(1, 2, 3, 5, 1, False),
(2, 5, 4, 3, 10, True),
(5, 4, 6, 7, 15, True),
(4, 3, 4, 2, 1, False)])
def test_order_is_approvable_according_to_daycare_capacity(self, dog_owner_1_id, dog_owner_2_id, delta_1, delta_2,
capacity, expected_result):
daycare = DayCare.create("[email protected]", "CapacityUserName", "CapacityPassword", "CapacityName",
"Changeable capacity", 100, capacity, "Merkaz", "Tel Aviv", "Capacity 123")
dog_owner_1 = DogOwner.objects.get(id=dog_owner_1_id)
dog_owner_2 = DogOwner.objects.get(id=dog_owner_2_id)

order1 = Order.create(start_date=timezone.now(), end_date=timezone.now() + datetime.timedelta(days=delta_1),
daycare_id=daycare, dog_owner_id=dog_owner_1, price_per_day=100)
order2 = Order.create(start_date=timezone.now(), end_date=timezone.now() + datetime.timedelta(days=delta_2),
daycare_id=daycare, dog_owner_id=dog_owner_2, price_per_day=100)

order1.approve_order()
assert order2.is_the_order_approvable() == expected_result

@pytest.mark.parametrize("new_status, expected_result", [(StatusOptions.Pending, True),
(StatusOptions.Canceled, False),
(StatusOptions.Approved, True),
(StatusOptions.OnGoing, False),
(StatusOptions.Finished, False)])
def test_order_cancellation_according_to_order_status(self, create_order, new_status, expected_result):
create_order.status = new_status
assert create_order.is_the_order_cancelable() == expected_result

@pytest.mark.parametrize("new_status, expected_result", [(StatusOptions.Pending, "Pending"),
(StatusOptions.Canceled, "Canceled"),
(StatusOptions.Approved, "Approved"),
(StatusOptions.OnGoing, "Ongoing"),
(StatusOptions.Finished, "Finished")])
def test_get_order_status_as_string(self, create_order, new_status, expected_result):
create_order.status = new_status
assert create_order.get_order_status() == expected_result

def test_get_daycare_capacity_in_dates_range(self, create_order):
original_capacity_list = Order.get_capacity_of_daycare_in_dates_range(create_order.daycare_id,
create_order.start_date,
create_order.end_date)
create_order.approve_order()
updated_capacity_list = Order.get_capacity_of_daycare_in_dates_range(create_order.daycare_id,
create_order.start_date,
create_order.end_date)
assert len(original_capacity_list) == len(updated_capacity_list)

equality_list = [updated_capacity_list[i] == original_capacity_list[i] + 1
for i in range(len(original_capacity_list))]
assert equality_list.count(True) == len(equality_list)
20 changes: 18 additions & 2 deletions orders/views.py
Original file line number Diff line number Diff line change
@@ -1,3 +1,19 @@
# from django.shortcuts import render
from django.contrib.auth.decorators import login_required
from django.shortcuts import render

# Create your views here.
from dogowner.models import DogOwner
from orders.models import Order


@login_required()
def orders(request):
if DogOwner.objects.filter(user=request.user).exists():
user = 'dog_owner'
else:
user = 'daycare'

context = {
'orders': Order.objects.all(),
'user': user,
}
return render(request, 'orders/orders.html', context)
8 changes: 8 additions & 0 deletions static/CSS/orders.css
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
td, th
{
text-align: center;
}

.container {
padding: 2rem 0;
}

0 comments on commit 7bb5e24

Please sign in to comment.