This is an automated email from the ASF dual-hosted git repository. rombert pushed a commit to annotated tag org.apache.sling.testing.jcr-mock-1.1.10 in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-testing-jcr-mock.git
commit f47dab8f950c0640393d278cada9eb79f7a1ab8b Author: Julian Sedding <[email protected]> AuthorDate: Thu Dec 11 11:31:04 2014 +0000 SLING-4237 - JCR Mock implementation should have "throws RepositoryException" git-svn-id: https://svn.apache.org/repos/asf/sling/trunk/testing/mocks/jcr-mock@1644602 13f79535-47bb-0310-9956-ffa450edef68 --- .../sling/testing/mock/jcr/AbstractItem.java | 16 ++-- .../apache/sling/testing/mock/jcr/ItemData.java | 1 + .../testing/mock/jcr/MockNamespaceRegistry.java | 8 +- .../apache/sling/testing/mock/jcr/MockNode.java | 94 +++++++++++----------- .../testing/mock/jcr/MockNodeTypeManager.java | 27 ++++--- .../testing/mock/jcr/MockObservationManager.java | 11 +-- .../sling/testing/mock/jcr/MockProperty.java | 20 ++--- .../sling/testing/mock/jcr/MockRepository.java | 9 ++- .../apache/sling/testing/mock/jcr/MockSession.java | 45 ++++++----- .../sling/testing/mock/jcr/MockWorkspace.java | 36 +++++---- 10 files changed, 139 insertions(+), 128 deletions(-) diff --git a/src/main/java/org/apache/sling/testing/mock/jcr/AbstractItem.java b/src/main/java/org/apache/sling/testing/mock/jcr/AbstractItem.java index f174514..9284fe5 100644 --- a/src/main/java/org/apache/sling/testing/mock/jcr/AbstractItem.java +++ b/src/main/java/org/apache/sling/testing/mock/jcr/AbstractItem.java @@ -41,12 +41,12 @@ abstract class AbstractItem implements Item { } @Override - public String getName() { + public String getName() throws RepositoryException { return this.itemData.getName(); } @Override - public String getPath() { + public String getPath() throws RepositoryException { return this.itemData.getPath(); } @@ -56,7 +56,7 @@ abstract class AbstractItem implements Item { } @Override - public Session getSession() { + public Session getSession()throws RepositoryException { return this.session; } @@ -78,7 +78,7 @@ abstract class AbstractItem implements Item { return this.session.getItem(ResourceUtil.getParent(getPath(), depth)); } - protected String makeAbsolutePath(final String relativePath) { + protected String makeAbsolutePath(final String relativePath) throws RepositoryException { String absolutePath = relativePath; // ensure the path is absolute and normalized if (!StringUtils.startsWith(absolutePath, "/")) { @@ -107,22 +107,22 @@ abstract class AbstractItem implements Item { // --- unsupported operations --- @Override - public void accept(final ItemVisitor visitor) { + public void accept(final ItemVisitor visitor) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public boolean isSame(final Item otherItem) { + public boolean isSame(final Item otherItem) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void refresh(final boolean keepChanges) { + public void refresh(final boolean keepChanges) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void save() { + public void save() throws RepositoryException { throw new UnsupportedOperationException(); } diff --git a/src/main/java/org/apache/sling/testing/mock/jcr/ItemData.java b/src/main/java/org/apache/sling/testing/mock/jcr/ItemData.java index 64ec7c1..87ff8c1 100644 --- a/src/main/java/org/apache/sling/testing/mock/jcr/ItemData.java +++ b/src/main/java/org/apache/sling/testing/mock/jcr/ItemData.java @@ -21,6 +21,7 @@ package org.apache.sling.testing.mock.jcr; import java.util.UUID; import javax.jcr.Item; +import javax.jcr.RepositoryException; import javax.jcr.Session; import javax.jcr.Value; import javax.jcr.nodetype.NodeType; diff --git a/src/main/java/org/apache/sling/testing/mock/jcr/MockNamespaceRegistry.java b/src/main/java/org/apache/sling/testing/mock/jcr/MockNamespaceRegistry.java index 91b194f..477a2fe 100644 --- a/src/main/java/org/apache/sling/testing/mock/jcr/MockNamespaceRegistry.java +++ b/src/main/java/org/apache/sling/testing/mock/jcr/MockNamespaceRegistry.java @@ -38,22 +38,22 @@ class MockNamespaceRegistry implements NamespaceRegistry { } @Override - public String getURI(final String prefix) { + public String getURI(final String prefix) throws RepositoryException { return this.namespacePrefixMapping.get(prefix); } @Override - public String getPrefix(final String uri) { + public String getPrefix(final String uri) throws RepositoryException { return this.namespacePrefixMapping.inverse().get(uri); } @Override - public void registerNamespace(final String prefix, final String uri) { + public void registerNamespace(final String prefix, final String uri) throws RepositoryException { this.namespacePrefixMapping.put(prefix, uri); } @Override - public void unregisterNamespace(final String prefix) { + public void unregisterNamespace(final String prefix) throws RepositoryException { this.namespacePrefixMapping.remove(prefix); } diff --git a/src/main/java/org/apache/sling/testing/mock/jcr/MockNode.java b/src/main/java/org/apache/sling/testing/mock/jcr/MockNode.java index 45a8060..3913479 100644 --- a/src/main/java/org/apache/sling/testing/mock/jcr/MockNode.java +++ b/src/main/java/org/apache/sling/testing/mock/jcr/MockNode.java @@ -77,7 +77,7 @@ class MockNode extends AbstractItem implements Node { public NodeIterator getNodes() throws RepositoryException { RangeIterator items = getMockedSession().listChildren(getPath(), new ItemFilter() { @Override - public boolean accept(final ItemData item) { + public boolean accept(final ItemData item) throws RepositoryException { return item.isNode(); } }); @@ -100,7 +100,7 @@ class MockNode extends AbstractItem implements Node { public PropertyIterator getProperties() throws RepositoryException { RangeIterator items = getMockedSession().listChildren(getPath(), new ItemFilter() { @Override - public boolean accept(final ItemData item) { + public boolean accept(final ItemData item) throws RepositoryException { return item.isProperty(); } }); @@ -126,12 +126,12 @@ class MockNode extends AbstractItem implements Node { } @Override - public String getIdentifier() { + public String getIdentifier() throws RepositoryException { return this.itemData.getUuid(); } @Override - public String getUUID() { + public String getUUID() throws RepositoryException { return getIdentifier(); } @@ -277,7 +277,7 @@ class MockNode extends AbstractItem implements Node { } @Override - public NodeType getPrimaryNodeType() { + public NodeType getPrimaryNodeType() throws RepositoryException { return this.itemData.getNodeType(); } @@ -308,212 +308,212 @@ class MockNode extends AbstractItem implements Node { // --- unsupported operations --- @Override - public Property setProperty(final String name, final Value value, final int type) { + public Property setProperty(final String name, final Value value, final int type) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public Property setProperty(final String name, final Value[] values, final int type) { + public Property setProperty(final String name, final Value[] values, final int type) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public Property setProperty(final String name, final String[] values, final int type) { + public Property setProperty(final String name, final String[] values, final int type) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public Property setProperty(final String name, final String value, final int type) { + public Property setProperty(final String name, final String value, final int type) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void addMixin(final String pMixinName) { + public void addMixin(final String pMixinName) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public boolean canAddMixin(final String pMixinName) { + public boolean canAddMixin(final String pMixinName) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void cancelMerge(final Version pVersion) { + public void cancelMerge(final Version pVersion) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public Version checkin() { + public Version checkin() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void checkout() { + public void checkout() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void doneMerge(final Version pVersion) { + public void doneMerge(final Version pVersion) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public Version getBaseVersion() { + public Version getBaseVersion() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public String getCorrespondingNodePath(final String workspaceName) { + public String getCorrespondingNodePath(final String workspaceName) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public NodeDefinition getDefinition() { + public NodeDefinition getDefinition() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public int getIndex() { + public int getIndex() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public Lock getLock() { + public Lock getLock() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public NodeType[] getMixinNodeTypes() { + public NodeType[] getMixinNodeTypes() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public PropertyIterator getReferences() { + public PropertyIterator getReferences() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public VersionHistory getVersionHistory() { + public VersionHistory getVersionHistory() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public boolean holdsLock() { + public boolean holdsLock() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public boolean isCheckedOut() { + public boolean isCheckedOut() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public boolean isLocked() { + public boolean isLocked() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public Lock lock(final boolean isDeep, final boolean isSessionScoped) { + public Lock lock(final boolean isDeep, final boolean isSessionScoped) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public NodeIterator merge(final String srcWorkspace, final boolean bestEffort) { + public NodeIterator merge(final String srcWorkspace, final boolean bestEffort) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void orderBefore(final String srcChildRelPath, final String destChildRelPath) { + public void orderBefore(final String srcChildRelPath, final String destChildRelPath) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void removeMixin(final String mixinName) { + public void removeMixin(final String mixinName) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void restore(final String versionName, final boolean removeExisting) { + public void restore(final String versionName, final boolean removeExisting) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void restore(final Version version, final boolean removeExisting) { + public void restore(final Version version, final boolean removeExisting) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void restore(final Version version, final String relPath, final boolean removeExisting) { + public void restore(final Version version, final String relPath, final boolean removeExisting) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void restoreByLabel(final String versionLabel, final boolean removeExisting) { + public void restoreByLabel(final String versionLabel, final boolean removeExisting) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void unlock() { + public void unlock() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void update(final String srcWorkspaceName) { + public void update(final String srcWorkspaceName) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void followLifecycleTransition(final String transition) { + public void followLifecycleTransition(final String transition) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public String[] getAllowedLifecycleTransistions() { + public String[] getAllowedLifecycleTransistions() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public NodeIterator getNodes(final String[] nameGlobs) { + public NodeIterator getNodes(final String[] nameGlobs) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public PropertyIterator getProperties(final String[] nameGlobs) { + public PropertyIterator getProperties(final String[] nameGlobs) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public PropertyIterator getReferences(final String name) { + public PropertyIterator getReferences(final String name) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public NodeIterator getSharedSet() { + public NodeIterator getSharedSet() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public PropertyIterator getWeakReferences() { + public PropertyIterator getWeakReferences() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public PropertyIterator getWeakReferences(final String name) { + public PropertyIterator getWeakReferences(final String name) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void removeShare() { + public void removeShare() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void removeSharedSet() { + public void removeSharedSet() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void setPrimaryType(final String pNodeTypeName) { + public void setPrimaryType(final String pNodeTypeName) throws RepositoryException { throw new UnsupportedOperationException(); } diff --git a/src/main/java/org/apache/sling/testing/mock/jcr/MockNodeTypeManager.java b/src/main/java/org/apache/sling/testing/mock/jcr/MockNodeTypeManager.java index edec788..3373c57 100644 --- a/src/main/java/org/apache/sling/testing/mock/jcr/MockNodeTypeManager.java +++ b/src/main/java/org/apache/sling/testing/mock/jcr/MockNodeTypeManager.java @@ -18,6 +18,7 @@ */ package org.apache.sling.testing.mock.jcr; +import javax.jcr.RepositoryException; import javax.jcr.nodetype.NodeDefinitionTemplate; import javax.jcr.nodetype.NodeType; import javax.jcr.nodetype.NodeTypeDefinition; @@ -32,13 +33,13 @@ import javax.jcr.nodetype.PropertyDefinitionTemplate; class MockNodeTypeManager implements NodeTypeManager { @Override - public NodeType getNodeType(String nodeTypeName) { + public NodeType getNodeType(String nodeTypeName) throws RepositoryException { // accept all node types and return a mock return new MockNodeType(nodeTypeName); } @Override - public boolean hasNodeType(String name) { + public boolean hasNodeType(String name) throws RepositoryException { // accept all node types return true; } @@ -46,57 +47,57 @@ class MockNodeTypeManager implements NodeTypeManager { // --- unsupported operations --- @Override - public NodeTypeIterator getAllNodeTypes() { + public NodeTypeIterator getAllNodeTypes() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public NodeTypeIterator getPrimaryNodeTypes() { + public NodeTypeIterator getPrimaryNodeTypes() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public NodeTypeIterator getMixinNodeTypes() { + public NodeTypeIterator getMixinNodeTypes() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public NodeTypeTemplate createNodeTypeTemplate() { + public NodeTypeTemplate createNodeTypeTemplate() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public NodeTypeTemplate createNodeTypeTemplate(NodeTypeDefinition ntd) { + public NodeTypeTemplate createNodeTypeTemplate(NodeTypeDefinition ntd) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public NodeDefinitionTemplate createNodeDefinitionTemplate() { + public NodeDefinitionTemplate createNodeDefinitionTemplate() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public PropertyDefinitionTemplate createPropertyDefinitionTemplate() { + public PropertyDefinitionTemplate createPropertyDefinitionTemplate() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public NodeType registerNodeType(NodeTypeDefinition ntd, boolean allowUpdate) { + public NodeType registerNodeType(NodeTypeDefinition ntd, boolean allowUpdate) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public NodeTypeIterator registerNodeTypes(NodeTypeDefinition[] ntds, boolean allowUpdate) { + public NodeTypeIterator registerNodeTypes(NodeTypeDefinition[] ntds, boolean allowUpdate) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void unregisterNodeType(String name) { + public void unregisterNodeType(String name) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void unregisterNodeTypes(String[] names) { + public void unregisterNodeTypes(String[] names) throws RepositoryException { throw new UnsupportedOperationException(); } diff --git a/src/main/java/org/apache/sling/testing/mock/jcr/MockObservationManager.java b/src/main/java/org/apache/sling/testing/mock/jcr/MockObservationManager.java index 4988fa0..b1ea937 100644 --- a/src/main/java/org/apache/sling/testing/mock/jcr/MockObservationManager.java +++ b/src/main/java/org/apache/sling/testing/mock/jcr/MockObservationManager.java @@ -31,23 +31,24 @@ class MockObservationManager implements ObservationManager { @Override public void addEventListener(final EventListener listener, final int eventTypes, final String absPath, - final boolean isDeep, final String[] uuid, final String[] nodeTypeName, final boolean noLocal) { + final boolean isDeep, final String[] uuid, final String[] nodeTypeName, final boolean noLocal) + throws RepositoryException { // do nothing } @Override - public void removeEventListener(final EventListener listener) { + public void removeEventListener(final EventListener listener) throws RepositoryException { // do nothing } // --- unsupported operations --- @Override - public EventListenerIterator getRegisteredEventListeners() { + public EventListenerIterator getRegisteredEventListeners() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void setUserData(final String userData) { + public void setUserData(final String userData) throws RepositoryException { throw new UnsupportedOperationException(); } @@ -58,7 +59,7 @@ class MockObservationManager implements ObservationManager { @Override public EventJournal getEventJournal(final int eventTypes, final String absPath, final boolean isDeep, - final String[] uuid, final String[] nodeTypeName) { + final String[] uuid, final String[] nodeTypeName) throws RepositoryException { throw new UnsupportedOperationException(); } diff --git a/src/main/java/org/apache/sling/testing/mock/jcr/MockProperty.java b/src/main/java/org/apache/sling/testing/mock/jcr/MockProperty.java index 2bc4182..cd8b7d0 100644 --- a/src/main/java/org/apache/sling/testing/mock/jcr/MockProperty.java +++ b/src/main/java/org/apache/sling/testing/mock/jcr/MockProperty.java @@ -51,7 +51,7 @@ class MockProperty extends AbstractItem implements Property { } } - private Value internalGetValue() throws ValueFormatException { + private Value internalGetValue() throws RepositoryException { if (this.itemData.getValues().length > 1) { throw new ValueFormatException(this + " is a multi-valued property, so it's values can only be retrieved as an array"); @@ -61,12 +61,12 @@ class MockProperty extends AbstractItem implements Property { } @Override - public Value getValue() throws ValueFormatException { + public Value getValue() throws RepositoryException { return internalGetValue(); } @Override - public Value[] getValues() { + public Value[] getValues() throws RepositoryException { Value[] valuesCopy = new Value[this.itemData.getValues().length]; for (int i = 0; i < this.itemData.getValues().length; i++) { valuesCopy[i] = this.itemData.getValues()[i]; @@ -75,13 +75,13 @@ class MockProperty extends AbstractItem implements Property { } @Override - public void setValue(final Value newValue) { + public void setValue(final Value newValue) throws RepositoryException { this.itemData.setValues(new Value[] { newValue }); this.itemData.setMultiple(false); } @Override - public void setValue(final Value[] newValues) { + public void setValue(final Value[] newValues) throws RepositoryException { Value[] values = new Value[newValues.length]; for (int i = 0; i < newValues.length; i++) { values[i] = newValues[i]; @@ -220,12 +220,12 @@ class MockProperty extends AbstractItem implements Property { } @Override - public boolean isMultiple() { + public boolean isMultiple() throws RepositoryException { return this.itemData.isMultiple(); } @Override - public PropertyDefinition getDefinition() { + public PropertyDefinition getDefinition() throws RepositoryException { return new MockPropertyDefinition(); } @@ -244,12 +244,12 @@ class MockProperty extends AbstractItem implements Property { // --- unsupported operations --- @Override - public Node getNode() { + public Node getNode() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public Property getProperty() { + public Property getProperty() throws RepositoryException { throw new UnsupportedOperationException(); } @@ -257,7 +257,7 @@ class MockProperty extends AbstractItem implements Property { @Override public boolean isMultiple() { - return MockProperty.this.isMultiple(); + return MockProperty.this.itemData.isMultiple(); } // --- unsupported operations --- diff --git a/src/main/java/org/apache/sling/testing/mock/jcr/MockRepository.java b/src/main/java/org/apache/sling/testing/mock/jcr/MockRepository.java index 4fb21ae..377ede6 100644 --- a/src/main/java/org/apache/sling/testing/mock/jcr/MockRepository.java +++ b/src/main/java/org/apache/sling/testing/mock/jcr/MockRepository.java @@ -23,6 +23,7 @@ import java.util.Map; import javax.jcr.Credentials; import javax.jcr.Repository; +import javax.jcr.RepositoryException; import javax.jcr.Session; import javax.jcr.SimpleCredentials; import javax.jcr.Value; @@ -45,22 +46,22 @@ class MockRepository implements Repository { } @Override - public Session login() { + public Session login() throws RepositoryException { return login(null, null); } @Override - public Session login(final String workspaceName) { + public Session login(final String workspaceName) throws RepositoryException { return login(null, workspaceName); } @Override - public Session login(final Credentials credentials) { + public Session login(final Credentials credentials) throws RepositoryException { return login(credentials, null); } @Override - public Session login(final Credentials credentials, final String workspaceName) { + public Session login(final Credentials credentials, final String workspaceName) throws RepositoryException { String userId = null; if (credentials instanceof SimpleCredentials) { userId = ((SimpleCredentials)credentials).getUserID(); diff --git a/src/main/java/org/apache/sling/testing/mock/jcr/MockSession.java b/src/main/java/org/apache/sling/testing/mock/jcr/MockSession.java index 8ec7408..18b344c 100644 --- a/src/main/java/org/apache/sling/testing/mock/jcr/MockSession.java +++ b/src/main/java/org/apache/sling/testing/mock/jcr/MockSession.java @@ -25,6 +25,7 @@ import java.util.List; import java.util.Map; import java.util.regex.Pattern; +import javax.jcr.AccessDeniedException; import javax.jcr.Credentials; import javax.jcr.Item; import javax.jcr.ItemNotFoundException; @@ -35,10 +36,14 @@ import javax.jcr.RangeIterator; import javax.jcr.Repository; import javax.jcr.RepositoryException; import javax.jcr.Session; +import javax.jcr.UnsupportedRepositoryOperationException; import javax.jcr.ValueFactory; import javax.jcr.Workspace; +import javax.jcr.lock.LockException; +import javax.jcr.nodetype.ConstraintViolationException; import javax.jcr.retention.RetentionManager; import javax.jcr.security.AccessControlManager; +import javax.jcr.version.VersionException; import org.apache.commons.lang3.StringUtils; import org.apache.jackrabbit.commons.iterator.RangeIteratorAdapter; @@ -65,7 +70,7 @@ class MockSession implements Session { } @Override - public ValueFactory getValueFactory() { + public ValueFactory getValueFactory() throws RepositoryException { return ValueFactoryImpl.getInstance(); } @@ -135,12 +140,12 @@ class MockSession implements Session { } @Override - public void removeItem(final String absPath) { + public void removeItem(final String absPath) throws RepositoryException { removeItemWithChildren(absPath); } @Override - public Node getRootNode() { + public Node getRootNode() throws RepositoryException { return (Node)this.items.get("/").getItem(this); } @@ -151,7 +156,7 @@ class MockSession implements Session { /** * Add item - * @param item item + * @param itemData item data * @throws RepositoryException */ void addItem(final ItemData itemData) throws RepositoryException { @@ -195,12 +200,12 @@ class MockSession implements Session { } @Override - public boolean hasPendingChanges() { + public boolean hasPendingChanges() throws RepositoryException { return false; } @Override - public boolean itemExists(final String absPath) { + public boolean itemExists(final String absPath) throws RepositoryException { return this.items.get(absPath) != null; } @@ -240,7 +245,7 @@ class MockSession implements Session { } @Override - public void save() { + public void save() throws RepositoryException { // do nothing } @@ -250,7 +255,7 @@ class MockSession implements Session { } @Override - public void checkPermission(final String absPath, final String actions) { + public void checkPermission(final String absPath, final String actions) throws RepositoryException { // always grant permission } @@ -262,25 +267,25 @@ class MockSession implements Session { @Override public void exportDocumentView(final String absPath, final ContentHandler contentHandler, final boolean skipBinary, - final boolean noRecurse) { + final boolean noRecurse) throws RepositoryException { throw new UnsupportedOperationException(); } @Override public void exportDocumentView(final String absPath, final OutputStream out, final boolean skipBinary, - final boolean noRecurse) { + final boolean noRecurse) throws RepositoryException { throw new UnsupportedOperationException(); } @Override public void exportSystemView(final String absPath, final ContentHandler contentHandler, final boolean skipBinary, - final boolean noRecurse) { + final boolean noRecurse) throws RepositoryException { throw new UnsupportedOperationException(); } @Override public void exportSystemView(final String absPath, final OutputStream out, final boolean skipBinary, - final boolean noRecurse) { + final boolean noRecurse) throws RepositoryException { throw new UnsupportedOperationException(); } @@ -295,7 +300,7 @@ class MockSession implements Session { } @Override - public ContentHandler getImportContentHandler(final String parentAbsPath, final int uuidBehavior) { + public ContentHandler getImportContentHandler(final String parentAbsPath, final int uuidBehavior) throws RepositoryException { throw new UnsupportedOperationException(); } @@ -305,12 +310,12 @@ class MockSession implements Session { } @Override - public Session impersonate(final Credentials credentials) { + public Session impersonate(final Credentials credentials) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void importXML(final String parentAbsPath, final InputStream in, final int uuidBehavior) { + public void importXML(final String parentAbsPath, final InputStream in, final int uuidBehavior) throws RepositoryException { throw new UnsupportedOperationException(); } @@ -325,7 +330,7 @@ class MockSession implements Session { } @Override - public void move(final String srcAbsPath, final String destAbsPath) { + public void move(final String srcAbsPath, final String destAbsPath) throws RepositoryException { throw new UnsupportedOperationException(); } @@ -335,22 +340,22 @@ class MockSession implements Session { } @Override - public AccessControlManager getAccessControlManager() { + public AccessControlManager getAccessControlManager() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public RetentionManager getRetentionManager() { + public RetentionManager getRetentionManager() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public boolean hasCapability(final String methodName, final Object target, final Object[] arguments) { + public boolean hasCapability(final String methodName, final Object target, final Object[] arguments) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public boolean hasPermission(final String absPath, final String actions) { + public boolean hasPermission(final String absPath, final String actions) throws RepositoryException { throw new UnsupportedOperationException(); } diff --git a/src/main/java/org/apache/sling/testing/mock/jcr/MockWorkspace.java b/src/main/java/org/apache/sling/testing/mock/jcr/MockWorkspace.java index 635f280..0e71284 100644 --- a/src/main/java/org/apache/sling/testing/mock/jcr/MockWorkspace.java +++ b/src/main/java/org/apache/sling/testing/mock/jcr/MockWorkspace.java @@ -21,6 +21,7 @@ package org.apache.sling.testing.mock.jcr; import java.io.InputStream; import javax.jcr.NamespaceRegistry; +import javax.jcr.RepositoryException; import javax.jcr.Session; import javax.jcr.Workspace; import javax.jcr.lock.LockManager; @@ -63,89 +64,90 @@ class MockWorkspace implements Workspace { } @Override - public NamespaceRegistry getNamespaceRegistry() { + public NamespaceRegistry getNamespaceRegistry() throws RepositoryException { return this.namespaceRegistry; } @Override - public ObservationManager getObservationManager() { + public ObservationManager getObservationManager() throws RepositoryException { return this.observationManager; } @Override - public NodeTypeManager getNodeTypeManager() { + public NodeTypeManager getNodeTypeManager() throws RepositoryException { return this.nodeTypeManager; } @Override - public QueryManager getQueryManager() { + public QueryManager getQueryManager() throws RepositoryException { return this.queryManager; } // --- unsupported operations --- @Override - public void copy(final String srcAbsPath, final String destAbsPath) { + public void copy(final String srcAbsPath, final String destAbsPath) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void copy(final String srcWorkspace, final String srcAbsPath, final String destAbsPath) { + public void copy(final String srcWorkspace, final String srcAbsPath, final String destAbsPath) + throws RepositoryException { throw new UnsupportedOperationException(); } @Override public void clone(final String srcWorkspace, final String srcAbsPath, final String destAbsPath, - final boolean removeExisting) { + final boolean removeExisting) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void move(final String srcAbsPath, final String destAbsPath) { + public void move(final String srcAbsPath, final String destAbsPath) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void restore(final Version[] versions, final boolean removeExisting) { + public void restore(final Version[] versions, final boolean removeExisting) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public LockManager getLockManager() { + public LockManager getLockManager() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public VersionManager getVersionManager() { + public VersionManager getVersionManager() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public String[] getAccessibleWorkspaceNames() { + public String[] getAccessibleWorkspaceNames() throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public ContentHandler getImportContentHandler(final String parentAbsPath, final int uuidBehavior) { + public ContentHandler getImportContentHandler(final String parentAbsPath, final int uuidBehavior) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void importXML(final String parentAbsPath, final InputStream in, final int uuidBehavior) { + public void importXML(final String parentAbsPath, final InputStream in, final int uuidBehavior) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void createWorkspace(final String name) { + public void createWorkspace(final String name) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void createWorkspace(final String name, final String srcWorkspace) { + public void createWorkspace(final String name, final String srcWorkspace) throws RepositoryException { throw new UnsupportedOperationException(); } @Override - public void deleteWorkspace(final String name) { + public void deleteWorkspace(final String name) throws RepositoryException { throw new UnsupportedOperationException(); } -- To stop receiving notification emails like this one, please contact "[email protected]" <[email protected]>.
