Consistent unit test header inclusion order

This patch consistently changes the inclusion order for unit test files
to the following:

 1. The header of the unit under test (using <> includes).
 2. The unit_test.hpp header (using "" includes).
 3. Any additional auxiliary test headers (using "" includes and sorted alphabetically).
 4. Additional system headers needed for the test (using <> includes and sorted alphabetically).
 5. Conditionally included system headers (using <> includes).

Putting the unit under test's header at the very beginning has the
advantage of also testing that the header is self-contained.  It also
makes it very quick to tell what unit is tested in this file.

tests/check.cpp

@@ -1,4 +1,5 @@
-#include "toml.hpp"
+#include <toml.hpp>
+
 #include <iostream>
 #include <iomanip>
 

tests/check_serialization.cpp

@@ -1,6 +1,7 @@
-#include "toml.hpp"
+#include <toml.hpp>
-#include <iostream>
+
 #include <iomanip>
+#include <iostream>
 
 int main(int argc, char **argv)
 {

tests/check_toml_test.cpp

@@ -1,6 +1,7 @@
-#include "toml.hpp"
+#include <toml.hpp>
-#include <iostream>
+
 #include <iomanip>
+#include <iostream>
 
 struct json_serializer
 {

tests/test_comments.cpp

@@ -1,7 +1,7 @@
-#include "unit_test.hpp"
-
 #include <toml.hpp>
 
+#include "unit_test.hpp"
+
 BOOST_AUTO_TEST_CASE(test_comment_before)
 {
     {

tests/test_datetime.cpp

@@ -1,7 +1,7 @@
-#include "unit_test.hpp"
-
 #include <toml/datetime.hpp>
 
+#include "unit_test.hpp"
+
 BOOST_AUTO_TEST_CASE(test_local_date)
 {
     const toml::local_date date(2018, toml::month_t::Jan, 1);

tests/test_error_detection.cpp

@@ -1,8 +1,9 @@
+#include <toml.hpp>
+
 #include "unit_test.hpp"
 
-#include <toml.hpp>
-#include <iostream>
 #include <fstream>
+#include <iostream>
 
 BOOST_AUTO_TEST_CASE(test_detect_empty_key)
 {

tests/test_expect.cpp

@@ -1,11 +1,12 @@
+#include <toml.hpp>
+
 #include "unit_test.hpp"
 
-#include <toml.hpp>
+#include <array>
+#include <deque>
+#include <list>
 #include <map>
 #include <unordered_map>
-#include <list>
-#include <deque>
-#include <array>
 
 BOOST_AUTO_TEST_CASE(test_expect)
 {

tests/test_extended_conversions.cpp

@@ -1,6 +1,7 @@
+#include <toml.hpp>
+
 #include "unit_test.hpp"
 
-#include <toml.hpp>
 #include <deque>
 #include <map>
 

tests/test_find.cpp

@@ -1,15 +1,17 @@
+#include <toml.hpp>
+
 #include "unit_test.hpp"
 
-#include <toml.hpp>
-#include <map>
-#include <unordered_map>
-#include <list>
-#include <deque>
 #include <array>
+#include <deque>
+#include <list>
+#include <map>
+#include <tuple>
+#include <unordered_map>
+
 #if TOML11_CPLUSPLUS_STANDARD_VERSION >= 201703L
 #include <string_view>
 #endif
-#include <tuple>
 
 using test_value_types = std::tuple<
     toml::basic_value<toml::discard_comments>,
@@ -823,4 +825,3 @@ BOOST_AUTO_TEST_CASE_TEMPLATE(test_get_toml_offset_datetime, value_type, test_va
         BOOST_TEST(tm.tm_sec ==            0);
     }
 }
-

tests/test_find_or.cpp

@@ -1,12 +1,14 @@
+#include <toml.hpp>
+
 #include "unit_test.hpp"
 
-#include <toml.hpp>
-#include <map>
-#include <unordered_map>
-#include <list>
-#include <deque>
 #include <array>
+#include <deque>
+#include <list>
+#include <map>
 #include <tuple>
+#include <unordered_map>
+
 #if TOML11_CPLUSPLUS_STANDARD_VERSION >= 201703L
 #include <string_view>
 #endif

tests/test_find_or_recursive.cpp

@@ -1,12 +1,14 @@
+#include <toml.hpp>
+
 #include "unit_test.hpp"
 
-#include <toml.hpp>
-#include <map>
-#include <unordered_map>
-#include <list>
-#include <deque>
 #include <array>
+#include <deque>
+#include <list>
+#include <map>
 #include <tuple>
+#include <unordered_map>
+
 #if TOML11_CPLUSPLUS_STANDARD_VERSION >= 201703L
 #include <string_view>
 #endif

tests/test_format_error.cpp

@@ -1,6 +1,7 @@
+#include <toml.hpp>
+
 #include "unit_test.hpp"
 
-#include <toml.hpp>
 #include <iostream>
 
 // to check it successfully compiles. it does not check the formatted string.

tests/test_get.cpp

@@ -1,12 +1,14 @@
+#include <toml.hpp>
+
 #include "unit_test.hpp"
 
-#include <toml.hpp>
-#include <map>
-#include <unordered_map>
-#include <list>
-#include <deque>
 #include <array>
+#include <deque>
+#include <list>
+#include <map>
 #include <tuple>
+#include <unordered_map>
+
 #if TOML11_CPLUSPLUS_STANDARD_VERSION >= 201703L
 #include <string_view>
 #endif

tests/test_get_or.cpp

@@ -1,12 +1,14 @@
+#include <toml.hpp>
+
 #include "unit_test.hpp"
 
-#include <toml.hpp>
-#include <map>
-#include <unordered_map>
-#include <list>
-#include <deque>
 #include <array>
+#include <deque>
+#include <list>
+#include <map>
 #include <tuple>
+#include <unordered_map>
+
 #if TOML11_CPLUSPLUS_STANDARD_VERSION >= 201703L
 #include <string_view>
 #endif

tests/test_lex_boolean.cpp

@@ -1,6 +1,6 @@
-#include "unit_test.hpp"
-
 #include <toml/lexer.hpp>
+
+#include "unit_test.hpp"
 #include "test_lex_aux.hpp"
 
 using namespace toml;

tests/test_lex_datetime.cpp

@@ -1,6 +1,6 @@
-#include "unit_test.hpp"
-
 #include <toml/lexer.hpp>
+
+#include "unit_test.hpp"
 #include "test_lex_aux.hpp"
 
 using namespace toml;

tests/test_lex_floating.cpp

@@ -1,9 +1,10 @@
-#include "unit_test.hpp"
-
 #include <toml/lexer.hpp>
-#include <limits>
+
+#include "unit_test.hpp"
 #include "test_lex_aux.hpp"
 
+#include <limits>
+
 using namespace toml;
 using namespace detail;
 

tests/test_lex_integer.cpp

@@ -1,6 +1,6 @@
-#include "unit_test.hpp"
-
 #include <toml/lexer.hpp>
+
+#include "unit_test.hpp"
 #include "test_lex_aux.hpp"
 
 using namespace toml;

tests/test_lex_key_comment.cpp

@@ -1,6 +1,6 @@
-#include "unit_test.hpp"
-
 #include <toml/lexer.hpp>
+
+#include "unit_test.hpp"
 #include "test_lex_aux.hpp"
 
 using namespace toml;

tests/test_lex_string.cpp

@@ -1,6 +1,6 @@
-#include "unit_test.hpp"
-
 #include <toml/lexer.hpp>
+
+#include "unit_test.hpp"
 #include "test_lex_aux.hpp"
 
 using namespace toml;

tests/test_literals.cpp

@@ -1,6 +1,7 @@
+#include <toml.hpp>
+
 #include "unit_test.hpp"
 
-#include <toml.hpp>
 #include <map>
 
 BOOST_AUTO_TEST_CASE(test_file_as_literal)

tests/test_parse_array.cpp

@@ -1,6 +1,6 @@
-#include "unit_test.hpp"
-
 #include <toml/parser.hpp>
+
+#include "unit_test.hpp"
 #include "test_parse_aux.hpp"
 
 using namespace toml;

tests/test_parse_boolean.cpp

@@ -1,6 +1,6 @@
-#include "unit_test.hpp"
-
 #include <toml/parser.hpp>
+
+#include "unit_test.hpp"
 #include "test_parse_aux.hpp"
 
 using namespace toml;

tests/test_parse_datetime.cpp

@@ -1,6 +1,6 @@
-#include "unit_test.hpp"
-
 #include <toml/parser.hpp>
+
+#include "unit_test.hpp"
 #include "test_parse_aux.hpp"
 
 using namespace toml;

tests/test_parse_file.cpp

@@ -1,10 +1,11 @@
+#include <toml.hpp>
+
 #include "unit_test.hpp"
 
-#include <toml.hpp>
-#include <iostream>
-#include <fstream>
-#include <map>
 #include <deque>
+#include <fstream>
+#include <iostream>
+#include <map>
 
 BOOST_AUTO_TEST_CASE(test_example)
 {

tests/test_parse_floating.cpp

@@ -1,9 +1,10 @@
-#include "unit_test.hpp"
-
 #include <toml/parser.hpp>
-#include <cmath>
+
+#include "unit_test.hpp"
 #include "test_parse_aux.hpp"
 
+#include <cmath>
+
 using namespace toml;
 using namespace detail;
 

tests/test_parse_inline_table.cpp

@@ -1,6 +1,6 @@
-#include "unit_test.hpp"
-
 #include <toml/parser.hpp>
+
+#include "unit_test.hpp"
 #include "test_parse_aux.hpp"
 
 using namespace toml;

tests/test_parse_integer.cpp

@@ -1,6 +1,6 @@
-#include "unit_test.hpp"
-
 #include <toml/parser.hpp>
+
+#include "unit_test.hpp"
 #include "test_parse_aux.hpp"
 
 using namespace toml;

tests/test_parse_key.cpp

@@ -1,6 +1,6 @@
-#include "unit_test.hpp"
-
 #include <toml/parser.hpp>
+
+#include "unit_test.hpp"
 #include "test_parse_aux.hpp"
 
 using namespace toml;

tests/test_parse_string.cpp

@@ -1,6 +1,6 @@
-#include "unit_test.hpp"
-
 #include <toml/parser.hpp>
+
+#include "unit_test.hpp"
 #include "test_parse_aux.hpp"
 
 using namespace toml;

tests/test_parse_table.cpp

@@ -1,7 +1,7 @@
-#include "unit_test.hpp"
-
-#include <toml/parser.hpp>
 #include <toml/get.hpp>
+#include <toml/parser.hpp>
+
+#include "unit_test.hpp"
 #include "test_parse_aux.hpp"
 
 using namespace toml;

tests/test_parse_table_key.cpp

@@ -1,6 +1,6 @@
-#include "unit_test.hpp"
-
 #include <toml/parser.hpp>
+
+#include "unit_test.hpp"
 #include "test_parse_aux.hpp"
 
 using namespace toml;

tests/test_parse_unicode.cpp

@@ -1,8 +1,9 @@
+#include <toml.hpp>
+
 #include "unit_test.hpp"
 
-#include <toml.hpp>
-#include <iostream>
 #include <fstream>
+#include <iostream>
 
 BOOST_AUTO_TEST_CASE(test_hard_example_unicode)
 {

tests/test_result.cpp

@@ -1,7 +1,8 @@
+#include <toml/result.hpp>
+
 #include "unit_test.hpp"
 
 #include <iostream>
-#include <toml/result.hpp>
 
 BOOST_AUTO_TEST_CASE(test_construct)
 {

tests/test_serialize_file.cpp

@@ -1,10 +1,11 @@
+#include <toml.hpp>
+
 #include "unit_test.hpp"
 
-#include <toml.hpp>
 #include <deque>
-#include <map>
-#include <iostream>
 #include <fstream>
+#include <iostream>
+#include <map>
 #include <sstream>
 
 template<typename Comment,

tests/test_string.cpp

@@ -1,7 +1,7 @@
-#include "unit_test.hpp"
-
 #include <toml.hpp>
 
+#include "unit_test.hpp"
+
 BOOST_AUTO_TEST_CASE(test_basic_string)
 {
     {

tests/test_traits.cpp

@@ -1,16 +1,16 @@
-#include "unit_test.hpp"
-
 #include <toml/types.hpp>
 
-#include <list>
+#include "unit_test.hpp"
-#include <forward_list>
+
-#include <deque>
 #include <array>
+#include <deque>
+#include <forward_list>
+#include <list>
 #include <map>
 #include <set>
+#include <string>
 #include <unordered_map>
 #include <unordered_set>
-#include <string>
 
 struct dummy_type{};
 

tests/test_utility.cpp

@@ -1,8 +1,9 @@
+#include <toml/utility.hpp>
+
 #include "unit_test.hpp"
 
-#include <toml/utility.hpp>
-#include <vector>
 #include <array>
+#include <vector>
 
 BOOST_AUTO_TEST_CASE(test_try_reserve)
 {

tests/test_value.cpp

@@ -1,6 +1,7 @@
+#include <toml.hpp>
+
 #include "unit_test.hpp"
 
-#include <toml.hpp>
 #include <map>
 #include <list>