Repository: lucenenet
Updated Branches:
  refs/heads/api-work 2ddcc94bf -> a6337320f


Lucene.Net.Tests.Spatial: Removed double-run of SpatialExample.Test() that was 
a hack put into JUnit because SpatialExample doesn't follow the conventions 
that were used to identify it as a test class.


Project: http://git-wip-us.apache.org/repos/asf/lucenenet/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucenenet/commit/a6c6e5d7
Tree: http://git-wip-us.apache.org/repos/asf/lucenenet/tree/a6c6e5d7
Diff: http://git-wip-us.apache.org/repos/asf/lucenenet/diff/a6c6e5d7

Branch: refs/heads/api-work
Commit: a6c6e5d7ed04f9b4f818046dd808ed44efaa8478
Parents: 2ddcc94
Author: Shad Storhaug <[email protected]>
Authored: Wed Mar 1 12:05:47 2017 +0700
Committer: Shad Storhaug <[email protected]>
Committed: Wed Mar 1 12:05:47 2017 +0700

----------------------------------------------------------------------
 src/Lucene.Net.Tests.Spatial/SpatialExample.cs    | 12 +++++++-----
 src/Lucene.Net.Tests.Spatial/TestTestFramework.cs | 18 +++++++++++-------
 2 files changed, 18 insertions(+), 12 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/lucenenet/blob/a6c6e5d7/src/Lucene.Net.Tests.Spatial/SpatialExample.cs
----------------------------------------------------------------------
diff --git a/src/Lucene.Net.Tests.Spatial/SpatialExample.cs 
b/src/Lucene.Net.Tests.Spatial/SpatialExample.cs
index 162c577..49b9321 100644
--- a/src/Lucene.Net.Tests.Spatial/SpatialExample.cs
+++ b/src/Lucene.Net.Tests.Spatial/SpatialExample.cs
@@ -39,11 +39,13 @@ namespace Lucene.Net.Spatial
     /// </summary>
     public class SpatialExample : LuceneTestCase
     {
-        //Note: Test invoked via TestTestFramework.spatialExample()
-        public static void Main(String[] args)
-        {
-            new SpatialExample().Test();
-        }
+        // LUCENENET specific: removed this because the test will run
+        // with only an attribute, it doesn't need to worry about a convention
+        ////Note: Test invoked via TestTestFramework.spatialExample()
+        //public static void Main(String[] args)
+        //{
+        //    new SpatialExample().Test();
+        //}
 
         [Test]
         public virtual void Test()

http://git-wip-us.apache.org/repos/asf/lucenenet/blob/a6c6e5d7/src/Lucene.Net.Tests.Spatial/TestTestFramework.cs
----------------------------------------------------------------------
diff --git a/src/Lucene.Net.Tests.Spatial/TestTestFramework.cs 
b/src/Lucene.Net.Tests.Spatial/TestTestFramework.cs
index 93268d0..6f1bafc 100644
--- a/src/Lucene.Net.Tests.Spatial/TestTestFramework.cs
+++ b/src/Lucene.Net.Tests.Spatial/TestTestFramework.cs
@@ -53,12 +53,16 @@ namespace Lucene.Net.Spatial
             assertEquals(SpatialOperation.Intersects, sf.args.Operation);
         }
 
-        [Test]
-        public virtual void SpatialExample_Mem()
-        {
-            //kind of a hack so that SpatialExample is tested despite
-            // it not starting or ending with "Test".
-            SpatialExample.Main(null);
-        }
+        // LUCENENET specific - we don't nee to worry about the naming 
convention
+        // because the [Test] attribute will cause the test to run regardless. 
+        // So, this is a duplicate test to SpatialExample.Test() and not 
needed.
+
+        //[Test]
+        //public virtual void SpatialExample_Mem()
+        //{
+        //    //kind of a hack so that SpatialExample is tested despite
+        //    // it not starting or ending with "Test".
+        //    SpatialExample.Main(null);
+        //}
     }
 }

Reply via email to