Still fixing the unittests...amule-svn-r10943
commit11a181ce2e91427ac1a00e5992f6437f4cabb007
authorupstream svn <svn@amule.org>
Mon, 1 Aug 2016 05:56:44 +0000 (1 05:56 +0000)
committerDévai Tamás <gonosztopi@amule.org>
Mon, 1 Aug 2016 07:02:32 +0000 (1 07:02 +0000)
tree91de41f4f209c2a239fd8768a3f410f74579c5a9
parent4e92cc1780971c526a3d3f7c2cf0e0a3f6a7873b
Still fixing the unittests...

- Fix argument order in ASSERT_EQUALS

  For a few tests the order of the arguments of ASSERT_EQUALS were swapped.
  Although it did not cause problems beacuse of the commutative nature of the
  equality operator, if such a test failed the reported error was incorrect
  and misleading.

  Also removed a duplicate test.

- Fix an unused parameter warning

- Fix unittest compilation with Boost enabled

- Fix check for '%e' conversion

  The documentation says only that the exponent is at least two digits, not that
  it is exactly two digits. It depends on the underlying C Runtime Library how
  many digits are used.

  The check itself is meant only to check if a wrong type argument is accepted
  and handled as if it was a matching type, and it still fulfills that
  requirement.

- Ensure we can read the source directory

  By checking early for the existence and readability of the source directory
  we can provide a different failure message from when the actual file I/O
  check fail.

  There are systems where the compiler's handling of non-ASCII characters is
  different from the OS's way. In such a system this test will fail if the
  source directory name contains non-ASCII characters.

- The order does matter, #2

  For some linker the order of the libraries on the command-line does matter.
.svn-revision
src/Logger.h
unittests/muleunit/main.cpp
unittests/tests/FormatTest.cpp
unittests/tests/Makefile.am
unittests/tests/Makefile.in
unittests/tests/NetworkFunctionsTest.cpp
unittests/tests/PathTest.cpp
unittests/tests/RangeMapTest.cpp
unittests/tests/TextFileTest.cpp