Simplify scm test parameters

This commit is contained in:
René Pfeuffer
2018-08-03 10:42:47 +02:00
parent 62f8509e85
commit e60bea5f08
6 changed files with 32 additions and 29 deletions

View File

@@ -39,11 +39,9 @@ import org.junit.Test;
import org.junit.runner.RunWith;
import org.junit.runners.Parameterized;
import org.junit.runners.Parameterized.Parameters;
import sonia.scm.util.IOUtil;
import sonia.scm.web.VndMediaType;
import java.io.IOException;
import java.util.ArrayList;
import java.util.Collection;
import static org.hamcrest.Matchers.equalTo;
@@ -64,17 +62,8 @@ public class RepositoriesITCase {
}
@Parameters(name = "{0}")
public static Collection<String[]> createParameters() {
Collection<String[]> params = new ArrayList<>();
params.add(new String[]{"git"});
params.add(new String[]{"svn"});
if (IOUtil.search("hg") != null) {
params.add(new String[]{"hg"});
}
return params;
public static Collection<String> createParameters() {
return ScmParameterizedIntegrationTestUtil.createParameters();
}
@After

View File

@@ -0,0 +1,21 @@
package sonia.scm.it;
import sonia.scm.util.IOUtil;
import java.util.ArrayList;
import java.util.Collection;
class ScmParameterizedIntegrationTestUtil {
static Collection<String> createParameters() {
Collection<String> params = new ArrayList<>();
params.add("git");
params.add("svn");
if (IOUtil.search("hg") != null) {
params.add("hg");
}
return params;
}
}

View File

@@ -162,22 +162,15 @@ public final class IntegrationTestUtil
client.getAddCommand().add(name);
}
/**
* Method description
*
*
* @return
*/
public static Collection<String[]> createRepositoryTypeParameters()
{
Collection<String[]> params = new ArrayList<>();
public static Collection<String> createRepositoryTypeParameters() {
Collection<String> params = new ArrayList<>();
params.add(new String[] { "git" });
params.add(new String[] { "svn" });
params.add("git");
params.add("svn" );
if (IOUtil.search("hg") != null)
{
params.add(new String[] { "hg" });
params.add("hg");
}
return params;

View File

@@ -200,7 +200,7 @@ public class RepositoryHookITCase extends AbstractAdminITCaseBase
* @return repository types test parameter
*/
@Parameters(name = "{0}")
public static Collection<String[]> createParameters()
public static Collection<String> createParameters()
{
return IntegrationTestUtil.createRepositoryTypeParameters();
}

View File

@@ -54,6 +54,7 @@ import static sonia.scm.it.IntegrationTestUtil.ADMIN_USERNAME;
import static sonia.scm.it.IntegrationTestUtil.commit;
import static sonia.scm.it.IntegrationTestUtil.createAdminClient;
import static sonia.scm.it.IntegrationTestUtil.createRandomFile;
import static sonia.scm.it.IntegrationTestUtil.createRepositoryTypeParameters;
import static sonia.scm.it.IntegrationTestUtil.createResource;
import static sonia.scm.it.IntegrationTestUtil.createTempDirectory;
import static sonia.scm.it.IntegrationTestUtil.readJson;
@@ -200,8 +201,7 @@ public class RepositoryITCaseBase
User noperm = UserTestData.createPerfect();
createUser(noperm);
IntegrationTestUtil.createRepositoryTypeParameters().stream().map(array -> array[0])
.forEach(t -> appendTestParameter(params, t, owner, write, read, noperm));
createRepositoryTypeParameters().forEach(t -> appendTestParameter(params, t, owner, write, read, noperm));
return params;
}

View File

@@ -42,7 +42,7 @@ import java.util.Collection;
public class RepositoryTypeITCaseBase {
@Parameters(name = "{0}")
public static Collection<String[]> createParameters() {
public static Collection<String> createParameters() {
return IntegrationTestUtil.createRepositoryTypeParameters();
}
}