text-based

Is your feature request related to a problem? Please describe.

Right now, this condition check on the delete() outcome has no value.

Describe the solution you'd like

I'd like to see the DefaultScript.delete() method check to make sure the deletion was done properly and then return a boolean value.

Or as @Griatch put it

Brief summary of issue / Description of requested feature:

We have one usage of Q_FOREACH. This had been discouraged for some time (since 2016-06-16).
https://github.com/Mudlet/Mudlet/blob/c5285d17615029a7af245948f7dad18c8c39fecc/src/Host.cpp#L1785

Steps to reproduce the issue / Reasons for adding feature:

1. Look at Qt documentation within Qt Creator:

`Q_FOREACH(variable, co

Refactor classes (including tests) to resolve dependencies out of the IoC container with $this->dep = app($class::class) in the constructor, instead of through the constructor method arguments.

Constructor arg lists get quite long (see PopulationCalculator), and needs fixing test constructors whenever this changes.

Beep, boop, I'm a bot! Th

The number of tests should probably be increased, and each test should cover all possible use cases of each commands. Certain existing tests might contain useless lines or need to check more use cases.