summaryrefslogtreecommitdiff
path: root/tests/oEmbedTest.php
diff options
context:
space:
mode:
authorBrion Vibber <brion@pobox.com>2010-11-15 12:58:00 -0800
committerBrion Vibber <brion@pobox.com>2010-11-15 12:58:00 -0800
commit0735ca86d27b774481f294c5500fd4f1d00db04c (patch)
tree9153fac800e3635679835100ecef72ecd3c8723b /tests/oEmbedTest.php
parent89d59936748ee55bff31f161e6a0f2a1cbd44635 (diff)
Add some data integrity checks on oembed tests (shows a bug on identi.ca test case -- missing width/height in photo data)
Diffstat (limited to 'tests/oEmbedTest.php')
-rw-r--r--tests/oEmbedTest.php12
1 files changed, 12 insertions, 0 deletions
diff --git a/tests/oEmbedTest.php b/tests/oEmbedTest.php
index d00963003..b5e441c42 100644
--- a/tests/oEmbedTest.php
+++ b/tests/oEmbedTest.php
@@ -69,6 +69,18 @@ class oEmbedTest extends PHPUnit_Framework_TestCase
try {
$data = oEmbedHelper::getObject($url);
$this->assertEquals($expectedType, $data->type);
+ if ($data->type == 'photo') {
+ $this->assertTrue(!empty($data->url), 'Photo must have a URL.');
+ $this->assertTrue(!empty($data->width), 'Photo must have a width.');
+ $this->assertTrue(!empty($data->height), 'Photo must have a height.');
+ } else if ($data->type == 'video') {
+ $this->assertTrue(!empty($data->html), 'Video must have embedding HTML.');
+ $this->assertTrue(!empty($data->thumbnail_url), 'Video should have a thumbnail.');
+ }
+ if (!empty($data->thumbnail_url)) {
+ $this->assertTrue(!empty($data->thumbnail_width), 'Thumbnail must list a width.');
+ $this->assertTrue(!empty($data->thumbnail_height), 'Thumbnail must list a height.');
+ }
} catch (Exception $e) {
if ($expectedType == 'none') {
$this->assertEquals($expectedType, 'none', 'Should not have data for this URL.');