From 05180574a95d534fe04e6bc8532b53dc84cb5674 Mon Sep 17 00:00:00 2001 From: Robin Rosenberg Date: Fri, 11 Jan 2013 00:19:27 +0100 Subject: [PATCH] Add test case for PathFilterGroup This is needed to prepare for optimization. Bug: 397185 Change-Id: If38507ef8cc5a161353cc07bf7cd1922f54d1f8a --- .../treewalk/filter/PathFilterGroupTest.java | 190 ++++++++++++++++++ 1 file changed, 190 insertions(+) create mode 100644 org.eclipse.jgit.test/tst/org/eclipse/jgit/treewalk/filter/PathFilterGroupTest.java diff --git a/org.eclipse.jgit.test/tst/org/eclipse/jgit/treewalk/filter/PathFilterGroupTest.java b/org.eclipse.jgit.test/tst/org/eclipse/jgit/treewalk/filter/PathFilterGroupTest.java new file mode 100644 index 000000000..5267e81a1 --- /dev/null +++ b/org.eclipse.jgit.test/tst/org/eclipse/jgit/treewalk/filter/PathFilterGroupTest.java @@ -0,0 +1,190 @@ +/* + * Copyright (C) 2013, Robin Rosenberg + * and other copyright owners as documented in the project's IP log. + * + * This program and the accompanying materials are made available + * under the terms of the Eclipse Distribution License v1.0 which + * accompanies this distribution, is reproduced below, and is + * available at http://www.eclipse.org/org/documents/edl-v10.php + * + * All rights reserved. + * + * Redistribution and use in source and binary forms, with or + * without modification, are permitted provided that the following + * conditions are met: + * + * - Redistributions of source code must retain the above copyright + * notice, this list of conditions and the following disclaimer. + * + * - Redistributions in binary form must reproduce the above + * copyright notice, this list of conditions and the following + * disclaimer in the documentation and/or other materials provided + * with the distribution. + * + * - Neither the name of the Eclipse Foundation, Inc. nor the + * names of its contributors may be used to endorse or promote + * products derived from this software without specific prior + * written permission. + * + * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND + * CONTRIBUTORS "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, + * INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES + * OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE + * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR + * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, + * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT + * NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; + * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER + * CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, + * STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) + * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF + * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. + */ + +package org.eclipse.jgit.treewalk.filter; + +import static org.junit.Assert.assertFalse; +import static org.junit.Assert.assertTrue; +import static org.junit.Assert.fail; + +import java.io.IOException; + +import org.eclipse.jgit.dircache.DirCache; +import org.eclipse.jgit.dircache.DirCacheEditor; +import org.eclipse.jgit.dircache.DirCacheEntry; +import org.eclipse.jgit.dircache.DirCacheIterator; +import org.eclipse.jgit.errors.IncorrectObjectTypeException; +import org.eclipse.jgit.errors.MissingObjectException; +import org.eclipse.jgit.errors.StopWalkException; +import org.eclipse.jgit.lib.FileMode; +import org.eclipse.jgit.lib.ObjectReader; +import org.eclipse.jgit.treewalk.TreeWalk; +import org.junit.Before; +import org.junit.Test; + +public class PathFilterGroupTest { + + private TreeFilter filter; + + @Before + public void setup() { + // @formatter:off + String[] paths = new String[] { + "/a", // never match + "/a/b", // never match + "a", + "b/c", + "c/d/e", + "c/d/f", + "d/e/f/g" + }; + // @formatter:on + filter = PathFilterGroup.createFromStrings(paths); + } + + @Test + public void testExact() throws MissingObjectException, + IncorrectObjectTypeException, IOException { + assertTrue(filter.include(fakeWalk("a"))); + assertTrue(filter.include(fakeWalk("b/c"))); + assertTrue(filter.include(fakeWalk("c/d/e"))); + assertTrue(filter.include(fakeWalk("c/d/f"))); + assertTrue(filter.include(fakeWalk("d/e/f/g"))); + } + + @Test + public void testNoMatchButClose() throws MissingObjectException, + IncorrectObjectTypeException, IOException { + assertFalse(filter.include(fakeWalk("a+"))); + assertFalse(filter.include(fakeWalk("b+/c"))); + assertFalse(filter.include(fakeWalk("c+/d/e"))); + assertFalse(filter.include(fakeWalk("c+/d/f"))); + assertFalse(filter.include(fakeWalk("c/d.a"))); + assertFalse(filter.include(fakeWalk("d+/e/f/g"))); + } + + @Test + public void testJustCommonPrefixIsNotMatch() throws MissingObjectException, + IncorrectObjectTypeException, IOException { + assertFalse(filter.include(fakeWalk("b/a"))); + assertFalse(filter.include(fakeWalk("b/d"))); + assertFalse(filter.include(fakeWalk("c/d/a"))); + assertFalse(filter.include(fakeWalk("d/e/e"))); + } + + @Test + public void testKeyIsPrefixOfFilter() throws MissingObjectException, + IncorrectObjectTypeException, IOException { + assertTrue(filter.include(fakeWalk("b"))); + assertTrue(filter.include(fakeWalk("c/d"))); + assertTrue(filter.include(fakeWalk("c/d"))); + assertTrue(filter.include(fakeWalk("c"))); + assertTrue(filter.include(fakeWalk("d/e/f"))); + assertTrue(filter.include(fakeWalk("d/e"))); + assertTrue(filter.include(fakeWalk("d"))); + } + + @Test + public void testFilterIsPrefixOfKey() throws MissingObjectException, + IncorrectObjectTypeException, IOException { + assertTrue(filter.include(fakeWalk("a/b"))); + assertTrue(filter.include(fakeWalk("b/c/d"))); + assertTrue(filter.include(fakeWalk("c/d/e/f"))); + assertTrue(filter.include(fakeWalk("c/d/f/g"))); + assertTrue(filter.include(fakeWalk("d/e/f/g/h"))); + } + + @Test + public void testStopWalk() throws MissingObjectException, + IncorrectObjectTypeException, IOException { + // Obvious + filter.include(fakeWalk("d/e/f/f")); + + // Obvious + try { + filter.include(fakeWalk("de")); + fail("StopWalkException expected"); + } catch (StopWalkException e) { + // good + } + + // less obvious due to git sorting order + filter.include(fakeWalk("d.")); + + // less obvious due to git sorting order + try { + filter.include(fakeWalk("d0")); + fail("StopWalkException expected"); + } catch (StopWalkException e) { + // good + } + + // non-ascii + try { + filter.include(fakeWalk("\u00C0")); + fail("StopWalkException expected"); + } catch (StopWalkException e) { + // good + } + } + + TreeWalk fakeWalk(final String path) throws IOException { + DirCache dc = DirCache.newInCore(); + DirCacheEditor dce = dc.editor(); + dce.add(new DirCacheEditor.PathEdit(path) { + + public void apply(DirCacheEntry ent) { + ent.setFileMode(FileMode.REGULAR_FILE); + } + }); + dce.finish(); + + TreeWalk ret = new TreeWalk((ObjectReader) null); + ret.reset(); + ret.setRecursive(true); + ret.addTree(new DirCacheIterator(dc)); + ret.next(); + return ret; + } + +}