]> git.stg.codes - stg.git/blobdiff - tests/test_reconnect_on_tariff_change.cpp
Simplify notifiers.
[stg.git] / tests / test_reconnect_on_tariff_change.cpp
index 60e36ffa64df3d0c15836d8dc7b7fb125741a786..c47bc1fdb2ee15b2cea5927650f6aec98a662fa9 100644 (file)
@@ -1,4 +1,4 @@
-#include "tut/tut.hpp"
+#define BOOST_TEST_MODULE STGReconnectOnTariffChange
 
 #include "stg/admin.h"
 #include "stg/user_property.h"
 #include "testusers.h"
 #include "testservices.h"
 
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wold-style-cast"
+#pragma GCC diagnostic ignored "-Wunused-parameter"
+#pragma GCC diagnostic ignored "-Wsign-compare"
+#pragma GCC diagnostic ignored "-Wparentheses"
+#include <boost/test/unit_test.hpp>
+#pragma GCC diagnostic pop
+
+volatile time_t stgTime = 0;
+
 namespace
 {
 
-class AFTER_CONNECTED_NOTIFIER : public STG::PropertyNotifierBase<bool> {
-public:
-    AFTER_CONNECTED_NOTIFIER()
-        : connects(0),
-          disconnects(0)
-    {}
-    void Notify(const bool & oldValue, const bool & newValue);
+class AfterConnectedNotifier : public STG::PropertyNotifierBase<bool>
+{
+    public:
+        AfterConnectedNotifier()
+            : m_connects(0),
+              m_disconnects(0)
+        {}
+
+        void notify(const bool& oldValue, const bool& newValue) override
+        {
+            if (!oldValue && newValue)
+                ++m_connects;
+            if (oldValue && !newValue)
+                ++m_disconnects;
+        }
 
-    size_t GetConnects() const { return connects; }
-    size_t GetDisconnects() const { return disconnects; }
+        size_t connects() const { return m_connects; }
+        size_t disconnects() const { return m_disconnects; }
 
-private:
-    size_t connects;
-    size_t disconnects;
+    private:
+        size_t m_connects;
+        size_t m_disconnects;
 };
 
-class TEST_SETTINGS_LOCAL : public TEST_SETTINGS {
+class Settings : public TestSettings
+{
     public:
-        TEST_SETTINGS_LOCAL(bool _reconnectOnTariffChange)
-            : TEST_SETTINGS(),
-              reconnectOnTariffChange(_reconnectOnTariffChange)
+        Settings(bool reconnectOnTariffChange)
+            : m_reconnectOnTariffChange(reconnectOnTariffChange)
         {}
 
-        bool GetReconnectOnTariffChange() const { return reconnectOnTariffChange; }
+        bool GetReconnectOnTariffChange() const { return m_reconnectOnTariffChange; }
 
     private:
-        bool reconnectOnTariffChange;
+        bool m_reconnectOnTariffChange;
 };
 
 }
 
-namespace tut
-{
-    struct reconnect_on_tariff_change_data {
-    };
-
-    typedef test_group<reconnect_on_tariff_change_data> tg;
-    tg reconnect_on_tariff_change_test_group("Reconnect on tariff change tests group");
+BOOST_AUTO_TEST_SUITE(ReconnectOnTariffChange)
 
-    typedef tg::object testobject;
-
-    template<>
-    template<>
-    void testobject::test<1>()
-    {
-        set_test_name("Check normal behaviour");
-
-        TEST_SETTINGS_LOCAL settings(false);
-        TEST_TARIFFS tariffs;
-        tariffs.ReadTariffs();
-        STG::Admin admin(STG::Priv(0xFFFF), {}, {});
-        TEST_STORE store;
-        TEST_AUTH auth;
-        TEST_USERS users;
-        TEST_SERVICES services;
-        STG::UserImpl user(&settings, &store, &tariffs, &admin, &users, services);
-
-        AFTER_CONNECTED_NOTIFIER connectionNotifier;
-
-        user.AddConnectedAfterNotifier(&connectionNotifier);
+BOOST_AUTO_TEST_CASE(NormalBehavior)
+{
+    Settings settings(false);
+    TestTariffs tariffs;
+    tariffs.ReadTariffs();
+    STG::Admin admin(STG::Priv(0xFFFF), {}, {});
+    TestStore store;
+    TestAuth auth;
+    TestUsers users;
+    TestServices services;
+    STG::UserImpl user(&settings, &store, &tariffs, &admin, &users, services);
 
-        STG::UserProperty<std::string> & tariffName = user.GetProperties().tariffName;
-        STG::UserProperty<STG::UserIPs> & ips = user.GetProperties().ips;
+    AfterConnectedNotifier connectionNotifier;
 
-        ips = STG::UserIPs::parse("*");
+    user.AddConnectedAfterNotifier(&connectionNotifier);
 
-        ensure_equals("user.connected = false", user.GetConnected(), false);
-        ensure_equals("connects = 0", connectionNotifier.GetConnects(), static_cast<size_t>(0));
-        ensure_equals("disconnects = 0", connectionNotifier.GetDisconnects(), static_cast<size_t>(0));
+    STG::UserProperty<std::string> & tariffName = user.GetProperties().tariffName;
+    STG::UserProperty<STG::UserIPs> & ips = user.GetProperties().ips;
 
-        ensure_equals("user.tariffName == NO_TARIFF_NAME", user.GetProperties().tariffName.ConstData(), NO_TARIFF_NAME);
+    ips = STG::UserIPs::parse("*");
 
-        user.Authorize(inet_strington("127.0.0.1"), 0, &auth);
-        user.Run();
+    BOOST_CHECK_EQUAL(user.GetConnected(), false);
+    BOOST_CHECK_EQUAL(connectionNotifier.connects(), static_cast<size_t>(0));
+    BOOST_CHECK_EQUAL(connectionNotifier.disconnects(), static_cast<size_t>(0));
 
-        ensure_equals("user.authorised_by = true", user.IsAuthorizedBy(&auth), true);
+    BOOST_CHECK_EQUAL(user.GetProperties().tariffName.ConstData(), NO_TARIFF_NAME);
 
-        ensure_equals("user.connected = true", user.GetConnected(), true);
-        ensure_equals("connects = 1", connectionNotifier.GetConnects(), static_cast<size_t>(1));
-        ensure_equals("disconnects = 0", connectionNotifier.GetDisconnects(), static_cast<size_t>(0));
+    user.Authorize(inet_strington("127.0.0.1"), 0, &auth);
+    user.Run();
 
-        tariffName = "test";
-        ensure_equals("user.tariffName == 'test'", user.GetProperties().tariffName.ConstData(), "test");
+    BOOST_CHECK_EQUAL(user.IsAuthorizedBy(&auth), true);
 
-        ensure_equals("user.authorised_by = true", user.IsAuthorizedBy(&auth), true);
+    BOOST_CHECK_EQUAL(user.GetConnected(), true);
+    BOOST_CHECK_EQUAL(connectionNotifier.connects(), static_cast<size_t>(1));
+    BOOST_CHECK_EQUAL(connectionNotifier.disconnects(), static_cast<size_t>(0));
 
-        ensure_equals("user.connected = true", user.GetConnected(), true);
-        ensure_equals("connects = 1", connectionNotifier.GetConnects(), static_cast<size_t>(1));
-        ensure_equals("disconnects = 0", connectionNotifier.GetDisconnects(), static_cast<size_t>(0));
-    }
+    tariffName = "test";
+    BOOST_CHECK_EQUAL(user.GetProperties().tariffName.ConstData(), "test");
 
+    BOOST_CHECK_EQUAL(user.IsAuthorizedBy(&auth), true);
 
-    template<>
-    template<>
-    void testobject::test<2>()
-    {
-        set_test_name("Check reconnect on tariff change");
+    BOOST_CHECK_EQUAL(user.GetConnected(), true);
+    BOOST_CHECK_EQUAL(connectionNotifier.connects(), static_cast<size_t>(1));
+    BOOST_CHECK_EQUAL(connectionNotifier.disconnects(), static_cast<size_t>(0));
+}
 
-        TEST_SETTINGS_LOCAL settings(true);
+BOOST_AUTO_TEST_CASE(Reconnect)
+{
+    Settings settings(true);
 
-        TEST_SETTINGS * s1 = &settings;
-        STG::Settings * s2 = &settings;
+    TestSettings * s1 = &settings;
+    STG::Settings * s2 = &settings;
 
-        ensure("settings.GetReconnectOnTariffChange() == true", settings.GetReconnectOnTariffChange());
-        ensure("s1->GetReconnectOnTariffChange() == true", s1->GetReconnectOnTariffChange());
-        ensure("s2->GetReconnectOnTariffChange() == true", s2->GetReconnectOnTariffChange());
+    BOOST_CHECK(settings.GetReconnectOnTariffChange());
+    BOOST_CHECK(s1->GetReconnectOnTariffChange());
+    BOOST_CHECK(s2->GetReconnectOnTariffChange());
 
-        TEST_TARIFFS tariffs;
-        STG::Admin admin(STG::Priv(0xFFFF), {}, {});
-        TEST_STORE store;
-        TEST_AUTH auth;
-        TEST_USERS users;
-        TEST_SERVICES services;
-        STG::UserImpl user(&settings, &store, &tariffs, &admin, &users, services);
+    TestTariffs tariffs;
+    STG::Admin admin(STG::Priv(0xFFFF), {}, {});
+    TestStore store;
+    TestAuth auth;
+    TestUsers users;
+    TestServices services;
+    STG::UserImpl user(&settings, &store, &tariffs, &admin, &users, services);
 
-        AFTER_CONNECTED_NOTIFIER connectionNotifier;
+    AfterConnectedNotifier connectionNotifier;
 
-        user.AddConnectedAfterNotifier(&connectionNotifier);
+    user.AddConnectedAfterNotifier(&connectionNotifier);
 
-        STG::UserProperty<std::string> & tariffName = user.GetProperties().tariffName;
-        STG::UserProperty<STG::UserIPs> & ips = user.GetProperties().ips;
+    STG::UserProperty<std::string> & tariffName = user.GetProperties().tariffName;
+    STG::UserProperty<STG::UserIPs> & ips = user.GetProperties().ips;
 
-        ips = STG::UserIPs::parse("*");
+    ips = STG::UserIPs::parse("*");
 
-        ensure_equals("user.connected = false", user.GetConnected(), false);
-        ensure_equals("connects = 0", connectionNotifier.GetConnects(), static_cast<size_t>(0));
-        ensure_equals("disconnects = 0", connectionNotifier.GetDisconnects(), static_cast<size_t>(0));
+    BOOST_CHECK_EQUAL(user.GetConnected(), false);
+    BOOST_CHECK_EQUAL(connectionNotifier.connects(), static_cast<size_t>(0));
+    BOOST_CHECK_EQUAL(connectionNotifier.disconnects(), static_cast<size_t>(0));
 
-        ensure_equals("user.tariffName == NO_TARIFF_NAME", user.GetProperties().tariffName.ConstData(), NO_TARIFF_NAME);
+    BOOST_CHECK_EQUAL(user.GetProperties().tariffName.ConstData(), NO_TARIFF_NAME);
 
-        user.Authorize(inet_strington("127.0.0.1"), 0, &auth);
-        user.Run();
+    user.Authorize(inet_strington("127.0.0.1"), 0, &auth);
+    user.Run();
 
-        ensure_equals("user.authorised_by = true", user.IsAuthorizedBy(&auth), true);
+    BOOST_CHECK_EQUAL(user.IsAuthorizedBy(&auth), true);
 
-        ensure_equals("user.connected = true", user.GetConnected(), true);
-        ensure_equals("connects = 1", connectionNotifier.GetConnects(), static_cast<size_t>(1));
-        ensure_equals("disconnects = 0", connectionNotifier.GetDisconnects(), static_cast<size_t>(0));
+    BOOST_CHECK_EQUAL(user.GetConnected(), true);
+    BOOST_CHECK_EQUAL(connectionNotifier.connects(), static_cast<size_t>(1));
+    BOOST_CHECK_EQUAL(connectionNotifier.disconnects(), static_cast<size_t>(0));
 
-        tariffName = "test";
-        ensure_equals("user.tariffName == 'test'", user.GetProperties().tariffName.ConstData(), "test");
+    tariffName = "test";
+    BOOST_CHECK_EQUAL(user.GetProperties().tariffName.ConstData(), "test");
 
-        ensure_equals("user.authorised_by = true", user.IsAuthorizedBy(&auth), true);
+    BOOST_CHECK_EQUAL(user.IsAuthorizedBy(&auth), true);
 
-        ensure_equals("user.connected = true", user.GetConnected(), true);
-        ensure_equals("connects = 2", connectionNotifier.GetConnects(), static_cast<size_t>(2));
-        ensure_equals("disconnects = 1", connectionNotifier.GetDisconnects(), static_cast<size_t>(1));
-    }
+    BOOST_CHECK_EQUAL(user.GetConnected(), true);
+    BOOST_CHECK_EQUAL(connectionNotifier.connects(), static_cast<size_t>(2));
+    BOOST_CHECK_EQUAL(connectionNotifier.disconnects(), static_cast<size_t>(1));
 }
 
-void AFTER_CONNECTED_NOTIFIER::Notify(const bool & oldValue, const bool & newValue)
-{
-    if (!oldValue && newValue)
-        ++connects;
-    if (oldValue && !newValue)
-        ++disconnects;
-}
+BOOST_AUTO_TEST_SUITE_END()