From 3b42785c57768e619ade198f7b4760e16288e0f3 Mon Sep 17 00:00:00 2001
From: Roland Reichwein <mail@reichwein.it>
Date: Sat, 14 Jan 2023 11:12:21 +0100
Subject: Added config tests, moved from BSD license to CC0

---
 tests/test-webserver.cpp   | 118 +++++++++++++++++++++++++++++++++++++++++++--
 tests/webserverprocess.cpp |  16 +++---
 tests/webserverprocess.h   |   6 +--
 3 files changed, 128 insertions(+), 12 deletions(-)

(limited to 'tests')

diff --git a/tests/test-webserver.cpp b/tests/test-webserver.cpp
index 5353c6a..72f2ab2 100644
--- a/tests/test-webserver.cpp
+++ b/tests/test-webserver.cpp
@@ -532,14 +532,14 @@ echo -ne "HTTP_HOST: $HTTP_HOST\r\n"
  BOOST_CHECK_EQUAL(result.second, "500 Bad Script: test2.sh/path1");
 }
 
-BOOST_FIXTURE_TEST_CASE(empty_config, Fixture)
+BOOST_FIXTURE_TEST_CASE(config_empty, Fixture)
 {
- WebserverProcess serverProcess{""};
+ WebserverProcess serverProcess{std::string{}};
  std::this_thread::sleep_for(std::chrono::milliseconds(50));
  BOOST_REQUIRE_EQUAL(serverProcess.is_running(), false);
 }
 
-BOOST_FIXTURE_TEST_CASE(incomplete_config, Fixture)
+BOOST_FIXTURE_TEST_CASE(config_incomplete, Fixture)
 {
  std::string webserver_config{R"CONFIG(<webserver>
  <user>www-data</user>
@@ -550,6 +550,118 @@ BOOST_FIXTURE_TEST_CASE(incomplete_config, Fixture)
  BOOST_REQUIRE_EQUAL(serverProcess.is_running(), false);
 }
 
+BOOST_FIXTURE_TEST_CASE(config_bad_syntax, Fixture)
+{
+ std::string config{R"CONFIG(<webserver>
+ <user>www-data</user>
+ <group>www-data</group>
+ <threads>10</threads>
+ <statisticspath>stats.db</statisticspath>
+ <plugin-directory>../plugins</plugin-directory>
+ <sites>
+  <site>
+   <name>localhost</name>
+   <host>ip6-localhost</host>
+   <host>localhost</host>
+   <host>127.0.0.1</host>
+   <host>[::1]</host>
+   <path requested="/">
+    <plugin>static-files</plugin>
+    <target>.</target>
+   </path>
+   <certpath>testchain.pem</certpath>
+   <keypath>testkey.pem</keypath>
+  </site>
+ </sites>
+ <sockets>
+  <socket>
+   <address>127.0.0.1</address>
+   <port>8080</port>
+   <protocol>http</protocol>
+   <site>localhost</site>
+  </socket>
+  <socket>
+   <address>::1</address>
+   <port>8080</port>
+   <protocol>http</protocol>
+   <site>localhost</site>
+  </socket>
+  <socket>
+   <address>127.0.0.1</address>
+   <port>8081</port>
+   <protocol>https</protocol>
+   <site>localhost</site>
+  </socket>
+  <socket>
+   <address>::1</address>
+   <port>8081</port>
+   <protocol>https</protocol>
+   <site>localhost</site>
+  <!-- </socket> -->
+ </sockets>
+</webserver>
+)CONFIG"};
+ WebserverProcess serverProcess{config, false};
+ std::this_thread::sleep_for(std::chrono::milliseconds(50));
+ BOOST_REQUIRE_EQUAL(serverProcess.is_running(), false);
+}
+
+BOOST_FIXTURE_TEST_CASE(config_bad_socket, Fixture)
+{
+ std::string config{R"CONFIG(<webserver>
+ <user>www-data</user>
+ <group>www-data</group>
+ <threads>10</threads>
+ <statisticspath>stats.db</statisticspath>
+ <plugin-directory>../plugins</plugin-directory>
+ <sites>
+  <site>
+   <name>localhost</name>
+   <host>ip6-localhost</host>
+   <host>localhost</host>
+   <host>127.0.0.1</host>
+   <host>[::1]</host>
+   <path requested="/">
+    <plugin>static-files</plugin>
+    <target>.</target>
+   </path>
+   <certpath>testchain.pem</certpath>
+   <keypath>testkey.pem</keypath>
+  </site>
+ </sites>
+ <sockets>
+  <socket>
+   <address>127.0.0.1</address>
+   <port>8080</port>
+   <protocol>http</protocol>
+   <site>localhost</site>
+  </socket>
+  <socket>
+   <address>::1</address>
+   <port>8080</port>
+   <protocol>http</protocol>
+   <site>localhost</site>
+  </socket>
+  <socket>
+   <address>127.0.0.1</address>
+   <port>8081</port>
+   <protocol>https</protocol>
+   <site>localhost</site>
+  </socket>
+  <socket>
+   <address>1234:5678::1</address>
+   <port>8081</port>
+   <protocol>https</protocol>
+   <site>localhost</site>
+  </socket>
+ </sockets>
+</webserver>
+)CONFIG"};
+ WebserverProcess serverProcess{config, false};
+ std::this_thread::sleep_for(std::chrono::milliseconds(50));
+ BOOST_REQUIRE_EQUAL(serverProcess.is_running(), false);
+}
+
 BOOST_FIXTURE_TEST_CASE(http_redirect, Fixture)
 {
  std::string webserver_config{R"CONFIG(<webserver>
diff --git a/tests/webserverprocess.cpp b/tests/webserverprocess.cpp
index f9ecdd3..31c9766 100644
--- a/tests/webserverprocess.cpp
+++ b/tests/webserverprocess.cpp
@@ -109,12 +109,14 @@ VZTqPHmb+db0rFA3XlAg2A==
  start();
 }
 
-WebserverProcess::WebserverProcess(const std::string& config): m_pid{}
+WebserverProcess::WebserverProcess(const std::string& config, bool wait_for_listener):
+ m_pid{}, m_wait_for_listener{wait_for_listener}
 {
  init(config);
 }
 
-WebserverProcess::WebserverProcess(): m_pid{}
+WebserverProcess::WebserverProcess(bool wait_for_listener):
+ m_pid{}, m_wait_for_listener{wait_for_listener}
 {
  std::string config{R"CONFIG(<webserver>
  <user>www-data</user>
@@ -190,10 +192,12 @@ void WebserverProcess::start()
   webserver(sizeof(argv) / sizeof(char*), argv);
   exit(0);
  }
-
- // wait for server to start up
- if (int port{port_from_config(m_config)}; port >= 0)
-  Process::wait_for_pid_listening_on(m_pid, port);
+ 
+ if (m_wait_for_listener) {
+  // wait for server to start up
+  if (int port{port_from_config(m_config)}; port >= 0)
+   Process::wait_for_pid_listening_on(m_pid, port);
+ }
  std::this_thread::sleep_for(std::chrono::milliseconds(20));
 }
 
diff --git a/tests/webserverprocess.h b/tests/webserverprocess.h
index 5593c70..3a4b7d6 100644
--- a/tests/webserverprocess.h
+++ b/tests/webserverprocess.h
@@ -14,8 +14,8 @@ class WebserverProcess
 private:
  void init(const std::string& config);
 public:
- WebserverProcess(const std::string& config);
- WebserverProcess();
+ WebserverProcess(const std::string& config, bool wait_for_listener = true);
+ WebserverProcess(bool wait_for_listener = true);
  ~WebserverProcess();
  void start();
  void stop();
@@ -25,6 +25,6 @@ public:
 private:
  pid_t m_pid;
  std::string m_config;
-
+ bool m_wait_for_listener;
 }; // class WebserverProcess
 
-- 
cgit v1.2.3