Clean up exception handling

There are a number of places where we allow RestconfDocumentedException
to be thrown, but we are returning a RestconfFuture. Make sure we catch
the exception and propagate it through the future.

JIRA: NETCONF-1157
Change-Id: Ia3a67e9469179302c0a9aabbc420a7a05fc8f08d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Eliminate InstanceIdentifierContext

Rework the way we lookup strategies and generally resolve ApiPath to
actual request paths.

The primary logic now rests with RestconfStrategy, which has a way to
recursively deal with yang-ext:mount steps, resulting in a strategy +
ApiPath tail.

These are then picked up by MdsalRestconfServer and routed to the target
strategy's actual method -- which then interprets the tail ApiPath as
appropriate.

JIRA: NETCONF-1157
Change-Id: I8a6391f5538ba5662bad1bef79186e9a1e08ae81
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Generalize ResourceBodyTest

Use proper payloads instead of mucking with internals. This shows that
negative case is completely unreachable.

JIRA: NETCONF-1157
Change-Id: I88371fe99362bec68462f6872e874bb7bbf89e5a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove duplicate container in network-topology

Task NETCONF-670 resolves a failed request to create a device
using the RPC create-device. This task added the grouping
"netconf-node-augmented-optional-fields" to the required RPC
"create-device" but unintentionally to the augmentation for
the list "node" in the network-topology model.

The same augmentation is already done in the
"netconf-node-optional" model.

This results in preparing the same augmentation from
two different models.

JIRA: NETCONF-1192
Change-Id: Ief7a449ffdaff5d42eb52856fac41e87a8e0274a
Signed-off-by: Peter Suna <peter.suna@pantheon.tech>

Generalize validTopLevelNodeName() test

Assert the same violation in JSON/XML tests, which shows this method is
that XML is doing its own thing.

Since these are exercising this code path as they should, we can hide
the method and inline it.

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

Inline json_sub_container.json

This is a simple resource, inline it to its two callers and eliminate
obsolete method which gets freed up.

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

Inline xml_sub_container.xml

This is a simple document, inline it directly in tests.

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

Inline foo.xml

This is a simplistic XML fragment, inline it.

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

Migrate AbstractResourceBodyTest to JUnit5

Clean up test interactions and migrate to JUnit5.

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

Do not hide DataGetParams.EMPTY

This is a simple constant, do not hide it, as we are already a
transparent record-based DTO.

JIRA: NETCONF-773
Change-Id: I103cdc4e8341f53078204fc0c9f4192f80df65c6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Inline RestconfApplication

Use an anonymous Application subclass to initialize the servlet, which
makes it obvious JaxRsNorthbound has now mdsal.dom.api dependencies.

JIRA: NETCONF-773
Change-Id: I25aea4b6fdfc3d6d3561387956f2e92ff057c6e7
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Refactor ReceiveEventsParams

This is a simple request DTO. Move it to restconf.server.api, where it
really belongs. Also rename it to EventStreamGetParams in keeping with
package naming.

JIRA: NETCONF-773
Change-Id: I06ca58bcf5dde64b9e8787b4d467a384a960596e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Cleanup restconf-nb dependencies

We have accumulated some warnings, clean them up.

JIRA: NETCONF-773
Change-Id: I69058c298a63b2f084c1ab2cce46474b71bd6c51
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Refactor ReadDataParams

Rename these to DataGetParams in keeping with restconf.server.api
package, where this DTO belongs.

JIRA: NETCONF-773
Change-Id: Ie11b1e09d8a45c392d57e5e69c5857aae88b8935
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Reorder QueryParams

odl-pretty-print is our extension, make sure it is filed in its
appropriate place.

JIRA: NETCONF-773
Change-Id: Ib57987e9cf33fe3e0959c6c4e14d23e94f92406d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Move RestconfSchemaSourceUrlProvider

This component is part of dealing with yang-library content, similar to
what we are doing for ietf-restconf, as both really should be transient
state.

Move it to restconf.server.mdsal, so it is co-located with other
datastore state-keeping components.

JIRA: NETCONF-773
Change-Id: I03da2c230123dbcbe04547e7304413c356f116b7
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Refactor root resource discovery

RootResourceDiscoveryServiceImpl is a mouthful and it is in a legacy
package.

Rename it to JaxRsWebHostMetadata, which is really what we are doing
here. Also eliminate RootFoundApplication by making it an anonymous
class in its sole user.

JIRA: NETCONF-773
Change-Id: I2dcaf136f5e8a324b5f31b72ad9bf5bcd7edbcd4
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove SSEApplication

Use an anonymous Application subclass to tightly bind to the single
user.

JIRA: NETCONF-773
Change-Id: I38ae98f2c8e720785fbd7687ba8766e86419f6f9
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove yang.model.api fakes for /restconf/operations

We are not emitting operatiions content through NormalizedNodes and
thefore do not need to mock anything.

For MD-SAL binding this use case is already handled by the parser,
which creates the magic nodes as needed.

For NETCONF binding this can potentially be lacking, as the remote
EffectiveModelContext may not even have ietf-restconf -- it might be
useful there, but we will re-introduce it if need be.

JIRA: NETCONF-773
Change-Id: Ia3f0f07804702d8046a775634a861e92c1761593
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Clean up ParserIdentifierTest

At this point this class tests InstanceIdentifierContext. Clean it up,
rename it and move it to the same package.

JIRA: NETCONF-1157
Change-Id: I3d0f6836d248a1255aeb619943761162e4e83168
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Eliminate ParserIdentifier

This is a useless class now. Move its sole method into
ParserIdentifierTest pending further specialization.

JIRA: NETCONF-1157
Change-Id: I6ae765a3934cac2d1139056a76d2f197d15bee59
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Eliminate IdentifierCodec

This is now a mostly-useless indirection to
YangInstanceIdentifierSerializer. Eliminate it by inlining serialization
into callers and tests in YangInstanceIdentifierSerializerTest.

JIRA: NETCONF-1157
Change-Id: I6a560a3ce520570e63ea6b4add335bafe36c4438
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Clean up YangInstanceIdentifierSerializer

Turn this class into a proper object, retaining DatabindContext and
having a serializePath() method.

JIRA: NETCONF-1157
Change-Id: I5c1b04346b17274c790fd02eb48cb0b952646f73
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Refactor YangInstanceIdentifierDeserializer

This class is really all about normalizing ApiPath into a
YangInstanceIdentifier (and related information).

Rename it to ApiPathNormalizer and move it to restconf.server.spi, where
so it can be used any and all RestconfServer implementations.

JIRA: NETCONF-1157
Change-Id: I593b5c468650535aca2423724557c0b2aabe8aa0
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove an obsolete FIXME

We already have ApiPath here, the FIXME is no longer relevant.

JIRA: NETCONF-773
Change-Id: I1c46c86d4e84b2193e4ce374cbf5d9b2a81be795
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove unused InstanceIdentifier factory methods

ofMountPoint{Path,Root}() are not used anywhere, remove them.

JIRA: NETCONF-1157
Change-Id: I33a4a653aab4b30add917e07bf7f9ecd25c55dce
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add DataPatchPath

This is a minimal capture of state we need, allowing us to reuse codecs.

Change-Id: Iefa54fef563ab1adac3a0cedd166d5aedbd45588
JIRA: NETCONF-1157
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Introduce DataPutPath

We need a way to identify resources, the PUT operation on top of /data
provides the semantics. This again allows us to share codecs.

JIRA: NETCONF-1157
Change-Id: I7099ce5e48d080b734705005c6331bd9a11a173c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add OperationsPostPath

This is a nice capture of DatabindContext and the logic we need to
decode OperationInputBody -- using which we can reuse JSON/XML codecs.

JIRA: NETCONF-1157
Change-Id: I6a7136c1c54293be22a85ef015992cc2fb349971
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Introduce restconf.server.api.DataPostPath

DataPostPath is a state capture of the context we get parsing the
ApiPath. Define in in restconf.server.api and use it in ChildBody.

This has the immediate benefit of reusing JSON/XML codecs.

JIRA: NETCONF-1157
Change-Id: I4177f963b09a732929c61617f56fd1fb24d55396
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Promote OperationOutput to restconf.server.api

If we rename this to OperationsPostResult, this goes very nicely with
OperationsGetResult.

JIRA: NETCONF-773
Change-Id: Idcc8d43fac0a7b0125dad692a70c60c9f7158bcd
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Use ActionEffectiveStatement.input()

Do not use YangConstants, just refer to the effective model.

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

Promote DatabindContext to restconf.server.api

The furst cut at our output will typically have NormalizedNode
structures and related DatabindContext, so so these can be driven home.

Duplicate DatabindProvider.currentContext() with
DatabindAware.databind().

JIRA: NETCONF-773
Change-Id: I62d10e8a912dfd62bbccaaa0cc52466caca7caa1
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Drop a FIXME

RestconfDocumentedExceptionMapper should end up being transmogrified
into a better error reporting tool. Drop a FIXME to that effect.

JIRA: NETCONF-1188
Change-Id: I58cd32b5c85c92cd3af8979037aa4bc199d25beb
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Use DatabindContext in instance identifier serialization

The first thing this path does is create a DataSchemaContextTree -- so
let's pass a DatabindContext, which has that readily available.

JIRA: NETCONF-1157
Change-Id: Ia66d13e4489b00a28404274c9841f227391876ba
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Use DatabindContext in deserialization path

Rather than passing a naked EffectiveModelContext, from which we create
a DataSchemaContextTree, pass down the readily-available
DatabindContext -- which provides the tree very cheaply.

JIRA: NETCONF-1157
Change-Id: I68c8478845392c9a1e2fc9fbeb92ed4803d93a5b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove InstanceIdentifierContext.getSchemaContext()

Use databind().modelContext() in callers, as they will want to be
shifting to using DatabindContext in some cases.

JIRA: NETCONF-1157
Change-Id: I2ec63f5b8e677f5806dafad5c2076dc8aba8fbf0
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Use DatabindContext in InstanceIdentifierContext

We are moving towards elimination of InstanceIdentifierContext, but
before we do that we need to make sure surrounding code passes around
DatabindContext instead of EffectiveModelContext.

This brings immediate an immediate benefit to YANG Patch body parsing,
as it can pick up the DataSchemaContextTree from DatabindContext.

JIRA: NETCONF-1157
Change-Id: I240cc150a7feab6e7344beefc863eb761d8e30d1
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Eliminate DOMDatabindProvider

We really need to integrate SourceResolve into RestconfStrategy, as we
really need to drive resolution from MdsalRestconfServer (and have
DatabindProvider really work only with EffectiveModelContext).

This also means RestconfStrategy has access to DatabindContext, which
brings immediate benefit of reusing DataSchemaContextTree instances
without a lookup-by-EffectiveModelContext.

JIRA: NETCONF-1157
Change-Id: I6954c1d168fa25d4ce6d261ae1fe1a23b1b0f6b0
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Disconnect WriterFieldsTranslator from InstanceIdentifierContext

We only need an EffectiveModelContext and target node. Communicate them
explicitly.

JIRA: NETCONF-1157
Change-Id: I5ac371b0086b19bcee06b0bfc5f9857267fb599a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add DatabindContext.schemaTree()

We need DataSchemaContextTree during binding of the URL. Retain a
reference in DatabindContext

JIRA: NETCONF-1157
Change-Id: If50b1880299f8b233f60e4198b77f7f11047e1dc
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Move DatabindProvider

Promote DatabindProvider to restconf.server.spi package, which is closer
to where it needs to sit (which is probably server.api, depending on
use).

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

Hide ParserIdentifier

We now only have a single production use or ParserIdentifier. Hide the
class and migrate out-of-package users.

JIRA: NETCONF-1157
Change-Id: I78b246cdb843e10bbb9d661c7a36aba1b4c3f46e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove ParserConstants

This class is not referenced anywhere, remove it.

JIRA: NETCONF-773
Change-Id: I3075910010fb9ca4dc48894a83b1ac9668c1a549
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Clean up RestconfSchemaSourceUrlProvider

Explicitly annotate injection constructor and instantiate it only when
needed.

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

Refactor modulesGET() methods

Make sure mountPath is passed to RestconfServer as ApiPath and make
source revision passed as a query parameter. This ends up
eliminating one reference to ParserIdentifier.

JIRA: NETCONF-1157
Change-Id: I4de5293126d3dc0c68b8ee132e5bfa56a1264008
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Eliminate RestconfSchemaServiceImpl

RestconfSchemaServiceImpl is serving YANG/YIN schema sources. While this
capability is not standardized, integrate it into RestconfServer as
modules{Yin,Yang}GET().

This turns out to be a major refactor, as the implementation was shoddy
-- it required local DOMSchemaService to expose
DOMYangTextSourceProvider extension and then it used that when talking
to mount points.

This refactor integrates the ability to lookup sources into
DatabindContext, with the corresponding DOMDatabindProvider wiring. If
no DOMYangTextSourceProvider is present, we call back to using
YangTextSnippet.

JIRA: NETCONF-773
Change-Id: I57689b2915eda0486c0bb5d5ba6de992b3107c0b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix channel inactive handling for SSH transport

Channel inactive handler require to re-transfer the event
down the pipeline in order for subsequent handler to be
triggered as well. SSH transport pipeline did contain two
channel inactive handlers both terminating the event
resulting *NetconfSession not handling channel termination
properly.

Fixed, case covered by test.

JIRA: NETCONF-1200
Change-Id: I2a78709909e1b8509b562a8fa0bf640c45c91432
Signed-off-by: Ruslan Kashapov <ruslan.kashapov@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Eliminate JaxRsApiPath

Bouncing through JaxRsApiPath is not entirely optimal, as we need to
expose that class -- which really is an implementation detail.

Eliminate the class in favor of JaxRsRestconf implementing
ParamConverterProvider for ApiPath.

While we could provide this support as an external component, inlining
it in JaxRsRestconf, which is the only relevant user, has the added
benefit of suppressing Jersey warnings about the resource not providing
any provider interfaces.

JIRA: NETCONF-1157
Change-Id: Id3d1cc5acf764bc03a46f792b3d8c2bf72c5bd85
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix /operations indexing

Guava's Table does not work for indexing here, as it cannot support null
keys, which we require for storing Revision. Switch to using explicit
nested maps.

JIRA: NETCONF-773
Change-Id: I5812191a070eb9e631afd962ecf1d99e537f3fc3
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add FIXMEs

ETag and Last-Modified should be part of the response payload maintained
on RestconfServer side. Add FIXMEs to reflect that.

JIRA: NETCONF-773
Change-Id: Ice8369fb6545569477a137079691c79e55f32927
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix operations{Json,Xml}GET definition

AsyncResponse should be @Suspended, otherwise we get a warnings from
Jersey that "A HTTP GET method, ..., should not consume any entity."

JIRA: NETCONF-773
Change-Id: I6ad989ae7bd8cd6a0303945d2bf6040c6fea8a5b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix JaxRsApiPath declaration

We are using this class to define our mapping, which means its
constructor must be public. Fixes Jersey reporting "No injection source
found for a parameter of type" fatal message.

JIRA: NETCONF-1157
Change-Id: I75031376ce3ee453d9a5d60b7917c34e57c831fe
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

ApiPath is a HierarchicalIdentifier

ApiPath is essentially a counterpart to YangInstanceIdentifier -- hence
it should provide same services.

JIRA: NETCONF-773
Change-Id: I1d35e43723c484a99c2630f81129ee43222c60b7
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Switch RestconfSchemaServiceImpl to RestconfFuture

We want to have async execution as we really can execute async method --
for example from SchemaSources. This is the first step, converting to
use RestconfFuture.

JIRA: NETCONF-718
Change-Id: Icf1c6bb505ba8dc37d78a248fe400690d49dfc3b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Relocate ParserIdentifier methods

Vast majority of methods here are implementation details of
RestconfSchemaServiceImpl. Move them there.

JIRA: NETCONF-773
Change-Id: I95836e3f6ee99f91c755f19ced3b3205ab2f7b56
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Clean up SchemaExportContext

SchemaExportContext is an output payload and we certainly can implement
it better than we have so far.

This is a first pass doing two things:
- disconnecting SchemaExportContext from mdsal.dom.api
- switches to use EffectiveStatements instead of SchemaNodes

Change-Id: Ic8cba496d8667fedffb92f5c1c482b5070719f01
JIRA: NETCONF-773
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate RestconfValidationTest to JUnit5

Migrate tests cases and improve assertions.

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

Eliminate nb.rfc8040.rests.services.api

This package contains just a specification and we can simply inline it
-- and move SchemaExportContext to nb.rfc8040.legacy pending further
refactor.

JIRA: NETCONF-773
Change-Id: Ia11b36dd66ca050cf8213d87e8d281250cd4de99
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

RestconfServer requires ApiPath identifiers

Parsing the request URL is the responsibility of the caller, hence we
should accept an ApiPath, not a plain String.

JAX-RS implementation uses JaxRxApiPath's default constructor to bind
with @PathParam semantics and report BadRequestException if anything
goes wrong.

JIRA: NETCONF-1157
Change-Id: Ie9dc79d30cbb4ea52643403f2762090a5e3b3aeb
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove ParserIdentifier.makeQNameFromIdentifier()

This method is used only in its associated tests, remove it and the
tests.

JIRA: NETCONF-1157
Change-Id: I15dc458ff6153376fd3eec22ab7cd6abd4e6e204
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Clean up ParserIdentifierTest

Clean up and improve assertions.

JIRA: NETCONF-1157
Change-Id: I15559113861fadd09f0feb8ad2781901f9d507dd
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Improve RestconfSchemaServiceTest

Add more assertions and perform general clean ups.

JIRA: NETCONF-1157
Change-Id: I3decca5b6dfeca2ab35adebbafdc73b086dbdb07
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Create tests for PUT "insert" query parameter

Created UT for PUT requests with: insert=last, insert=first,
insert=before, insert=after query parameters.

JIRA: NETCONF-1163
Change-Id: I80f55f0d3f5d91535dccce7c43e7581b3de6feb5
Signed-off-by: Yaroslav Lastivka <yaroslav.lastivka@pantheon.tech>

Use Timer in NetconfNodeHandler

We are using ScheduledExecutorService to scheduler timers. Since we are
already passing down Timer to other timeouts, also migrate
NetconfNodeHandler to use it.

JIRA: NETCONF-1106
Change-Id: I7b231cd4c91b67eff8c8937124a04338ffc8bbe0
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Eliminate space escaping

We want to be spec-compliant, and a simple space is not a special
character, so we should not be escaping it on output.

Fixup tests which require us escaping spaces and convert them to JUnit5.

JIRA: NETCONF-1157
Change-Id: Id550a4f9d17b77bec3fbb12de89bbf1f775b8e8f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Use Timer for KeepaliveSalFacade

Our timers typically do not get to fire, which means our performance is
dominated by task setup/teardown.

Improve this by using a Netty Timer, which is optimized for timeouts
which do not typically occur.

JIRA: NETCONF-1199
Change-Id: Iabb89f863a8c447f0e9fcc6981127893f8aa255f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Establish ietf-yang-library revision from model context

Rather than hard-coding, report the latest revision available in the
context.

JIRA: NETCONF-773
Change-Id: I0981b6efba576b393eea1f43db5417df4cf3b527
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

Add ApiPath.toString()

We are about to push ApiPath deeper into stack. This requires reporting
ApiPath values to end user -- for which we need a toString() method.

This rehosts PERCENT_ESCAPER, as we really need it to emit key values.

JIRA: NETCONF-1157
Change-Id: Ia602dacc78eb9f6b0edd9801998dcc5bd60817ea
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

OpenApi: Do not introduce JUnit 4

Do not use JUnit 4 AfterClass annotation with our JUnit 5 test cases.

JIRA: NETCONF-938
Change-Id: Ie959671a110ba37da6f87410edbd4493766d6612
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

Update ietf-yang-library to 2019-01-04

Update ietf-yang-library to 2019-01-04 where needed.
It is also required to add ietf-datastores, ietf-inet-types
and ietf-yang-types as its dependency.

Remove ietf-yang-library YANG model where the module is not needed.
It means its not required in effective model context and/or not
expected by tests.

Change-Id: I34c854cebabeedd19ce4ec3a5ab1d7ef2eb91b24
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

OpenApi: Rework tests to parameterized

Reworked tests which are expecting JSON output to paramaterized and
reduced code duplications.

JIRA: NETCONF-938
Change-Id: I98d5d4725811da42c630f779f65d4c447afc7fdc
Signed-off-by: lubos-cicut <lubos.cicut@pantheon.tech>

OpenApi: Improve test coverage

Created new tests which are expecting JSON outputs. Created suites for
models we are using in current unit tests.

JIRA: NETCONF-938
Change-Id: I8445ac3754b9e4cb604be36f517f1b75abfc27e6
Signed-off-by: lubos-cicut <lubos.cicut@pantheon.tech>

RestconfServer is strictly asynchronous

Make sure all methods are returning RestconfFuture, meaning everything
is potentially asynchronous and every call fail in a well-documented
way.

Take the time to create an assertion framework in AbstractRestconfTest,
which makes testing easy and understandable.

JIRA: NETCONF-718
Change-Id: I852f78568ed14ba527284cf835bfdee5ec024452
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Annotate RestconfFuture returns

Both of() and failed() are returing a non-null instance, annotate that
fact.

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

Move DOMDatabindProvider

DOMDatabindProvider is a logical part of MD-SAL-based server, move it to
server.mdsal. Also document restconf.server.mdsal package a bit.

JIRA: NETCONF-773
Change-Id: Idf72b75eeadf9953d69caf5925f276261e380600
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Split up MediaTypes

We really want two sets of independent constants:
- plain Strings in restconf.api.MediaTypes
- JAX-RS MediaTypes in restconf.nb.jaxrs

JIRA: NETCONF-773
Change-Id: I11fcf0694bd2d7d3d8f75024b3aa5a43f84c6b31
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Introduce restconf.nb.jaxrs package

We have reached a place where we can clean up JAX-RS northbound
components without worrying about weird dependencies.

Refactor RestconfImpl into restconf.nb.jaxrs.JaxRsRestconf.

JIRA: NETCONF-773
Change-Id: I75dbe87f2276be12157616779ec78b988863daac
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Move MdsalRestconfServer

MdsalRestconfServer is now properly isolated, move it to
restconf.server.mdsal.

This hides getStrategy() from tests, which is forcing us to convert them
to proper end-to-end integration tests.

JIRA: NETCONF-773
Change-Id: I5e2729f2b9113d610886972cb7a50f28e40bcd4e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Introduce RestconfServer.dataPOST()

POST on /data has two modes, Invoke Operation and Create Resource,
which we can neatly capture in RestconfServer.

This eliminates the final bits of RestconfDataServiceImpl, completing
indirection through RestconfServer.

JIRA: NETCONF-773
Change-Id: I7338d6f49688bc2769a1fc5e952553478566965a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add DataPostResult

POST to /data can result in multiple different results. Add
DataPostResult encapsulate the possiblities.

Also introduce CreateResource, which is the result of POST in Operation
Create Mode -- moving some of the worries away from JAX-RS layer's
postData().

JIRA: NETCONF-773
Change-Id: I56dcde72630d53290f002e9b6258924175bb3072
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

More NetconfCommandUtils cleanup

We have forgotten a commented-out block of code. Eliminate it and
simplify isIpValid().

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

Rename OperationsContent

A better name for this interface is OperationsGetResult, mirroring
DataPutResult.

JIRA: NETCONF-773
Change-Id: Ic00d867373fd635ac2d7de78d6d87100a8a433c3
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Introduce RestconfServer.dataPUT() operations

Migrate PUT operations on /data to MdsalRestconfServer/RestconfImpl.
This temporarily duplicates Insert parsing logic, which will be
eliminated in a follow-up patch.

JIRA: NETCONF-773
Change-Id: I8692fede3238cdcca283dfc7d9bc8e123b18aea7
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Introduce RestconfServer methods for YANG Patch

YANG Patch is a quite simple request, easily defined by RestconfServer.
This allows us to rehost JAX-RS methods to RestconfImpl.

JIRA: NETCONF-773
Change-Id: I8532685a52f7b4989125f52c758c956faebe41ac
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Clean up getStatusCode()

Do not use else branches and simplify error evaluation. Rename it ot
statusOf() and put it into JaxRsCallback which is using it.

JIRA: NETCONF-773
Change-Id: I8e40f87f174a1ab62b6c5d9188f202a5f85f6028
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add RestconfServer.dataPATCH()

Plain patch method is a rather simple operation, define it in
RestconfServer and migrate JAX-RS implementation to RestconfImpl.

JIRA: NETCONF-773
Change-Id: I26be45eb6eaa77d52df6197bcb43a6d0ee12df94
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add RestconfServer.dataDELETE()

Add a the delete operation and move its implementation into
RestconfImpl. Also unify test mocking.

JIRA: NETCONF-773
Change-Id: Iecbe8219fdd72266b78e466859b593ff341f177b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Move bindResourceRequest()

This is a shared utility method destined to become an implementation
detail. Move it to MdsalRestconfServer.

JIRA: NETCONF-773
Change-Id: I819412e8dc9cabffaf01558b658ee5f6b892a3cd
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Move /data GET methods to RestconfImpl

The methods are extremely simple wrappers around RestconfServer, move
them to RestconfImpl. Also splits up the related tests, migrating them
to JUnit5.

JIRA: NETCONF-773
Change-Id: I73bbbadcdd02cbca703cae290e3e779911c2295a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Introduce asynchronous RestconfServer.readData()

Refactor logic so as to reside in MdsalRestconfServer, interfaced with
RestconfFuture -- which is forcing us to use AsyncResponse.

JIRA: NETCONF-773
JIRA: NETCONF-1155
Change-Id: Idc8207f0c2ac4486f9382d1649131852e76e5965
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Move action invocation to MdsalRestconfServer

This code is quite clear-cut and JAX-RS free, move it to
MdsalRestconfServer. The actual interface is still TBD.

This allows us to clean up JaxRsNorthbound of quite a few unnecessary
services.

JIRA: NETCONF-773
Change-Id: If030a80eaab55dd4cb5ccc6889c94b7cc06e97cd
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Eliminate RestconfOperationsServiceImpl

This class is a simple forwarded to MdsalRestconfServer, just like
RestconfImpl. Move all its methods to RestconfImpl, improving their
naming consistency.

JIRA: NETCONF-773
Change-Id: I04306d99473fcbbe8474026684b4d185a4cedb74
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix single-item OperationsContent

We are returning the wrong value when asked for a specific operation.
Fix that. Since we are here, also simplify XML emission to use simple
entries with default namespace -- making the output more consistent
between queries.

JIRA: NETCONF-773
Change-Id: I4cbb67278b9e2227f9b1b9272f066942d4c9108f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Refactor OperationsContent

Turn the enumeration into a plain DTO which can serialize itself into
XML/JSON -- allowing it to be promoted to restconf.server.api.

While we are in the area, fix the semantics here:
- do not consider ActionEffectiveStatement, as that is handled through
  a different service
- report 404 when the operation points to a non-RPC

JIRA: NETCONF-773
Change-Id: I69fd5865c89ea2b727befdf4cedb8cddf8418002
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Rehost OperationsContent.bodyFor()

Rehost formatting logic to MdsalRestconfServer, reducing API/impl
friction.

JIRA: NETCONF-773
Change-Id: Ib0fdebad4ff8c7f4309eace3e177707a873288bb
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Codify operationsGET

GET on /operations is well-defined, express it in RestconfServer,
migrating callers of OperationsContent.bodyFor().

JIRA: NETCONF-773
Change-Id: I64b713fdddac2dc3b1e4166a26ea554d23e3cfa1
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Document RestconfServer.invokeRpc()

This should be called operationsPOST() to keep with flexible naming.
Document the behaviour and tie it to RFC8040.

JIRA: NETCONF-773
Change-Id: I05f77706de74e7fb2af57060317246d7773e92ad
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Codify RestconfServer.yangLibraryVersionGET()

The inquiry to the supported ietf-yang-library version is a well-defined
RESTCONF operation. Codify our implementations through RestconfServer.

JIRA: NETCONF-773
Change-Id: Ie969a9d4897c232769005d3c628826401febecbe
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Clean up wadl-generator

Migrate to JUnit5 and clean up reported issues.

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