20250213 Weekly Development Meeting #134
Replies: 3 comments
-
Torrust TrackerI've continued working on the tracker core overhaul mainly adding tests. 98.13% of lines are covered. Basically what it's not covered are errors when SQL queries fail in the database drivers. That should only happen when there are problems with the database schema. For example, schema was altered without changing the SQL statements in the code. I have finished what I planned to do, however I'm considering adding some integration tests. Opened issues:
Issues:
Most of the work was just adding tests. I would only highlight that I have started using the testcontainers crate to run the MySQL container instead of writing custom code as we did for the TrackerContainer. Tests for the MySQL driver: https://github.com/torrust/torrust-tracker/blob/develop/packages/tracker-core/src/databases/driver/mysql.rs#L267 To run the tests for the MySQL driver you have to add this env var: TORRUST_TRACKER_CORE_RUN_MYSQL_DRIVER_TEST=true cargo test That's because I don't want to run those tests in the tracker docker image build. While I was adding tests to the database drivers I found 3 bugs. This is the final coverage report. I did not add tests for the dependencies of the
Note: time spent on Torrust 32,5h (including weekly meeting and meeting agenda). |
Beta Was this translation helpful? Give feedback.
-
Index
|
Beta Was this translation helpful? Give feedback.
-
Beta Was this translation helpful? Give feedback.
-
Location:
https://meeting.blockfinance-eco.li/nautiluscyberneeringweeklydevmeeting
Expected duration: 2 to 4 hours with breaks
Communicated attendants:
Agenda:
During the meeting every team member will take their turns and follow this sequence.
The topics for further discussion will be noted in advance by each the team member and to the meetings agenda.
set another meeting date for the discussion of these or to hold the meeting hereafter.
Also the persons to be part of the meeting will be decided on so that the others will be free to continue their own work.
Beta Was this translation helpful? Give feedback.
All reactions