Merge pull request #1415 from scm-manager/feature/lookup_api

Feature/lookup api
This commit is contained in:
René Pfeuffer
2020-11-12 09:42:23 +01:00
committed by GitHub
10 changed files with 307 additions and 2 deletions

View File

@@ -6,6 +6,9 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
## Unreleased
### Added
- Lookup command which provides further repository information ([#1415](https://github.com/scm-manager/scm-manager/pull/1415))
### Fixed
- Error on repository initialization with least-privilege user ([#1414](https://github.com/scm-manager/scm-manager/pull/1414))
- Adhere to git quiet flag ([#1421](https://github.com/scm-manager/scm-manager/pull/1421))

View File

@@ -57,5 +57,10 @@ public enum Command
/**
* @since 2.0
*/
MODIFICATIONS, MERGE, DIFF_RESULT, BRANCH, MODIFY;
MODIFICATIONS, MERGE, DIFF_RESULT, BRANCH, MODIFY,
/**
* @since 2.10.0
*/
LOOKUP;
}

View File

@@ -0,0 +1,51 @@
/*
* MIT License
*
* Copyright (c) 2020-present Cloudogu GmbH and Contributors
*
* Permission is hereby granted, free of charge, to any person obtaining a copy
* of this software and associated documentation files (the "Software"), to deal
* in the Software without restriction, including without limitation the rights
* to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
* copies of the Software, and to permit persons to whom the Software is
* furnished to do so, subject to the following conditions:
*
* The above copyright notice and this permission notice shall be included in all
* copies or substantial portions of the Software.
*
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
* AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
* SOFTWARE.
*/
package sonia.scm.repository.api;
import sonia.scm.repository.spi.LookupCommand;
import sonia.scm.repository.spi.LookupCommandRequest;
import java.util.Optional;
/**
* The lookup command executes a lookup for additional repository information.
*
* @since 2.10.0
*/
public class LookupCommandBuilder {
private final LookupCommand lookupCommand;
public LookupCommandBuilder(LookupCommand lookupCommand) {
this.lookupCommand = lookupCommand;
}
public <T> Optional<T> lookup(Class<T> type, String... args) {
LookupCommandRequest<T> request = new LookupCommandRequest<>();
request.setType(type);
request.setArgs(args);
return lookupCommand.lookup(request);
}
}

View File

@@ -430,6 +430,20 @@ public final class RepositoryService implements Closeable {
return new ModifyCommandBuilder(provider.getModifyCommand(), workdirProvider, eMail);
}
/**
* The lookup command executes a lookup which returns additional information for the repository.
*
* @return instance of {@link LookupCommandBuilder}
* @throws CommandNotSupportedException if the command is not supported
* by the implementation of the repository service provider.
* @since 2.10.0
*/
public LookupCommandBuilder getLookupCommand() {
LOG.debug("create lookup command for repository {}",
repository.getNamespaceAndName());
return new LookupCommandBuilder(provider.getLookupCommand());
}
/**
* Returns true if the command is supported by the repository service.
*

View File

@@ -0,0 +1,38 @@
/*
* MIT License
*
* Copyright (c) 2020-present Cloudogu GmbH and Contributors
*
* Permission is hereby granted, free of charge, to any person obtaining a copy
* of this software and associated documentation files (the "Software"), to deal
* in the Software without restriction, including without limitation the rights
* to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
* copies of the Software, and to permit persons to whom the Software is
* furnished to do so, subject to the following conditions:
*
* The above copyright notice and this permission notice shall be included in all
* copies or substantial portions of the Software.
*
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
* AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
* SOFTWARE.
*/
package sonia.scm.repository.spi;
import java.util.Optional;
public interface LookupCommand {
/**
* Executes lookup for given parameters.
*
* @param request Arguments provided for the lookup.
* @return Result of provided type.
*/
<T> Optional<T> lookup(LookupCommandRequest<T> request);
}

View File

@@ -0,0 +1,35 @@
/*
* MIT License
*
* Copyright (c) 2020-present Cloudogu GmbH and Contributors
*
* Permission is hereby granted, free of charge, to any person obtaining a copy
* of this software and associated documentation files (the "Software"), to deal
* in the Software without restriction, including without limitation the rights
* to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
* copies of the Software, and to permit persons to whom the Software is
* furnished to do so, subject to the following conditions:
*
* The above copyright notice and this permission notice shall be included in all
* copies or substantial portions of the Software.
*
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
* AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
* SOFTWARE.
*/
package sonia.scm.repository.spi;
import lombok.Getter;
import lombok.Setter;
@Getter
@Setter
public class LookupCommandRequest<T> {
private Class<T> type;
private String[] args;
}

View File

@@ -274,4 +274,12 @@ public abstract class RepositoryServiceProvider implements Closeable
{
throw new CommandNotSupportedException(Command.MODIFY);
}
/**
* @since 2.10.0
*/
public LookupCommand getLookupCommand()
{
throw new CommandNotSupportedException(Command.LOOKUP);
}
}

View File

@@ -0,0 +1,64 @@
/*
* MIT License
*
* Copyright (c) 2020-present Cloudogu GmbH and Contributors
*
* Permission is hereby granted, free of charge, to any person obtaining a copy
* of this software and associated documentation files (the "Software"), to deal
* in the Software without restriction, including without limitation the rights
* to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
* copies of the Software, and to permit persons to whom the Software is
* furnished to do so, subject to the following conditions:
*
* The above copyright notice and this permission notice shall be included in all
* copies or substantial portions of the Software.
*
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
* AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
* SOFTWARE.
*/
package sonia.scm.repository.spi;
import lombok.extern.slf4j.Slf4j;
import org.tmatesoft.svn.core.SVNException;
import org.tmatesoft.svn.core.io.SVNRepository;
import java.util.Optional;
@Slf4j
public class SvnLookupCommand extends AbstractSvnCommand implements LookupCommand {
protected SvnLookupCommand(SvnContext context) {
super(context);
}
@Override
public <T> Optional<T> lookup(LookupCommandRequest<T> request) {
try {
if (request.getArgs().length > 1 && "propget".equalsIgnoreCase(request.getArgs()[0])) {
return lookupProps(request);
}
} catch (SVNException e) {
log.error("Lookup failed: ", e);
}
return Optional.empty();
}
private <T> Optional<T> lookupProps(LookupCommandRequest<T> request) throws SVNException {
if (request.getArgs()[1].equalsIgnoreCase("uuid")) {
if (!request.getType().equals(String.class)) {
throw new IllegalArgumentException("uuid can only be returned as String");
}
SVNRepository repository = context.open();
return Optional.of((T) repository.getRepositoryUUID(true));
}
log.debug("No result found on lookup");
return Optional.empty();
}
}

View File

@@ -46,7 +46,7 @@ public class SvnRepositoryServiceProvider extends RepositoryServiceProvider
//J-
public static final Set<Command> COMMANDS = ImmutableSet.of(
Command.BLAME, Command.BROWSE, Command.CAT, Command.DIFF,
Command.LOG, Command.BUNDLE, Command.UNBUNDLE, Command.MODIFY
Command.LOG, Command.BUNDLE, Command.UNBUNDLE, Command.MODIFY, Command.LOOKUP
);
//J+
@@ -148,14 +148,21 @@ public class SvnRepositoryServiceProvider extends RepositoryServiceProvider
return new SvnLogCommand(context);
}
@Override
public ModificationsCommand getModificationsCommand() {
return new SvnModificationsCommand(context);
}
@Override
public ModifyCommand getModifyCommand() {
return new SvnModifyCommand(context, workingCopyFactory);
}
@Override
public LookupCommand getLookupCommand() {
return new SvnLookupCommand(context);
}
/**
* Method description
*

View File

@@ -0,0 +1,80 @@
/*
* MIT License
*
* Copyright (c) 2020-present Cloudogu GmbH and Contributors
*
* Permission is hereby granted, free of charge, to any person obtaining a copy
* of this software and associated documentation files (the "Software"), to deal
* in the Software without restriction, including without limitation the rights
* to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
* copies of the Software, and to permit persons to whom the Software is
* furnished to do so, subject to the following conditions:
*
* The above copyright notice and this permission notice shall be included in all
* copies or substantial portions of the Software.
*
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
* AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
* SOFTWARE.
*/
package sonia.scm.repository.spi;
import org.junit.jupiter.api.Test;
import org.junit.jupiter.api.extension.ExtendWith;
import org.mockito.InjectMocks;
import org.mockito.Mock;
import org.mockito.junit.jupiter.MockitoExtension;
import org.tmatesoft.svn.core.SVNException;
import org.tmatesoft.svn.core.io.SVNRepository;
import java.util.Optional;
import static org.assertj.core.api.Assertions.assertThat;
import static org.mockito.Mockito.when;
@ExtendWith(MockitoExtension.class)
class SvnLookupCommandTest {
@Mock
SvnContext context;
@Mock
SVNRepository svnRepository;
@InjectMocks
SvnLookupCommand command;
@Test
void shouldReturnEmptyOptional() {
LookupCommandRequest request = new LookupCommandRequest();
request.setType(String.class);
request.setArgs(new String[]{"propget"});
Optional<Object> result = command.lookup(request);
assertThat(result).isNotPresent();
}
@Test
void shouldReturnRepositoryUUID() throws SVNException {
String uuid = "trillian-hitchhiker-42";
when(context.open()).thenReturn(svnRepository);
when(svnRepository.getRepositoryUUID(true)).thenReturn(uuid);
LookupCommandRequest request = new LookupCommandRequest();
request.setType(String.class);
request.setArgs(new String[]{"propget", "uuid", "/"});
Optional<Object> result = command.lookup(request);
assertThat(result).isPresent();
assertThat(result.get())
.isInstanceOf(String.class)
.isEqualTo(uuid);
}
}