Changes:
* CHECK_STRIKES macro now takes a comma-delimited list of strike counts for when units have a different number of strikes for different weapons, and updates the existing usage in the attacks tests
* Fixed a typo in the attacks_zero test
* Adds tests for berserk as a weapon special ability.
Berserk's handling differs from attacks as a weapon special ability:
* A value less than 1 is treated as effectively infinite rounds of combat (undocumented)
* The cumulative attribute is handled differently - in some cases it sums the values instead of using the highest single value whereas for attacks it always uses the highest single value
Also now exclude data/tests/ from scons pot-update since it was hitting the argument limit for number of arguments to a script.
...for strings, too
this is done by a) scanning for all .lua files additionally to all
.cfg files in FINDCFG and b) by adding the equivalent of a "commented
out" #textdomain DOMAIN declaration
next step: testing if it really does work as expected this should also
fix bug #15629 since the objectives are now *only* declared via some
lua magic
sorry that the names will appear in both textdomains, but since they
do appear in the ingame help, too, there is no clean way to prevent it
since the editor is only allowed to read data from wesnoth-lib and
wesnoth-editor
