Reactor requires org.eclipse.jdt.annotation

We are using these, let's not rely on them coming in transitively.

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

Use coerceStatemntArgument for YinElementStatement

yin-element requires an argument, it is fair to coerce it and get
rid of a nullness warning.

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

StorageSpecific.storageType is private and final

Somehow this field escaped our usual attention, make sure it's
properly final and private.

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

Add requires org.slf4j

We are directly using logging, do not rely on 'requires transitive'.

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

Recheck fields when cleaning up listeners

We may end up in a situation where we trigger multiple times, in
which case we can end up with mutated state and therefore cannot
rely on fields being non-null.

JIRA: YANGTOOLS-1155
Change-Id: I5607843d1e02483faa387a5630f3c4c06a3be62a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Centralize Abstract{Input,Output}StatementSupport.parseArgumentValue()

As we have reduced the number of implementations of this method,
by introducing the default implementation, we are down to three
possible implementations under BaseSchemaTreeStatementSupport.

Add a level of indirection through Function<QNameModule, QName>, which
allows us to combine the two implementations into a single one residing
in BaseOperationContainerStatementSupport.

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

Add BaseSchemaTreeStatementSupport.parseArgumentValue()

Almost all subclasses of BaseSchemaTreeStatementSupport do the same
thing for parseArgumentValue(). Centralize the default behaviour there,
while allowing input/output statements to do their own thing.

JIRA: YANGTOOLS-1154
Change-Id: I7a3daf3a9c90eefadc1cad8043af3a2e7b9065e0
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Move implementation of OnStatementAdded to single class

Create BaseSchemaTreeStatementSupport that implements
onStatementAdded and migrate all the usage from
StatementSupport classes.

JIRA: YANGTOOLS-1154
Change-Id: I3815d69f8621b9559470fea35fa4d427a717acc4
Signed-off-by: miroslav.kovac <miroslav.kovac@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add OnDemandSchemaTreeStorageNode

In order to correctly implement schema dependencies, we need to
invoke a callback from namespace prerequisites back to
StatementContextBase and its subclasses.

The conduit here is NamespaceBehavior.getFrom(), which is invoked
with a namespace key. For ChildSchemaNodeNamespace this ends up
the QName matching the argument the corresponding
SchemaTreeEffectiveStatement will end up having.

Introduce OnDemandSchemaTreeStorageNode, which acts as an extension
to normal NamespaceStorageNode behavior -- it allows an
implementation to supply a QName-based child statement to be
returned.

Since StatementContextBase is a NamespaceStorageNode, its subclasses
can now additionally implement this interface to receive the
callback.

JIRA: YANGTOOLS-784
Change-Id: I0e6f621eb39b341af61005b2334174e814c18875
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Revert "Add OnDemandSchemaTreeStorageNode"

This reverts commit 81aa05143b9cdc4cea1bf39dad0ae937525f40e5, as it
is miss-filed.

JIRA: YANGTOOLS-874
Change-Id: I68b8ad39af88a100dc8c97b148426025bbb4edc6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add OnDemandSchemaTreeStorageNode

In order to correctly implement schema dependencies, we need to
invoke a callback from namespace prerequisites back to
StatementContextBase and its subclasses.

The conduit here is NamespaceBehavior.getFrom(), which is invoked
with a namespace key. For ChildSchemaNodeNamespace this ends up
the QName matching the argument the corresponding
SchemaTreeEffectiveStatement will end up having.

Introduce OnDemandSchemaTreeStorageNode, which acts as an extension
to normal NamespaceStorageNode behavior -- it allows an
implementation to supply a QName-based child statement to be
returned.

Since StatementContextBase is a NamespaceStorageNode, its subclasses
can now additionally implement this interface to receive the
callback.

JIRA: YANGTOOLS-874
Change-Id: I9badab99764cc182074420eee05e3e21bb7dca0a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert rfc8528-data-util to JPMS

This is a simplistic module, convert it.

JIRA: YANGTOOLS-934
Change-Id: Ie104b142cf268a79a19cbe5713be721076457904
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Minimize surefire overrides

We just need Mockito to be able to pick up our mocks, for which
we can just increase their visibility.

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

Add faster NormalizedNodes.findNode() alternatives

End users are going through iterators needlessly when they want
to touch just a single PathArgument. Add methods which side-step
the intermediate List.

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

Speed up NormalizedNodes.getDirectChild()

Deal with positive matches first, let ValueNode be dealt with using
the default path.

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

Move yang-data-jaxen to attic top-level directory

yang-data-jaxen is a nice prototype, but we really need a better
executor. For now it is still useful and needs to be kept until
we have a replacement.

Change-Id: I72d7e24e887fccae6fcd60b13f7bc1303d0d5186
JIRA: YANGTOOLS-1149
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert rfc7952-data-util to JPMS

This is a simplistic module, convert it.

JIRA: YANGTOOLS-934
Change-Id: I20b645ac1fbe04bfbb36e54e850f08513b6cfa34
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert yang-data-util to JPMS

This is a simplistic module, convert it.

JIRA: YANGTOOLS-934
Change-Id: Ie81fa0e53ea91f35303d1b1c36b0db97893345ef
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert rfc8528-data-api to JPMS

This is a simplistic module, convert it. While converting it, an
unneeded dependency on rfc8528-model-api is also eliminated,
which crops up in rfc8528-data-util.

JIRA: YANGTOOLS-934
Change-Id: Iaf9cb15e61eb1b035762843b4a59ea10a87d7233
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert yang-parser-rfc7950 to JPMS

With all the preparatory work going in before, this one is quite
easy.

JIRA: YANGTOOLS-1151
Change-Id: I9ee0d53b32b2fa2e99709cb8425b72d6c3097133
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert yang-parser-api to JPMS module

This again extremely simple, allowing further progress.

JIRA: YANGTOOLS-934
Change-Id: Idc52eaf14ec0e74e600ff8e729ae2db780c82d03
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert yang-parser-reactor to a JPMS module

The reactor does not have that many dependencies, convert it.

JIRA: YANGTOOLS-1151
Change-Id: Ic3ff663c2a217e14c55dea214671903e21549dc7
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert yang-parser-spi to JPMS

This is simple enough, and even if we need to move packages, that
is something we can do later.

JIRA: YANGTOOLS-1151
Change-Id: I2f31980b00c59de77b15e36fa39b39c0cbf943a2
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Deprecate YangModelDependencyInfo.forResource()

With JPMS we cannot guarantee resources are accessible as that
depends on their path. Let users deal with acquiring the resource
and expose forYangText(), which takes YangTextSchemaSource.

Migrate the test class to use StmtTextUtils.sourceForResource(),
which is updated to guarantee YangStatementStreamSource.

JIRA: YANGTOOLS-1151
Change-Id: If0e381853e66701cb0312fb9bea5c6d5845266be
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Deprecate ModifierImpl methods

Eclipse is noticing that deprecated implementations are not marked
as deprecated. Mark them now.

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

Add a missing @Override

Noticed by Eclipse: this method is no longer @Beta, either.

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

Fix a raw type warning

This is a simple violation, fix it.

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

Workaround test model loading issues

Going to a JPMS module means our resources are subject to encapsulation
-- which means things like YangTextSchemaSource.forResource() actually
depend on where the resource if located.

If its location ends up being in a valid package, that request will fail,
as JRE will consider the resource encapsulated and will reject external
module's attempt to meddle with it (such as yang.model.api).

As it turns out some of models are matching this check and we load them
through such facilities. In some cases we can re-route in a way which
makes things less verbose -- and we do that in this patch.

JIRA: YANGTOOLS-1151
Change-Id: Ib55cadd9eba840980eeba3b9004d8f9c4101e33d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Change StmtTestUtils.sourceForResource()

We have a number of test resources which are considered in JPMS
world -- and hence we cannot rely on utilities to load them, as
those utilities are not authorized to use them.

Since we do not want to update massive amounts of tests, just change
how we load resources -- go through Class.getResource() and turn
that into a File and use that. That way StmtTestUtils is the caller
and it is inside our module and is allowed to do that.

JIRA: YANGTOOLS-1151
Change-Id: I6cd43e7cc7ddc1b6c3cbac5875dd9716d43be684
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fixup resource loading in yang-parser-rfc7950

Going towards modular world requires proper context for resource
loading, as we will be on our class loader. Make sure we centralize
the policy in StmtTestUtils. This removes a bit of verbosity from
individual tests.

JIRA: YANGTOOLS-1151
Change-Id: I981245ba93826222950a764e2f9556421e90fc95
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert yang-xpath-impl into a JPMS module

This is a bit more involved, as we are using a different
javax.inject and do not use osgi.cmpn but
org.osgi.service.component.annotations directly.

The latter results in a dependency on an automatic module, but
that is not really a problem as we never touch them at runtime.

Since we do not want to export out implementation directly, and
it still needs to be discoverable via reflection, we dedicate
org.opendaylight.yang.xpath.impl.di package for that purpose.
It will come in handy for other injection frameworks as well.

JIRA: YANGTOOLS-1145
Change-Id: Ie80b02eba551fa89c334f20484e386d808316ca3
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Rename distribution-karaf to karaf

Shorter name for easier use. It's not like its heavily used.

JIRA: YANGTOOLS-1149
Change-Id: I09a66b0c367a3f6e635c91e7419a3e77b04064f9
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove integration-test directory

This is an old remnant of days way past and is not used anywhere.
Remove it to clean up our directory structure.

JIRA: YANGTOOLS-1149
Change-Id: Ib4ab2ebe4a2b41b0071e8a22786673f7bddb2b40
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Move validation tools

Let's create a new tools/ top level directory, so people can
find things more easily.

JIRA: YANGTOOLS-1149
Change-Id: I4a88a9c82a1a9d4112b1a8e1053fef529f873a86
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Move yang-plugin artifacts

We have a plugin top-level directory, move the plugin
things there. This reduces clutter in yang/ and makes for
a more navigable tree.

JIRA: YANGTOOLS-1149
Change-Id: Ifab0a8abda1ca183a3d730367d3da75ab752bc30
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Switch plugin-generator-api to JPMS module

Switch from being an automatic module to being a full module.

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

Add generalized FileGenerator interface

This patch adds a maven-independent interface for code generators,
which captures enough semantics to make mapping usable. We will be
implementing this interface in place of the original SPI.

JIRA: YANGTOOLS-1146
Change-Id: I3daef73d3fa71e74a9678979fd00417579f60695
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add a test for resolving names of submodules

There is a weird interaction going on somewhere in the system,
this patch adds a test to see what is going on.

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

Convert rfc8040-model-api to JPMS

This is a simplistic module, convert it.

JIRA: YANGTOOLS-934
Change-Id: Ie3b6348bb986b50f335f4bd5182cdfffcd9882a9
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert rfc6643-model-api to JPMS

This is a simplistic module, convert it.

JIRA: YANGTOOLS-934
Change-Id: Id1b24eddcb34c332e77032d88aac2366f062e143
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert rfc6536-model-api to JPMS

This is a simplistic module, convert it.

JIRA: YANGTOOLS-934
Change-Id: I8c3c032a6687ee25b96056541997e7d091df267b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert rfc6241-model-api to JPMS

This is a simplistic module, convert it.

JIRA: YANGTOOLS-934
Change-Id: I7e78151d978c97d5b6adfb2182018401e6716495
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert openconfig-model-api to JPMS

This is a simplistic module, convert it.

JIRA: YANGTOOLS-934
Change-Id: I01101cb1e291933ae0787b8d5af1ce70fb77394b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix odlext-model-api conversion

I920432d50409fad36c3567eaa02d55a64331048d was missing the pom.xml
updates. Fix that up.

JIRA: YANGTOOLS-934
Change-Id: Ia2b299c72038fd09190937a73f584a3d943bff93
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add a test reproducer

This issue does not reproduce on any stream, add a test case just
to prove it.

JIRA: YANGTOOLS-1039
Change-Id: Idce40f02d1311db4eef6642e0c3281da058523b1
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert yang-model-util to JPMS

This is one of our core dependencies, convert it so that we can
proceed further. Note that UTs require a block of pom.xml magic
to work.

JIRA: YANGTOOLS-934
Change-Id: Ifdd94495717d19138a1cf2580cde1dd0e9418e10
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove unneeded type specification

Eclipse is informing us we are overly verbose, fix that.

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

Convert rfc8528-model-api to JPMS

This is a simplistic module, convert it.

JIRA: YANGTOOLS-934
Change-Id: I920432d50409fad36c3567eaa02d55a64331048d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert rfc8528-model-api to JPMS

This is a simplistic module, convert it.

JIRA: YANGTOOLS-934
Change-Id: I17ba630c16eae524a88b83e98c387bb9cb62bf7b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert rfc7952-data-api to JPMS

This is extremely simple artifact, convert it to JPMS. Note that
this also shows unneded dependency on rfc7952-model-api, which is
removed.

JIRA: YANGTOOLS-934
Change-Id: I384b83ee85ff786821fdcf85d489e17417dc7bae
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert rfc7952-model-api to JPMS

This is a simplistic module, convert it.

JIRA: YANGTOOLS-934
Change-Id: Iec8520cf1e7ce2044d2a5df622ea473a8fcdc038
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert yang-data-api to a JPMS module

This drops plugin configuration in favor of declaring a module.

JIRA: YANGTOOLS-934
Change-Id: Id96ce4f0a85d9c957f694ca0125f38979f4654e6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Make yang-model-api a JPMS module

This converts yang-model-api to an explicit module, expressing what
it really does. This unfornately exposes the state of affairs re.
JPMS testing and requires us to use explicit opens to get mockito
working.

JIRA: YANGTOOLS-934
Change-Id: I80de90ea903300624e64c57859dc451a59381644
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Turn yang-xpath-api into an explicit JPMS module

This is a simple API module, convert it to an explicit module.

JIRA: YANGTOOLS-934
Change-Id: I7e6ac24051df9971a6e224c27ebeee6753cac01c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump versions 6.0.1-SNAPSHOT

This starts the enxt development iteration.

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

Expose SchemaNodeIdentifier-based JSON access

RESTCONF is using SchemaPath-based instantiation quite heavily,
some of which is getting triggered with operations and their
SchemaNodeIdentifier.

To ease the pain of that interaction, add methods which take
SchemaNodeIdentifier.Absolute. This allows users to minimize
conversions.

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

Expose SchemaNodeIdentifier-based XML access

NETCONF is using SchemaPath-based instantiation quite heavily,
some of which is getting triggered with operations and their
SchemaNodeIdentifier.

To ease the pain of that interaction, add methods which take
SchemaNodeIdentifier.Absolute. This allows users to minimize
conversions.

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

Add SchemaPath->SchemaNodeIdentifier conversions

There are a number of users who are still crossing
SchemaPath->SchemaNodeIdentifier boundary. Add a utility method
for making that task easier.

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

Promote Absolute.{last,first}NodeIdentifier

SchemaNodeIdentifier.Descendant has to have at least one
element just as Absolute. Promote utility methods to allow
users pick them.

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

Make YangNamespaceContextNamespace global

The crux of the problem with not being able to resolve prefixes comes
from the fact we end up reusing the same context for both submodule
and module. Turn the namespace into a global one, keyed by the root
context.

JIRA: YANGTOOLS-1138
Change-Id: Ibdd1b4c2427fcf62a341e9ff15dddbd055ffa6fb
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Refactor AbstractEffectiveDocumentedNodeWithStatus

We want to be passing down declared statement from support, so as
to make statement lifecycle predictable and driven by the reactor.

JIRA: YANGTOOLS-1065
Change-Id: I18a135f979f18ed81c53b7957d876252e10e59cd
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

More updates to AbstractDeclaredStatement

Migrate RegularPresenceStatement, fix BitStatement implementations
a bit and mark evolution for next major version.

JIRA: YANGTOOLS-1065
Change-Id: Iadf1859ac163d78c4252bb9bb8a3ad95ed15fc1a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Refactor bit/if-feature statements

Do not use StmtContext when instantiating these statements, which
also shows we can improve footprint of BitStatement implementation.

JIRA: YANGTOOLS-1065
Change-Id: I2beeee04705f18d68b93fed9d77a759788cdb7b3
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Refactor simple statements

Statements which have obvious argument/rawArgument interactions
are refactored to make it clear they have no StmtContext interactions.

JIRA: YANGTOOLS-1065
Change-Id: I217f3fed816783a6581a7d84e93754011a5abcdb
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Deprecate StmtContext constructors of AbstractDeclaredStatement

The only reason for taking a parser.spi.meta dependency is unpacking
of argument/rawArgument. Provide altenative constructors and deprecate
use of StmtContext.

JIRA: YANGTOOLS-1065
Change-Id: If518edc28043fca4f9fcd9b714f8022e687261c0
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove SchemaContextProvider

This class has been superseded by EffectiveModelContextProvider,
remove it.

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

Simplify Module/Submodule statement argument usage

Instead of decoupling statement argument, we can use raw argument,
which is guaranteed to be a String.

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

Use UnqualifiedQName for {Module,Submodule}Statement

UnqualifiedQName is the best approximation of a validated string
we have, even if it has a clunky name. Redefine module-like statements
to take that as an argument, forcing proper validation during
parse.

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

Push out AbstractMountPointContextFactory FIXMEs

We do not have the time to start interacting with XPath, push FIXMEs
out to next major release.

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

Push out uses/refine integration FIXMEs out

We do not have the time to address the indirection here, push it out
to the next release.

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

Rename NamespaceRevisionAware to QNameModuleAware

We have a concept which ties together a namespace and a revision,
use that instead of atomizing them again.

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

Push FIXMEs out

A number of FIXMEs will not be addressed in 6.0.0, push them out
to 7.0.0.

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

Graduate most YangInstanceIdentifier-hosted methods

We have a number of @Beta methods, which have proven to be okay,
remove the @Beta moniker.

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

Fix NumericalRestrictions.getRange() definition

While the interface is not implented, we want the default method
to be accurate.

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

Remove RevisionAwareXPath

Both 'must' and 'when' statements need to have their expressions
parsed now. This allows us to remove an indirection in the form
of RevisionAwareXPath, and just use
YangXPathExpression.QualifiedBound for result.

We have a number of non-compliant models in our test suite, which
are now fixed up.

JIRA: YANGTOOLS-1137
Change-Id: I89c6498f855b6f436052e186e159343d504e3fe4
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Refactor PatternEffectiveStatement

Add PatternExpression so we do not end up confusing constraints with
actual arguments. This allows us to eliminate explicit implementations
of PatternConstraint, as we can easily have PatternEffectiveStatement
implement them.

This also means we can no longer ignore broken patterns, but rather
need to report them. Most obvious offenders are our aging test models.

JIRA: YANGTOOLS-1136
Change-Id: I875299b6b95947b90f0c75ee9a76c81e27b50dc7
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Always use lazy leaf nodes

Remove the global knob which controls eager/lazy leaf nodes, meaning
we will always treat them as transient.

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

Migrate default method implementations to model.api

Since we are bridging to EffectiveStatements, there are a few methods
we can implement by deferring to EffectiveStatement.argument().

JIRA: YANGTOOLS-1120
Change-Id: If3b0f3815f92641278d8696690835acac3cde221
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Split out input/output schema nodes

The need to specialize to statements is forcing our hand a little,
we need to have specialized SchemaNodes to cover input/output case.

This creates a ContainerLike schema node interface, which acts as
the common super-interface.

JIRA: YANGTOOLS-1119
Change-Id: I4c8480f34cdc42dccf8335a965ac4018b7453504
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Separate out Module and Submodule interfaces

Introduce ModuleLike to hold the common information and specialize
the two interfaces properly. This allows them to be properly tied
to EffectiveStatement world.

JIRA: YANGTOOLS-1119
Change-Id: I4eee11311c6aad70de16f235da935a42b84874e0
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Replace UniqueConstraint with UniqueEffectiveStatement

As noted in the FIXME, this @Beta class has served its purpose.
Rather than bridging it, perform a direct replacement with
UniqueEffectiveStatement.

JIRA: YANGTOOLS-1119
Change-Id: I022f940b859224a58b5d706fa5a63e8acd7f6c5a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add EffectiveStatementEquivalent

Our implementation classes provide both EffectiveStatement world
as well as SchemaNode world view in a single object. As we are
making the transition we need to formalize this contract, so
downstreams can operate on both old and new views.

To make this easier, we mark vast majority of constructs with a
new interface, EffectiveStatementEquivalent, which allows users
to get EffectiveStatement view of a particular SchemaNode. With
this interface users can provide a single EffectiveStatement-based
implementation and provide a convenience method, or directly ask
their users to perform the conversion.

A few statements are not transitioned yet, as their mapping is
ambiguous.

JIRA: YANGTOOLS-1119
Change-Id: I8aa6627912498faf8d88873f7db08fe0ab524adf
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Rename UnknownSchemaNode.isAddedByAugmentation()

As noted in the FIXME:, we want to unify handling and naming of
these methods with CopyableNode.

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

Remove legacy toString() overrides

This is preparation for unified toString(), once it gets specified.

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

Hide CompositeNodeDataWithSchema.addChild()

These methods have been deprecated and are no longer used from
the outside world, hide them.

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

Remove use of MockitoAnnotations.initMocks()

Use explicit runners instead.

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

Enrich YinExportUtils

Add ModuleEffectiveStatement-based exporter.

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

Remove SchemaContextFactory

This has been superseded by EffectiveModelContextFactory three
releases ago, remove it now.

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

Propagate EffectiveModelContext to more places

We are bent on changing our code on effective statements, make sure
we propagate EffectiveModelContext to more places.

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

Include Uint.ZERO in lower saturation bounds

This is a slight performance optimization, use constant zero for
zero values.

Change-Id: I4cc3e2e8656746b8fce7a43ddb42df8865221653
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 390508b6c2cffb9c8089f641758bc39e2ee19f2d)

Add Uint saturated converters

There are a number of places where we need to perform a saturated
conversion, i.e. when we have a 'long' count and need to put it into
a Uint32 field. Add methods to deal with that.

JIRA: YANGTOOLS-1135
Change-Id: Ia0bb2e9f0400d975ce1f4b150704dcf526d3bc82
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 254b2d0b480ae2ddec8b5db9b97cb5fef084d93c)

Remove BaseTypes.isInt8() and friends

These methods are only used in mdsal's binding generator, remove
them for rehosting.

JIRA: YANGTOOLS-1099
Change-Id: Ic80f9f3aa99d82ba0ad19fc5e9275032e1f46f84
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Use Uint constants in base types

Rather than using valueOf(), use exposed ZERO and MAX_VALUE,
improving initialization speed just a little.

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

Convert uint tests to assertThrows()

We have a number of tests which can be consolidated, use assertThrows.
This flushes out a bad test case, which is corrected.

Change-Id: I3b0b760cb89a4dbd93ebe44af2ed917b0698fc31
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit b355455a078fbe3043c34bbe4f1ceab39aef11a6)

Remove util.OptionalBoolean

This class is not used anywhere, remove it.

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

Set description

Upstream helpfully sets a generic description, which cascades for
example to Sonar. Add a proper description instead.

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

Remove use of TypeDefinition.getPath()

Do not use SchemaPath when comparing types. Just check QName and
corresponding attributes.

JIRA: YANGTOOLS-1098
Change-Id: I748dccfc62d67ced6d2abba6f2b02054919d3e5d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump odlparent to 8.0.0

Upstream version has been released, adopt it.

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

Upgrade odlparent to 8.0.0-SNAPSHOT

Pick up latest ugprades.

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

Do not rely on default charset for test resources

Modernizer is pointing out a fair number of improvements, implement
them.

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