Fix skipped/broken unit tests

There is a bunch of skipped tests, some of them were marked to be skipped in 643ebf57. It would be nice to enable them as they could prevent bugs such as the one in #7466 (closed).

assigned to @rajmund_hruska

By Rajmund Hruška on 2024-12-26T14:07:23

added C-Test Done 0% P4 S-New T-Task labels

changed milestone to %Backlog

Back Relation 'blocks' from: 7466

Relation 'relates' to: 7544

Relation Blocked - Bug 7466 Viewing details of abuse group in reports/dashboard results in internal server error

By Rajmund Hruška on 2021-08-08T14:41:03

In the last couple of days I have been trying to fix the skipped tests. I found out that I don't really know the purpose of vial, it seems to have the same functionality as hawat.

There is still one test marked to be skipped, it's lib/vial/blueprints/auth_api/test/__init__.py. I think I managed to fix a couple of tests, though.

Also, there are some tests which for some reason are not tested with make tests command. For example, lib/vial/blueprints/groups/test/__init__.py contains a test which results in error (the error is solved in #7466 (closed)), but when running make tests this error never appears.

By Rajmund Hruška on 2021-08-11T17:32:17

added Done 60% S-In Progress labels and removed Done 0% S-New labels

In one of the meetings, we talked about removing/merging vial and @Jan_Mach said he will do it. All of the skipped tests are in vial. Should I try fixing those tests before the merge is done?

By Rajmund Hruška on 2022-01-03T16:41:24

added S-Feedback label and removed S-In Progress label

Rajmund Hruska wrote in https://gitlab.cesnet.cz/709/migrace/homeproj/mentat-project-05/mentat-ng/-/issues/7494#note_200502:

In one of the meetings, we talked about removing/merging vial and @Jan_Mach said he will do it. All of the skipped tests are in vial. Should I try fixing those tests before the merge is done?

If all of them are from Vial, it's task for Vial merging (I guess at #7544 Mek will decide, whether those tests have their opposite in Hawat, or should be adapted, or should be ditched).

By Pavel Kácha on 2022-01-04T16:36:11

assigned to @Jan_Mach and unassigned @rajmund_hruska

By Pavel Kácha on 2024-12-26T14:24:01

Relation Relates - Task 7544 Resect Vial

By Pavel Kácha on 2022-01-04T16:36:50

assigned to @rajmund_hruska and unassigned @Jan_Mach

By Rajmund Hruška on 2024-12-26T14:24:02

added Done 100% S-Resolved labels and removed Done 60% S-Feedback labels

changed milestone to %2.10

All tests passed on Buildbot.

By Rajmund Hruška on 2022-07-27T15:39:35

added S-Closed label and removed S-Resolved label

closed

marked this issue as related to #7466 (closed)

By Rajmund Hruška on 2024-12-26T14:23:55