]> git.lizzy.rs Git - dragonfireclient.git/blobdiff - src/test.cpp
Schematics: Fix core.schematic_create()
[dragonfireclient.git] / src / test.cpp
index bc0692a78325f05ee5fcfb6fac7d2d81b4619081..402bae95cb33277813d3d1e5e75134f281db0a75 100644 (file)
@@ -1,6 +1,6 @@
 /*
-Minetest-c55
-Copyright (C) 2010 celeron55, Perttu Ahola <celeron55@gmail.com>
+Minetest
+Copyright (C) 2013 celeron55, Perttu Ahola <celeron55@gmail.com>
 
 This program is free software; you can redistribute it and/or modify
 it under the terms of the GNU Lesser General Public License as published by
@@ -22,9 +22,8 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 #include "debug.h"
 #include "map.h"
 #include "player.h"
-#include "main.h"
 #include "socket.h"
-#include "connection.h"
+#include "network/connection.h"
 #include "serialization.h"
 #include "voxel.h"
 #include "collision.h"
@@ -36,12 +35,15 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 #include "settings.h"
 #include "log.h"
 #include "util/string.h"
+#include "filesys.h"
 #include "voxelalgorithms.h"
 #include "inventory.h"
 #include "util/numeric.h"
 #include "util/serialize.h"
 #include "noise.h" // PseudoRandom used for random data for compression
-#include "clientserver.h" // LATEST_PROTOCOL_VERSION
+#include "network/networkprotocol.h" // LATEST_PROTOCOL_VERSION
+#include "profiler.h"
+#include <algorithm>
 
 /*
        Asserts that the exception occurs
@@ -57,7 +59,7 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 #define UTEST(x, fmt, ...)\
 {\
        if(!(x)){\
-               LOGLINEF(LMT_ERROR, "Test (%s) failed: " fmt, #x, ##__VA_ARGS__);\
+               dstream << "Test (" #x ") failed: " fmt << std::endl; \
                test_failed = true;\
        }\
 }
@@ -68,20 +70,18 @@ with this program; if not, write to the Free Software Foundation, Inc.,
        A few item and node definitions for those tests that need them
 */
 
-#define CONTENT_STONE 0
-#define CONTENT_GRASS 0x800
-#define CONTENT_TORCH 100
+static content_t CONTENT_STONE;
+static content_t CONTENT_GRASS;
+static content_t CONTENT_TORCH;
 
 void define_some_nodes(IWritableItemDefManager *idef, IWritableNodeDefManager *ndef)
 {
-       content_t i;
        ItemDefinition itemdef;
        ContentFeatures f;
 
        /*
                Stone
        */
-       i = CONTENT_STONE;
        itemdef = ItemDefinition();
        itemdef.type = ITEM_NODE;
        itemdef.name = "default:stone";
@@ -97,12 +97,11 @@ void define_some_nodes(IWritableItemDefManager *idef, IWritableNodeDefManager *n
                f.tiledef[i].name = "default_stone.png";
        f.is_ground_content = true;
        idef->registerItem(itemdef);
-       ndef->set(i, f);
+       CONTENT_STONE = ndef->set(f.name, f);
 
        /*
                Grass
        */
-       i = CONTENT_GRASS;
        itemdef = ItemDefinition();
        itemdef.type = ITEM_NODE;
        itemdef.name = "default:dirt_with_grass";
@@ -120,12 +119,11 @@ void define_some_nodes(IWritableItemDefManager *idef, IWritableNodeDefManager *n
                f.tiledef[i].name = "default_dirt.png^default_grass_side.png";
        f.is_ground_content = true;
        idef->registerItem(itemdef);
-       ndef->set(i, f);
+       CONTENT_GRASS = ndef->set(f.name, f);
 
        /*
                Torch (minimal definition for lighting tests)
        */
-       i = CONTENT_TORCH;
        itemdef = ItemDefinition();
        itemdef.type = ITEM_NODE;
        itemdef.name = "default:torch";
@@ -136,7 +134,7 @@ void define_some_nodes(IWritableItemDefManager *idef, IWritableNodeDefManager *n
        f.sunlight_propagates = true;
        f.light_source = LIGHT_MAX-1;
        idef->registerItem(itemdef);
-       ndef->set(i, f);
+       CONTENT_TORCH = ndef->set(f.name, f);
 }
 
 struct TestBase
@@ -149,48 +147,395 @@ struct TestBase
 
 struct TestUtilities: public TestBase
 {
+       inline float ref_WrapDegrees180(float f)
+       {
+               // This is a slower alternative to the wrapDegrees_180() function;
+               // used as a reference for testing
+               float value = fmodf(f + 180, 360);
+               if (value < 0)
+                       value += 360;
+               return value - 180;
+       }
+
+       inline float ref_WrapDegrees_0_360(float f)
+       {
+               // This is a slower alternative to the wrapDegrees_0_360() function;
+               // used as a reference for testing
+               float value = fmodf(f, 360);
+               if (value < 0)
+                       value += 360;
+               return value < 0 ? value + 360 : value;
+       }
+
+
        void Run()
        {
-               /*infostream<<"wrapDegrees(100.0) = "<<wrapDegrees(100.0)<<std::endl;
-               infostream<<"wrapDegrees(720.5) = "<<wrapDegrees(720.5)<<std::endl;
-               infostream<<"wrapDegrees(-0.5) = "<<wrapDegrees(-0.5)<<std::endl;*/
-               UASSERT(fabs(wrapDegrees(100.0) - 100.0) < 0.001);
-               UASSERT(fabs(wrapDegrees(720.5) - 0.5) < 0.001);
-               UASSERT(fabs(wrapDegrees(-0.5) - (-0.5)) < 0.001);
-               UASSERT(fabs(wrapDegrees(-365.5) - (-5.5)) < 0.001);
+               UASSERT(fabs(modulo360f(100.0) - 100.0) < 0.001);
+               UASSERT(fabs(modulo360f(720.5) - 0.5) < 0.001);
+               UASSERT(fabs(modulo360f(-0.5) - (-0.5)) < 0.001);
+               UASSERT(fabs(modulo360f(-365.5) - (-5.5)) < 0.001);
+
+               for (float f = -720; f <= -360; f += 0.25) {
+                       UASSERT(fabs(modulo360f(f) - modulo360f(f + 360)) < 0.001);
+               }
+
+               for (float f = -1440; f <= 1440; f += 0.25) {
+                       UASSERT(fabs(modulo360f(f) - fmodf(f, 360)) < 0.001);
+                       UASSERT(fabs(wrapDegrees_180(f) - ref_WrapDegrees180(f)) < 0.001);
+                       UASSERT(fabs(wrapDegrees_0_360(f) - ref_WrapDegrees_0_360(f)) < 0.001);
+                       UASSERT(wrapDegrees_0_360(fabs(wrapDegrees_180(f) - wrapDegrees_0_360(f))) < 0.001);
+               }
+
                UASSERT(lowercase("Foo bAR") == "foo bar");
+               UASSERT(trim("\n \t\r  Foo bAR  \r\n\t\t  ") == "Foo bAR");
+               UASSERT(trim("\n \t\r    \r\n\t\t  ") == "");
                UASSERT(is_yes("YeS") == true);
                UASSERT(is_yes("") == false);
                UASSERT(is_yes("FAlse") == false);
-               const char *ends[] = {"abc", "c", "bc", NULL};
+               UASSERT(is_yes("-1") == true);
+               UASSERT(is_yes("0") == false);
+               UASSERT(is_yes("1") == true);
+               UASSERT(is_yes("2") == true);
+               const char *ends[] = {"abc", "c", "bc", "", NULL};
                UASSERT(removeStringEnd("abc", ends) == "");
                UASSERT(removeStringEnd("bc", ends) == "b");
                UASSERT(removeStringEnd("12c", ends) == "12");
                UASSERT(removeStringEnd("foo", ends) == "");
+               UASSERT(urlencode("\"Aardvarks lurk, OK?\"")
+                               == "%22Aardvarks%20lurk%2C%20OK%3F%22");
+               UASSERT(urldecode("%22Aardvarks%20lurk%2C%20OK%3F%22")
+                               == "\"Aardvarks lurk, OK?\"");
+               UASSERT(padStringRight("hello", 8) == "hello   ");
+               UASSERT(str_equal(narrow_to_wide("abc"), narrow_to_wide("abc")));
+               UASSERT(str_equal(narrow_to_wide("ABC"), narrow_to_wide("abc"), true));
+               UASSERT(trim("  a") == "a");
+               UASSERT(trim("   a  ") == "a");
+               UASSERT(trim("a   ") == "a");
+               UASSERT(trim("") == "");
+               UASSERT(mystoi("123", 0, 1000) == 123);
+               UASSERT(mystoi("123", 0, 10) == 10);
+               std::string test_str;
+               test_str = "Hello there";
+               str_replace(test_str, "there", "world");
+               UASSERT(test_str == "Hello world");
+               test_str = "ThisAisAaAtest";
+               str_replace(test_str, 'A', ' ');
+               UASSERT(test_str == "This is a test");
+               UASSERT(string_allowed("hello", "abcdefghijklmno") == true);
+               UASSERT(string_allowed("123", "abcdefghijklmno") == false);
+               UASSERT(string_allowed_blacklist("hello", "123") == true);
+               UASSERT(string_allowed_blacklist("hello123", "123") == false);
+               UASSERT(wrap_rows("12345678",4) == "1234\n5678");
+               UASSERT(is_number("123") == true);
+               UASSERT(is_number("") == false);
+               UASSERT(is_number("123a") == false);
+               UASSERT(is_power_of_two(0) == false);
+               UASSERT(is_power_of_two(1) == true);
+               UASSERT(is_power_of_two(2) == true);
+               UASSERT(is_power_of_two(3) == false);
+               for (int exponent = 2; exponent <= 31; ++exponent) {
+                       UASSERT(is_power_of_two((1 << exponent) - 1) == false);
+                       UASSERT(is_power_of_two((1 << exponent)) == true);
+                       UASSERT(is_power_of_two((1 << exponent) + 1) == false);
+               }
+               UASSERT(is_power_of_two((u32)-1) == false);
+       }
+};
+
+struct TestPath: public TestBase
+{
+       // adjusts a POSIX path to system-specific conventions
+       // -> changes '/' to DIR_DELIM
+       // -> absolute paths start with "C:\\" on windows
+       std::string p(std::string path)
+       {
+               for(size_t i = 0; i < path.size(); ++i){
+                       if(path[i] == '/'){
+                               path.replace(i, 1, DIR_DELIM);
+                               i += std::string(DIR_DELIM).size() - 1; // generally a no-op
+                       }
+               }
+
+               #ifdef _WIN32
+               if(path[0] == '\\')
+                       path = "C:" + path;
+               #endif
+
+               return path;
+       }
+
+       void Run()
+       {
+               std::string path, result, removed;
+
+               /*
+                       Test fs::IsDirDelimiter
+               */
+               UASSERT(fs::IsDirDelimiter('/') == true);
+               UASSERT(fs::IsDirDelimiter('A') == false);
+               UASSERT(fs::IsDirDelimiter(0) == false);
+               #ifdef _WIN32
+               UASSERT(fs::IsDirDelimiter('\\') == true);
+               #else
+               UASSERT(fs::IsDirDelimiter('\\') == false);
+               #endif
+
+               /*
+                       Test fs::PathStartsWith
+               */
+               {
+                       const int numpaths = 12;
+                       std::string paths[numpaths] = {
+                               "",
+                               p("/"),
+                               p("/home/user/minetest"),
+                               p("/home/user/minetest/bin"),
+                               p("/home/user/.minetest"),
+                               p("/tmp/dir/file"),
+                               p("/tmp/file/"),
+                               p("/tmP/file"),
+                               p("/tmp"),
+                               p("/tmp/dir"),
+                               p("/home/user2/minetest/worlds"),
+                               p("/home/user2/minetest/world"),
+                       };
+                       /*
+                               expected fs::PathStartsWith results
+                               0 = returns false
+                               1 = returns true
+                               2 = returns false on windows, true elsewhere
+                               3 = returns true on windows, false elsewhere
+                               4 = returns true if and only if
+                                   FILESYS_CASE_INSENSITIVE is true
+                       */
+                       int expected_results[numpaths][numpaths] = {
+                               {1,2,0,0,0,0,0,0,0,0,0,0},
+                               {1,1,0,0,0,0,0,0,0,0,0,0},
+                               {1,1,1,0,0,0,0,0,0,0,0,0},
+                               {1,1,1,1,0,0,0,0,0,0,0,0},
+                               {1,1,0,0,1,0,0,0,0,0,0,0},
+                               {1,1,0,0,0,1,0,0,1,1,0,0},
+                               {1,1,0,0,0,0,1,4,1,0,0,0},
+                               {1,1,0,0,0,0,4,1,4,0,0,0},
+                               {1,1,0,0,0,0,0,0,1,0,0,0},
+                               {1,1,0,0,0,0,0,0,1,1,0,0},
+                               {1,1,0,0,0,0,0,0,0,0,1,0},
+                               {1,1,0,0,0,0,0,0,0,0,0,1},
+                       };
+
+                       for (int i = 0; i < numpaths; i++)
+                       for (int j = 0; j < numpaths; j++){
+                               /*verbosestream<<"testing fs::PathStartsWith(\""
+                                       <<paths[i]<<"\", \""
+                                       <<paths[j]<<"\")"<<std::endl;*/
+                               bool starts = fs::PathStartsWith(paths[i], paths[j]);
+                               int expected = expected_results[i][j];
+                               if(expected == 0){
+                                       UASSERT(starts == false);
+                               }
+                               else if(expected == 1){
+                                       UASSERT(starts == true);
+                               }
+                               #ifdef _WIN32
+                               else if(expected == 2){
+                                       UASSERT(starts == false);
+                               }
+                               else if(expected == 3){
+                                       UASSERT(starts == true);
+                               }
+                               #else
+                               else if(expected == 2){
+                                       UASSERT(starts == true);
+                               }
+                               else if(expected == 3){
+                                       UASSERT(starts == false);
+                               }
+                               #endif
+                               else if(expected == 4){
+                                       UASSERT(starts == (bool)FILESYS_CASE_INSENSITIVE);
+                               }
+                       }
+               }
+
+               /*
+                       Test fs::RemoveLastPathComponent
+               */
+               UASSERT(fs::RemoveLastPathComponent("") == "");
+               path = p("/home/user/minetest/bin/..//worlds/world1");
+               result = fs::RemoveLastPathComponent(path, &removed, 0);
+               UASSERT(result == path);
+               UASSERT(removed == "");
+               result = fs::RemoveLastPathComponent(path, &removed, 1);
+               UASSERT(result == p("/home/user/minetest/bin/..//worlds"));
+               UASSERT(removed == p("world1"));
+               result = fs::RemoveLastPathComponent(path, &removed, 2);
+               UASSERT(result == p("/home/user/minetest/bin/.."));
+               UASSERT(removed == p("worlds/world1"));
+               result = fs::RemoveLastPathComponent(path, &removed, 3);
+               UASSERT(result == p("/home/user/minetest/bin"));
+               UASSERT(removed == p("../worlds/world1"));
+               result = fs::RemoveLastPathComponent(path, &removed, 4);
+               UASSERT(result == p("/home/user/minetest"));
+               UASSERT(removed == p("bin/../worlds/world1"));
+               result = fs::RemoveLastPathComponent(path, &removed, 5);
+               UASSERT(result == p("/home/user"));
+               UASSERT(removed == p("minetest/bin/../worlds/world1"));
+               result = fs::RemoveLastPathComponent(path, &removed, 6);
+               UASSERT(result == p("/home"));
+               UASSERT(removed == p("user/minetest/bin/../worlds/world1"));
+               result = fs::RemoveLastPathComponent(path, &removed, 7);
+               #ifdef _WIN32
+               UASSERT(result == "C:");
+               #else
+               UASSERT(result == "");
+               #endif
+               UASSERT(removed == p("home/user/minetest/bin/../worlds/world1"));
+
+               /*
+                       Now repeat the test with a trailing delimiter
+               */
+               path = p("/home/user/minetest/bin/..//worlds/world1/");
+               result = fs::RemoveLastPathComponent(path, &removed, 0);
+               UASSERT(result == path);
+               UASSERT(removed == "");
+               result = fs::RemoveLastPathComponent(path, &removed, 1);
+               UASSERT(result == p("/home/user/minetest/bin/..//worlds"));
+               UASSERT(removed == p("world1"));
+               result = fs::RemoveLastPathComponent(path, &removed, 2);
+               UASSERT(result == p("/home/user/minetest/bin/.."));
+               UASSERT(removed == p("worlds/world1"));
+               result = fs::RemoveLastPathComponent(path, &removed, 3);
+               UASSERT(result == p("/home/user/minetest/bin"));
+               UASSERT(removed == p("../worlds/world1"));
+               result = fs::RemoveLastPathComponent(path, &removed, 4);
+               UASSERT(result == p("/home/user/minetest"));
+               UASSERT(removed == p("bin/../worlds/world1"));
+               result = fs::RemoveLastPathComponent(path, &removed, 5);
+               UASSERT(result == p("/home/user"));
+               UASSERT(removed == p("minetest/bin/../worlds/world1"));
+               result = fs::RemoveLastPathComponent(path, &removed, 6);
+               UASSERT(result == p("/home"));
+               UASSERT(removed == p("user/minetest/bin/../worlds/world1"));
+               result = fs::RemoveLastPathComponent(path, &removed, 7);
+               #ifdef _WIN32
+               UASSERT(result == "C:");
+               #else
+               UASSERT(result == "");
+               #endif
+               UASSERT(removed == p("home/user/minetest/bin/../worlds/world1"));
+
+               /*
+                       Test fs::RemoveRelativePathComponent
+               */
+               path = p("/home/user/minetest/bin");
+               result = fs::RemoveRelativePathComponents(path);
+               UASSERT(result == path);
+               path = p("/home/user/minetest/bin/../worlds/world1");
+               result = fs::RemoveRelativePathComponents(path);
+               UASSERT(result == p("/home/user/minetest/worlds/world1"));
+               path = p("/home/user/minetest/bin/../worlds/world1/");
+               result = fs::RemoveRelativePathComponents(path);
+               UASSERT(result == p("/home/user/minetest/worlds/world1"));
+               path = p(".");
+               result = fs::RemoveRelativePathComponents(path);
+               UASSERT(result == "");
+               path = p("./subdir/../..");
+               result = fs::RemoveRelativePathComponents(path);
+               UASSERT(result == "");
+               path = p("/a/b/c/.././../d/../e/f/g/../h/i/j/../../../..");
+               result = fs::RemoveRelativePathComponents(path);
+               UASSERT(result == p("/a/e"));
        }
 };
 
+#define TEST_CONFIG_TEXT_BEFORE               \
+       "leet = 1337\n"                           \
+       "leetleet = 13371337\n"                   \
+       "leetleet_neg = -13371337\n"              \
+       "floaty_thing = 1.1\n"                    \
+       "stringy_thing = asd /( Â¤%&(/\" BLÖÄRP\n" \
+       "coord = (1, 2, 4.5)\n"                   \
+       "      # this is just a comment\n"        \
+       "this is an invalid line\n"               \
+       "asdf = {\n"                              \
+       "       a   = 5\n"                            \
+       "       bb  = 2.5\n"                          \
+       "       ccc = \"\"\"\n"                       \
+       "testy\n"                                 \
+       "   testa   \n"                           \
+       "\"\"\"\n"                                \
+       "\n"                                      \
+       "}\n"                                     \
+       "blarg = \"\"\" \n"                       \
+       "some multiline text\n"                   \
+       "     with leading whitespace!\n"         \
+       "\"\"\"\n"                                \
+       "np_terrain = 5, 40, (250, 250, 250), 12341, 5, 0.7, 2.4\n" \
+       "zoop = true"
+
+#define TEST_CONFIG_TEXT_AFTER                \
+       "leet = 1337\n"                           \
+       "leetleet = 13371337\n"                   \
+       "leetleet_neg = -13371337\n"              \
+       "floaty_thing = 1.1\n"                    \
+       "stringy_thing = asd /( Â¤%&(/\" BLÖÄRP\n" \
+       "coord = (1, 2, 4.5)\n"                   \
+       "      # this is just a comment\n"        \
+       "this is an invalid line\n"               \
+       "asdf = {\n"                              \
+       "       a   = 5\n"                            \
+       "       bb  = 2.5\n"                          \
+       "       ccc = \"\"\"\n"                       \
+       "testy\n"                                 \
+       "   testa   \n"                           \
+       "\"\"\"\n"                                \
+       "\n"                                      \
+       "}\n"                                     \
+       "blarg = \"\"\" \n"                       \
+       "some multiline text\n"                   \
+       "     with leading whitespace!\n"         \
+       "\"\"\"\n"                                \
+       "np_terrain = {\n"                        \
+       "       flags = defaults\n"                   \
+       "       lacunarity = 2.4\n"                   \
+       "       octaves = 6\n"                        \
+       "       offset = 3.5\n"                       \
+       "       persistence = 0.7\n"                  \
+       "       scale = 40\n"                         \
+       "       seed = 12341\n"                       \
+       "       spread = (250,250,250)\n"             \
+       "}\n"                                     \
+       "zoop = true\n"                           \
+       "coord2 = (1,2,3.3)\n"                    \
+       "floaty_thing_2 = 1.2\n"                  \
+       "groupy_thing = {\n"                      \
+       "       animals = cute\n"                     \
+       "       num_apples = 4\n"                     \
+       "       num_oranges = 53\n"                   \
+       "}\n"
+
 struct TestSettings: public TestBase
 {
        void Run()
        {
+               try {
                Settings s;
+
                // Test reading of settings
-               s.parseConfigLine("leet = 1337");
-               s.parseConfigLine("leetleet = 13371337");
-               s.parseConfigLine("leetleet_neg = -13371337");
-               s.parseConfigLine("floaty_thing = 1.1");
-               s.parseConfigLine("stringy_thing = asd /( Â¤%&(/\" BLÖÄRP");
-               s.parseConfigLine("coord = (1, 2, 4.5)");
+               std::istringstream is(TEST_CONFIG_TEXT_BEFORE);
+               s.parseConfigLines(is);
+
                UASSERT(s.getS32("leet") == 1337);
                UASSERT(s.getS16("leetleet") == 32767);
                UASSERT(s.getS16("leetleet_neg") == -32768);
+
                // Not sure if 1.1 is an exact value as a float, but doesn't matter
                UASSERT(fabs(s.getFloat("floaty_thing") - 1.1) < 0.001);
                UASSERT(s.get("stringy_thing") == "asd /( Â¤%&(/\" BLÖÄRP");
                UASSERT(fabs(s.getV3F("coord").X - 1.0) < 0.001);
                UASSERT(fabs(s.getV3F("coord").Y - 2.0) < 0.001);
                UASSERT(fabs(s.getV3F("coord").Z - 4.5) < 0.001);
+
                // Test the setting of settings too
                s.setFloat("floaty_thing_2", 1.2);
                s.setV3F("coord2", v3f(1, 2, 3.3));
@@ -199,6 +544,69 @@ struct TestSettings: public TestBase
                UASSERT(fabs(s.getV3F("coord2").X - 1.0) < 0.001);
                UASSERT(fabs(s.getV3F("coord2").Y - 2.0) < 0.001);
                UASSERT(fabs(s.getV3F("coord2").Z - 3.3) < 0.001);
+
+               // Test settings groups
+               Settings *group = s.getGroup("asdf");
+               UASSERT(group != NULL);
+               UASSERT(s.getGroupNoEx("zoop", group) == false);
+               UASSERT(group->getS16("a") == 5);
+               UASSERT(fabs(group->getFloat("bb") - 2.5) < 0.001);
+
+               Settings *group3 = new Settings;
+               group3->set("cat", "meow");
+               group3->set("dog", "woof");
+
+               Settings *group2 = new Settings;
+               group2->setS16("num_apples", 4);
+               group2->setS16("num_oranges", 53);
+               group2->setGroup("animals", group3);
+               group2->set("animals", "cute"); //destroys group 3
+               s.setGroup("groupy_thing", group2);
+
+               // Test set failure conditions
+               UASSERT(s.set("Zoop = Poop\nsome_other_setting", "false") == false);
+               UASSERT(s.set("sneaky", "\"\"\"\njabberwocky = false") == false);
+               UASSERT(s.set("hehe", "asdfasdf\n\"\"\"\nsomething = false") == false);
+
+               // Test multiline settings
+               UASSERT(group->get("ccc") == "testy\n   testa   ");
+
+               UASSERT(s.get("blarg") ==
+                       "some multiline text\n"
+                       "     with leading whitespace!");
+
+               // Test NoiseParams
+               UASSERT(s.getEntry("np_terrain").is_group == false);
+
+               NoiseParams np;
+               UASSERT(s.getNoiseParams("np_terrain", np) == true);
+               UASSERT(fabs(np.offset - 5) < 0.001);
+               UASSERT(fabs(np.scale - 40) < 0.001);
+               UASSERT(fabs(np.spread.X - 250) < 0.001);
+               UASSERT(fabs(np.spread.Y - 250) < 0.001);
+               UASSERT(fabs(np.spread.Z - 250) < 0.001);
+               UASSERT(np.seed == 12341);
+               UASSERT(np.octaves == 5);
+               UASSERT(fabs(np.persist - 0.7) < 0.001);
+
+               np.offset  = 3.5;
+               np.octaves = 6;
+               s.setNoiseParams("np_terrain", np);
+
+               UASSERT(s.getEntry("np_terrain").is_group == true);
+
+               // Test writing
+               std::ostringstream os(std::ios_base::binary);
+               is.clear();
+               is.seekg(0);
+
+               UASSERT(s.updateConfigObject(is, os, "", 0) == true);
+               //printf(">>>> expected config:\n%s\n", TEST_CONFIG_TEXT_AFTER);
+               //printf(">>>> actual config:\n%s\n", os.str().c_str());
+               UASSERT(os.str() == TEST_CONFIG_TEXT_AFTER);
+               } catch (SettingNotFoundException &e) {
+                       UASSERT(!"Setting not found!");
+               }
        }
 };
 
@@ -220,7 +628,7 @@ struct TestSerialization: public TestBase
                UASSERT(serializeWideString(L"") == mkstr("\0\0"));
                UASSERT(serializeLongString("") == mkstr("\0\0\0\0"));
                UASSERT(serializeJsonString("") == "\"\"");
-               
+
                std::string teststring = "Hello world!";
                UASSERT(serializeString(teststring) ==
                        mkstr("\0\14Hello world!"));
@@ -346,12 +754,12 @@ struct TestCompress: public TestBase
                fromdata[1]=5;
                fromdata[2]=5;
                fromdata[3]=1;
-               
+
                std::ostringstream os(std::ios_base::binary);
                compress(fromdata, os, 0);
 
                std::string str_out = os.str();
-               
+
                infostream<<"str_out.size()="<<str_out.size()<<std::endl;
                infostream<<"TestCompress: 1,5,5,1 -> ";
                for(u32 i=0; i<str_out.size(); i++)
@@ -402,12 +810,12 @@ struct TestCompress: public TestBase
                fromdata[1]=5;
                fromdata[2]=5;
                fromdata[3]=1;
-               
+
                std::ostringstream os(std::ios_base::binary);
-               compress(fromdata, os, SER_FMT_VER_HIGHEST);
+               compress(fromdata, os, SER_FMT_VER_HIGHEST_READ);
 
                std::string str_out = os.str();
-               
+
                infostream<<"str_out.size()="<<str_out.size()<<std::endl;
                infostream<<"TestCompress: 1,5,5,1 -> ";
                for(u32 i=0; i<str_out.size(); i++)
@@ -419,7 +827,7 @@ struct TestCompress: public TestBase
                std::istringstream is(str_out, std::ios_base::binary);
                std::ostringstream os2(std::ios_base::binary);
 
-               decompress(is, os2, SER_FMT_VER_HIGHEST);
+               decompress(is, os2, SER_FMT_VER_HIGHEST_READ);
                std::string str_out2 = os2.str();
 
                infostream<<"decompress: ";
@@ -462,8 +870,8 @@ struct TestCompress: public TestBase
                                        <<os_decompressed.str().size()<<std::endl;
                        std::string str_decompressed = os_decompressed.str();
                        UTEST(str_decompressed.size() == data_in.size(), "Output size not"
-                                       " equal (output: %i, input: %i)",
-                                       str_decompressed.size(), data_in.size());
+                                       " equal (output: %u, input: %u)",
+                                       (unsigned int)str_decompressed.size(), (unsigned int)data_in.size());
                        for(u32 i=0; i<size && i<str_decompressed.size(); i++){
                                UTEST(str_decompressed[i] == data_in[i],
                                                "index out[%i]=%i differs from in[%i]=%i",
@@ -477,13 +885,12 @@ struct TestMapNode: public TestBase
 {
        void Run(INodeDefManager *nodedef)
        {
-               MapNode n;
+               MapNode n(CONTENT_AIR);
 
-               // Default values
                UASSERT(n.getContent() == CONTENT_AIR);
                UASSERT(n.getLight(LIGHTBANK_DAY, nodedef) == 0);
                UASSERT(n.getLight(LIGHTBANK_NIGHT, nodedef) == 0);
-               
+
                // Transparency
                n.setContent(CONTENT_AIR);
                UASSERT(nodedef->get(n).light_propagates == true);
@@ -503,44 +910,44 @@ struct TestVoxelManipulator: public TestBase
                VoxelArea a(v3s16(-1,-1,-1), v3s16(1,1,1));
                UASSERT(a.index(0,0,0) == 1*3*3 + 1*3 + 1);
                UASSERT(a.index(-1,-1,-1) == 0);
-               
+
                VoxelArea c(v3s16(-2,-2,-2), v3s16(2,2,2));
                // An area that is 1 bigger in x+ and z-
                VoxelArea d(v3s16(-2,-2,-3), v3s16(3,2,2));
-               
-               core::list<VoxelArea> aa;
+
+               std::list<VoxelArea> aa;
                d.diff(c, aa);
-               
+
                // Correct results
-               core::array<VoxelArea> results;
+               std::vector<VoxelArea> results;
                results.push_back(VoxelArea(v3s16(-2,-2,-3),v3s16(3,2,-3)));
                results.push_back(VoxelArea(v3s16(3,-2,-2),v3s16(3,2,2)));
 
                UASSERT(aa.size() == results.size());
-               
+
                infostream<<"Result of diff:"<<std::endl;
-               for(core::list<VoxelArea>::Iterator
-                               i = aa.begin(); i != aa.end(); i++)
+               for(std::list<VoxelArea>::const_iterator
+                               i = aa.begin(); i != aa.end(); ++i)
                {
                        i->print(infostream);
                        infostream<<std::endl;
-                       
-                       s32 j = results.linear_search(*i);
-                       UASSERT(j != -1);
-                       results.erase(j, 1);
+
+                       std::vector<VoxelArea>::iterator j = std::find(results.begin(), results.end(), *i);
+                       UASSERT(j != results.end());
+                       results.erase(j);
                }
 
 
                /*
                        VoxelManipulator
                */
-               
+
                VoxelManipulator v;
 
                v.print(infostream, nodedef);
 
                infostream<<"*** Setting (-1,0,-1)=2 ***"<<std::endl;
-               
+
                v.setNodeNoRef(v3s16(-1,0,-1), MapNode(CONTENT_GRASS));
 
                v.print(infostream, nodedef);
@@ -556,7 +963,7 @@ struct TestVoxelManipulator: public TestBase
                infostream<<"*** Adding area ***"<<std::endl;
 
                v.addArea(a);
-               
+
                v.print(infostream, nodedef);
 
                UASSERT(v.getNode(v3s16(-1,0,-1)).getContent() == CONTENT_GRASS);
@@ -582,7 +989,7 @@ struct TestVoxelAlgorithms: public TestBase
                        }
                        VoxelArea a(v3s16(0,0,0), v3s16(2,2,2));
                        {
-                               core::map<v3s16, bool> light_sources;
+                               std::set<v3s16> light_sources;
                                voxalgo::setLight(v, a, 0, ndef);
                                voxalgo::SunlightPropagateResult res = voxalgo::propagateSunlight(
                                                v, a, true, light_sources, ndef);
@@ -593,7 +1000,7 @@ struct TestVoxelAlgorithms: public TestBase
                        }
                        v.setNodeNoRef(v3s16(0,0,0), MapNode(CONTENT_STONE));
                        {
-                               core::map<v3s16, bool> light_sources;
+                               std::set<v3s16> light_sources;
                                voxalgo::setLight(v, a, 0, ndef);
                                voxalgo::SunlightPropagateResult res = voxalgo::propagateSunlight(
                                                v, a, true, light_sources, ndef);
@@ -602,7 +1009,7 @@ struct TestVoxelAlgorithms: public TestBase
                                                == LIGHT_SUN);
                        }
                        {
-                               core::map<v3s16, bool> light_sources;
+                               std::set<v3s16> light_sources;
                                voxalgo::setLight(v, a, 0, ndef);
                                voxalgo::SunlightPropagateResult res = voxalgo::propagateSunlight(
                                                v, a, false, light_sources, ndef);
@@ -612,7 +1019,7 @@ struct TestVoxelAlgorithms: public TestBase
                        }
                        v.setNodeNoRef(v3s16(1,3,2), MapNode(CONTENT_STONE));
                        {
-                               core::map<v3s16, bool> light_sources;
+                               std::set<v3s16> light_sources;
                                voxalgo::setLight(v, a, 0, ndef);
                                voxalgo::SunlightPropagateResult res = voxalgo::propagateSunlight(
                                                v, a, true, light_sources, ndef);
@@ -621,7 +1028,7 @@ struct TestVoxelAlgorithms: public TestBase
                                                == 0);
                        }
                        {
-                               core::map<v3s16, bool> light_sources;
+                               std::set<v3s16> light_sources;
                                voxalgo::setLight(v, a, 0, ndef);
                                voxalgo::SunlightPropagateResult res = voxalgo::propagateSunlight(
                                                v, a, false, light_sources, ndef);
@@ -635,14 +1042,14 @@ struct TestVoxelAlgorithms: public TestBase
                                v.setNodeNoRef(v3s16(1,-1,2), n);
                        }
                        {
-                               core::map<v3s16, bool> light_sources;
+                               std::set<v3s16> light_sources;
                                voxalgo::setLight(v, a, 0, ndef);
                                voxalgo::SunlightPropagateResult res = voxalgo::propagateSunlight(
                                                v, a, true, light_sources, ndef);
                                UASSERT(res.bottom_sunlight_valid == true);
                        }
                        {
-                               core::map<v3s16, bool> light_sources;
+                               std::set<v3s16> light_sources;
                                voxalgo::setLight(v, a, 0, ndef);
                                voxalgo::SunlightPropagateResult res = voxalgo::propagateSunlight(
                                                v, a, false, light_sources, ndef);
@@ -654,14 +1061,14 @@ struct TestVoxelAlgorithms: public TestBase
                                v.setNodeNoRef(v3s16(1,-1,2), n);
                        }
                        {
-                               core::map<v3s16, bool> light_sources;
+                               std::set<v3s16> light_sources;
                                voxalgo::setLight(v, a, 0, ndef);
                                voxalgo::SunlightPropagateResult res = voxalgo::propagateSunlight(
                                                v, a, true, light_sources, ndef);
                                UASSERT(res.bottom_sunlight_valid == false);
                        }
                        {
-                               core::map<v3s16, bool> light_sources;
+                               std::set<v3s16> light_sources;
                                voxalgo::setLight(v, a, 0, ndef);
                                voxalgo::SunlightPropagateResult res = voxalgo::propagateSunlight(
                                                v, a, false, light_sources, ndef);
@@ -669,7 +1076,7 @@ struct TestVoxelAlgorithms: public TestBase
                        }
                        v.setNodeNoRef(v3s16(1,3,2), MapNode(CONTENT_IGNORE));
                        {
-                               core::map<v3s16, bool> light_sources;
+                               std::set<v3s16> light_sources;
                                voxalgo::setLight(v, a, 0, ndef);
                                voxalgo::SunlightPropagateResult res = voxalgo::propagateSunlight(
                                                v, a, true, light_sources, ndef);
@@ -697,16 +1104,16 @@ struct TestVoxelAlgorithms: public TestBase
                                v.setNode(v3s16(1,1,2), n);
                        }
                        {
-                               core::map<v3s16, bool> light_sources;
-                               core::map<v3s16, u8> unlight_from;
+                               std::set<v3s16> light_sources;
+                               std::map<v3s16, u8> unlight_from;
                                voxalgo::clearLightAndCollectSources(v, a, LIGHTBANK_DAY,
                                                ndef, light_sources, unlight_from);
                                //v.print(dstream, ndef, VOXELPRINT_LIGHT_DAY);
                                UASSERT(v.getNode(v3s16(0,1,1)).getLight(LIGHTBANK_DAY, ndef)
                                                == 0);
-                               UASSERT(light_sources.find(v3s16(1,1,1)) != NULL);
+                               UASSERT(light_sources.find(v3s16(1,1,1)) != light_sources.end());
                                UASSERT(light_sources.size() == 1);
-                               UASSERT(unlight_from.find(v3s16(1,1,2)) != NULL);
+                               UASSERT(unlight_from.find(v3s16(1,1,2)) != unlight_from.end());
                                UASSERT(unlight_from.size() == 1);
                        }
                }
@@ -754,7 +1161,7 @@ struct TestInventory: public TestBase
                "Empty\n"
                "EndInventoryList\n"
                "EndInventory\n";
-               
+
                std::string serialized_inventory_2 =
                "List main 32\n"
                "Width 5\n"
@@ -792,7 +1199,7 @@ struct TestInventory: public TestBase
                "Empty\n"
                "EndInventoryList\n"
                "EndInventory\n";
-               
+
                Inventory inv(idef);
                std::istringstream is(serialized_inventory, std::ios::binary);
                inv.deSerialize(is);
@@ -868,7 +1275,7 @@ struct TestMapBlock: public TestBase
        void Run()
        {
                TC parent;
-               
+
                MapBlock b(&parent, v3s16(1,1,1));
                v3s16 relpos(MAP_BLOCKSIZE, MAP_BLOCKSIZE, MAP_BLOCKSIZE);
 
@@ -880,7 +1287,7 @@ struct TestMapBlock: public TestBase
                UASSERT(b.getBox().MaxEdge.Y == MAP_BLOCKSIZE*2-1);
                UASSERT(b.getBox().MinEdge.Z == MAP_BLOCKSIZE);
                UASSERT(b.getBox().MaxEdge.Z == MAP_BLOCKSIZE*2-1);
-               
+
                UASSERT(b.isValidPosition(v3s16(0,0,0)) == true);
                UASSERT(b.isValidPosition(v3s16(-1,0,0)) == false);
                UASSERT(b.isValidPosition(v3s16(-1,-142,-2341)) == false);
@@ -894,7 +1301,7 @@ struct TestMapBlock: public TestBase
                */
                /*UASSERT(b.getSizeNodes() == v3s16(MAP_BLOCKSIZE,
                                MAP_BLOCKSIZE, MAP_BLOCKSIZE));*/
-               
+
                // Changed flag should be initially set
                UASSERT(b.getModified() == MOD_STATE_WRITE_NEEDED);
                b.resetModified();
@@ -911,7 +1318,7 @@ struct TestMapBlock: public TestBase
                        UASSERT(b.getNode(v3s16(x,y,z)).getLight(LIGHTBANK_DAY) == 0);
                        UASSERT(b.getNode(v3s16(x,y,z)).getLight(LIGHTBANK_NIGHT) == 0);
                }
-               
+
                {
                        MapNode n(CONTENT_AIR);
                        for(u16 z=0; z<MAP_BLOCKSIZE; z++)
@@ -921,7 +1328,7 @@ struct TestMapBlock: public TestBase
                                b.setNode(v3s16(x,y,z), n);
                        }
                }
-                       
+
                /*
                        Parent fetch functions
                */
@@ -929,7 +1336,7 @@ struct TestMapBlock: public TestBase
                parent.node.setContent(5);
 
                MapNode n;
-               
+
                // Positions in the block should still be valid
                UASSERT(b.isValidPositionParent(v3s16(0,0,0)) == true);
                UASSERT(b.isValidPositionParent(v3s16(MAP_BLOCKSIZE-1,MAP_BLOCKSIZE-1,MAP_BLOCKSIZE-1)) == true);
@@ -940,7 +1347,7 @@ struct TestMapBlock: public TestBase
                UASSERT(b.isValidPositionParent(v3s16(-121,2341,0)) == false);
                UASSERT(b.isValidPositionParent(v3s16(-1,0,0)) == false);
                UASSERT(b.isValidPositionParent(v3s16(MAP_BLOCKSIZE-1,MAP_BLOCKSIZE-1,MAP_BLOCKSIZE)) == false);
-               
+
                {
                        bool exception_thrown = false;
                        try{
@@ -972,7 +1379,7 @@ struct TestMapBlock: public TestBase
                //TODO: Update to new system
                /*UASSERT(b.getNodeTile(p) == 4);
                UASSERT(b.getNodeTile(v3s16(-1,-1,0)) == 5);*/
-               
+
                /*
                        propagateSunlight()
                */
@@ -1102,29 +1509,29 @@ struct TestMapSector: public TestBase
                        if(position_valid == false)
                                throw InvalidPositionException();
                };
-               
+
                virtual u16 nodeContainerId() const
                {
                        return 666;
                }
        };
-       
+
        void Run()
        {
                TC parent;
                parent.position_valid = false;
-               
+
                // Create one with no heightmaps
                ServerMapSector sector(&parent, v2s16(1,1));
-               
+
                UASSERT(sector.getBlockNoCreateNoEx(0) == 0);
                UASSERT(sector.getBlockNoCreateNoEx(1) == 0);
 
                MapBlock * bref = sector.createBlankBlock(-2);
-               
+
                UASSERT(sector.getBlockNoCreateNoEx(0) == 0);
                UASSERT(sector.getBlockNoCreateNoEx(-2) == bref);
-               
+
                //TODO: Check for AlreadyExistsException
 
                /*bool exception_thrown = false;
@@ -1292,26 +1699,104 @@ struct TestSocket: public TestBase
        void Run()
        {
                const int port = 30003;
-               UDPSocket socket;
-               socket.Bind(port);
+               Address address(0, 0, 0, 0, port);
+               Address bind_addr(0, 0, 0, 0, port);
+               Address address6((IPv6AddressBytes*) NULL, port);
 
-               const char sendbuffer[] = "hello world!";
-               socket.Send(Address(127,0,0,1,port), sendbuffer, sizeof(sendbuffer));
+               /*
+                * Try to use the bind_address for servers with no localhost address
+                * For example: FreeBSD jails
+                */
+               std::string bind_str = g_settings->get("bind_address");
+               try {
+                       bind_addr.Resolve(bind_str.c_str());
+
+                       if (!bind_addr.isIPv6()) {
+                               address = bind_addr;
+                       }
+               } catch (ResolveError &e) {
+               }
 
-               sleep_ms(50);
+               // IPv6 socket test
+               if (g_settings->getBool("enable_ipv6")) {
+                       UDPSocket socket6;
+
+                       if (!socket6.init(true, true)) {
+                               /* Note: Failing to create an IPv6 socket is not technically an
+                                  error because the OS may not support IPv6 or it may
+                                  have been disabled. IPv6 is not /required/ by
+                                  minetest and therefore this should not cause the unit
+                                  test to fail
+                               */
+                               dstream << "WARNING: IPv6 socket creation failed (unit test)"
+                                       << std::endl;
+                       } else {
+                               const char sendbuffer[] = "hello world!";
+                               IPv6AddressBytes bytes;
+                               bytes.bytes[15] = 1;
+
+                               socket6.Bind(address6);
+
+                               try {
+                                       socket6.Send(Address(&bytes, port), sendbuffer, sizeof(sendbuffer));
+
+                                       sleep_ms(50);
+
+                                       char rcvbuffer[256] = { 0 };
+                                       Address sender;
+
+                                       for(;;) {
+                                               if (socket6.Receive(sender, rcvbuffer, sizeof(rcvbuffer )) < 0)
+                                                       break;
+                                       }
+                                       //FIXME: This fails on some systems
+                                       UASSERT(strncmp(sendbuffer, rcvbuffer, sizeof(sendbuffer)) == 0);
+                                       UASSERT(memcmp(sender.getAddress6().sin6_addr.s6_addr,
+                                                       Address(&bytes, 0).getAddress6().sin6_addr.s6_addr, 16) == 0);
+                               }
+                               catch (SendFailedException &e) {
+                                       errorstream << "IPv6 support enabled but not available!"
+                                                   << std::endl;
+                               }
+                       }
+               }
 
-               char rcvbuffer[256];
-               memset(rcvbuffer, 0, sizeof(rcvbuffer));
-               Address sender;
-               for(;;)
+               // IPv4 socket test
                {
-                       int bytes_read = socket.Receive(sender, rcvbuffer, sizeof(rcvbuffer));
-                       if(bytes_read < 0)
-                               break;
+                       UDPSocket socket(false);
+                       socket.Bind(address);
+
+                       const char sendbuffer[] = "hello world!";
+                       /*
+                        * If there is a bind address, use it.
+                        * It's useful in container environments
+                        */
+                       if (address != Address(0, 0, 0, 0, port)) {
+                               socket.Send(address, sendbuffer, sizeof(sendbuffer));
+                       }
+                       else
+                               socket.Send(Address(127, 0, 0 ,1, port), sendbuffer, sizeof(sendbuffer));
+
+                       sleep_ms(50);
+
+                       char rcvbuffer[256] = { 0 };
+                       Address sender;
+                       for(;;) {
+                               if (socket.Receive(sender, rcvbuffer, sizeof(rcvbuffer)) < 0)
+                                       break;
+                       }
+                       //FIXME: This fails on some systems
+                       UASSERT(strncmp(sendbuffer, rcvbuffer, sizeof(sendbuffer)) == 0);
+
+                       if (address != Address(0, 0, 0, 0, port)) {
+                               UASSERT(sender.getAddress().sin_addr.s_addr ==
+                                               address.getAddress().sin_addr.s_addr);
+                       }
+                       else {
+                               UASSERT(sender.getAddress().sin_addr.s_addr ==
+                                               Address(127, 0, 0, 1, 0).getAddress().sin_addr.s_addr);
+                       }
                }
-               //FIXME: This fails on some systems
-               UASSERT(strncmp(sendbuffer, rcvbuffer, sizeof(sendbuffer))==0);
-               UASSERT(sender.getAddress() == Address(127,0,0,1, 0).getAddress());
        }
 };
 
@@ -1330,7 +1815,7 @@ struct TestConnection: public TestBase
                SharedBuffer<u8> data1(1);
                data1[0] = 100;
                Address a(127,0,0,1, 10);
-               u16 seqnum = 34352;
+               const u16 seqnum = 34352;
 
                con::BufferedPacket p1 = con::makePacket(a, data1,
                                proto_id, peer_id, channel);
@@ -1347,7 +1832,7 @@ struct TestConnection: public TestBase
                UASSERT(readU16(&p1.data[4]) == peer_id);
                UASSERT(readU8(&p1.data[6]) == channel);
                UASSERT(readU8(&p1.data[7]) == data1[0]);
-               
+
                //infostream<<"initial data1[0]="<<((u32)data1[0]&0xff)<<std::endl;
 
                SharedBuffer<u8> p2 = con::makeReliablePacket(data1, seqnum);
@@ -1409,37 +1894,56 @@ struct TestConnection: public TestBase
 
                Handler hand_server("server");
                Handler hand_client("client");
-               
+
+               Address address(0, 0, 0, 0, 30001);
+               Address bind_addr(0, 0, 0, 0, 30001);
+               /*
+                * Try to use the bind_address for servers with no localhost address
+                * For example: FreeBSD jails
+                */
+               std::string bind_str = g_settings->get("bind_address");
+               try {
+                       bind_addr.Resolve(bind_str.c_str());
+
+                       if (!bind_addr.isIPv6()) {
+                               address = bind_addr;
+                       }
+               } catch (ResolveError &e) {
+               }
+
                infostream<<"** Creating server Connection"<<std::endl;
-               con::Connection server(proto_id, 512, 5.0, &hand_server);
-               server.Serve(30001);
-               
+               con::Connection server(proto_id, 512, 5.0, false, &hand_server);
+               server.Serve(address);
+
                infostream<<"** Creating client Connection"<<std::endl;
-               con::Connection client(proto_id, 512, 5.0, &hand_client);
+               con::Connection client(proto_id, 512, 5.0, false, &hand_client);
 
                UASSERT(hand_server.count == 0);
                UASSERT(hand_client.count == 0);
-               
+
                sleep_ms(50);
-               
-               Address server_address(127,0,0,1, 30001);
+
+               Address server_address(127, 0, 0, 1, 30001);
+               if (address != Address(0, 0, 0, 0, 30001)) {
+                       server_address = bind_addr;
+               }
+
                infostream<<"** running client.Connect()"<<std::endl;
                client.Connect(server_address);
 
                sleep_ms(50);
-               
+
                // Client should not have added client yet
                UASSERT(hand_client.count == 0);
-               
+
                try
                {
-                       u16 peer_id;
-                       SharedBuffer<u8> data;
-                       infostream<<"** running client.Receive()"<<std::endl;
-                       u32 size = client.Receive(peer_id, data);
-                       infostream<<"** Client received: peer_id="<<peer_id
-                                       <<", size="<<size
-                                       <<std::endl;
+                       NetworkPacket pkt;
+                       infostream << "** running client.Receive()" << std::endl;
+                       client.Receive(&pkt);
+                       infostream << "** Client received: peer_id=" << pkt.getPeerId()
+                                       << ", size=" << pkt.getSize()
+                                       << std::endl;
                }
                catch(con::NoIncomingDataException &e)
                {
@@ -1450,45 +1954,43 @@ struct TestConnection: public TestBase
                UASSERT(hand_client.last_id == 1);
                // Server should not have added client yet
                UASSERT(hand_server.count == 0);
-               
-               sleep_ms(50);
+
+               sleep_ms(100);
 
                try
                {
-                       u16 peer_id;
-                       SharedBuffer<u8> data;
-                       infostream<<"** running server.Receive()"<<std::endl;
-                       u32 size = server.Receive(peer_id, data);
-                       infostream<<"** Server received: peer_id="<<peer_id
-                                       <<", size="<<size
-                                       <<std::endl;
+                       NetworkPacket pkt;
+                       infostream << "** running server.Receive()" << std::endl;
+                       server.Receive(&pkt);
+                       infostream<<"** Server received: peer_id=" << pkt.getPeerId()
+                                       << ", size=" << pkt.getSize()
+                                       << std::endl;
                }
                catch(con::NoIncomingDataException &e)
                {
                        // No actual data received, but the client has
                        // probably been connected
                }
-               
+
                // Client should be the same
                UASSERT(hand_client.count == 1);
                UASSERT(hand_client.last_id == 1);
                // Server should have the client
                UASSERT(hand_server.count == 1);
                UASSERT(hand_server.last_id == 2);
-               
+
                //sleep_ms(50);
 
                while(client.Connected() == false)
                {
                        try
                        {
-                               u16 peer_id;
-                               SharedBuffer<u8> data;
-                               infostream<<"** running client.Receive()"<<std::endl;
-                               u32 size = client.Receive(peer_id, data);
-                               infostream<<"** Client received: peer_id="<<peer_id
-                                               <<", size="<<size
-                                               <<std::endl;
+                               NetworkPacket pkt;
+                               infostream << "** running client.Receive()" << std::endl;
+                               client.Receive(&pkt);
+                               infostream << "** Client received: peer_id=" << pkt.getPeerId()
+                                               << ", size=" << pkt.getSize()
+                                               << std::endl;
                        }
                        catch(con::NoIncomingDataException &e)
                        {
@@ -1497,198 +1999,92 @@ struct TestConnection: public TestBase
                }
 
                sleep_ms(50);
-               
+
                try
                {
-                       u16 peer_id;
-                       SharedBuffer<u8> data;
-                       infostream<<"** running server.Receive()"<<std::endl;
-                       u32 size = server.Receive(peer_id, data);
-                       infostream<<"** Server received: peer_id="<<peer_id
-                                       <<", size="<<size
-                                       <<std::endl;
+                       NetworkPacket pkt;
+                       infostream << "** running server.Receive()" << std::endl;
+                       server.Receive(&pkt);
+                       infostream << "** Server received: peer_id=" << pkt.getPeerId()
+                                       << ", size=" << pkt.getSize()
+                                       << std::endl;
                }
                catch(con::NoIncomingDataException &e)
                {
                }
-#if 1
+
                /*
                        Simple send-receive test
                */
                {
-                       /*u8 data[] = "Hello World!";
-                       u32 datasize = sizeof(data);*/
-                       SharedBuffer<u8> data = SharedBufferFromString("Hello World!");
+                       NetworkPacket pkt;
+                       pkt.putRawPacket((u8*) "Hello World !", 14, 0);
 
-                       infostream<<"** running client.Send()"<<std::endl;
-                       client.Send(PEER_ID_SERVER, 0, data, true);
-
-                       sleep_ms(50);
+                       Buffer<u8> sentdata = pkt.oldForgePacket();
 
-                       u16 peer_id;
-                       SharedBuffer<u8> recvdata;
-                       infostream<<"** running server.Receive()"<<std::endl;
-                       u32 size = server.Receive(peer_id, recvdata);
-                       infostream<<"** Server received: peer_id="<<peer_id
-                                       <<", size="<<size
-                                       <<", data="<<*data
-                                       <<std::endl;
-                       UASSERT(memcmp(*data, *recvdata, data.getSize()) == 0);
-               }
-#endif
-               u16 peer_id_client = 2;
-#if 0
-               /*
-                       Send consequent packets in different order
-                       Not compatible with new Connection, thus commented out.
-               */
-               {
-                       //u8 data1[] = "hello1";
-                       //u8 data2[] = "hello2";
-                       SharedBuffer<u8> data1 = SharedBufferFromString("hello1");
-                       SharedBuffer<u8> data2 = SharedBufferFromString("Hello2");
-
-                       Address client_address =
-                                       server.GetPeerAddress(peer_id_client);
-                       
-                       infostream<<"*** Sending packets in wrong order (2,1,2)"
-                                       <<std::endl;
-                       
-                       u8 chn = 0;
-                       con::Channel *ch = &server.getPeer(peer_id_client)->channels[chn];
-                       u16 sn = ch->next_outgoing_seqnum;
-                       ch->next_outgoing_seqnum = sn+1;
-                       server.Send(peer_id_client, chn, data2, true);
-                       ch->next_outgoing_seqnum = sn;
-                       server.Send(peer_id_client, chn, data1, true);
-                       ch->next_outgoing_seqnum = sn+1;
-                       server.Send(peer_id_client, chn, data2, true);
+                       infostream<<"** running client.Send()"<<std::endl;
+                       client.Send(PEER_ID_SERVER, 0, &pkt, true);
 
                        sleep_ms(50);
 
-                       infostream<<"*** Receiving the packets"<<std::endl;
+                       NetworkPacket recvpacket;
+                       infostream << "** running server.Receive()" << std::endl;
+                       server.Receive(&recvpacket);
+                       infostream << "** Server received: peer_id=" << pkt.getPeerId()
+                                       << ", size=" << pkt.getSize()
+                                       << ", data=" << (const char*)pkt.getU8Ptr(0)
+                                       << std::endl;
 
-                       u16 peer_id;
-                       SharedBuffer<u8> recvdata;
-                       u32 size;
+                       Buffer<u8> recvdata = pkt.oldForgePacket();
 
-                       infostream<<"** running client.Receive()"<<std::endl;
-                       peer_id = 132;
-                       size = client.Receive(peer_id, recvdata);
-                       infostream<<"** Client received: peer_id="<<peer_id
-                                       <<", size="<<size
-                                       <<", data="<<*recvdata
-                                       <<std::endl;
-                       UASSERT(size == data1.getSize());
-                       UASSERT(memcmp(*data1, *recvdata, data1.getSize()) == 0);
-                       UASSERT(peer_id == PEER_ID_SERVER);
-                       
-                       infostream<<"** running client.Receive()"<<std::endl;
-                       peer_id = 132;
-                       size = client.Receive(peer_id, recvdata);
-                       infostream<<"** Client received: peer_id="<<peer_id
-                                       <<", size="<<size
-                                       <<", data="<<*recvdata
-                                       <<std::endl;
-                       UASSERT(size == data2.getSize());
-                       UASSERT(memcmp(*data2, *recvdata, data2.getSize()) == 0);
-                       UASSERT(peer_id == PEER_ID_SERVER);
-                       
-                       bool got_exception = false;
-                       try
-                       {
-                               infostream<<"** running client.Receive()"<<std::endl;
-                               peer_id = 132;
-                               size = client.Receive(peer_id, recvdata);
-                               infostream<<"** Client received: peer_id="<<peer_id
-                                               <<", size="<<size
-                                               <<", data="<<*recvdata
-                                               <<std::endl;
-                       }
-                       catch(con::NoIncomingDataException &e)
-                       {
-                               infostream<<"** No incoming data for client"<<std::endl;
-                               got_exception = true;
-                       }
-                       UASSERT(got_exception);
-               }
-#endif
-#if 0
-               /*
-                       Send large amounts of packets (infinite test)
-                       Commented out because of infinity.
-               */
-               {
-                       infostream<<"Sending large amounts of packets (infinite test)"<<std::endl;
-                       int sendcount = 0;
-                       for(;;){
-                               int datasize = myrand_range(0,5)==0?myrand_range(100,10000):myrand_range(0,100);
-                               infostream<<"datasize="<<datasize<<std::endl;
-                               SharedBuffer<u8> data1(datasize);
-                               for(u16 i=0; i<datasize; i++)
-                                       data1[i] = i/4;
-                               
-                               int sendtimes = myrand_range(1,10);
-                               for(int i=0; i<sendtimes; i++){
-                                       server.Send(peer_id_client, 0, data1, true);
-                                       sendcount++;
-                               }
-                               infostream<<"sendcount="<<sendcount<<std::endl;
-                               
-                               //int receivetimes = myrand_range(1,20);
-                               int receivetimes = 20;
-                               for(int i=0; i<receivetimes; i++){
-                                       SharedBuffer<u8> recvdata;
-                                       u16 peer_id = 132;
-                                       u16 size = 0;
-                                       bool received = false;
-                                       try{
-                                               size = client.Receive(peer_id, recvdata);
-                                               received = true;
-                                       }catch(con::NoIncomingDataException &e){
-                                       }
-                               }
-                       }
+                       UASSERT(memcmp(*sentdata, *recvdata, recvdata.getSize()) == 0);
                }
-#endif
+
+               u16 peer_id_client = 2;
                /*
                        Send a large packet
                */
                {
                        const int datasize = 30000;
-                       SharedBuffer<u8> data1(datasize);
-                       for(u16 i=0; i<datasize; i++){
-                               data1[i] = i/4;
+                       NetworkPacket pkt(0, datasize);
+                       for (u16 i=0; i<datasize; i++) {
+                               pkt << (u8) i/4;
                        }
 
                        infostream<<"Sending data (size="<<datasize<<"):";
-                       for(int i=0; i<datasize && i<20; i++){
+                       for(int i=0; i<datasize && i<20; i++) {
                                if(i%2==0) infostream<<" ";
                                char buf[10];
-                               snprintf(buf, 10, "%.2X", ((int)((const char*)*data1)[i])&0xff);
+                               snprintf(buf, 10, "%.2X", ((int)((const char*)pkt.getU8Ptr(0))[i])&0xff);
                                infostream<<buf;
                        }
-                       if(datasize>20)
-                               infostream<<"...";
-                       infostream<<std::endl;
-                       
-                       server.Send(peer_id_client, 0, data1, true);
+                       if(datasize > 20)
+                               infostream << "...";
+                       infostream << std::endl;
+
+                       Buffer<u8> sentdata = pkt.oldForgePacket();
+
+                       server.Send(peer_id_client, 0, &pkt, true);
 
                        //sleep_ms(3000);
-                       
-                       SharedBuffer<u8> recvdata;
-                       infostream<<"** running client.Receive()"<<std::endl;
+
+                       Buffer<u8> recvdata;
+                       infostream << "** running client.Receive()" << std::endl;
                        u16 peer_id = 132;
                        u16 size = 0;
                        bool received = false;
                        u32 timems0 = porting::getTimeMs();
-                       for(;;){
+                       for(;;) {
                                if(porting::getTimeMs() - timems0 > 5000 || received)
                                        break;
-                               try{
-                                       size = client.Receive(peer_id, recvdata);
+                               try {
+                                       NetworkPacket pkt;
+                                       client.Receive(&pkt);
+                                       size = pkt.getSize();
+                                       peer_id = pkt.getPeerId();
+                                       recvdata = pkt.oldForgePacket();
                                        received = true;
-                               }catch(con::NoIncomingDataException &e){
+                               } catch(con::NoIncomingDataException &e) {
                                }
                                sleep_ms(10);
                        }
@@ -1708,37 +2104,67 @@ struct TestConnection: public TestBase
                                infostream<<"...";
                        infostream<<std::endl;
 
-                       UASSERT(memcmp(*data1, *recvdata, data1.getSize()) == 0);
+                       UASSERT(memcmp(*sentdata, *recvdata, recvdata.getSize()) == 0);
                        UASSERT(peer_id == PEER_ID_SERVER);
                }
-               
+
                // Check peer handlers
                UASSERT(hand_client.count == 1);
                UASSERT(hand_client.last_id == 1);
                UASSERT(hand_server.count == 1);
                UASSERT(hand_server.last_id == 2);
-               
-               //assert(0);
        }
 };
 
-#define TEST(X)\
-{\
+struct TestProfiler : public TestBase
+{
+       void Run()
+       {
+               Profiler p;
+
+               p.avg("Test1", 1.f);
+               UASSERT(p.getValue("Test1") == 1.f);
+
+               p.avg("Test1", 2.f);
+               UASSERT(p.getValue("Test1") == 1.5f);
+
+               p.avg("Test1", 3.f);
+               UASSERT(p.getValue("Test1") == 2.f);
+
+               p.avg("Test1", 486.f);
+               UASSERT(p.getValue("Test1") == 123.f);
+
+               p.avg("Test1", 8);
+               UASSERT(p.getValue("Test1") == 100.f);
+
+               p.avg("Test1", 700);
+               UASSERT(p.getValue("Test1") == 200.f);
+
+               p.avg("Test1", 10000);
+               UASSERT(p.getValue("Test1") == 1600.f);
+
+               p.avg("Test2", 123.56);
+               p.avg("Test2", 123.58);
+
+               UASSERT(p.getValue("Test2") == 123.57f);
+       }
+};
+
+#define TEST(X) do {\
        X x;\
        infostream<<"Running " #X <<std::endl;\
        x.Run();\
        tests_run++;\
        tests_failed += x.test_failed ? 1 : 0;\
-}
+} while (0)
 
-#define TESTPARAMS(X, ...)\
-{\
+#define TESTPARAMS(X, ...) do {\
        X x;\
        infostream<<"Running " #X <<std::endl;\
        x.Run(__VA_ARGS__);\
        tests_run++;\
        tests_failed += x.test_failed ? 1 : 0;\
-}
+} while (0)
 
 void run_tests()
 {
@@ -1746,18 +2172,22 @@ void run_tests()
 
        int tests_run = 0;
        int tests_failed = 0;
-       
+
        // Create item and node definitions
        IWritableItemDefManager *idef = createItemDefManager();
        IWritableNodeDefManager *ndef = createNodeDefManager();
        define_some_nodes(idef, ndef);
 
+       log_set_lev_silence(LMT_ERROR, true);
+
        infostream<<"run_tests() started"<<std::endl;
        TEST(TestUtilities);
+       TEST(TestPath);
        TEST(TestSettings);
        TEST(TestCompress);
        TEST(TestSerialization);
        TEST(TestNodedefSerialization);
+       TEST(TestProfiler);
        TESTPARAMS(TestMapNode, ndef);
        TESTPARAMS(TestVoxelManipulator, ndef);
        TESTPARAMS(TestVoxelAlgorithms, ndef);
@@ -1767,17 +2197,23 @@ void run_tests()
        TEST(TestCollision);
        if(INTERNET_SIMULATOR == false){
                TEST(TestSocket);
-               dout_con<<"=== BEGIN RUNNING UNIT TESTS FOR CONNECTION ==="<<std::endl;
+               dout_con << "=== BEGIN RUNNING UNIT TESTS FOR CONNECTION ===" << std::endl;
                TEST(TestConnection);
-               dout_con<<"=== END RUNNING UNIT TESTS FOR CONNECTION ==="<<std::endl;
+               dout_con << "=== END RUNNING UNIT TESTS FOR CONNECTION ===" << std::endl;
        }
-       if(tests_failed == 0){
-               infostream<<"run_tests(): "<<tests_failed<<" / "<<tests_run<<" tests failed."<<std::endl;
-               infostream<<"run_tests() passed."<<std::endl;
+
+       log_set_lev_silence(LMT_ERROR, false);
+
+       delete idef;
+       delete ndef;
+
+       if(tests_failed == 0) {
+               actionstream << "run_tests(): " << tests_failed << " / " << tests_run << " tests failed." << std::endl;
+               actionstream << "run_tests() passed." << std::endl;
                return;
        } else {
-               errorstream<<"run_tests(): "<<tests_failed<<" / "<<tests_run<<" tests failed."<<std::endl;
-               errorstream<<"run_tests() aborting."<<std::endl;
+               errorstream << "run_tests(): " << tests_failed << " / " << tests_run << " tests failed." << std::endl;
+               errorstream << "run_tests() aborting." << std::endl;
                abort();
        }
 }