[asterisk-commits] mjordan: trunk r397631 - in /trunk: ./ tests/test_bucket.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Sun Aug 25 13:12:44 CDT 2013


Author: mjordan
Date: Sun Aug 25 13:12:42 2013
New Revision: 397631

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=397631
Log:
Fix bucket unit tests

After the review for buckets was completed (r2715), the handling of names in
the bucket core was deferred to the wizards. As such, the bucket unit tests
cannot expect that passing a URI with a scheme specified but no actual resource
name will automatically fail. The tests have been updated to not make this
check.
........

Merged revisions 397630 from http://svn.asterisk.org/svn/asterisk/branches/12

Modified:
    trunk/   (props changed)
    trunk/tests/test_bucket.c

Propchange: trunk/
------------------------------------------------------------------------------
--- branch-12-merged (original)
+++ branch-12-merged Sun Aug 25 13:12:42 2013
@@ -1,1 +1,1 @@
-/branches/12:1-397614,397621,397628
+/branches/12:1-397614,397621,397628,397630

Modified: trunk/tests/test_bucket.c
URL: http://svnview.digium.com/svn/asterisk/trunk/tests/test_bucket.c?view=diff&rev=397631&r1=397630&r2=397631
==============================================================================
--- trunk/tests/test_bucket.c (original)
+++ trunk/tests/test_bucket.c Sun Aug 25 13:12:42 2013
@@ -170,11 +170,6 @@
 		return AST_TEST_FAIL;
 	}
 
-	if ((bucket = ast_bucket_alloc("test://"))) {
-		ast_test_status_update(test, "Allocated a bucket with no name\n");
-		return AST_TEST_FAIL;
-	}
-
 	if (!(bucket = ast_bucket_alloc("test:///tmp/bob"))) {
 		ast_test_status_update(test, "Failed to allocate bucket\n");
 		return AST_TEST_FAIL;
@@ -374,11 +369,6 @@
 
 	if ((file = ast_bucket_file_alloc(""))) {
 		ast_test_status_update(test, "Allocated a file with no URI provided\n");
-		return AST_TEST_FAIL;
-	}
-
-	if ((file = ast_bucket_file_alloc("test://"))) {
-		ast_test_status_update(test, "Allocated a file with no name\n");
 		return AST_TEST_FAIL;
 	}
 




More information about the asterisk-commits mailing list