Tests: Skip Version.CURRENT for static bwc indexes

The current version is normally a snapshot while in development.
However, when the release process changes the snapshot flag to false,
this causes the static bwc tests to fail because they cannot
find an index for the current version.  Instead, this change
skips the current version, because there is no need to test
a verion's bwc against itself.

closes #10292
closes #10293

Conflicts:
	src/test/java/org/elasticsearch/bwcompat/OldIndexBackwardsCompatibilityTests.java

Author: rjernst

src/test/java/org/elasticsearch/bwcompat/OldIndexBackwardsCompatibilityTests.java

@@ -55,9 +55,9 @@
 
 @TimeoutSuite(millis = 40 * TimeUnits.MINUTE)
 public class OldIndexBackwardsCompatibilityTests extends StaticIndexBackwardCompatibilityTest {
-    
+
     static List<String> indexes;
-    
+
     @BeforeClass
     public static void initIndexes() throws Exception {
         indexes = new ArrayList<>();
@@ -70,19 +70,20 @@ public static void initIndexes() throws Exception {
         }
         Collections.sort(indexes);
     }
-    
+
     public void testAllVersionsTested() throws Exception {
         SortedSet<String> expectedVersions = new TreeSet<>();
         for (java.lang.reflect.Field field : Version.class.getDeclaredFields()) {
             if (Modifier.isStatic(field.getModifiers()) && field.getType() == Version.class) {
                 Version v = (Version)field.get(Version.class);
-                if (v.snapshot()) continue;
-                if (v.before(Version.V_0_20_0_RC1)) continue;
+                if (v.snapshot()) continue;  // snapshots are unreleased, so there is no backcompat yet
+                if (v.onOrBefore(Version.V_0_20_0_RC1)) continue; // we can only test back one major lucene version
+                if (v.equals(Version.CURRENT)) continue; // the current version is always compatible with itself
 
                 expectedVersions.add("index-" + v.toString() + ".zip");
             }
         }
-        
+
         for (String index : indexes) {
             if (expectedVersions.remove(index) == false) {
                 logger.warn("Old indexes tests contain extra index: " + index);
@@ -128,11 +129,11 @@ void assertOldIndexWorks(String index) throws Exception {
         assertDeleteByQueryWorked(version);
         unloadIndex();
     }
-    
+
     Version extractVersion(String index) {
         return Version.fromString(index.substring(index.indexOf('-') + 1, index.lastIndexOf('.')));
     }
-    
+
     boolean isLatestLuceneVersion(Version version) {
         return version.luceneVersion.major == Version.CURRENT.luceneVersion.major &&
                version.luceneVersion.minor == Version.CURRENT.luceneVersion.minor;
@@ -144,7 +145,7 @@ void assertBasicSearchWorks() {
         ElasticsearchAssertions.assertNoFailures(searchRsp);
         long numDocs = searchRsp.getHits().getTotalHits();
         logger.info("Found " + numDocs + " in old index");
-        
+
         searchReq.addSort("long_sort", SortOrder.ASC);
         ElasticsearchAssertions.assertNoFailures(searchReq.get());
 
@@ -203,7 +204,7 @@ void assertDeleteByQueryWorked(Version version) throws Exception {
         SearchRequestBuilder searchReq = client().prepareSearch("test").setQuery(QueryBuilders.queryStringQuery("long_sort:[10 TO 20]"));
         assertEquals(0, searchReq.get().getHits().getTotalHits());
     }
-    
+
     void assertUpgradeWorks(boolean alreadyLatest) throws Exception {
         HttpRequestBuilder httpClient = httpClient();
 

src/test/java/org/elasticsearch/bwcompat/RestoreBackwardsCompatTests.java

@@ -72,6 +72,7 @@ public void restoreOldSnapshots() throws Exception {
                 Version v = (Version) field.get(Version.class);
                 if (v.snapshot()) continue;
                 if (v.onOrBefore(Version.V_1_0_0_Beta1)) continue;
+                if (v.equals(Version.CURRENT)) continue;
 
                 expectedVersions.add(v.toString());
             }