Add EntrySet/KeySet spliterators

Spliterators do not allow modification of underlying map, hence
we can always instantiate an immutable iterator and flag appropriate
characteristics.

This is an initial cut, which is not perfect: we still should create
dedicated spliterators, which do not instantie entries nor perform
lookahead and have optimized trySplit().

Change-Id: Iad1a8f4b30d5a996484b21c9be589bf80ee7d584
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Revert "Revert "BUG-7262: Operational data tree should enforce mandatory nodes""

This reverts commit 7e8657de5bbcbcb098eca56df3445d155f20bb50.

Change-Id: Ie976e7b5c64e53c812b0913ef9fb38cb3049a5ca
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Cleanup checkstyle warnings and turn enforcement on in yang-data-impl

This patch cleans up bunch of old code to comply with checkstyle rules
and flips the switch so we do not regress.

Change-Id: I2b314d2a672e9b0b7fb225004fd1f3ce770f098a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Allow shared JSONNNSWriter use

JSONNormalizedNodeStreamWriter.createNestedWriter() gives the
impression that it can be used with an externally-controlled
JsonWriter, e.g. to build a JSON document containing several
serialised NormalizedNode instances. However, since it closes the
JsonWriter when closed itself, it limits the possible uses: currently,
a writer ends up having the same lifetime as a JsonWriter, even though
it doesn’t really have the same scope (because of the SchemaPath in
particular).

This patch changes the behaviour so that the provided JsonWriter has
an independent lifecycle (and needs to be closed by callers). I will
follow up with patches to the few users in ODL.

Change-Id: I9c597fe96937867cffdcbeeddd4b1507de20f2ee
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bug 2444 - Yin export of all YANG statements

Currently yang-export exports effective world of yang into yin,
what is not correct, because lot of information is not accesible
in effective world anymore.
So the correct solution is to export declared world of yang,
where all information is accessible and in raw form and that is
exactly what we need.
Export of effective world is still retained for cases when
we don't have access to declared form of supplied module or
we want to emit also instantiated statements (e.g. statements added
by uses or augment)

Change-Id: Ic2bc559623d9e72c59e2ed7f6a797cd67933ada6
Signed-off-by: Peter Kajsa <pkajsa@cisco.com>

Bug 2444 - add missing API of some declared statements

Fix of missing API of some declared statements according to
RFC7950. This API is necessary to proper export of declared statements
to Yin.

Change-Id: Icc0209656751b00c5cc7401f7b95d8979af0fa0d
Signed-off-by: Peter Kajsa <pkajsa@cisco.com>

Fix deleteWriteFooBar2ndLevelEmptyContainerTest

- this unit is to test delete-write while the next one is delete-merge,
here its body is duplicated with deleteMergeFooBar2ndLevelEmptyContainerTest

Change-Id: I7bafd002859e23de2d89748fc71c876cb4ab674c
Signed-off-by: Jie Han <han.jie@zte.com.cn>

Add option to setup output file for logs in validator

  * added option -o (--output) to setup path of output file
    for logs

Change-Id: I42ba358932c4cf20fa11400490c1d11e2a0f6046
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>

Rename yang-system-test module to yang-model-validator

  * yang-model-validator better describes purpose of the module

Change-Id: I7353c23afcfa7f53b37e428564f783565b41f4c6
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>

Minor JavaDoc fixes in CountingRejectedExecutionHandler

Change-Id: I4bfab54a6c80e751a46fff840f763d1e7fe6aeb0
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

BUG-2389: Remove obsolete XML parser classes

yang-data-codec-xml supersedes this parser classes, which have been
deprecated. This patch removes them.

Change-Id: I73778eb0505e402224f0adb46e0fd6477d667522
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump odlparent 2.0.4 to 2.0.5

Change-Id: I11d5ca5ff224338a5543f13edbe398e545bc8038
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Do not use String.replace()

This forces a Pattern compilation, which we do not really want.
Pre-compile the patterns instead.

Change-Id: Ibcb9f911f4a4089f375264b75ce2f5d9c5f46447
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Further yang-parser-impl checkstyle fixes

Brings the violation count to below 500.

Change-Id: I22ee943c06bf04f3692a511f703573a0a57ec0ba
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove Named{File,Byte,}InputStream

These utility classes are no longer useful, remove them.

Change-Id: If9e96e978cff5c6a69ac8bfb1c63717c1762d3de
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove deprecated Yin/YangStatementSourceImpl

These classes have been superseded by Yin/YangStatementStreamSource,
which have better semantics. Remove them and fixup former call sites.

Change-Id: I9f16e62da8cf38ce35703de6a93b1d5d9dcdf2e8
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Rename MainNode methods

CAS_PREV -> casPrev
READ_PREV -> readPrev
WRITE_PREV -> writePrev

Change-Id: Ia918db3c68931d585a5f2922b0923e939814d015
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Rename recursive INode methods

Change names to conform to Java standards:
- rec_insert -> recInsert
- rec_insertif -> recInsertIf
- rec_lookup -> recLookup
- rec_remove -> recRemove

Change-Id: Ic11da9ac2ae96d8029d0bb08464033bfe027ee0b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove BasicNode.string()

This method is not used anywhere, remove it and take all commented-out
support code with it.

Change-Id: I24720c16f562fd391894484566903f1a526ad202
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove trailing spaces in resource files

Remove trailing spaces to pass the checkstyle rules included in [1].

[1] https://git.opendaylight.org/gerrit/#/c/48710/

Change-Id: I2a6ec71294ee9b8e8720393c1867d36df4e29b01
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>

Remove yang.parser.util.ModuleDependencySort

This class has been moved to yang.model.util, remove it from its old
place. Also removes YangValidationException.

Change-Id: Ic30985769d486720f971a9a70422b62e9ceb00bb
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove NodeWrappedType and TopologicalSort

These have been migrated to utilities, remove them from parser proper.

Change-Id: I543c06ee36fe92b1f0570b781b3581b253802c4f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Throw SourceException instead of invalid fraction-digits

We have a source reference, hence use SourceException.throwIf() instead
of Preconditions.checkArgument().

Change-Id: I0e41ad3b6069dfd4801ca645c54682cbd442f58b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Throw InferenceException on invalid identity reference

We have source context available, hence use InferenceException.throwIfNull()
instead of Preconditions.checkArgument().

Change-Id: I12034275f9a6a4e4bb6e53f58ef111d5129edf1b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Another round of checkstyle fixes

Brings down the number of violations to under 700.

Change-Id: I8da94e59b13fa109832b6d903045c352ad47786c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Next round of checkstyle fixes in yang-parser-impl

Another batch, bringing the number of violation to under 800.

Change-Id: I2d39b73747c952f07f633822d46f79326f9355e0
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Next round of yang-parser-impl checkstyle fixes

This patch fixes up more violations.

Change-Id: I4aed5811d3118229a8275f053dc3e4aa5538f2a3
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Cleanup some checkstyle warnings

This is incremental improvement so we start converging on being able
to enforce checkstyle.

Change-Id: I0870f80407fe54000be34bbfc8e41c15e7ce087a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bug 7437: Fix yang-system-test logging without the --verbose option

yang-system-test artifact has the --verbose option which provides
more detailed logging output of the parsing process. However,
the logging ouput is full of DEBUG level messages coming from
the YANG statement parser even without the --verbose option.

This happens because the yang-system-test uses java.util.logging for
logging events and sets the level of logging only for that Logger.
However, the YANG statement parser uses ch.qos.logback backend for
logging, so we need to set the log level for that instead.

Therefore, replace the java.util.logging in yang-system-test with
ch.qos.logback backend.

Change-Id: I14ac775228506e4ed22cd1b55e7efec1898fa27f
Signed-off-by: Igor Foltin <igor.foltin@pantheon.tech>

Cleanup YangToSourceProcessor

Use java.util.Objects.requireNonNull() instead of
Preconditions.checkNotNull().

Simplify and speed up delta change detection.

Change-Id: I3c41d3245554ac41127042c8285c03c5a5666865
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix checkstyle in yang-parser-spi

This is public contract, clean up checkstyle issues and expand documentation a bit.

Change-Id: Iead665f037bb0396bd0ff294d54c427a0e82168c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix yang-common/pom.xml whitespace

This corrects whitespace to 4 spaces as needed.

Change-Id: I9f8764385ee953cf17b48a1a6edc64815df78ab4
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

BUG-7052: split out yang-parser-spi

Parser SPI classes have no further dependencies on other parser
components. Split them out into a separate artifact so we do not
accidentaly re-introduce an inappropriate dependency.

Change-Id: I56ea4ed1962d898c807203a2b82d048c0200548c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Make SimpleDateFormatUtil non-lenient

Converting an illegal revision, like 2017-16-05, results in incorrect
results in that the resulting Date will reflect 2018-04-05, which is
no a reversible transformation -- leading to hard to fix issues when
a String and Date representation meet.

Make sure we do not allow such invalid revisions through and fixup
bad test models.

Change-Id: I138cb5357cd6898bd64b745fbfed5d05b593e1e1
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Get rid of commons-io

Java 8 gives us everything we need to search for files, hence there
is no need to use commons-io, remove it.

Change-Id: I034ba44936c97929c7c72d8e2725acacc0fa131f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

BUG-7568: Use YangTextSchemaSource to emit schema files

To properly address the issue of META-INF/yang files not having their
revisions we need to encapsulate minimal project initialization state,
which gives us properly-resolved revisions.

We then refactor the execution flow to first inspect files to
determine whan needs to happen and then execute.

Change-Id: Icdf8069c72b24ad4cb196ebd79de194023eba9e4
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Avoid module namespace/submodule name conflict when using lib sources

The issue comes up when using yang-system-test artifact as a single
YANG file validator:

A parent module or a submodule is specified as its argument
and the same directory is specified as one of the library dirs
using the -p option.

The parent module/submodule ends up being added twice to the parsed
sources which causes a module namespace/submodule name collision later
in the parsing process.

Therefore remove the conflicting lib sources from the collection of
required lib sources before they cause any trouble.

Change-Id: Id41d9f57006adec7ff658904792f9bc319b8c11e
Signed-off-by: Igor Foltin <igor.foltin@pantheon.tech>

Override toString() in Yang/YinStatementStreamSource

Yang/YinStatementStreamSource objects are used in log messages
within the YANG statement parser. In order to provide useful
information they need to have the toString() method overridden.

Change-Id: I2bfdda8a6a58256e6f4e1e113b4a5a8174c73ef7
Signed-off-by: Igor Foltin <igor.foltin@pantheon.tech>

BUG-7568: refactor yang-maven-plugin

Refactor the plugin to not use named input streams, but rather
instantiate YangTextSchemaSources. Along with the use of
YangTextSchemaContextResolver this eliminates the need for weird
path-based tricks to get local modules.

Change-Id: Ifa1139e867b773662be64ee51bf85a46be7ea619
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bug 7051 - Refactoring of UnknownStatements handling

Refactoring of UknownStatements handling due to undesired
dependecies of StatementSupport on stmt.reactor package.

Change-Id: Ic58dbe4cdda5f7252e4e17a11173a57ad5ad1db6
Signed-off-by: Peter Kajsa <pkajsa@cisco.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bug 7051 - Refactoring of implicit case statements creation

Refactoring of implicit case statements creation due to
undesired dependencies of StatementSupport on stmt.reactor
package.

Change-Id: I306c33ad284dd2aeeb551b72ba0fd795a0761d9e
Signed-off-by: Peter Kajsa <pkajsa@cisco.com>

Add the parent module of a submodule to the set of required lib sources

This issue is related to the following use case of YANG statement parser:
If we add a YANG submodule as a main source and its parent module
as a library source, YANG statement parser does not add the parent module
to the set of requires lib sources. This causes an Exception as the parser
cannot resolve belongs-to statement in the submodule.

This patch fixes the issue.

Change-Id: Icbfddbbbecec310f0f7de17456e6a12dab8167d6
Signed-off-by: Igor Foltin <igor.foltin@pantheon.tech>

Bug 8713 - BGP models not compatible with leafref context

Lookup of parent module should be performed based on the first path
argument from root not the last one.

Change-Id: Ic8e0dc901056caf93e4fc5266334434bd067d9ab
Signed-off-by: Peter Kajsa <pkajsa@cisco.com>

Revert "Cleanup old concepts"

This reverts commit a54529b3f105d0a51343f746d3a03ede31958f40.

Change-Id: I4cb084c7f2081b78a96e9a134222f468f750c8f6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Cleanup old concepts

RpcResult is in a rather bad place, as it should be part of mdsal.
Deprecate it and mark SimpleDataFormatUtil for migration.

Change-Id: Id7b8752cbe54c4bd332b146df6e278c6f8cee545
Signed-off-by: Robert Varga <rovarga@cisco.com>

Revert "BUG-7568: refactor yang-maven-plugin"

This reverts commit ab09dfe2691b4ab5d5937b124ae3afc199a561b7.

Change-Id: Idb128323149613fae8d06298b39cdcf40b0ca690
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Revert "BUG-7568: Use YangTextSchemaSource to emit schema files"

This reverts commit bc1c0861e1c229cfee7465b84b5fd15eb462a3bf.

Change-Id: I7b35dd553f181382ac587ee335dafddeb5807ebd
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Revert "Get rid of commons-io"

This reverts commit e8ac56902440704082470574abf0cd159f66293b.

Change-Id: Iffb89d4c9f0572ead4c64e5249844eb875a93824
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Get rid of commons-io

Java 8 gives us everything we need to search for files, hence there
is no need to use commons-io, remove it.

Change-Id: Ieab83e52189992f68616e8246320ca7685a87cd5
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

BUG-7568: Use YangTextSchemaSource to emit schema files

To properly address the issue of META-INF/yang files not having their
revisions we need to encapsulate minimal project initialization state,
which gives us properly-resolved revisions.

We then refactor the execution flow to first inspect files to
determine whan needs to happen and then execute.

Change-Id: I3257475f0b267adf0e4d5cd469829a267cfc68aa
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

BUG-7568: refactor yang-maven-plugin

Refactor the plugin to not use named input streams, but rather
instantiate YangTextSchemaSources. Along with the use of
YangTextSchemaContextResolver this eliminates the need for weird
path-based tricks to get local modules.

Change-Id: I27403270cbacf567ad2182b60f265cc6bb3f83e7
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

BUG-8043: eliminate ConstraintFactory

This factory is getting in the way and it really is a useless abstraction,
as we can achieve the same with simple subclass contract.

Change-Id: I3fb50c4bcce08ec79ac710d2ca0bb9e4292ffc53
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

BUG-8043: eliminate LenghtStatement.getValue()

DeclaredStatement.argument() is an alias, no need have another method
for the same thing.

Change-Id: I9d6da9d640165a286f5c7ffdd30b20b019d0635a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix benchmark compilation and runtime

Benchmark has been broken for some time, fix it up.

Change-Id: If2e1f3292a110947a1428a096de20e90591b0f6d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

BUG-8043: introduce RangeRestrictedTypeDefinition

{Decimal,Integer,UnsignedInteger}TypeDefinition share the same method
and semantics, capture them in RangeRestrictedTypeDefinition.

Change-Id: I5fa6dbf5efdcc3797abc628a98e034602b8285ad
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

BUG-8043: introduce LengthRestrictedTypeDefinition

{Binary,String}TypeDefinition share the same method and semantics,
capture them in LengthRestrictedTypeDefinition.

Change-Id: Id913e79e2d2c9a2090721c1ed6e5bee8f309051c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Kill websocket-client

This artifact is a useless experiment, which we do not package
for a long time. Kill it finally.

Change-Id: I9c0fe90065a75809b715606ce857856bd568af42
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Enable checkstyle on yang-data-codec-gson

Gneeral cleanup and enable checkstyle.

Change-Id: Ib63901ef099b1b457504e2b8e11341ab357a394a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove antlr version override

antlr is pre-configured from odlparent, no need to override its
version.

Change-Id: I86a83c22bea34f95688c63141a30454dd82b4a96
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Activate checkstyle in yang-system-test

Fix a few offences and flip the switch.

Change-Id: I0c0601d3db712b3d9ae8dd6ca53195885d7d81ab
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Enforce checkstyle on yang-test-util

Fixup a few warnings and flip the switch.

Change-Id: If82a193f6c3287edb24ef6e19ed78c823d48f736
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bug 7246 - Fix of SchemaTracker initialization and lookup of schema nodes

SchemaUtils methods perform lookup in both the namespace of groupings and
the namespace of all leafs, leaf-lists, lists, containers, choices, rpcs,
actions, notifications, anydatas, and anyxmls. In consequence, performed
lookups are ambiguous due to possible name conflicts between these namespaces.

Change-Id: Icd3e141b21d0adaf0126b539454e640c0ea0b5a6
Signed-off-by: Peter Kajsa <pkajsa@cisco.com>

Cleanup SystemTestUtils

Remove use of deprecated class. Also convert to using collections.

Change-Id: I15eeee89628dfedb2ca0baf9a9b4f318801daddb
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Throw SourceException instead of IllegalArgumentException

TypeUtils already considers StatementSourceReference, so instead
of home-grown format use SourceException.throwIf().

Change-Id: Ifc81db735d72bb7c98b94c66ac013f8dcc47413c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Mark getIdentity() as deprecated

Super interface marks this as deprecated, implementations should do
the same.

Change-Id: Ic48fe6064e2f0a96b4f35e8cfcd499764451e995
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

BUG-8043: AbstractConstraintEffectiveStatement update

This forms the baseline for implementing sub-interfaces of
ConstraintMetaDefinition, so it should itself implement that
interface.

Change-Id: Idcd9819c7262c236c39c96b8294b128de9f0410e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Use Guava's Interner instead of ObjectCache

Guava's Interner does pretty much the same thing, no need to use
ObjectCache.

Change-Id: I4ee79155aeac465bfd59392656f555c3509459fd
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Do not instantiate needless objects in augment

Non-mutable versions of accessors are cheaper and should be used
wherever we do not modify state.

Change-Id: I200d843b830af59d407da346d0b09aa4909a4467
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

BUG-7052: move copy operation to StatementContextBase

The copy operation works on top of StatementContextBase concepts
aside from argument adaptation. Move the copy operation around,
cleaning up some of the access patterns along the way.

Change-Id: I34413c7d2c4d314503e9c9adffc7e2f0d2ea2531
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bug 9005 - scope of model import prefix should be module/submodule

Yang parser includes all substatements of a submodule into a parent
module including all import statements of the submodule, what causes
mismatch of parent module imports.

Change-Id: I5c0b45348ca5e9f3ae54351c93e940135fe2b99d
Signed-off-by: Peter Kajsa <pkajsa@cisco.com>

Remove yangtools.util.Identifiables

This class does not hold anything useful in Java 8 world and was
deprecated. Remove it.

Change-Id: Id5c870a64f203adda12f9ca697531223c2d5b379
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix AnyXml node handling

Fixup the mixup of calling instanceof on an Optional.

Change-Id: I184e2ad7535129cf19c41dfd5c235285e82e9f28
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Revert "BUG-7262: Operational data tree should enforce mandatory nodes"

This reverts commit 8befab93f3eb3683ddf68665542bf64c3f36f9f7.

Downstreams are not disconnected from master yet and this exposes
issues with mis-alignment of models and implementation.

Change-Id: Ifc0a9113b7985011be0eff1853d7668e85ece403
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Cleanup testutils

Cleans up sonar warnings and re-organizes code a bit.

Change-Id: I99761f484cd992d994f1bca7c681190a055793cd
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Simplify DeviateStatementImpl.addStatement()

There is a copy&paste line here which can be eliminated with some
application of structured programming.

Change-Id: If3fffc7f07648de1456a8fbe02feba44ed40b0e9
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Refactor argument adaptation

Adapting arguments is really a StatementSupport-level operation, and
should not be part of a specific reactor implementation.

This patch creates the appropriate interfaces required to have
StatementSupport classes a say when arguments are adapted.

Change-Id: Ifa3e25b96d9f451a53a99a30be2573abe7ee744c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

BUG-7052: remove StmtContext.createCopy()

The two StmtContext.createCopy() methods are the final bits of
spi.meta dependending on stmt.reactor.

This patch eliminates them by introducing Mutable.childCopyOf()
methods, which perform the same function, except the are hosted
in the new parent object, which internally knows it is
a StatementContextBase.

Change-Id: I16978eb62851295a5c922224a051d200c5e32abc
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

BUG-7262: Operational data tree should enforce mandatory nodes

This flips the switch on enforcement of mandatory nodes in operational
data trees by default.

Change-Id: Id9c44d119a324ea711c36cea9a87728d0cbcc3ae
Signed-off-by: Robert Varga <rovarga@cisco.com>

Remove unused property declaration yang.codegen...

yang-codegen-version does not appear to be used anywhere so remove it.

Change-Id: Ib775144a7f38e3fd44256033467fb09456bd1eae
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Eliminate LeafRefPathParseException

LeafRefPathParseException is never caught and used only internally.
Refactor internals to return an Optional() and propagate an IAE instead.

Change-Id: Ibd35390b1481e16a73980d0b5aaf10b929ae282c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Use ValueNode in LeafRefValidation

ValueNode is the common ancestor of LeafNode and LeafsetEntryNode,
so we can do a single instanceof check.

Change-Id: I3faf13cae56896e234520c8d7498cf85e0e742a6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump versions by (x+1).y.z for next dev cycle

YangTools versions are bumped as follows:

1) 0.y.z-SNAPSHOT versions are bumped to 1.0.0-SNAPSHOT
2) 1.y.z-SNAPSHOT versions are bumped to 2.0.0-SNAPSHOT

Change-Id: Iab0b1edc3b72d7d5918db235fc2d9209862f94ed
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Bug 8691 - Update yang-parser-impl grammars

Update of YangStatementLexer.g4 after migration to antlr 4.7.

Change-Id: Ie2482c1b155a0119862f855738c5f55d8df938c8
Signed-off-by: Peter Kajsa <pkajsa@cisco.com>

Remove duplicate version declaration

odlparent is declaring this for use, no need to repeat it here.

Change-Id: Ic426ae0fa01f14f90a4c4355106e2420794de1dd
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bug 8922 - Evaluation of if-features is done regardless of ancestors

Evaluation of if-features for a statement is done regardless of its
ancestors. This is not correct and if an ancestor of a statement is
not supported by features, then current statement should be
unsupported too. In other words, if a statement is not supported by
features, then all its children should be unsupported too.

Change-Id: Ieb8a3c32849808a4492f518f4481aaef45c11cae
Signed-off-by: Peter Kajsa <pkajsa@cisco.com>

Make sure binary values are properly wrapped

Leaking byte[] in case binary leaves an avenue for data modification.
Disallow that by always cloning byte[].

Change-Id: Ic3cde28a4af19800d00f0f1e1ab9084b6e6951ef
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump odlparent 2.0.2 to 2.0.4

Change-Id: I17495da43f8cbde07cabcb9d9657737af2e450de
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Bug 8940: Fix XML parsing of anyxml nodes

The new XML parser unescapes already escaped special characters
in anyxml nodes which triggers an exception when loading parsed
anyxml data into org.w3c.dom.Document object.

Use the Guava's XmlEscapers to properly handle special escaped
characters in anyxml nodes.

Change-Id: Ifa79f7d6f6736253afc5092afded5f118bf84ab5
Signed-off-by: Igor Foltin <igor.foltin@pantheon.tech>

Enforce no split packages

This adds a guard to prevent accidental split packages.

Change-Id: I063ec1ee6090c60d177c22c9ad8cbeafb54f70ce
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Reduce CheckedFuture proliferation

We are not really using checkedGet() here, hence we can switch
to ListenableFuture.

Change-Id: I7c8e7bbcdc6fe180a88d27079196b9a69cc67dd0
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove unneeded version declaration

These versions are declared upstream, no need to repeat them here.

Change-Id: Iac58a882ebbb57206348b48165f013b87aca9ff6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bug 8831 - Yang 1.1 default values are not checked correctly

When a leaf has a union type, and a default that does not correspond
to the first member of that union type, the default is not processed
correctly and IllegalStateException is thrown during the check of if-feature
statements. The same bug occurs also in case, when the default is a
number, which is in the range of the first member of the union etc..

Change-Id: Ic3b165eb5c4416dee2216f47c240ce596e3f7dd2
Signed-off-by: Peter Kajsa <pkajsa@cisco.com>

Cleanup JSONCodecFactory.create() users

JSONCodecFactory.create() is deprecated, use getShared() instead.

Change-Id: I2abf0350391e067e3668760f88776130cf14f74e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Cleanup SchemaTracker logic

Do not perform duplicate null checks.

Change-Id: If1fa8631e9e6d93fa2063bea0c395d3fdcba87ea
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Use Objects.requireNonNull instead of Preconditions.checkNotNull

This is a Java-8 equivalent, use it to reduce dependecies on Guava.

Change-Id: Id4ec468b8908fb4d7a272af607c65b3b0bbc0583
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix/suppress checkstyle warnings

Some of the argument names are coming from superclasses, add suppression
of those. Others can be easily renamed.

Change-Id: I75e746287fc2278790091185d967dfcc2d16d94e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Share codepath between snapshot methods

Eliminate a bit of code duplication by creating a snapshot()
methods which does the looping. Also fixes up some violations
and makes the contract more explicit.

Change-Id: Ic02f9c148b85cbbef53ab632a22bc68f1c16f6b2
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix of Module.getPrefix() to return prefix also in case of submodule

Module.getPrefix() returns null for all submodules.
However, in case of submodule a prefix for module's
namespace is defined in belongs-to statement.
This information is required by yang-export where
namespaces need to be binded to corresponding prefixes.

Change-Id: Ifa5406654021fd017b442b6c351a4e0b4f5fd55d
Signed-off-by: Peter Kajsa <pkajsa@cisco.com>

Downgrade stax2-api to 3.1.4

As it turns out, downstream is using jackson, which is bound
to 3.1.x version of stax2-api. Downgrade to match.

Change-Id: I7a86037abceac8499986af11392da5112570d838
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add XmlParserStream.traverse(DOMSource)

This adds an alternative traversal based on DOMSource. This is needed
as it seems XMLStreamReaders based on DOMSource are in short supply and
in some cases we are interpreting a part of an already-parsed Document
as a NormalizedNode.

Change-Id: I3e21daa144ca90d868f71b3f5af318f4c35cc24f
Signed-off-by: Igor Foltin <igor.foltin@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>