tag tests cont.
This commit is contained in:
parent
f30146b521
commit
cf2d8ef7a6
|
@ -959,7 +959,6 @@ function handle_tag($a, &$body, &$inform, &$str_tags, $profile_uid, $tag) {
|
|||
else
|
||||
$newname = $r[0]['name'];
|
||||
//add person's id to $inform
|
||||
$inform_id = 'cid:' . $r[0]['id'];
|
||||
if(strlen($inform))
|
||||
$inform .= ',';
|
||||
$inform .= 'cid:' . $r[0]['id'];
|
||||
|
|
|
@ -139,9 +139,19 @@ class GetTagsTest extends PHPUnit_Framework_TestCase {
|
|||
$str_tags='';
|
||||
handle_tag($this->a, $text, $inform, $str_tags, 11, $tags[0]);
|
||||
|
||||
$this->assertEquals("cid:15", $inform);
|
||||
$this->assertEquals("@[url=http://justatest.de]Mike Lastname[/url]", $str_tags);
|
||||
$this->assertEquals("hi @[url=http://justatest.de]Mike Lastname[/url].because", $text);
|
||||
// (mike) - This is a tricky case.
|
||||
// we support mentions as in @mike@example.com - which contains a period.
|
||||
// This shouldn't match anything unless you have a contact named "Mike.because".
|
||||
// We may need another test for "@Mike. because" - which should return the contact
|
||||
// as we ignore trailing periods in tags.
|
||||
|
||||
// $this->assertEquals("cid:15", $inform);
|
||||
// $this->assertEquals("@[url=http://justatest.de]Mike Lastname[/url]", $str_tags);
|
||||
// $this->assertEquals("hi @[url=http://justatest.de]Mike Lastname[/url].because", $text);
|
||||
|
||||
$this->assertEquals("", $inform);
|
||||
$this->assertEquals("", $str_tags);
|
||||
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -253,7 +263,8 @@ class GetTagsTest extends PHPUnit_Framework_TestCase {
|
|||
|
||||
$this->assertEquals("Test with @[url=http://justatest.de]Mike Lastname[/url] id tag", $text);
|
||||
$this->assertEquals("@[url=http://justatest.de]Mike Lastname[/url]", $str_tags);
|
||||
$this->assertEquals("cid:15", $inform);
|
||||
// this test may produce two cid:15 entries - which is OK because duplicates are pruned before delivery
|
||||
$this->assertTrue(strstr($inform,"cid:15"));
|
||||
}
|
||||
|
||||
/**
|
||||
|
|
Loading…
Reference in New Issue
Block a user