X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=test%2Fjob_test.cc;h=5a554145b4fe2ff7a030b1b3fec87d3028dc64de;hb=3c86edba7f02de3a65c7a58937f11429050b3e10;hp=a3ad1bb6c5936589e475d1d2cf4078d38fc1e2e7;hpb=373f010a7f04add1f49169cbaa60cb7ae5f508d4;p=dcpomatic.git

diff --git a/test/job_test.cc b/test/job_test.cc
index a3ad1bb6c..5a554145b 100644
--- a/test/job_test.cc
+++ b/test/job_test.cc
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2012 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2012-2015 Carl Hetherington <cth@carlh.net>
 
     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
@@ -17,6 +17,10 @@
 
 */
 
+/** @file  test/job_test.cc
+ *  @brief Basic tests of Job and JobManager.
+ */
+
 #include <boost/test/unit_test.hpp>
 #include "lib/job.h"
 #include "lib/job_manager.h"
@@ -28,8 +32,8 @@ using boost::shared_ptr;
 class TestJob : public Job
 {
 public:
-	TestJob (shared_ptr<Film> f)
-		: Job (f)
+	TestJob (shared_ptr<Film> film)
+		: Job (film)
 	{
 
 	}
@@ -44,7 +48,7 @@ public:
 
 	void run ()
 	{
-		while (1) {
+		while (true) {
 			if (finished ()) {
 				return;
 			}
@@ -54,14 +58,18 @@ public:
 	string name () const {
 		return "";
 	}
+
+	string json_name () const {
+		return "";
+	}
 };
 
 BOOST_AUTO_TEST_CASE (job_manager_test)
 {
-	shared_ptr<Film> f;
+	shared_ptr<Film> film;
 
 	/* Single job */
-	shared_ptr<TestJob> a (new TestJob (f));
+	shared_ptr<TestJob> a (new TestJob (film));
 
 	JobManager::instance()->add (a);
 	dcpomatic_sleep (1);