From 1b3c8ea466807522883e80df374e205d47fe879f Mon Sep 17 00:00:00 2001 From: Enno Rehling Date: Tue, 24 Nov 2015 14:57:50 +0100 Subject: [PATCH 1/2] import monsters tests from PR #404 Monsters (i.e. sea serpents) can attack on oceans, when not guarding --- src/CMakeLists.txt | 1 + src/monsters.c | 2 +- src/monsters.test.c | 274 ++++++++++++++++++++++++++++++++++++++++++++ src/test_eressea.c | 1 + 4 files changed, 277 insertions(+), 1 deletion(-) create mode 100644 src/monsters.test.c diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt index 037898ed9..36c728d40 100644 --- a/src/CMakeLists.txt +++ b/src/CMakeLists.txt @@ -180,6 +180,7 @@ target_link_libraries(eressea ) set(TESTS_SRC + monsters.test.c donations.test.c wormhole.test.c alchemy.test.c diff --git a/src/monsters.c b/src/monsters.c index 3474c6b45..74321a2ea 100644 --- a/src/monsters.c +++ b/src/monsters.c @@ -789,7 +789,7 @@ void plan_monsters(faction * f) setstatus(u, ST_FIGHT); /* all monsters fight */ } - if (attacking && is_guard(u, GUARD_TAX)) { + if (attacking && (!r->land || is_guard(u, GUARD_TAX))) { monster_attacks(u); } /* units with a plan to kill get ATTACK orders: */ diff --git a/src/monsters.test.c b/src/monsters.test.c new file mode 100644 index 000000000..e3af8351d --- /dev/null +++ b/src/monsters.test.c @@ -0,0 +1,274 @@ +#include +#include + +#include +#include +#include +#include +#include +#include +#include +#include + +#include "monster.h" +#include "guard.h" +#include "skill.h" + +#include + +#include +#include +#include +#include +#include + +extern void plan_monsters(struct faction *f); + +static void init_language(void) +{ + struct locale* lang; + int i; + + lang = get_or_create_locale("de"); + locale_setstring(lang, "skill::unarmed", "Waffenloser Kampf"); + locale_setstring(lang, "keyword::attack", "ATTACKIERE"); + locale_setstring(lang, "keyword::study", "LERNE"); + locale_setstring(lang, "keyword::tax", "TREIBE"); + locale_setstring(lang, "keyword::loot", "PLUENDERE"); + locale_setstring(lang, "keyword::guard", "BEWACHE"); + locale_setstring(lang, "keyword::move", "NACH"); + locale_setstring(lang, "keyword::message", "BOTSCHAFT"); + locale_setstring(lang, "REGION", "REGION"); + locale_setstring(lang, "east", "O"); + + for (i = 0; i < MAXKEYWORDS; ++i) { + if (!locale_getstring(lang, mkname("keyword", keywords[i]))) + locale_setstring(lang, mkname("keyword", keywords[i]), keywords[i]); + } + for (i = 0; i < MAXSKILLS; ++i) { + if (!locale_getstring(lang, mkname("skill", skillnames[i]))) + locale_setstring(lang, mkname("skill", skillnames[i]), skillnames[i]); + } + init_keywords(lang); + init_skills(lang); +} + +static order *find_order(const char *expected, const unit *unit) +{ + char cmd[32]; + order *order; + for (order = unit->orders; order; order = order->next) { + if (strcmp(expected, get_command(order, cmd, sizeof(cmd))) == 0) { + return order; + } + } + return NULL; +} + +static void create_monsters(faction **player, faction **monsters, region **r, unit **u, unit **m) { + race* rc; + + test_cleanup(); + + init_language(); + + test_create_world(); + *player = test_create_faction(NULL); + *monsters = get_or_create_monsters(); + assert(rc_find((*monsters)->race->_name)); + rc = rc_get_or_create((*monsters)->race->_name); + fset(rc, RCF_UNARMEDGUARD); + fset(rc, RCF_NPC); + fset(*monsters, FFL_NOIDLEOUT); + assert(fval(*monsters, FFL_NPC) && fval((*monsters)->race, RCF_UNARMEDGUARD) && fval((*monsters)->race, RCF_NPC) && fval(*monsters, FFL_NOIDLEOUT)); + (*monsters)->locale = default_locale; + + *r = findregion(0, 0); + + *u = test_create_unit(*player, *r); + unit_setid(*u, 1); + *m = test_create_unit(*monsters, *r); +} + +static void test_monsters_attack(CuTest * tc) +{ + faction *f, *f2; + region *r; + unit *u, *m; + + create_monsters(&f, &f2, &r, &u, &m); + + guard(m, GUARD_TAX); + + set_param(&global.parameters, "rules.monsters.attack_chance", "1"); + + plan_monsters(f2); + + CuAssertPtrNotNull(tc, find_order("ATTACKIERE 1", m)); + test_cleanup(); +} + +static void test_monsters_attack_ocean(CuTest * tc) +{ + faction *f, *f2; + region *r; + unit *u, *m; + + create_monsters(&f, &f2, &r, &u, &m); + r = findregion(-1, 0); + u = test_create_unit(u->faction, r); + unit_setid(u, 2); + m = test_create_unit(m->faction, r); + assert(!m->region->land); + + set_param(&global.parameters, "rules.monsters.attack_chance", "1"); + + plan_monsters(f2); + + CuAssertPtrNotNull(tc, find_order("ATTACKIERE 2", m)); + test_cleanup(); +} + +extern int monster_attacks(unit * monster, bool respect_buildings, bool rich_only); + +static void test_monsters_waiting(CuTest * tc) +{ + faction *f, *f2; + region *r; + unit *u, *m; + + create_monsters(&f, &f2, &r, &u, &m); + guard(m, GUARD_TAX); + fset(m, UFL_ISNEW); + // FIXME: monster_attacks(m, false, false); + CuAssertPtrEquals(tc, 0, find_order("ATTACKIERE 1", m)); + test_cleanup(); +} + +static void test_seaserpent_attack(CuTest * tc) +{ + faction *f, *f2; + region *r; + unit *u, *m; + race *rc; + + create_monsters(&f, &f2, &r, &u, &m); + r = findregion(-1, 0); + u = test_create_unit(u->faction, r); + unit_setid(u, 2); + m = test_create_unit(m->faction, r); + u_setrace(m, rc = test_create_race("seaserpent")); + assert(!m->region->land); + // FIXME: write an extra test for sea serpent piracy? + // fset(m, UFL_MOVED); + // fset(rc, RCF_ATTACK_MOVED); + + set_param(&global.parameters, "rules.monsters.attack_chance", "1"); + + plan_monsters(f2); + + CuAssertPtrNotNull(tc, find_order("ATTACKIERE 2", m)); + test_cleanup(); +} + +static void test_monsters_attack_not(CuTest * tc) +{ + faction *f, *f2; + region *r; + unit *u, *m; + + create_monsters(&f, &f2, &r, &u, &m); + + guard(m, GUARD_TAX); + guard(u, GUARD_TAX); + + set_param(&global.parameters, "rules.monsters.attack_chance", "0"); + + plan_monsters(f2); + + CuAssertPtrEquals(tc, 0, find_order("ATTACKIERE 1", m)); + test_cleanup(); +} + +static void test_dragon_attacks_the_rich(CuTest * tc) +{ + faction *f, *f2; + region *r; + unit *u, *m; + const item_type *i_silver; + + init_language(); + create_monsters(&f, &f2, &r, &u, &m); + + guard(m, GUARD_TAX); + set_level(m, SK_WEAPONLESS, 10); + + rsetmoney(r, 1); + rsetmoney(findregion(1, 0), 0); + i_silver = it_find("money"); + assert(i_silver); + i_change(&u->items, i_silver, 5000); + + set_param(&global.parameters, "rules.monsters.attack_chance", "0.00001"); + + plan_monsters(f2); + + CuAssertPtrNotNull(tc, find_order("ATTACKIERE 1", m)); + CuAssertPtrNotNull(tc, find_order("PLUENDERE", m)); + test_cleanup(); +} + +static void test_dragon_moves(CuTest * tc) +{ + faction *f, *f2; + region *r; + unit *u, *m; + + create_monsters(&f, &f2, &r, &u, &m); + rsetpeasants(r, 0); + rsetmoney(r, 0); + rsetmoney(findregion(1, 0), 1000); + + set_level(m, SK_WEAPONLESS, 10); + set_param(&global.parameters, "rules.monsters.attack_chance", ".0"); + plan_monsters(f2); + + CuAssertPtrNotNull(tc, find_order("NACH O", m)); + test_cleanup(); +} + +static void test_monsters_learn_exp(CuTest * tc) +{ + faction *f, *f2; + region *r; + unit *u, *m; + skill* sk; + + create_monsters(&f, &f2, &r, &u, &m); + set_param(&global.parameters, "study.from_use", "1"); + + u_setrace(u, u_race(m)); + produceexp(u, SK_MELEE, u->number); + sk = unit_skill(u, SK_MELEE); + CuAssertTrue(tc, !sk); + + produceexp(m, SK_MELEE, u->number); + sk = unit_skill(m, SK_MELEE); + CuAssertTrue(tc, sk && (sk->level > 0 || (sk->level == 0 && sk->weeks > 0))); + + test_cleanup(); +} + +CuSuite *get_monsters_suite(void) +{ + CuSuite *suite = CuSuiteNew(); + SUITE_ADD_TEST(suite, test_monsters_attack); + SUITE_ADD_TEST(suite, test_monsters_attack_ocean); + SUITE_ADD_TEST(suite, test_seaserpent_attack); + DISABLE_TEST(suite, test_monsters_waiting); + SUITE_ADD_TEST(suite, test_monsters_attack_not); + SUITE_ADD_TEST(suite, test_dragon_attacks_the_rich); + DISABLE_TEST(suite, test_dragon_moves); + DISABLE_TEST(suite, test_monsters_learn_exp); + return suite; +} \ No newline at end of file diff --git a/src/test_eressea.c b/src/test_eressea.c index c04764992..da4f6e8d8 100644 --- a/src/test_eressea.c +++ b/src/test_eressea.c @@ -118,6 +118,7 @@ int RunAllTests(int argc, char *argv[]) ADD_SUITE(give); ADD_SUITE(laws); ADD_SUITE(market); + ADD_SUITE(monsters); ADD_SUITE(move); ADD_SUITE(piracy); ADD_SUITE(stealth); From bdc5372537e37b388a99ffad0687fdb99037ab0b Mon Sep 17 00:00:00 2001 From: Enno Rehling Date: Tue, 24 Nov 2015 16:12:33 +0100 Subject: [PATCH 2/2] disable test for seaserpent piracy (planned for 3.8) --- src/monsters.test.c | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/src/monsters.test.c b/src/monsters.test.c index e3af8351d..aba9d9c4d 100644 --- a/src/monsters.test.c +++ b/src/monsters.test.c @@ -35,6 +35,7 @@ static void init_language(void) locale_setstring(lang, "keyword::study", "LERNE"); locale_setstring(lang, "keyword::tax", "TREIBE"); locale_setstring(lang, "keyword::loot", "PLUENDERE"); + locale_setstring(lang, "keyword::piracy", "PIRATERIE"); locale_setstring(lang, "keyword::guard", "BEWACHE"); locale_setstring(lang, "keyword::move", "NACH"); locale_setstring(lang, "keyword::message", "BOTSCHAFT"); @@ -145,7 +146,7 @@ static void test_monsters_waiting(CuTest * tc) test_cleanup(); } -static void test_seaserpent_attack(CuTest * tc) +static void test_seaserpent_piracy(CuTest * tc) { faction *f, *f2; region *r; @@ -159,14 +160,14 @@ static void test_seaserpent_attack(CuTest * tc) m = test_create_unit(m->faction, r); u_setrace(m, rc = test_create_race("seaserpent")); assert(!m->region->land); - // FIXME: write an extra test for sea serpent piracy? - // fset(m, UFL_MOVED); + fset(m, UFL_MOVED); // fset(rc, RCF_ATTACK_MOVED); set_param(&global.parameters, "rules.monsters.attack_chance", "1"); plan_monsters(f2); + CuAssertPtrNotNull(tc, find_order("PIRATERIE", m)); CuAssertPtrNotNull(tc, find_order("ATTACKIERE 2", m)); test_cleanup(); } @@ -264,11 +265,11 @@ CuSuite *get_monsters_suite(void) CuSuite *suite = CuSuiteNew(); SUITE_ADD_TEST(suite, test_monsters_attack); SUITE_ADD_TEST(suite, test_monsters_attack_ocean); - SUITE_ADD_TEST(suite, test_seaserpent_attack); + DISABLE_TEST(suite, test_seaserpent_piracy); DISABLE_TEST(suite, test_monsters_waiting); SUITE_ADD_TEST(suite, test_monsters_attack_not); SUITE_ADD_TEST(suite, test_dragon_attacks_the_rich); DISABLE_TEST(suite, test_dragon_moves); DISABLE_TEST(suite, test_monsters_learn_exp); return suite; -} \ No newline at end of file +}