Author: sisbell
Date: Fri Dec 21 13:12:39 2007
New Revision: 606318

URL: http://svn.apache.org/viewvc?rev=606318&view=rev
Log:
Added back in another unit test. It is now platform aware.

Modified:
    
incubator/nmaven/trunk/components/maven-dotnet-compiler/src/main/java/org/apache/maven/dotnet/compiler/impl/DotnetCompilerContextImpl.java
    
incubator/nmaven/trunk/components/maven-dotnet-compiler/src/test/java/org/apache/maven/dotnet/compiler/impl/DotnetCompilerContextImplTest.java

Modified: 
incubator/nmaven/trunk/components/maven-dotnet-compiler/src/main/java/org/apache/maven/dotnet/compiler/impl/DotnetCompilerContextImpl.java
URL: 
http://svn.apache.org/viewvc/incubator/nmaven/trunk/components/maven-dotnet-compiler/src/main/java/org/apache/maven/dotnet/compiler/impl/DotnetCompilerContextImpl.java?rev=606318&r1=606317&r2=606318&view=diff
==============================================================================
--- 
incubator/nmaven/trunk/components/maven-dotnet-compiler/src/main/java/org/apache/maven/dotnet/compiler/impl/DotnetCompilerContextImpl.java
 (original)
+++ 
incubator/nmaven/trunk/components/maven-dotnet-compiler/src/main/java/org/apache/maven/dotnet/compiler/impl/DotnetCompilerContextImpl.java
 Fri Dec 21 13:12:39 2007
@@ -345,7 +345,7 @@
             for ( String token : tokens )
             {
                 File gacRoot = new File( new File( token ).getParentFile(), 
"lib/mono/gac/" );
-                if ( ( !assemblyExistsCheck && !gacRoot.exists() ) || 
gacRoot.exists() )
+                if ( !assemblyExistsCheck || gacRoot.exists() )
                 {
                     return gacRoot.getAbsolutePath();
                 }

Modified: 
incubator/nmaven/trunk/components/maven-dotnet-compiler/src/test/java/org/apache/maven/dotnet/compiler/impl/DotnetCompilerContextImplTest.java
URL: 
http://svn.apache.org/viewvc/incubator/nmaven/trunk/components/maven-dotnet-compiler/src/test/java/org/apache/maven/dotnet/compiler/impl/DotnetCompilerContextImplTest.java?rev=606318&r1=606317&r2=606318&view=diff
==============================================================================
--- 
incubator/nmaven/trunk/components/maven-dotnet-compiler/src/test/java/org/apache/maven/dotnet/compiler/impl/DotnetCompilerContextImplTest.java
 (original)
+++ 
incubator/nmaven/trunk/components/maven-dotnet-compiler/src/test/java/org/apache/maven/dotnet/compiler/impl/DotnetCompilerContextImplTest.java
 Fri Dec 21 13:12:39 2007
@@ -165,11 +165,13 @@
                     } ) );
     }
 
-   // @Test
+    @Test
     public void testInit_WithGacGenericAndNovellAndPath()
         throws InitializationException, IOException
     {
-        System.setProperty( "PATH", "/Mono/bin" );
+        String monoRoot =  Vendor.getDefaultVendorForOS().equals( 
Vendor.NOVELL ) ? "/Mono" : "C:\\Program Files\\Mono";
+
+        System.setProperty( "PATH", monoRoot + File.separator + "bin");
         DotnetCompilerContextImpl ctx = new DotnetCompilerContextImpl();
         MavenProject project = new MavenProject();
 
@@ -194,14 +196,14 @@
         Set<Artifact> libraries = ctx.getLibraryDependencies();
         assertThat( libraries.size(), equalTo( 1 ) );
         assertThat( ( (Artifact) libraries.toArray()[0] 
).getFile().getAbsolutePath(), equalTo(
-            "C:\\Program 
Files\\Mono\\lib\\mono\\gac\\artifactId\\1.0__dsfajkdsfajdfs\\artifactId.dll" ) 
);
+            monoRoot + 
replaceFileSeparator("\\lib\\mono\\gac\\artifactId\\1.0__dsfajkdsfajdfs\\artifactId.dll"
 )) );
     }
 
-  //  @Test
+    //  @Test
     public void testInit_WithGacGenericAndNovellAndMonoRoot()
         throws InitializationException, IOException
     {
-        System.setProperty( "MONO_ROOT", "C:\\Program Files\\Mono\\bin" );
+        System.setProperty( "MONO_ROOT", "/Program Files/Mono/bin" );
         DotnetCompilerContextImpl ctx = new DotnetCompilerContextImpl();
         MavenProject project = new MavenProject();
 
@@ -262,5 +264,10 @@
         config.setVendor( Vendor.MICROSOFT );
 
         return config;
+    }
+
+        private static String replaceFileSeparator(String value)
+    {
+        return value.replaceAll( "[/\\\\]+", "\\" + File.separator);
     }
 }


Reply via email to