Test: make sure TranslogTests.randomNonTranslogPatternString returns a reference to a file

It can return a directory now (like /  or ..)
This commit is contained in:
Boaz Leskes 2015-09-29 07:53:31 +02:00
parent 148265bd16
commit c1ab2dc244
1 changed files with 6 additions and 22 deletions

View File

@ -29,7 +29,6 @@ import org.apache.lucene.util.LuceneTestCase;
import org.apache.lucene.util.TestUtil; import org.apache.lucene.util.TestUtil;
import org.elasticsearch.ElasticsearchException; import org.elasticsearch.ElasticsearchException;
import org.elasticsearch.Version; import org.elasticsearch.Version;
import org.elasticsearch.bwcompat.OldIndexBackwardsCompatibilityIT;
import org.elasticsearch.common.bytes.BytesArray; import org.elasticsearch.common.bytes.BytesArray;
import org.elasticsearch.common.io.FileSystemUtils; import org.elasticsearch.common.io.FileSystemUtils;
import org.elasticsearch.common.io.stream.BytesStreamOutput; import org.elasticsearch.common.io.stream.BytesStreamOutput;
@ -53,25 +52,9 @@ import java.io.InputStream;
import java.nio.ByteBuffer; import java.nio.ByteBuffer;
import java.nio.channels.FileChannel; import java.nio.channels.FileChannel;
import java.nio.charset.Charset; import java.nio.charset.Charset;
import java.nio.file.DirectoryStream; import java.nio.file.*;
import java.nio.file.Files; import java.util.*;
import java.nio.file.InvalidPathException; import java.util.concurrent.*;
import java.nio.file.Path;
import java.nio.file.StandardOpenOption;
import java.util.ArrayList;
import java.util.Collections;
import java.util.HashSet;
import java.util.List;
import java.util.Locale;
import java.util.Map;
import java.util.Set;
import java.util.concurrent.ArrayBlockingQueue;
import java.util.concurrent.BlockingQueue;
import java.util.concurrent.BrokenBarrierException;
import java.util.concurrent.CopyOnWriteArrayList;
import java.util.concurrent.CountDownLatch;
import java.util.concurrent.CyclicBarrier;
import java.util.concurrent.TimeUnit;
import java.util.concurrent.atomic.AtomicBoolean; import java.util.concurrent.atomic.AtomicBoolean;
import java.util.concurrent.atomic.AtomicInteger; import java.util.concurrent.atomic.AtomicInteger;
import java.util.concurrent.atomic.AtomicLong; import java.util.concurrent.atomic.AtomicLong;
@ -188,8 +171,9 @@ public class TranslogTests extends ESTestCase {
validPathString = false; validPathString = false;
string = randomRealisticUnicodeOfCodepointLength(randomIntBetween(min, max)); string = randomRealisticUnicodeOfCodepointLength(randomIntBetween(min, max));
try { try {
translogDir.resolve(string); final Path resolved = translogDir.resolve(string);
validPathString = true; // some strings (like '/' , '..') do not refer to a file, which we this method should return
validPathString = resolved.getFileName() != null;
} catch (InvalidPathException ex) { } catch (InvalidPathException ex) {
// some FS don't like our random file names -- let's just skip these random choices // some FS don't like our random file names -- let's just skip these random choices
} }