projects
/
dcpomatic.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Protect the public API of Player with a mutex, since
[dcpomatic.git]
/
test
/
client_server_test.cc
diff --git
a/test/client_server_test.cc
b/test/client_server_test.cc
index e21f41c7998163ec7c5290e1b6aa0b3ba16797aa..592a7e294fe2c55caa10804515d17a6de46f81b8 100644
(file)
--- a/
test/client_server_test.cc
+++ b/
test/client_server_test.cc
@@
-20,6
+20,7
@@
/** @file test/client_server_test.cc
* @brief Test the server class.
/** @file test/client_server_test.cc
* @brief Test the server class.
+ * @ingroup specific
*
* Create a test image and then encode it using the standard mechanism
* and also using a EncodeServer object running on localhost. Compare the resulting
*
* Create a test image and then encode it using the standard mechanism
* and also using a EncodeServer object running on localhost. Compare the resulting
@@
-42,15
+43,16
@@
using std::list;
using boost::shared_ptr;
using boost::thread;
using boost::optional;
using boost::shared_ptr;
using boost::thread;
using boost::optional;
+using boost::weak_ptr;
using dcp::Data;
void
do_remote_encode (shared_ptr<DCPVideo> frame, EncodeServerDescription description, Data locally_encoded)
{
Data remotely_encoded;
using dcp::Data;
void
do_remote_encode (shared_ptr<DCPVideo> frame, EncodeServerDescription description, Data locally_encoded)
{
Data remotely_encoded;
- BOOST_
CHECK
_NO_THROW (remotely_encoded = frame->encode_remotely (description, 60));
+ BOOST_
REQUIRE
_NO_THROW (remotely_encoded = frame->encode_remotely (description, 60));
- BOOST_
CHECK
_EQUAL (locally_encoded.size(), remotely_encoded.size());
+ BOOST_
REQUIRE
_EQUAL (locally_encoded.size(), remotely_encoded.size());
BOOST_CHECK_EQUAL (memcmp (locally_encoded.data().get(), remotely_encoded.data().get(), locally_encoded.size()), 0);
}
BOOST_CHECK_EQUAL (memcmp (locally_encoded.data().get(), remotely_encoded.data().get(), locally_encoded.size()), 0);
}
@@
-69,7
+71,7
@@
BOOST_AUTO_TEST_CASE (client_server_test_rgb)
p += image->stride()[0];
}
p += image->stride()[0];
}
- shared_ptr<Image> sub_image (new Image (AV_PIX_FMT_
RGB
A, dcp::Size (100, 200), true));
+ shared_ptr<Image> sub_image (new Image (AV_PIX_FMT_
BGR
A, dcp::Size (100, 200), true));
p = sub_image->data()[0];
for (int y = 0; y < 200; ++y) {
uint8_t* q = p;
p = sub_image->data()[0];
for (int y = 0; y < 200; ++y) {
uint8_t* q = p;
@@
-93,11
+95,13
@@
BOOST_AUTO_TEST_CASE (client_server_test_rgb)
dcp::Size (1998, 1080),
EYES_BOTH,
PART_WHOLE,
dcp::Size (1998, 1080),
EYES_BOTH,
PART_WHOLE,
- ColourConversion ()
+ ColourConversion(),
+ weak_ptr<Content>(),
+ optional<Frame>()
)
);
)
);
- pvf->set_
subtitle
(PositionImage (sub_image, Position<int> (50, 60)));
+ pvf->set_
text
(PositionImage (sub_image, Position<int> (50, 60)));
shared_ptr<DCPVideo> frame (
new DCPVideo (
shared_ptr<DCPVideo> frame (
new DCPVideo (
@@
-114,12
+118,13
@@
BOOST_AUTO_TEST_CASE (client_server_test_rgb)
EncodeServer* server = new EncodeServer (log, true, 2);
EncodeServer* server = new EncodeServer (log, true, 2);
- new thread (boost::bind (&EncodeServer::run, server));
+
thread* server_thread =
new thread (boost::bind (&EncodeServer::run, server));
/* Let the server get itself ready */
dcpomatic_sleep (1);
/* Let the server get itself ready */
dcpomatic_sleep (1);
- EncodeServerDescription description ("localhost", 2);
+ /* "localhost" rather than "127.0.0.1" here fails on docker; go figure */
+ EncodeServerDescription description ("127.0.0.1", 1, SERVER_LINK_VERSION);
list<thread*> threads;
for (int i = 0; i < 8; ++i) {
list<thread*> threads;
for (int i = 0; i < 8; ++i) {
@@
-134,6
+139,9
@@
BOOST_AUTO_TEST_CASE (client_server_test_rgb)
delete *i;
}
delete *i;
}
+ server->stop ();
+ server_thread->join ();
+ delete server_thread;
delete server;
}
delete server;
}
@@
-148,7
+156,7
@@
BOOST_AUTO_TEST_CASE (client_server_test_yuv)
}
}
}
}
- shared_ptr<Image> sub_image (new Image (AV_PIX_FMT_
RGB
A, dcp::Size (100, 200), true));
+ shared_ptr<Image> sub_image (new Image (AV_PIX_FMT_
BGR
A, dcp::Size (100, 200), true));
uint8_t* p = sub_image->data()[0];
for (int y = 0; y < 200; ++y) {
uint8_t* q = p;
uint8_t* p = sub_image->data()[0];
for (int y = 0; y < 200; ++y) {
uint8_t* q = p;
@@
-172,11
+180,13
@@
BOOST_AUTO_TEST_CASE (client_server_test_yuv)
dcp::Size (1998, 1080),
EYES_BOTH,
PART_WHOLE,
dcp::Size (1998, 1080),
EYES_BOTH,
PART_WHOLE,
- ColourConversion ()
+ ColourConversion(),
+ weak_ptr<Content>(),
+ optional<Frame>()
)
);
)
);
- pvf->set_
subtitle
(PositionImage (sub_image, Position<int> (50, 60)));
+ pvf->set_
text
(PositionImage (sub_image, Position<int> (50, 60)));
shared_ptr<DCPVideo> frame (
new DCPVideo (
shared_ptr<DCPVideo> frame (
new DCPVideo (
@@
-193,12
+203,13
@@
BOOST_AUTO_TEST_CASE (client_server_test_yuv)
EncodeServer* server = new EncodeServer (log, true, 2);
EncodeServer* server = new EncodeServer (log, true, 2);
- new thread (boost::bind (&EncodeServer::run, server));
+
thread* server_thread =
new thread (boost::bind (&EncodeServer::run, server));
/* Let the server get itself ready */
dcpomatic_sleep (1);
/* Let the server get itself ready */
dcpomatic_sleep (1);
- EncodeServerDescription description ("localhost", 2);
+ /* "localhost" rather than "127.0.0.1" here fails on docker; go figure */
+ EncodeServerDescription description ("127.0.0.1", 2, SERVER_LINK_VERSION);
list<thread*> threads;
for (int i = 0; i < 8; ++i) {
list<thread*> threads;
for (int i = 0; i < 8; ++i) {
@@
-213,6
+224,9
@@
BOOST_AUTO_TEST_CASE (client_server_test_yuv)
delete *i;
}
delete *i;
}
+ server->stop ();
+ server_thread->join ();
+ delete server_thread;
delete server;
}
delete server;
}
@@
-238,7
+252,9
@@
BOOST_AUTO_TEST_CASE (client_server_test_j2k)
dcp::Size (1998, 1080),
EYES_BOTH,
PART_WHOLE,
dcp::Size (1998, 1080),
EYES_BOTH,
PART_WHOLE,
- ColourConversion ()
+ ColourConversion(),
+ weak_ptr<Content>(),
+ optional<Frame>()
)
);
)
);
@@
-264,7
+280,9
@@
BOOST_AUTO_TEST_CASE (client_server_test_j2k)
dcp::Size (1998, 1080),
EYES_BOTH,
PART_WHOLE,
dcp::Size (1998, 1080),
EYES_BOTH,
PART_WHOLE,
- PresetColourConversion::all().front().conversion
+ PresetColourConversion::all().front().conversion,
+ weak_ptr<Content>(),
+ optional<Frame>()
)
);
)
);
@@
-283,12
+301,13
@@
BOOST_AUTO_TEST_CASE (client_server_test_j2k)
EncodeServer* server = new EncodeServer (log, true, 2);
EncodeServer* server = new EncodeServer (log, true, 2);
- new thread (boost::bind (&EncodeServer::run, server));
+
thread* server_thread =
new thread (boost::bind (&EncodeServer::run, server));
/* Let the server get itself ready */
dcpomatic_sleep (1);
/* Let the server get itself ready */
dcpomatic_sleep (1);
- EncodeServerDescription description ("localhost", 2);
+ /* "localhost" rather than "127.0.0.1" here fails on docker; go figure */
+ EncodeServerDescription description ("127.0.0.1", 2, SERVER_LINK_VERSION);
list<thread*> threads;
for (int i = 0; i < 8; ++i) {
list<thread*> threads;
for (int i = 0; i < 8; ++i) {
@@
-303,5
+322,8
@@
BOOST_AUTO_TEST_CASE (client_server_test_j2k)
delete *i;
}
delete *i;
}
+ server->stop ();
+ server_thread->join ();
+ delete server_thread;
delete server;
}
delete server;
}