X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=opendaylight%2Fblueprint%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fblueprint%2Fext%2FDataStoreAppConfigDefaultXMLReader.java;h=6f323716ee481a2bb5e015b9b57e985fdfaf8d3b;hb=6c95898df7a932ba15f12ca900ed0646a1e165f8;hp=57f3abce85ea5b0da1bf3b66377a40c097d7f4f1;hpb=821944277049bbb3949021626844ef7a80101f70;p=controller.git diff --git a/opendaylight/blueprint/src/main/java/org/opendaylight/controller/blueprint/ext/DataStoreAppConfigDefaultXMLReader.java b/opendaylight/blueprint/src/main/java/org/opendaylight/controller/blueprint/ext/DataStoreAppConfigDefaultXMLReader.java index 57f3abce85..6f323716ee 100644 --- a/opendaylight/blueprint/src/main/java/org/opendaylight/controller/blueprint/ext/DataStoreAppConfigDefaultXMLReader.java +++ b/opendaylight/blueprint/src/main/java/org/opendaylight/controller/blueprint/ext/DataStoreAppConfigDefaultXMLReader.java @@ -7,22 +7,24 @@ */ package org.opendaylight.controller.blueprint.ext; -import com.google.common.base.Optional; import com.google.common.base.Strings; import com.google.common.io.Resources; import java.io.IOException; import java.io.InputStream; +import java.net.URISyntaxException; import java.net.URL; -import org.opendaylight.controller.sal.core.api.model.SchemaService; +import java.util.Optional; +import javax.xml.parsers.ParserConfigurationException; +import javax.xml.stream.XMLStreamException; import org.opendaylight.mdsal.binding.dom.codec.api.BindingNormalizedNodeSerializer; +import org.opendaylight.mdsal.dom.api.DOMSchemaService; import org.opendaylight.yangtools.util.xml.UntrustedXML; import org.opendaylight.yangtools.yang.binding.DataObject; import org.opendaylight.yangtools.yang.common.QName; import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier; import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode; -import org.opendaylight.yangtools.yang.data.impl.codec.xml.XmlUtils; -import org.opendaylight.yangtools.yang.data.impl.schema.transform.dom.parser.DomToNormalizedNodeParserFactory; import org.opendaylight.yangtools.yang.model.api.DataSchemaNode; +import org.opendaylight.yangtools.yang.model.api.EffectiveModelContext; import org.opendaylight.yangtools.yang.model.api.Module; import org.opendaylight.yangtools.yang.model.api.SchemaContext; import org.slf4j.Logger; @@ -44,14 +46,15 @@ public class DataStoreAppConfigDefaultXMLReader { private final String logName; private final String defaultAppConfigFileName; - private final SchemaService schemaService; + private final DOMSchemaService schemaService; private final BindingNormalizedNodeSerializer bindingSerializer; private final BindingContext bindingContext; private final ConfigURLProvider inputStreamProvider; @FunctionalInterface public interface FallbackConfigProvider { - NormalizedNode get(SchemaContext schemaContext, DataSchemaNode dataSchema); + NormalizedNode get(EffectiveModelContext schemaContext, DataSchemaNode dataSchema) throws IOException, + XMLStreamException, ParserConfigurationException, SAXException, URISyntaxException; } @FunctionalInterface @@ -60,12 +63,12 @@ public class DataStoreAppConfigDefaultXMLReader { } public DataStoreAppConfigDefaultXMLReader( - String logName, - String defaultAppConfigFileName, - SchemaService schemaService, - BindingNormalizedNodeSerializer bindingSerializer, - BindingContext bindingContext, - ConfigURLProvider inputStreamProvider) { + final String logName, + final String defaultAppConfigFileName, + final DOMSchemaService schemaService, + final BindingNormalizedNodeSerializer bindingSerializer, + final BindingContext bindingContext, + final ConfigURLProvider inputStreamProvider) { this.logName = logName; this.defaultAppConfigFileName = defaultAppConfigFileName; @@ -76,21 +79,22 @@ public class DataStoreAppConfigDefaultXMLReader { } public DataStoreAppConfigDefaultXMLReader( - Class testClass, - String defaultAppConfigFileName, - SchemaService schemaService, - BindingNormalizedNodeSerializer bindingSerializer, - Class klass) { + final Class testClass, + final String defaultAppConfigFileName, + final DOMSchemaService schemaService, + final BindingNormalizedNodeSerializer bindingSerializer, + final Class klass) { this(testClass.getName(), defaultAppConfigFileName, schemaService, bindingSerializer, BindingContext.create(testClass.getName(), klass, null), appConfigFileName -> Optional.of(getURL(testClass, defaultAppConfigFileName))); } - private static URL getURL(Class testClass, String defaultAppConfigFileName) { + private static URL getURL(final Class testClass, final String defaultAppConfigFileName) { return Resources.getResource(testClass, defaultAppConfigFileName); } - public T createDefaultInstance() throws ConfigXMLReaderException { + public T createDefaultInstance() throws ConfigXMLReaderException, ParserConfigurationException, XMLStreamException, + IOException, SAXException, URISyntaxException { return createDefaultInstance((schemaContext, dataSchema) -> { throw new IllegalArgumentException("Failed to read XML " + "(not creating model from defaults as runtime would, for better clarity in tests)"); @@ -98,39 +102,30 @@ public class DataStoreAppConfigDefaultXMLReader { } @SuppressWarnings("unchecked") - public T createDefaultInstance(FallbackConfigProvider fallback) throws ConfigXMLReaderException { + public T createDefaultInstance(final FallbackConfigProvider fallback) throws ConfigXMLReaderException, + URISyntaxException, ParserConfigurationException, XMLStreamException, SAXException, IOException { YangInstanceIdentifier yangPath = bindingSerializer.toYangInstanceIdentifier(bindingContext.appConfigPath); LOG.debug("{}: Creating app config instance from path {}, Qname: {}", logName, yangPath, bindingContext.bindingQName); - if (schemaService == null) { - throw new ConfigXMLReaderException( - String.format("%s: Could not obtain the SchemaService OSGi service", logName)); - } + checkNotNull(schemaService, "%s: Could not obtain the SchemaService OSGi service", logName); - SchemaContext schemaContext = schemaService.getGlobalContext(); + EffectiveModelContext schemaContext = schemaService.getGlobalContext(); - Module module = schemaContext.findModuleByNamespaceAndRevision(bindingContext.bindingQName.getNamespace(), - bindingContext.bindingQName.getRevision()); - if (module == null) { - throw new ConfigXMLReaderException( - String.format("%s: Could not obtain the module schema for namespace %s, revision %s", - logName, bindingContext.bindingQName.getNamespace(), bindingContext.bindingQName.getRevision())); - } + Module module = schemaContext.findModule(bindingContext.bindingQName.getModule()).orElse(null); + checkNotNull(module, "%s: Could not obtain the module schema for namespace %s, revision %s", + logName, bindingContext.bindingQName.getNamespace(), bindingContext.bindingQName.getRevision()); - DataSchemaNode dataSchema = module.getDataChildByName(bindingContext.bindingQName); - if (dataSchema == null) { - throw new ConfigXMLReaderException( - String.format("%s: Could not obtain the schema for %s", logName, - bindingContext.bindingQName)); - } + QName qname = bindingContext.bindingQName; + DataSchemaNode dataSchema = module.findDataChildByName(qname).orElseThrow( + () -> new ConfigXMLReaderException(logName + ": Could not obtain the schema for " + qname)); - if (!bindingContext.schemaType.isAssignableFrom(dataSchema.getClass())) { - throw new ConfigXMLReaderException( - String.format("%s: Expected schema type %s for %s but actual type is %s", logName, - bindingContext.schemaType, bindingContext.bindingQName, dataSchema.getClass())); - } + checkNotNull(dataSchema, "%s: Could not obtain the schema for %s", logName, bindingContext.bindingQName); + + checkCondition(bindingContext.schemaType.isAssignableFrom(dataSchema.getClass()), + "%s: Expected schema type %s for %s but actual type is %s", logName, + bindingContext.schemaType, bindingContext.bindingQName, dataSchema.getClass()); NormalizedNode dataNode = parsePossibleDefaultAppConfigXMLFile(schemaContext, dataSchema); if (dataNode == null) { @@ -138,17 +133,27 @@ public class DataStoreAppConfigDefaultXMLReader { } DataObject appConfig = bindingSerializer.fromNormalizedNode(yangPath, dataNode).getValue(); - if (appConfig == null) { - // This shouldn't happen but need to handle it in case... - throw new ConfigXMLReaderException( - String.format("%s: Could not create instance for app config binding %s", - logName, bindingContext.appConfigBindingClass)); - } else { - return (T) appConfig; + + // This shouldn't happen but need to handle it in case... + checkNotNull(appConfig, "%s: Could not create instance for app config binding %s", logName, + bindingContext.appConfigBindingClass); + + return (T) appConfig; + } + + private static void checkNotNull(final Object reference, final String errorMessageFormat, + final Object... formatArgs) throws ConfigXMLReaderException { + checkCondition(reference != null, errorMessageFormat, formatArgs); + } + + private static void checkCondition(final boolean expression, final String errorMessageFormat, + final Object... formatArgs) throws ConfigXMLReaderException { + if (!expression) { + throw new ConfigXMLReaderException(String.format(errorMessageFormat, formatArgs)); } } - private NormalizedNode parsePossibleDefaultAppConfigXMLFile(final SchemaContext schemaContext, + private NormalizedNode parsePossibleDefaultAppConfigXMLFile(final EffectiveModelContext schemaContext, final DataSchemaNode dataSchema) throws ConfigXMLReaderException { String appConfigFileName = defaultAppConfigFileName; @@ -157,13 +162,10 @@ public class DataStoreAppConfigDefaultXMLReader { appConfigFileName = moduleName + "_" + bindingContext.bindingQName.getLocalName() + ".xml"; } - DomToNormalizedNodeParserFactory parserFactory = DomToNormalizedNodeParserFactory.getInstance( - XmlUtils.DEFAULT_XML_CODEC_PROVIDER, schemaContext); - Optional optionalURL; try { optionalURL = inputStreamProvider.getURL(appConfigFileName); - } catch (IOException e) { + } catch (final IOException e) { String msg = String.format("%s: Could not getURL()", logName); LOG.error(msg, e); throw new ConfigXMLReaderException(msg, e); @@ -175,12 +177,13 @@ public class DataStoreAppConfigDefaultXMLReader { try (InputStream is = url.openStream()) { Document root = UntrustedXML.newDocumentBuilder().parse(is); NormalizedNode dataNode = bindingContext.parseDataElement(root.getDocumentElement(), dataSchema, - parserFactory); + schemaContext); LOG.debug("{}: Parsed data node: {}", logName, dataNode); return dataNode; - } catch (SAXException | IOException e) { + } catch (final IOException | SAXException | XMLStreamException | ParserConfigurationException + | URISyntaxException e) { String msg = String.format("%s: Could not read/parse app config %s", logName, url); LOG.error(msg, e); throw new ConfigXMLReaderException(msg, e);