First draft. Tested search path adding mechanism.

7 files changed, 168 insertions, 12 deletions

diff --git a/CMakeLists.txt b/CMakeLists.txt
index d824b86..6651069 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -32,10 +32,11 @@ OPTION(BUILD_DOCUMENTATION "Create and install the HTML based API documentation
 # Enable C++11 and all warnings
 SET(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -pedantic-errors -std=c++11")
 
-# Include expat and mozjs-24 via PkgConfig
+# Include boost (filesystem) expat and mozjs-24 via PkgConfig
 FIND_PACKAGE(PkgConfig REQUIRED)
 PKG_CHECK_MODULES(MOZJS REQUIRED mozjs-24)
 PKG_CHECK_MODULES(EXPAT REQUIRED expat)
+FIND_PACKAGE(Boost COMPONENTS system filesystem REQUIRED)
 
 ################################################################################
 # Inclusion of doxygen                                                         #
@@ -85,6 +86,7 @@ INCLUDE_DIRECTORIES(
 	src/
 	${MOZJS_INCLUDE_DIRS}
 	${EXPAT_INCLUDE_DIRS}
+	${Boost_INCLUDE_DIR}
 )
 
 # Link directories
@@ -121,6 +123,15 @@ ADD_LIBRARY(ousia_core
 	src/core/variant/Variant
 )
 
+ADD_LIBRARY(ousia_boost
+	src/plugins/boost/FileLocator
+)
+
+TARGET_LINK_LIBRARIES(ousia_boost
+	ousia_core
+	${Boost_LIBRARIES}
+)
+
 ADD_LIBRARY(ousia_css
 	src/plugins/css/CSSParser
 )
@@ -181,6 +192,16 @@ IF(TEST)
 		ousia_core
 	)
 
+	ADD_EXECUTABLE(ousia_test_boost
+		test/plugins/boost/FileLocatorTest
+	)
+
+	TARGET_LINK_LIBRARIES(ousia_test_boost
+		${GTEST_LIBRARIES}
+		ousia_core
+		ousia_boost
+	)
+
 	ADD_EXECUTABLE(ousia_test_css
 		test/plugins/css/CSSParserTest
 	)
diff --git a/src/core/ResourceLocator.hpp b/src/core/ResourceLocator.hpp
index 36930e9..926b312 100644
--- a/src/core/ResourceLocator.hpp
+++ b/src/core/ResourceLocator.hpp
@@ -39,11 +39,11 @@ public:
 	 */
 	enum class Type {
 		// A Domain description
-		DOMAIN,
+		DOMAIN_DESC,
 		// An ECMA/JavaScript
 		SCRIPT,
 		// A Type System
-		TYPESYSTEM,
+		TYPESYS,
 		// TODO: Aren't documents and attribute descriptors missing?
 		// TODO: What is the purpose of these two?
 		GENERIC_MODULE,
diff --git a/src/plugins/boost/FileLocator.cpp b/src/plugins/boost/FileLocator.cpp
new file mode 100644
index 0000000..fdbe4d6
--- /dev/null
+++ b/src/plugins/boost/FileLocator.cpp
@@ -0,0 +1,56 @@
+/*
+    Ousía
+    Copyright (C) 2014, 2015  Benjamin Paaßen, Andreas Stöckel
+
+    This program is free software: you can redistribute it and/or modify
+    it under the terms of the GNU General Public License as published by
+    the Free Software Foundation, either version 3 of the License, or
+    (at your option) any later version.
+
+    This program is distributed in the hope that it will be useful,
+    but WITHOUT ANY WARRANTY; without even the implied warranty of
+    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+    GNU General Public License for more details.
+
+    You should have received a copy of the GNU General Public License
+    along with this program.  If not, see <http://www.gnu.org/licenses/>.
+*/
+
+#include "FileLocator.hpp"
+
+#include <fstream>
+
+namespace ousia {
+
+void FileLocator::addSearchPath(const boost::filesystem::path &path,
+                                std::set<ResourceLocator::Type> types)
+{
+	for (auto &type : types) {
+		// retrieve the path vector for the given type.
+		auto it = searchPaths.find(type);
+		if (it != searchPaths.end()) {
+			it->second.push_back(path);
+		} else {
+			std::vector<boost::filesystem::path> v{path};
+			searchPaths.insert({type, v});
+		}
+	}
+}
+
+ResourceLocator::Location FileLocator::locate(const std::string &path,
+                                              const std::string &relativeTo,
+                                              const Type type) const
+{
+	// TODO: Implement Properly
+	ResourceLocator::Location l(false, *this, type, "");
+	return l;
+}
+
+std::unique_ptr<std::istream> FileLocator::stream(
+    const std::string &location) const
+{
+	std::unique_ptr<std::istream> ifs {
+	    new std::ifstream(location)};
+	return std::move(ifs);
+}
+}
diff --git a/src/plugins/boost/FileLocator.hpp b/src/plugins/boost/FileLocator.hpp
index 50a77db..41826b8 100644
--- a/src/plugins/boost/FileLocator.hpp
+++ b/src/plugins/boost/FileLocator.hpp
@@ -19,10 +19,14 @@
 #ifndef _OUSIA_FILE_LOCATOR_HPP_
 #define _OUSIA_FILE_LOCATOR_HPP_
 
+#include <core/ResourceLocator.hpp>
+
 #include <map>
 #include <set>
+#include <vector>
+
+#include <boost/filesystem/path.hpp>
 
-#include <core/ResourceLocator.hpp>
 
 namespace ousia {
 
@@ -36,10 +40,10 @@ namespace ousia {
  */
 class FileLocator : public ResourceLocator {
 private:
-	map<ResourceLocator::Type, std::vector<std::string>> searchPaths;
+	std::map<ResourceLocator::Type, std::vector<boost::filesystem::path>> searchPaths;
 
 public:
-	FileLocator() searchpaths() {}
+	FileLocator() : searchPaths() {}
 
 	/**
 	 * Adds a search paths for the given types.
@@ -49,9 +53,19 @@ public:
 	 *              resources of the specified types at the given path in the
 	 *              future.
 	 */
-	void addSearchPath(const std::string &path,
+	void addSearchPath(const boost::filesystem::path &path,
 	                   std::set<ResourceLocator::Type> types);
 
+	/**
+	 * Returns the backing map containing all search paths for a given type.
+	 * This is read-only.
+	 */
+	const std::map<ResourceLocator::Type, std::vector<boost::filesystem::path>> &
+	getSearchPaths() const
+	{
+		return searchPaths;
+	}
+
 	Location locate(const std::string &path, const std::string &relativeTo,
 	                const Type type) const override;
 
diff --git a/test/core/RegistryTest.cpp b/test/core/RegistryTest.cpp
index e06011a..4450227 100644
--- a/test/core/RegistryTest.cpp
+++ b/test/core/RegistryTest.cpp
@@ -54,9 +54,9 @@ TEST(Registry, locateResource)
 	instance.registerResourceLocator(&locator);
 	
 	ResourceLocator::Location location =
-	    instance.locateResource("path", "", ResourceLocator::Type::DOMAIN);
+	    instance.locateResource("path", "", ResourceLocator::Type::DOMAIN_DESC);
 	ASSERT_TRUE(location.found);
-	ASSERT_EQ(ResourceLocator::Type::DOMAIN, location.type);
+	ASSERT_EQ(ResourceLocator::Type::DOMAIN_DESC, location.type);
 	ASSERT_EQ("path", location.location);
 }
 
diff --git a/test/core/ResourceLocatorTest.cpp b/test/core/ResourceLocatorTest.cpp
index 38cfc3e..ebb164d 100644
--- a/test/core/ResourceLocatorTest.cpp
+++ b/test/core/ResourceLocatorTest.cpp
@@ -48,9 +48,9 @@ TEST(ResourceLocator, locate)
 {
 	TestResourceLocator instance;
 	ResourceLocator::Location location =
-	    instance.locate("path", "", ResourceLocator::Type::DOMAIN);
+	    instance.locate("path", "", ResourceLocator::Type::DOMAIN_DESC);
 	ASSERT_TRUE(location.found);
-	ASSERT_EQ(ResourceLocator::Type::DOMAIN, location.type);
+	ASSERT_EQ(ResourceLocator::Type::DOMAIN_DESC, location.type);
 	ASSERT_EQ("path", location.location);
 }
 
@@ -58,7 +58,7 @@ TEST(ResourceLocator, stream)
 {
 	TestResourceLocator instance;
 	ResourceLocator::Location location =
-	    instance.locate("path", "", ResourceLocator::Type::DOMAIN);
+	    instance.locate("path", "", ResourceLocator::Type::DOMAIN_DESC);
 	std::unique_ptr<std::istream> is = location.stream();
 
 	std::string str;
diff --git a/test/plugins/boost/FileLocatorTest.cpp b/test/plugins/boost/FileLocatorTest.cpp
new file mode 100644
index 0000000..7730492
--- /dev/null
+++ b/test/plugins/boost/FileLocatorTest.cpp
@@ -0,0 +1,65 @@
+/*
+    Ousía
+    Copyright (C) 2014, 2015  Benjamin Paaßen, Andreas Stöckel
+
+    This program is free software: you can redistribute it and/or modify
+    it under the terms of the GNU General Public License as published by
+    the Free Software Foundation, either version 3 of the License, or
+    (at your option) any later version.
+
+    This program is distributed in the hope that it will be useful,
+    but WITHOUT ANY WARRANTY; without even the implied warranty of
+    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+    GNU General Public License for more details.
+
+    You should have received a copy of the GNU General Public License
+    along with this program.  If not, see <http://www.gnu.org/licenses/>.
+*/
+
+#include <gtest/gtest.h>
+
+#include <plugins/boost/FileLocator.hpp>
+
+namespace ousia {
+TEST(FileLocator, testAddSearchPath)
+{
+	FileLocator instance;
+	ASSERT_EQ(0, instance.getSearchPaths().size());
+
+	// Add one path for three types.
+	instance.addSearchPath(
+	    ".", {ResourceLocator::Type::DOMAIN_DESC, ResourceLocator::Type::SCRIPT,
+	          ResourceLocator::Type::TYPESYS});
+
+	ASSERT_EQ(3, instance.getSearchPaths().size());
+
+	auto it =
+	    instance.getSearchPaths().find(ResourceLocator::Type::DOMAIN_DESC);
+
+	ASSERT_EQ(1, it->second.size());
+	ASSERT_EQ(".", it->second[0].generic_string());
+
+	it = instance.getSearchPaths().find(ResourceLocator::Type::SCRIPT);
+
+	ASSERT_EQ(1, it->second.size());
+	ASSERT_EQ(".", it->second[0].generic_string());
+
+	it = instance.getSearchPaths().find(ResourceLocator::Type::TYPESYS);
+
+	ASSERT_EQ(1, it->second.size());
+	ASSERT_EQ(".", it->second[0].generic_string());
+
+	// Add another path for only one of those types.
+
+	instance.addSearchPath("..", {ResourceLocator::Type::DOMAIN_DESC});
+
+	ASSERT_EQ(3, instance.getSearchPaths().size());
+
+	auto it =
+	    instance.getSearchPaths().find(ResourceLocator::Type::DOMAIN_DESC);
+
+	ASSERT_EQ(2, it->second.size());
+	ASSERT_EQ(".", it->second[0].generic_string());
+	ASSERT_EQ("..", it->second[1].generic_string());
+}
+}