You are viewing a plain text version of this content. The canonical link for it is here.
Posted to doxia-commits@maven.apache.org by lt...@apache.org on 2011/01/15 18:42:55 UTC
svn commit: r1059377 - in
/maven/doxia/doxia-sitetools/trunk/doxia-decoration-model/src:
main/java/org/apache/maven/doxia/site/decoration/inheritance/
test/java/org/apache/maven/doxia/site/decoration/inheritance/
Author: ltheussl
Date: Sat Jan 15 17:42:55 2011
New Revision: 1059377
URL: http://svn.apache.org/viewvc?rev=1059377&view=rev
Log:
more tests
Modified:
maven/doxia/doxia-sitetools/trunk/doxia-decoration-model/src/main/java/org/apache/maven/doxia/site/decoration/inheritance/URIPathDescriptor.java
maven/doxia/doxia-sitetools/trunk/doxia-decoration-model/src/test/java/org/apache/maven/doxia/site/decoration/inheritance/URIPathDescriptorTest.java
Modified: maven/doxia/doxia-sitetools/trunk/doxia-decoration-model/src/main/java/org/apache/maven/doxia/site/decoration/inheritance/URIPathDescriptor.java
URL: http://svn.apache.org/viewvc/maven/doxia/doxia-sitetools/trunk/doxia-decoration-model/src/main/java/org/apache/maven/doxia/site/decoration/inheritance/URIPathDescriptor.java?rev=1059377&r1=1059376&r2=1059377&view=diff
==============================================================================
--- maven/doxia/doxia-sitetools/trunk/doxia-decoration-model/src/main/java/org/apache/maven/doxia/site/decoration/inheritance/URIPathDescriptor.java (original)
+++ maven/doxia/doxia-sitetools/trunk/doxia-decoration-model/src/main/java/org/apache/maven/doxia/site/decoration/inheritance/URIPathDescriptor.java Sat Jan 15 17:42:55 2011
@@ -208,8 +208,7 @@ public class URIPathDescriptor
}
private static boolean sameSite( final URI baseURI, final URI newBaseURI )
- {
- final boolean equalScheme = ( baseURI.getScheme() == null ? newBaseURI.getScheme() == null
+ {final boolean equalScheme = ( newBaseURI.getScheme() == null ? false
: baseURI.getScheme().equalsIgnoreCase( newBaseURI.getScheme() ) );
final boolean equalHost = ( baseURI.getHost() == null ? newBaseURI.getHost() == null
: baseURI.getHost().equalsIgnoreCase( newBaseURI.getHost() ) );
Modified: maven/doxia/doxia-sitetools/trunk/doxia-decoration-model/src/test/java/org/apache/maven/doxia/site/decoration/inheritance/URIPathDescriptorTest.java
URL: http://svn.apache.org/viewvc/maven/doxia/doxia-sitetools/trunk/doxia-decoration-model/src/test/java/org/apache/maven/doxia/site/decoration/inheritance/URIPathDescriptorTest.java?rev=1059377&r1=1059376&r2=1059377&view=diff
==============================================================================
--- maven/doxia/doxia-sitetools/trunk/doxia-decoration-model/src/test/java/org/apache/maven/doxia/site/decoration/inheritance/URIPathDescriptorTest.java (original)
+++ maven/doxia/doxia-sitetools/trunk/doxia-decoration-model/src/test/java/org/apache/maven/doxia/site/decoration/inheritance/URIPathDescriptorTest.java Sat Jan 15 17:42:55 2011
@@ -32,6 +32,8 @@ import junit.framework.TestCase;
public class URIPathDescriptorTest
extends TestCase
{
+ private static final String maven = "http://maven.apache.org/";
+
/**
* Test of constructor, of class URIPathDescriptor.
*
@@ -40,10 +42,12 @@ public class URIPathDescriptorTest
public void testConstructor()
throws Exception
{
- final String expected = "http://maven.apache.org/doxia";
+ final String expected = maven + "doxia";
- final URIPathDescriptor path = new URIPathDescriptor( "http://maven.apache.org/", "doxia" );
+ final URIPathDescriptor path = new URIPathDescriptor( maven, "doxia" );
assertEquals( expected, path.toString() );
+ assertEquals( maven, path.getBaseURI().toString() );
+ assertEquals( "doxia", path.getLink().toString() );
URIPathDescriptor compare = new URIPathDescriptor( "http://maven.apache.org", "/doxia" );
assertEquals( expected, compare.toString() );
@@ -54,15 +58,17 @@ public class URIPathDescriptorTest
compare = new URIPathDescriptor( "http://maven.apache.org/doxia", "" );
assertEquals( expected + "/", compare.toString() );
- try
- {
- compare = new URIPathDescriptor( "/doxia", "http://maven.apache.org" );
- fail();
- }
- catch ( IllegalArgumentException ex )
- {
- assertNotNull( ex );
- }
+ compare = new URIPathDescriptor( "file:///C:\\Foo\\bar1", "" );
+ assertEquals( "file:/C:/Foo/bar1/", compare.toString() );
+
+ compare = new URIPathDescriptor( "file:///C:/Documents%20and%20Settings/foo/", "bar" );
+ assertEquals( "file:/C:/Documents%20and%20Settings/foo/bar", compare.toString() );
+
+ compare = new URIPathDescriptor( "C:\\Foo\\bar", "bar" );
+ assertEquals( "C:/Foo/bar/bar", compare.toString() ); // NOTE: C: is the scheme here!
+
+ assertFailure( "/doxia", maven );
+ assertFailure( "file:///C:/Documents and Settings/foo/", "bar" );
}
/**
@@ -73,17 +79,19 @@ public class URIPathDescriptorTest
public void testResolveLink()
throws Exception
{
- URIPathDescriptor oldPath = new URIPathDescriptor( "http://maven.apache.org/", "source" );
- assertEquals( "http://maven.apache.org/source", oldPath.resolveLink().toString() );
+ final String expected = maven + "source";
+
+ URIPathDescriptor oldPath = new URIPathDescriptor( maven, "source" );
+ assertEquals( expected, oldPath.resolveLink().toString() );
- oldPath = new URIPathDescriptor( "http://maven.apache.org/", "source/" );
- assertEquals( "http://maven.apache.org/source/", oldPath.resolveLink().toString() );
+ oldPath = new URIPathDescriptor( maven, "source/" );
+ assertEquals( expected + "/", oldPath.resolveLink().toString() );
- oldPath = new URIPathDescriptor( "http://maven.apache.org/", "/source" );
- assertEquals( "http://maven.apache.org/source", oldPath.resolveLink().toString() );
+ oldPath = new URIPathDescriptor( maven, "/source" );
+ assertEquals( expected, oldPath.resolveLink().toString() );
oldPath = new URIPathDescriptor( "http://maven.apache.org", "source" );
- assertEquals( "http://maven.apache.org/source", oldPath.resolveLink().toString() );
+ assertEquals( expected, oldPath.resolveLink().toString() );
}
/**
@@ -94,17 +102,19 @@ public class URIPathDescriptorTest
public void testRebaseLink()
throws Exception
{
- URIPathDescriptor oldPath = new URIPathDescriptor( "http://maven.apache.org/", "source" );
+ URIPathDescriptor oldPath = new URIPathDescriptor( maven, "source" );
assertEquals( "../source", oldPath.rebaseLink( "http://maven.apache.org/doxia/" ).toString() );
+ assertEquals( "http://maven.apache.org/source", oldPath.rebaseLink( null ).toString() );
+ assertEquals( "http://maven.apache.org/source", oldPath.rebaseLink( "C:/Documents and Settings/" ).toString() );
oldPath = new URIPathDescriptor( "scp://people.apache.org/", "source" );
assertEquals( "../source", oldPath.rebaseLink( "scp://people.apache.org/doxia" ).toString() );
- oldPath = new URIPathDescriptor( "http://maven.apache.org/", "banner/left" );
+ oldPath = new URIPathDescriptor( maven, "banner/left" );
assertEquals( "../banner/left", oldPath.rebaseLink( "http://maven.apache.org/doxia/" ).toString() );
oldPath = new URIPathDescriptor( "http://jakarta.apache.org/", "banner/left" );
- assertEquals( "http://jakarta.apache.org/banner/left", oldPath.rebaseLink( "http://maven.apache.org/" ).toString() );
+ assertEquals( "http://jakarta.apache.org/banner/left", oldPath.rebaseLink( maven ).toString() );
}
/**
@@ -115,10 +125,10 @@ public class URIPathDescriptorTest
public void testRelativizeLink()
throws Exception
{
- URIPathDescriptor path = new URIPathDescriptor( "http://maven.apache.org/", "source" );
+ URIPathDescriptor path = new URIPathDescriptor( maven, "source" );
assertEquals( "source", path.relativizeLink().toString() );
- path = new URIPathDescriptor( "http://maven.apache.org/", "http://maven.apache.org/source" );
+ path = new URIPathDescriptor( maven, "http://maven.apache.org/source" );
assertEquals( "source", path.relativizeLink().toString() );
path = new URIPathDescriptor( "http://maven.apache.org/doxia/", "http://maven.apache.org/source/" );
@@ -136,7 +146,7 @@ public class URIPathDescriptorTest
public void testSameSite()
throws Exception
{
- final URIPathDescriptor path = new URIPathDescriptor( "http://maven.apache.org/", "doxia" );
+ final URIPathDescriptor path = new URIPathDescriptor( maven, "doxia" );
assertTrue( path.sameSite( new URI( "http://maven.apache.org/" ) ) );
assertTrue( path.sameSite( new URI( "http://maven.apache.org" ) ) );
@@ -148,5 +158,24 @@ public class URIPathDescriptorTest
assertFalse( path.sameSite( new URI( "https://maven.apache.org/" ) ) );
assertFalse( path.sameSite( new URI( "http://ant.apache.org/" ) ) );
assertFalse( path.sameSite( new URI( "http://maven.apache.org:80" ) ) );
+ assertFalse( path.sameSite( new URI( "/usr/share/bin/" ) ) );
+ assertFalse( path.sameSite( new URI( "http:///maven.apache.org/" ) ) );
+
+ final URIPathDescriptor nullHost = new URIPathDescriptor( "http:///maven.apache.org/", "doxia" );
+ assertTrue( nullHost.sameSite( new URI( "http:///maven.apache.org/" ) ) );
+ assertFalse( nullHost.sameSite( new URI( "http://maven.apache.org/" ) ) );
+ }
+
+ private static void assertFailure( final String base, final String link )
+ {
+ try
+ {
+ final URIPathDescriptor test = new URIPathDescriptor( base, link );
+ fail( "Should fail: " + test.toString() );
+ }
+ catch ( IllegalArgumentException ex )
+ {
+ assertNotNull( ex );
+ }
}
}