added some tests

build.xml

@@ -1,16 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
 
 <project name="friendica" default="test">
-	<!-- set up include directories, this is necessary for the tests to work -->
-	<php>
-		set_include_path(
-					get_include_path() . PATH_SEPARATOR
-					. 'include' . PATH_SEPARATOR 
-					. 'library' . PATH_SEPARATOR 
-					. 'library/phpsec' . PATH_SEPARATOR 
-					. '.' );
-	</php>
-	
 	<!-- ====================================================== -->
 	<!-- Target: clean-test -->
 	<!-- deletes directories with old test reports -->
@@ -32,16 +22,17 @@
 	<!-- this target runs all test files -->
 	<!-- =================================== -->
 	<target name="test" depends="prepare-test">
-		<coverage-setup database="./report/coverage-database">
+		<!-- coverage-setup database="./report/coverage-database">
 			<fileset dir=".">
 				<include name="**/*.php" />
 				<exclude name="*test.php"/>
-				<exclude name="./index.php"/>
+				<exclude name="index.php"/>
-				<exclude name="./library/**"/>
+				<exclude name="library/**"/>
 				<exclude name="doc/**"/>
+				<exclude name=".."/>
 			</fileset>
-		</coverage-setup>
+		</coverage-setup -->
-		<phpunit printsummary="true" >
+		<phpunit printsummary="true">
 			<batchtest>
 				<fileset dir="tests">
 					<include name="*test.php" />
@@ -50,9 +41,9 @@
 			<formatter type="xml" todir="report" outfile="testlog.xml" />
 		</phpunit>
 		<phpunitreport infile="report/testlog.xml" todir="report" />
-		<coverage-report outfile="report/coverage-database">
+		<!-- coverage-report outfile="report/coverage-database">
 		      <report todir="report" styledir="/home/phing/etc" />
-		    </coverage-report>
+		    </coverage-report -->
 	</target>
 
 	<!-- ===================================================== -->
@@ -75,6 +66,9 @@
 		<docblox title="Friendica API" destdir="./doc/api">
 			<fileset dir=".">
 				<include name="**/*.php" />
+				<include name="README"/>
+				<include name="INSTALL.txt"/>
+				<include name="LICENSE"/>
 			</fileset>
 		</docblox>
 	</target>

tests/xss_filter_test.php

@@ -1,16 +1,26 @@
 <?php
 /**
-* Tests, without pHPUnit by now
+ * Tests, without pHPUnit by now
-* @package test.util
+ * @package test.util
-*/
+ */
 
-require_once('include/text.php'); 
+require_once("include/template_processor.php");
+require_once('include/text.php');
 
 class AntiXSSTest extends PHPUnit_Framework_TestCase {
 
-/**
+	public function setUp() {
-* test no tags
+		set_include_path(
-*/
+				get_include_path() . PATH_SEPARATOR
+				. 'include' . PATH_SEPARATOR
+				. 'library' . PATH_SEPARATOR
+				. 'library/phpsec' . PATH_SEPARATOR
+				. '.' );
+	}
+
+	/**
+	 * test no tags
+	 */
 	public function testEscapeTags() {
 		$invalidstring='<submit type="button" onclick="alert(\'failed!\');" />';
 
@@ -53,12 +63,12 @@ class AntiXSSTest extends PHPUnit_Framework_TestCase {
 		$autoname1=autoname(-23);
 		$this->assertEquals(0, count($autoname1));
 	}
-	
+
-// 	public function testAutonameMaxLength() {
+	// 	public function testAutonameMaxLength() {
-// 		$autoname2=autoname(PHP_INT_MAX);
+	// 		$autoname2=autoname(PHP_INT_MAX);
-// 		$this->assertEquals(PHP_INT_MAX, count($autoname2));
+	// 		$this->assertEquals(PHP_INT_MAX, count($autoname2));
-// 	}
+	// 	}
-	
+
 	public function testAutonameLength1() {
 		$autoname3=autoname(1);
 		$this->assertEquals(1, count($autoname3));
@@ -68,7 +78,7 @@ class AntiXSSTest extends PHPUnit_Framework_TestCase {
 	 *xmlify and unxmlify
 	 */
 	public function testXmlify() {
-		$text="<tag>I want to break\n this!11!<?hard?></tag>"; 
+		$text="<tag>I want to break\n this!11!<?hard?></tag>";
 		$xml=xmlify($text); //test whether it actually may be part of a xml document
 		$retext=unxmlify($text);
 
@@ -85,7 +95,7 @@ class AntiXSSTest extends PHPUnit_Framework_TestCase {
 		$this->assertEquals(12, hex2bin(bin2hex(12)));
 		$this->assertEquals(PHP_INT_MAX, hex2bin(bin2hex(PHP_INT_MAX)));
 	}
-	
+
 	/**
 	 * test expand_acl
 	 */
@@ -93,7 +103,7 @@ class AntiXSSTest extends PHPUnit_Framework_TestCase {
 		$text="<1><2><3>";
 		$this->assertEquals(array(1, 2, 3), expand_acl($text));
 	}
-	
+
 	public function testExpandAclBigNumber() {
 		$text="<1><279012><15>";
 		$this->assertEquals(array(1, 279012, 15), expand_acl($text));
@@ -133,19 +143,19 @@ class AntiXSSTest extends PHPUnit_Framework_TestCase {
 		$text="Another> invalid> string>"; //should be invalid
 		$this->assertEquals(array(), expand_acl($text));
 	}
-	
+
 	public function testExpandAclOpenOnly() {
 		$text="<Another< invalid string<"; //should be invalid
 		$this->assertEquals(array(), expand_acl($text));
 	}
-	
+
 	public function testExpandAclNoMatching1() {
 		$text="<Another<> invalid <string>"; //should be invalid
 		$this->assertEquals(array(), expand_acl($text));
 	}
-	
+
 	public function testExpandAclNoMatching2() {
-		$text="<1>2><3>"; 
+		$text="<1>2><3>";
 		$this->assertEquals(array(), expand_acl($text));
 	}
 
@@ -166,7 +176,7 @@ class AntiXSSTest extends PHPUnit_Framework_TestCase {
 		$this->assertTrue(attribute_contains($testAttr, "class3"));
 		$this->assertFalse(attribute_contains($testAttr, "class2"));
 	}
-	
+
 	public function testAttributeContainsEmpty() {
 		$testAttr="";
 		$this->assertFalse(attribute_contains($testAttr, "class2"));
@@ -176,17 +186,71 @@ class AntiXSSTest extends PHPUnit_Framework_TestCase {
 		$testAttr="--... %\$ä() /(=?}";
 		$this->assertFalse(attribute_contains($testAttr, "class2"));
 	}
-	
+
 	/**
 	 * test get_tags
 	 */
+	public function testGetTagsShortPerson() {
+		$text="hi @Mike"; 
+		
+		$tags=get_tags($text);
+		
+		$this->assertEquals("@Mike", $tags[0]);
+	}
+	
+	public function testGetTagsShortTag() {
+		$text="This is a #test_case";
+	
+		$tags=get_tags($text);
+	
+		$this->assertEquals("#test_case", $tags[0]);
+	}
+	
+	public function testGetTagsShortTagAndPerson() {
+		$text="hi @Mike This is a #test_case";
+	
+		$tags=get_tags($text);
+	
+		$this->assertEquals("@Mike", $tags[0]);
+		$this->assertEquals("#test_case", $tags[1]);
+	}
+	
+	public function testGetTagsShortTagAndPersonSpecialChars() {
+		$text="hi @Mike, This is a #test_case.";
+	
+		$tags=get_tags($text);
+	
+		$this->assertEquals("@Mike", $tags[0]);
+		$this->assertEquals("#test_case", $tags[1]);
+	}
+	
+	public function testGetTagsPersonOnly() {
+		$text="@Mike I saw the Theme Dev group was created."; 
+		
+		$tags=get_tags($text);
+		
+		$this->assertEquals("@Mike", $tags[0]);
+	}
+	
+	public function testGetTags2Persons1TagSpecialChars() {
+		$text="hi @Mike, I'm just writing #test_cases, so"
+		." so @somebody@friendica.com may change #things.";
+	
+		$tags=get_tags($text);
+	
+		$this->assertEquals("@Mike", $tags[0]);
+		$this->assertEquals("#test_cases", $tags[1]);
+		$this->assertEquals("@somebody@friendica.com", $tags[2]);
+		$this->assertEquals("#things", $tags[3]);
+	}
+	
 	public function testGetTags() {
 		$text="hi @Mike, I'm just writing #test_cases, "
 		." so @somebody@friendica.com may change #things. Of course I "
 		."look for a lot of #pitfalls, like #tags at the end of a sentence "
 		."@comment. I hope noone forgets about @fullstops.because that might"
 		." break #things. @Mike@campino@friendica.eu is also #nice, isn't it? "
-		."Now, add a @first_last tag. "; 
+		."Now, add a @first_last tag. ";
 		//check whether this are all variants (no, auto-stuff is missing).
 
 		$tags=get_tags($text);
@@ -210,8 +274,8 @@ class AntiXSSTest extends PHPUnit_Framework_TestCase {
 		$tags=get_tags("");
 		$this->assertEquals(0, count($tags));
 	}
-//function qp, quick and dirty??
+	//function qp, quick and dirty??
-//get_mentions
+	//get_mentions
-//get_contact_block, bis Zeile 538
+	//get_contact_block, bis Zeile 538
 }
 ?>

util/db_update.php

@@ -26,7 +26,7 @@ echo "New DB VERSION: " . DB_UPDATE_VERSION . "\n";
 
 if($build != DB_UPDATE_VERSION) {
 	echo "Updating database...";
-	check_config();
+	check_config($a);
 	echo "Done\n";
 }