#142-mockup-room-service #259

a24antej · 2026-04-13T11:18:21Z

a24antej commented

2026-04-13 11:18:21 +00:00

Har kommit så långt att man kan få fram data till frontend, har exempel View som kan användas för testning

a24antej added 18 commits

2026-04-13 11:18:22 +00:00

created the mockup services for room booking 5304ff0507

added the containers to the compose file bb48208ec9

fixed and tested build and compose db03e91bc6

fixed docker files 96db8d6d67

updated compose.yaml to access database e4c6148549

started with roomBookingMockup 668eb0a86e

Merge branch '#142-mockup-room-service' of https://git.webug.se/Andras/BoundlessFlowCampus2K into #142-mockup-room-service 22c5e5403d

fixed a start for readRoomBookingMockup e130a418b1

first version on roomBookingService bf77413602

fixed the whole roombooking 73f132b524

added start time and end time to the roomBooking f95cdaaab8

fixed SQL syntax error 139b1b39a3

Started on restructuring the roomBooking service 3ce447f4e9

Needs testing and implementation the use of database

tried to fix database, not work 811e3e60a9

Might need to create more containers, at least one that can act as a reader that uploads the data to the database

fixed a start on database for roomBooking 103fe922f2

it can only create a predetermined amount of bookings and can only request all of the bookings in the database

Cleaned upp old code 300f9c3d8f

removed first verison and started to dokument the code

added frontend af64926159

some cleanup and commenting 57244d3349

a24antej added the

backEnd

needs review

labels

2026-04-13 11:21:11 +00:00

a24antej was assigned by c24elipe

2026-04-13 12:16:04 +00:00

a24sakma reviewed

2026-04-13 12:45:29 +00:00

a24sakma left a comment

I did briefly review on as I was reading the code for reusing of BookingDbHandler.
I have not run the code on my machine as I have many changes on my local but if someone else can run it then it looks good to me :)

I did briefly review on as I was reading the code for reusing of BookingDbHandler. I have not run the code on my machine as I have many changes on my local but if someone else can run it then it looks good to me :)

roomBookingApiMockup/Program.cs

					
				@ -0,0 +4,4 @@

				using Dapper;

				using System.Text.Json;

				string connectionString = "Host=PostgreSQL;Port=5432;Database=sensors;Username=postgres;Password=AFnRTQac7e4c";

I have added a environment variable for this in my branch we can reuse that after I have pushed my branch.

a24sakma marked this conversation as resolved

roomBookingApiMockup/Program.cs Outdated

					
				@ -0,0 +91,4 @@

				});

Unused spaces can be removed. 94 to 98

a24antej marked this conversation as resolved

roomBookingApiMockup/README.MD

					
				@ -0,0 +15,4 @@

				{"bookingNr":"2","building":"C","roomNr":691,"startTime":"2026-04-18T02:15:00","endTime":"2026-04-18T04:45:00"},

				{"bookingNr":"1","building":"A","roomNr":462,"startTime":"2026-04-20T23:30:00","endTime":"2026-04-21T00:30:00"}

				]

				-----------------------------------------

Very good README

a24sakma marked this conversation as resolved

roomBookingApiMockup/roomBookingApiMockup.http Outdated

					
				@ -0,0 +1,6 @@

				@roomBookingApiMockup_HostAddress = http://localhost:5226

				GET {{roomBookingApiMockup_HostAddress}}/weatherforecast/

I think this comes when you create the project can be safely removed.
I removed from ParkingService and it works fine: Andras/BoundlessFlowCampus2K@b4b4a06c4d

I think this comes when you create the project can be safely removed. I removed from ParkingService and it works fine: https://git.webug.se/Andras/BoundlessFlowCampus2K/commit/b4b4a06c4d50b98104d1491f4a2a65daacac156c

a24antej marked this conversation as resolved

roomBookingDbHandler/Program.cs Outdated

					
				@ -0,0 +5,4 @@

				using System.Data.SQLite;

				namespace SensorRead

Shall this not be renamed to
namespace RoomBookingRead

Shall this not be renamed to `namespace RoomBookingRead`

a24antej marked this conversation as resolved

a24antej added 1 commit

2026-04-13 13:06:03 +00:00

fixed according to review c1f23af199

c24elipe requested review from b25marem

2026-04-13 13:57:42 +00:00

a24sakma commented

2026-04-13 14:31:31 +00:00

Testing the build:
The docker file one in the roomBookingApiMockup shall be renamed from DockerFIle to Dockerfile otherwise this command does not work: docker build -t roombookingapimockup .

Add this command docker build -t roombookingdbhandler . also in the README of the roomBookingDbHandler even though we have the build script since the current buildScript in linux requires that the image exist at least once then the script works

add this lines to UnixBuild.sh for Linux users: I will send the code below to DM as it here when I paste here it interprets the # as Header :

Before dockerCompose comment

Testing the build: The docker file one in the roomBookingApiMockup shall be renamed from `DockerFIle to Dockerfile `otherwise this command does not work: `docker build -t roombookingapimockup .` Add this command `docker build -t roombookingdbhandler .` also in the README of the roomBookingDbHandler even though we have the build script since the current buildScript in linux requires that the image exist at least once then the script works add this lines to UnixBuild.sh for Linux users: I will send the code below to DM as it here when I paste here it interprets the # as Header : `Before dockerCompose comment`

Screenshot from 2026-04-13 16-30-26.png

78 KiB

a24sakma commented

2026-04-13 14:36:58 +00:00

After adjusting the last comments I'all approve.

Screenshot from 2026-04-13 16-36-18.png

49 KiB

a24antej added 1 commit

2026-04-13 14:40:37 +00:00

fixed build linux and removed dockerfile because name wont change 4b9ea867cd

a24antej added 1 commit

2026-04-13 14:45:05 +00:00

added to README and re-created the dockerfile e1f742b3d1

a24sakma commented

2026-04-13 14:49:54 +00:00

@c24elipe @b24johka @a24antej

It looks good to me now approved 👍!

@c24elipe @b24johka @a24antej It looks good to me now approved 👍!

c24elipe requested review from a24sakma

2026-04-13 14:57:02 +00:00

b25marem commented

2026-04-13 16:13:02 +00:00

Tested building the Docker image for roomBookingApiMockup. The build completed successfully without errors.

image.png

197 KiB

b25marem commented

2026-04-13 16:15:30 +00:00

Tested building the Docker image for roomBookingDbHandler. The build completed successfully without errors.

image.png

48 KiB

b25marem commented

2026-04-13 16:51:31 +00:00

The test was performed using docker compose up --build.The result shows that booking data is generated,sent via MQTT, received by the middleware, and stored in PostgreSQL.This confirms that the system works correctly.

image.png

53 KiB

b25marem commented

2026-04-14 06:30:15 +00:00

I tried to verify the room booking flow together with the frontend, but I was not able to fully test the frontend end-to-end. From the logs, roomBookingApiMockup keeps printing "no data to upload waiting...", and roomBookingDbHandler eventually hits a HttpClient.Timeout after 100 seconds.

Additionally, when testing the API through nginx using /api/roombooking/all, it returns 404.

I’m not completely sure if I’m testing this correctly or if there is an issue in the setup, but based on the behavior it seems like the room booking backend is not consistently providing data. Because of that, I couldn’t properly verify the frontend part yet.

I tried to verify the room booking flow together with the frontend, but I was not able to fully test the frontend end-to-end. From the logs, roomBookingApiMockup keeps printing "no data to upload waiting...", and roomBookingDbHandler eventually hits a HttpClient.Timeout after 100 seconds. Additionally, when testing the API through nginx using /api/roombooking/all, it returns 404. I’m not completely sure if I’m testing this correctly or if there is an issue in the setup, but based on the behavior it seems like the room booking backend is not consistently providing data. Because of that, I couldn’t properly verify the frontend part yet.

b25marem added the

reviewed

label

2026-04-14 06:51:12 +00:00

a24antej commented

2026-04-14 06:51:18 +00:00

the backend is set to only provide 4 bookings and the "no data to upload waiting..." is the db handler waiting for potential more data, do not currently know how to work around it because the db hanlder runs in an forever loop.
need to know more how MQTT works before i can come upp with a better solution

the backend is set to only provide 4 bookings and the "no data to upload waiting..." is the db handler waiting for potential more data, do not currently know how to work around it because the db hanlder runs in an forever loop. need to know more how MQTT works before i can come upp with a better solution

👍 1

a24antej commented

2026-04-14 06:57:28 +00:00

when starting the project i used the windowsBuild script, there could be something that you have missed from that.

b25marem commented

2026-04-14 07:08:46 +00:00

Thanks, I used the build script as well, but I’ll go through it again to make sure I didn’t miss anything

b25marem commented

2026-04-14 08:00:14 +00:00

I ran the setup again using WindowsBuild.bat, and now everything works. The booking data shows up in the frontend, so it seems my previous issue was just due to my local setup.I think the issue earlier was that my local setup wasn’t fully initialized correctly.

image.png

112 KiB

b24johka commented

2026-04-14 08:26:29 +00:00

A question has been raised regarding this pull request, why would the conf file in entrypoint be changed? we have noticed that the other mockup generator did not change this file.

a24antej commented

2026-04-14 08:30:57 +00:00

this is to set this as a api(like tempService), this way you can reach it from the frontend like this "http://localhost/api/roombooking/all"

a24sakma commented

2026-04-14 08:36:42 +00:00

I just want to mention Parking Api also has this changed in the entry point. This is needed in order to communicate with the frontend.
https://git.webug.se/Andras/BoundlessFlowCampus2K/src/branch/team_1_week_1/entryPoint/nginx.conf

~~~It was just not mentioned in the pr comments. My bad. I can update the comment.~~ It was mentioned on point 7

I just want to mention Parking Api also has this changed in the entry point. This is needed in order to communicate with the frontend. https://git.webug.se/Andras/BoundlessFlowCampus2K/src/branch/team_1_week_1/entryPoint/nginx.conf ~~It was just not mentioned in the pr comments. My bad. I can update the comment.~ It was mentioned on point 7

Screenshot from 2026-04-14 10-33-33.png

63 KiB