Mercurial > hg > thermostat
view dependency-tool/command/src/test/java/com/redhat/thermostat/tools/dependency/internal/actions/ListDependenciesActionTest.java @ 2573:006ac47d1506
Fix Version Checks in Dependency Analysis
Fixes the dependency analysis code in Launcher and the
Dependency analyzer command
- Parses version strings from the OSGI manifest
- When building the dependency graph, ensures that
the correct versions of each bundle are used
Reviewed-by: neugens, jkang
Review-thread: http://icedtea.classpath.org/pipermail/thermostat/2016-December/021832.html
author | Joshua Matsuoka <jmatsuok@redhat.com> |
---|---|
date | Thu, 26 Jan 2017 12:14:51 -0500 |
parents | e7fd8adb5f9c |
children |
line wrap: on
line source
/* * Copyright 2012-2017 Red Hat, Inc. * * This file is part of Thermostat. * * Thermostat is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published * by the Free Software Foundation; either version 2, or (at your * option) any later version. * * Thermostat is distributed in the hope that it will be useful, but * WITHOUT ANY WARRANTY; without even the implied warranty of * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU * General Public License for more details. * * You should have received a copy of the GNU General Public License * along with Thermostat; see the file COPYING. If not see * <http://www.gnu.org/licenses/>. * * Linking this code with other modules is making a combined work * based on this code. Thus, the terms and conditions of the GNU * General Public License cover the whole combination. * * As a special exception, the copyright holders of this code give * you permission to link this code with independent modules to * produce an executable, regardless of the license terms of these * independent modules, and to copy and distribute the resulting * executable under terms of your choice, provided that you also * meet, for each linked independent module, the terms and conditions * of the license of that module. An independent module is a module * which is not derived from or based on this code. If you modify * this code, you may extend this exception to your version of the * library, but you are not obligated to do so. If you do not wish * to do so, delete this exception statement from your version. */ package com.redhat.thermostat.tools.dependency.internal.actions; import com.redhat.thermostat.collections.graph.Node; import com.redhat.thermostat.common.cli.CommandContext; import com.redhat.thermostat.tools.dependency.internal.BundleProperties; import com.redhat.thermostat.tools.dependency.internal.JarLocations; import com.redhat.thermostat.tools.dependency.internal.PathProcessorHandler; import com.redhat.thermostat.tools.dependency.internal.Utils; import com.redhat.thermostat.tools.dependency.internal.utils.TestHelper; import org.junit.Assert; import org.junit.Before; import org.junit.Test; import org.mockito.Mockito; import java.io.File; import java.nio.file.Path; import java.util.ArrayList; import java.util.List; /** */ public class ListDependenciesActionTest { private File underneathTheBridge; private JarLocations paths; private CommandContext ctx; private TestUtils utils; private Path a; private Path b; private Path c; private Path d; private Path e; private Path c1; private static class TestUtils extends Utils { List<Node> results = new ArrayList<>(); public void printHeader(CommandContext ctx, String name) {} public void print(CommandContext ctx, Object x) { if (x instanceof Node) { results.add((Node) x); } } public void cannotAccessPathError(CommandContext ctx, Path path) {} public void cannotFindAttributeError(CommandContext ctx, BundleProperties property, Path path) {} } @Before public void setUp() throws Exception { ctx = Mockito.mock(CommandContext.class); utils = new TestUtils(); Utils.initSingletonForTest(utils); underneathTheBridge = TestHelper.createTestDirectory(); paths = new JarLocations(); paths.getLocations().add(underneathTheBridge.toPath()); a = TestHelper.createJar("a", null, underneathTheBridge.toPath()); b = TestHelper.createJar("b", "a", underneathTheBridge.toPath()); c = TestHelper.createJar("c", "b", underneathTheBridge.toPath()); d = TestHelper.createJar("d", "b,c", underneathTheBridge.toPath()); e = TestHelper.createJar("e", "d", underneathTheBridge.toPath()); c1 = TestHelper.createJar("c1", "c2", underneathTheBridge.toPath()); TestHelper.createJar("c2", "c3", underneathTheBridge.toPath()); TestHelper.createJar("c3", "c1", underneathTheBridge.toPath()); } @Test public void testInboundDependenciesOnLibraryWithoutImports() { PathProcessorHandler handler = new PathProcessorHandler(paths); ListDependenciesAction.execute(handler, a.toString(), ctx); Assert.assertTrue(utils.results.isEmpty()); } @Test public void testOutboundDependenciesOnLibraryWithMultipleDependencies() { PathProcessorHandler handler = new PathProcessorHandler(paths); ListDependenciesAction.execute(handler, d.toString(), ctx); Assert.assertFalse(utils.results.isEmpty()); Assert.assertEquals(3, utils.results.size()); Assert.assertEquals(a.toString(), utils.results.get(0).getName()); Assert.assertEquals(b.toString(), utils.results.get(1).getName()); Assert.assertEquals(c.toString(), utils.results.get(2).getName()); } @Test public void testOutboundDependenciesOnRootLibrary() { PathProcessorHandler handler = new PathProcessorHandler(paths); ListDependenciesAction.execute(handler, a.toString(), ctx, true); Assert.assertFalse(utils.results.isEmpty()); Assert.assertEquals(4, utils.results.size()); Assert.assertEquals(b.toString(), utils.results.get(0).getName()); Assert.assertEquals(c.toString(), utils.results.get(1).getName()); Assert.assertEquals(d.toString(), utils.results.get(2).getName()); Assert.assertEquals(e.toString(), utils.results.get(3).getName()); } // Test that an exception is thrown when a cycle is detected. @Test(expected = IllegalStateException.class) public void testCyclicDependencies() { PathProcessorHandler handler = new PathProcessorHandler(paths); ListDependenciesAction.execute(handler, c1.toString(), ctx, true); } }