-
Notifications
You must be signed in to change notification settings - Fork 962
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove stale BWC tests #12874
Remove stale BWC tests #12874
Conversation
Both of these tests have been disabled for quiet a long time. While `TestManyPointsInOldIndex` looks indeed stale, `TestIndexWriterOnOldIndex` is not a more general test.
} | ||
|
||
public void testCheckOldIndex() throws IOException { | ||
assumeTrue("Reenable when 7.0 is released", false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doh! So I guess this test never got enabled, and now the index it is testing (which is apparently nowhere to be found in our resources) is too ancient for modern Lucene to read.
public class TestIndexWriterOnOldIndex extends LuceneTestCase { | ||
|
||
public void testOpenModeAndCreatedVersion() throws IOException { | ||
assumeTrue("Reenable when 8.0 is released", false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sheesh ... this makes me wonder what other assumeTrue
are lurking that we failed to re-enable on XYZ release!
This PR has not had activity in the past 2 weeks, labeling it as stale. If the PR is waiting for review, notify the dev@lucene.apache.org list. Thank you for your contribution! |
Both of these tests have been disabled for quiet a long time. While `TestManyPointsInOldIndex` looks indeed stale, `TestIndexWriterOnOldIndex` is not a more general test.
Both of these tests have been disabled for quiet a long time. While `TestManyPointsInOldIndex` looks indeed stale, `TestIndexWriterOnOldIndex` is not a more general test.
Both of these tests have been disabled for quiet a long time. While
TestManyPointsInOldIndex
looks indeed stale,TestIndexWriterOnOldIndex
is not a more general test.