diff options
Diffstat (limited to 'domain')
| -rw-r--r-- | domain/include/turns/domain/participant.hpp | 12 | ||||
| -rw-r--r-- | domain/include/turns/domain/turn_order.hpp | 3 | ||||
| -rw-r--r-- | domain/src/participant.cpp | 27 | ||||
| -rw-r--r-- | domain/src/turn_order.cpp | 4 | ||||
| -rw-r--r-- | domain/tests/participant.cpp | 52 | ||||
| -rw-r--r-- | domain/tests/turn_order.cpp | 46 |
6 files changed, 108 insertions, 36 deletions
diff --git a/domain/include/turns/domain/participant.hpp b/domain/include/turns/domain/participant.hpp index 3f249d5..d845c77 100644 --- a/domain/include/turns/domain/participant.hpp +++ b/domain/include/turns/domain/participant.hpp @@ -1,6 +1,8 @@ #ifndef TURNS_DOMAIN_PARTICIPANT_HPP #define TURNS_DOMAIN_PARTICIPANT_HPP +#include "turns/domain/disposition.hpp" + #include <compare> #include <glibmm/object.h> @@ -14,12 +16,17 @@ namespace turns::domain struct participant : Glib::Object { - auto static create(Glib::ustring name, float priority) -> Glib::RefPtr<participant>; + auto static create(Glib::ustring name, float priority, disposition disposition) -> Glib::RefPtr<participant>; - participant(Glib::ustring name, float priority); + participant(Glib::ustring name, float priority, disposition disposition); auto operator<=>(participant const & other) const noexcept -> std::partial_ordering; + auto property_disposition() -> Glib::PropertyProxy<disposition>; + auto property_disposition() const -> Glib::PropertyProxy_ReadOnly<disposition>; + auto get_disposition() const noexcept -> disposition; + auto set_disposition(disposition value) -> void; + auto property_name() -> Glib::PropertyProxy<Glib::ustring>; auto property_name() const -> Glib::PropertyProxy_ReadOnly<Glib::ustring>; auto get_name() const -> Glib::ustring; @@ -31,6 +38,7 @@ namespace turns::domain auto set_priority(float value) -> void; private: + Glib::Property<disposition> m_disposition; Glib::Property<Glib::ustring> m_name; Glib::Property<float> m_priority; }; diff --git a/domain/include/turns/domain/turn_order.hpp b/domain/include/turns/domain/turn_order.hpp index ba9bc3a..d7f8821 100644 --- a/domain/include/turns/domain/turn_order.hpp +++ b/domain/include/turns/domain/turn_order.hpp @@ -1,6 +1,7 @@ #ifndef TURNS_DOMAIN_TURN_ORDER_HPP #define TURNS_DOMAIN_TURN_ORDER_HPP +#include "turns/domain/disposition.hpp" #include "turns/domain/participant.hpp" #include <giomm/liststore.h> @@ -20,7 +21,7 @@ namespace turns::domain auto static create() -> Glib::RefPtr<turn_order>; auto append(Glib::RefPtr<participant> item) -> void; - auto append(Glib::ustring const & name, float priority) -> void; + auto append(Glib::ustring const & name, float priority, disposition disposition) -> void; auto remove(Glib::RefPtr<participant> item) -> void; private: diff --git a/domain/src/participant.cpp b/domain/src/participant.cpp index 08c6deb..5265eb3 100644 --- a/domain/src/participant.cpp +++ b/domain/src/participant.cpp @@ -8,13 +8,14 @@ namespace turns::domain { - auto participant::create(Glib::ustring name, float priority) -> Glib::RefPtr<participant> + auto participant::create(Glib::ustring name, float priority, disposition disposition) -> Glib::RefPtr<participant> { - return Glib::make_refptr_for_instance(new participant{name, priority}); + return Glib::make_refptr_for_instance(new participant{name, priority, disposition}); } - participant::participant(Glib::ustring name, float priority) + participant::participant(Glib::ustring name, float priority, disposition disposition) : Glib::ObjectBase{typeid(participant)} + , m_disposition{*this, "disposition", disposition} , m_name{*this, "name", name} , m_priority{*this, "priority", priority} { @@ -25,6 +26,16 @@ namespace turns::domain return m_priority <=> other.m_priority; } + auto participant::get_disposition() const noexcept -> disposition + { + return m_disposition; + } + + auto participant::set_disposition(disposition value) -> void + { + m_disposition = value; + } + auto participant::get_name() const -> Glib::ustring { return m_name; @@ -45,6 +56,16 @@ namespace turns::domain m_priority = value; } + auto participant::property_disposition() -> Glib::PropertyProxy<disposition> + { + return m_disposition.get_proxy(); + } + + auto participant::property_disposition() const -> Glib::PropertyProxy_ReadOnly<disposition> + { + return m_disposition.get_proxy(); + } + auto participant::property_name() -> Glib::PropertyProxy<Glib::ustring> { return m_name.get_proxy(); diff --git a/domain/src/turn_order.cpp b/domain/src/turn_order.cpp index eddd468..4f1ed3b 100644 --- a/domain/src/turn_order.cpp +++ b/domain/src/turn_order.cpp @@ -42,9 +42,9 @@ namespace turns::domain } } - auto turn_order::append(Glib::ustring const & name, float priority) -> void + auto turn_order::append(Glib::ustring const & name, float priority, disposition disposition) -> void { - auto participant = participant::create(name, priority); + auto participant = participant::create(name, priority, disposition); participant->property_priority().signal_changed().connect([this] { sort(comparator); }); append(participant); } diff --git a/domain/tests/participant.cpp b/domain/tests/participant.cpp index 6db98f7..dd244f4 100644 --- a/domain/tests/participant.cpp +++ b/domain/tests/participant.cpp @@ -1,4 +1,5 @@ #include "turns/domain/participant.hpp" +#include "turns/domain/disposition.hpp" #include <catch2/catch_test_macros.hpp> @@ -13,11 +14,52 @@ namespace turns::domain::tests { auto constexpr constructed_name = "Vana Thistletop"; auto constexpr constructed_priority = 17; - auto instance = participant{constructed_name, constructed_priority}; + auto constexpr constructed_disposition = disposition::friendly; + auto instance = participant{constructed_name, constructed_priority, constructed_disposition}; SECTION("can be created") { - REQUIRE(participant::create(constructed_name, constructed_priority)); + REQUIRE(participant::create(constructed_name, constructed_priority, constructed_disposition)); + } + + SECTION("allows access to its disposition via the associated accessors") + { + SECTION("allowing to get it") + { + REQUIRE(instance.get_disposition() == constructed_disposition); + } + + SECTION("allowing to get it via a constant object") + { + auto const & cref = instance; + REQUIRE(cref.get_disposition() == constructed_disposition); + } + + SECTION("allowing to set it") + { + instance.set_disposition(disposition::hostile); + REQUIRE(instance.get_disposition() == disposition::hostile); + } + } + + SECTION("allows access to its disposition via the associated property") + { + SECTION("allowing to get it") + { + REQUIRE(instance.property_disposition() == constructed_disposition); + } + + SECTION("allowing to get it via a constant object") + { + auto const & cref = instance; + REQUIRE(cref.property_disposition() == constructed_disposition); + } + + SECTION("allowing to set it") + { + instance.property_disposition() = disposition::hostile; + REQUIRE(instance.get_disposition() == disposition::hostile); + } } SECTION("allows access to its name via the associated accessors") @@ -102,9 +144,9 @@ namespace turns::domain::tests SECTION("can be compared with another participant") { - auto equivalent_instance = participant{"Equivalent", constructed_priority}; - auto lesser_instance = participant{"Lesser", constructed_priority - 1}; - auto greater_instance = participant{"Greater", constructed_priority + 1}; + auto equivalent_instance = participant{"Equivalent", constructed_priority, constructed_disposition}; + auto lesser_instance = participant{"Lesser", constructed_priority - 1, constructed_disposition}; + auto greater_instance = participant{"Greater", constructed_priority + 1, constructed_disposition}; SECTION("yielding std::partial_ordering::equivalent for itself") { diff --git a/domain/tests/turn_order.cpp b/domain/tests/turn_order.cpp index 7172ffc..f1f2653 100644 --- a/domain/tests/turn_order.cpp +++ b/domain/tests/turn_order.cpp @@ -27,19 +27,19 @@ namespace turns::domain::tests SECTION("accepts a new participant in form of a refptr") { - instance.append(participant::create("Honey Bunches of Oats", 12)); + instance.append(participant::create("Honey Bunches of Oats", 12, disposition::friendly)); REQUIRE(instance.get_n_items() == 1); } SECTION("accepts a new participant in form of components") { - instance.append("River along the Field", 14); + instance.append("River along the Field", 14, disposition::friendly); REQUIRE(instance.get_n_items() == 1); } SECTION("does nothing when trying to remove an item by refptr if no items were added beforehand") { - instance.remove(participant::create("Patch in the Forest", 3)); + instance.remove(participant::create("Patch in the Forest", 3, disposition::friendly)); REQUIRE(instance.get_n_items() == 0); } @@ -51,7 +51,7 @@ namespace turns::domain::tests SECTION("allows the removal of an item by refptr if the same item was added beforehand") { - auto item = participant::create("Blank Canvas", 23); + auto item = participant::create("Blank Canvas", 23, disposition::friendly); instance.append(item); instance.remove(item); REQUIRE(instance.get_n_items() == 0); @@ -59,9 +59,9 @@ namespace turns::domain::tests SECTION("does nothing when trying to remove an item by refptr that was not added beforehand") { - auto item = participant::create("Blank Canvas", 23); + auto item = participant::create("Blank Canvas", 23, disposition::friendly); instance.append(item); - instance.remove(participant::create("Spell of Rain", 6)); + instance.remove(participant::create("Spell of Rain", 6, disposition::friendly)); REQUIRE(instance.get_n_items() == 1); } @@ -69,22 +69,22 @@ namespace turns::domain::tests { SECTION("when appending the higher one last") { - instance.append(participant::create("Snow on the Field", 2)); - instance.append(participant::create("Bees behind the Cottage", 8)); + instance.append(participant::create("Snow on the Field", 2, disposition::friendly)); + instance.append(participant::create("Bees behind the Cottage", 8, disposition::friendly)); REQUIRE(instance.get_item(0)->get_name() == "Bees behind the Cottage"); } SECTION("when appending the higher one first") { - instance.append(participant::create("Bees behind the Cottage", 8)); - instance.append(participant::create("Snow on the Field", 2)); + instance.append(participant::create("Bees behind the Cottage", 8, disposition::friendly)); + instance.append(participant::create("Snow on the Field", 2, disposition::friendly)); REQUIRE(instance.get_item(0)->get_name() == "Bees behind the Cottage"); } SECTION("keeping the insertion order when appending items with equal priority") { - instance.append(participant::create("Snow on the Field", 8)); - instance.append(participant::create("Bees behind the Cottage", 8)); + instance.append(participant::create("Snow on the Field", 8, disposition::friendly)); + instance.append(participant::create("Bees behind the Cottage", 8, disposition::friendly)); REQUIRE(instance.get_item(0)->get_name() == "Snow on the Field"); } } @@ -93,29 +93,29 @@ namespace turns::domain::tests { SECTION("when appending the higher one last") { - instance.append("Tree Blossom", 6); - instance.append("Fish in the River", 12); + instance.append("Tree Blossom", 6, disposition::friendly); + instance.append("Fish in the River", 12, disposition::friendly); REQUIRE(instance.get_item(0)->get_name() == "Fish in the River"); } SECTION("when appending the higher one first") { - instance.append("Fish in the River", 12); - instance.append("Tree Blossom", 6); + instance.append("Fish in the River", 12, disposition::friendly); + instance.append("Tree Blossom", 6, disposition::friendly); REQUIRE(instance.get_item(0)->get_name() == "Fish in the River"); } SECTION("keeping the insertion order when appending items with equal priority") { - instance.append("Fish in the River", 6); - instance.append("Tree Blossom", 6); + instance.append("Fish in the River", 6, disposition::friendly); + instance.append("Tree Blossom", 6, disposition::friendly); REQUIRE(instance.get_item(0)->get_name() == "Fish in the River"); } } SECTION("does not accept the same item twice by the same refptr") { - auto item = participant::create("Angelic Berry", 9); + auto item = participant::create("Angelic Berry", 9, disposition::friendly); instance.append(item); instance.append(item); REQUIRE(instance.get_n_items() == 1); @@ -123,8 +123,8 @@ namespace turns::domain::tests SECTION("does not accept the same item twice by different refptrs") { - auto item_one = participant::create("Misty Meadow", 14.2); - auto item_two = participant::create("Misty Meadow", 14.2); + auto item_one = participant::create("Misty Meadow", 14.2, disposition::friendly); + auto item_two = participant::create("Misty Meadow", 14.2, disposition::friendly); instance.append(item_one); instance.append(item_two); REQUIRE(instance.get_n_items() == 1); @@ -132,8 +132,8 @@ namespace turns::domain::tests SECTION("does not accept the same item twice by components") { - instance.append("Frozen Apple Tree", 2.1); - instance.append("Frozen Apple Tree", 2.1); + instance.append("Frozen Apple Tree", 2.1, disposition::friendly); + instance.append("Frozen Apple Tree", 2.1, disposition::friendly); REQUIRE(instance.get_n_items() == 1); } } |
