How we'd like to recieve the soluion?

Clone this repository and upload it as a new public repository in your github account
Create a new branch in your repository
Create a pull request with the requested functionality to unchanged master branch in your repository
Share link to the PR with us

Backend C# developer project

Background

We as a vacation rental software provide a solution for the vacation rental hosts to create their rentals and assign received bookings to them. We have to control the current rental availability to avoid overbookings. Term rental refers to anything that could be rented: house, flat, room, bed in room etc. We allow to define how many number of units in this rental type is available for rental, it is represented by unit. For example we can have a rental with 2 units, and this rental represents a room - it means that the host is renting out 2 rooms.

Our customers want to retrieve the calendar which shows their current rental occupation. To provide this functionality we deliver the API.

The API

This project includes an API that can be consumed in order to update the aforementioned information about the host vacation rentals. It contains:

RentalsController which needs to be used first for the rental creation. It provides the following endpoints: POST api/v1/rentals - to create the host rental GET api/v1/rentals/{id} - to retrieve the rental information for the given rental id
BookingsController which needs to be used to create a booking for the existing rental. It provides the following endpoints: POST api/v1/bookings - to create new booking for the existing rental GET api/v1/bookings/{id} - to retrieve the booking information for the given booking id
CalendarController which needs to be used to retrieve the availability of an existing rental. It provides the following endpoints: GET api/v1/calendar - to retrieve the booking information for the given query parameters (rental id, start date, number of nights)
Additionally we provide the endpoint
GET /swagger - it's a swagger documentation for the API (will open on process run)

As it is presented the information for the contacts is only stored in memory and therefore we lose all information when the service restarts.

The tasks

Our customers requested a new feature related to the coronavirus restriction. They have to clean each rental unit after their guest occupies it. To achieve this we should block additional X days after each booking. Block means to not allow to have any booking for the same unit during this time. The number of days will be defined by the host.

E.g.: 1 day preparation time: For any booking without another booking after, an extra day is blocked for the unit where the existing booking occurs.

One booking always occupies only one unit. The rental can have many units. So a rental with 3 units is fully occupied for the selected night when it has (3 bookings) OR (2 bookings and 1 preparation time period) OR (1 booking and 2 preparation time periods) OR (3 preparation time periods).

The contracts with API consumers are already defined. You have to extend existing endpoints to fulfill new definitions and provide the requested functionality.

A new Unit property is added to the Booking calendar model and to the new PreparationTime calendar model. This property is a number that references one of the available units in the Rental. For example, if the Rental has a total of 3 units, the Unit property can accept the values 1, 2, or 3. It is irrelevant whether the Booking occupies unit 1, 2 or 3, as long as its associated PreparationTime occupies the same Unit number as the Booking.

If the calendar model contains Booking with Unit 2 then it means that Booking is booked at Unit number 2. It does NOT mean that the booking occupies two units.

a) extend POST api/v1/rentals endpoint request body by adding PreparationTimeInDays property:

POST api/v1/vacationrental/rentals { Units: 3, PreparationTimeInDays: 2 }

b) extend GET api/v1/calendar response by adding the occupied unit number to Bookings collection (property Unit) and by adding collection of preparation time objects (property PreparationTimes) - each object should have Unit number property:

{ Dates: [ { Date: “2020-01-01”, Bookings: [ { Id: 2, Unit: 1 } ], PreparationTimes: [] }, { Date: “2020-01-02”, Bookings: [ { Id: 2, Unit: 1 } ], PreparationTimes: [] }, { Date: “2020-01-03”, Bookings: [], PreparationTimes: [ { Unit: 1 } ] } ] }

Please consider existing code as legacy. Improve it. Please focus on readability, maintainability and self-explanatory. Unit tests are more than welcome.
Optional (nice to have). Our customers requested the possibility of updating their existing rentals. They would like to be able to change the number of units and the length of preparation time. Please provide a new endpoint which allows them to do it:

PUT api/v1/rentals/{id} { Units: 3, PreparationTimeInDays: 2 }

If the length of preparation time is changed then it should be updated for all existing bookings. The request should fail if overlapping between existing bookings and/or their preparation times occurs due to a decrease of the number of units or an increase of the length of preparation time.

What we are looking for

How you refactor the existing code to make it more readable and structured
How you implement new features and understand the requirements (feel free to ask in case of doubts)
How you test the correctness of your solution

Restrictions

Please do not modify the API contract
- Do not rename the exposed model's properties
- Do not modify the API routes
Besides it changing/adding everything is allowed, feel free to add new projects, libraries etc. - everything what you consider as useful
If there is something that you want us to notice in your approach please add a text file describing it