Test for region deletion

[JoeyStk]

Short description

This PR adds a test for region deletion

Proposed changes

• Test to delete all regions
• Test that all regions which are mirrored can't be deleted

Side effects

• There might be some?

Resolved issues

Fixes: #3050

---

Pull Request Review Guidelines

[MizukiTemma]

Thank you for adding test 💪 I have some suggestions, please find in the comments 💬

[david-venhoff]

Do you remember when deleting a region caused internal server errors? Ideally we should catch these with the new tests

[JoeyStk]

Do you remember when deleting a region caused internal server errors? Ideally we should catch these with the new tests

These are the three Issues/PRs that I could find:

• Deleting regions not possible if media library has nested structure #1749 & Fix region deletion error if media library has nested structure #1778
• Region and user deletion not working #653 & Fix region and user deletion #654
• IntegrityError when deleting region with feedback #2462 & Fix IntegrityError when deleting region with multiple feedback #2506

I will add commits in the next days that also test for these use cases :)

[JoeyStk]

As I'm busy with a lot of other issues/projects please feel free to contribute to this PR. I don't see myself going back to this PR anytime soon :(

[MizukiTemma]

I'll take over then this PR.

Changes and checks so far sugegsted are now applied.

[jarlhengstmengel]

Thanks! Looks good to me :)

[jarlhengstmengel]

There is a typo here with "Region" instead "Regionen", but it's also in the translation file so I think it's probably just copied from there?

[MizukiTemma]

Hmm, I think singular is correct here, as delete_region handles actually one single region 👀

[david-venhoff]

Suggested change

	"Die Region konnte nicht gelöscht werden, weil die folgenden Seiten in anderen Region gespiegelt werden:"
	"Die Region konnte nicht gelöscht werden, weil die folgenden Seiten in anderen Regionen gespiegelt werden:"

I think @jarlhengstmengel was referring to the second occurrence, which should actually be plural.

[MizukiTemma]

@jarlhengstmengel @david-venhoff
Ah 🙈 Yes, it must be "Regionen" there. #3302