From 9b73b3c7dc7c012b53ea21a2ade35f6087223402 Mon Sep 17 00:00:00 2001
From: Paul Davis <paul@linuxaudiosystems.com>
Date: Thu, 11 Jul 2013 11:11:45 -0400
Subject: [PATCH] Use common test function to create temporary session

---
 .gitignore                               | 1 -
 libs/ardour/test/test_needing_session.cc | 9 ++++++---
 libs/ardour/wscript                      | 2 ++
 3 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/.gitignore b/.gitignore
index 48a95b8e4b..f2251ba204 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,7 +1,6 @@
 *.[oa]
 *.o
 *.os
-*.template
 *.binding
 *.so
 *.dylib
diff --git a/libs/ardour/test/test_needing_session.cc b/libs/ardour/test/test_needing_session.cc
index a6220ded12..181d391e01 100644
--- a/libs/ardour/test/test_needing_session.cc
+++ b/libs/ardour/test/test_needing_session.cc
@@ -1,3 +1,5 @@
+#include <glibmm/miscutils.h>
+
 #include "midi++/manager.h"
 #include "pbd/compose.h"
 #include "pbd/enumwriter.h"
@@ -5,6 +7,7 @@
 #include "ardour/audioengine.h"
 #include "test_needing_session.h"
 #include "test_util.h"
+#include "test_common.h"
 
 using namespace std;
 using namespace ARDOUR;
@@ -13,9 +16,9 @@ using namespace PBD;
 void
 TestNeedingSession::setUp ()
 {
-	string const test_session_path = "libs/ardour/test/test_session";
-	system (string_compose ("rm -rf %1", test_session_path).c_str());
-	_session = load_session (test_session_path, "test_session");
+	const string session_name("test_session");
+	std::string new_session_dir = Glib::build_filename (new_test_output_dir(), session_name);
+	_session = load_session (new_session_dir, "test_session");
 }
 
 void
diff --git a/libs/ardour/wscript b/libs/ardour/wscript
index 2b16ed3a87..131dda520b 100644
--- a/libs/ardour/wscript
+++ b/libs/ardour/wscript
@@ -483,6 +483,7 @@ def build(bld):
             create_ardour_test_program(bld, obj.includes, 'region_naming', 'test_region_naming', ['test/region_naming_test.cc'])
             create_ardour_test_program(bld, obj.includes, 'control_surface', 'test_control_surfaces', ['test/control_surfaces_test.cc'])
             create_ardour_test_program(bld, obj.includes, 'mtdm_test', 'test_mtdm', ['test/mtdm_test.cc'])
+            create_ardour_test_program(bld, obj.includes, 'session_test', 'test_session', ['test/session_test.cc'])
 
         test_sources  = '''
             test/automation_list_property_test.cc
@@ -499,6 +500,7 @@ def build(bld):
             test/region_naming_test.cc
             test/control_surfaces_test.cc
             test/mtdm_test.cc
+            test/session_test.cc
         '''.split()
 
 # Tests that don't work