Refactor NormalizedNodePayload

Turn this class into a record, with complete non-null semantics.

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

Fix action invocation response

We should be checking for null here as well, which eliminates the last
user of NormalizedNodePayload.ofNullable().

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

Eliminate NormalizedNodePayload.headers

Convert the only user working with this facility to return a Response,
where it can set the location header directly.

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

Do not use InstanceIdentifierContext in postData()

The variables being passed are trivial, refactor internal wiring to pass
appropriate variables instead.

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

Reduce InstanceIdentifierContext proliferation

Use local variable type inference to ease future migrations.

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

Do not use InstanceIdentifierContext in NormalizedNodePayload

All writers have been proven to just need the Inference, ditch the
useless indirection through InstanceIdentifierContext.

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

Refactor AbstractNormalizedNodeBodyWriter

AbstractNormalizedNodeBodyWriter so that the three common dispatches are
handled in common code. This also modernizes them and pulls common bits
together.

This allows us to also switch the dispatch logic, so it operates on
SchemaInferenceStack instead of on SchemaNode.

All of this ends up showing that there is a bug in operation output
code, which causes us to emit an unnecessary declaration.

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

Do not use InstanceIdentifierContext in OperationsContent

We are really operating on an Inference, remove the indirection.

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

OperationsContent requires a context

All callers are guaranteed to have a non-null context, just ditch the
check.

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

Share common parts of AbstractNormalizedNodeBodyWriter

The two specializations are doing similar things, let's move them closer
together by sharing the common bits. This has a nice effect of reducing
JAX-RS exposure a tiny bit.

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

Clean up XmlNormalizedNodeBodyWriter

We are hunting down the actual state used from
InstanceIdentifierContext, this patch gets us a bit closer and it also
shows we have some oddities going on.

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

Clean up JsonNormalizedNodeBodyWriter

We are hunting down the actual state used from
InstanceIdentifierContext, this patch gets us a bit closer and it also
shows we have some oddities going on.

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

Fix JsonPatchBody javadoc warnings

We are using javadoc for private methods. That is wasteful, as the only
real information is a single sentence. Turn that into an ordinary
comment.

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

Remove InstanceIdentifierContext from PatchBody parsing

We only need the path encoded from URL and the EffectiveModelContext.
This also splits the parsing and dispatch to simplifying the code flow.

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

Move InstanceIdentifierContext

This is a legacy container, move it to restconf-nb, which is the only
place where it is being used.

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

Disconnect PatchStatusContext from InstanceIdentifierContext

All we need here is a reference to EffectiveModeContext, but later we
should be getting a DatabindContext.

Also highlight an encoding problem, where we are doing the wrong thing
based on YangInstanceIdentitier.toString().

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

Migrate Collections references

Use List.of() instead of Collections. Also be smarter about treating
arrays.

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

Improve NetconfDataTreeService documentation

Remove XML escapes and spring {@code} elements.

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

Add description to POST requests

Add description to POST request (paths/…/post/description)
that states that user in example payload sees only the first
data node child of the resource to be created and that
this is in accordance with RFC 8040 which allows us to
create only one resource in POST request.

JIRA: NETCONF-1137
Change-Id: I8549cf860deb5e069a017c9e7247c0bce311e588
Signed-off-by: lubos-cicut <lubos.cicut@pantheon.tech>
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

OpenApi: Remove incorrect list POST requests

Our logic creates POST requests for lists with keys, containing
multiple resources in payload. This does not align with the expected
structure for creating child resources in the YANG model.

Added condition that prevents the creation of POST requests
that has list as a last element.

JIRA: NETCONF-1101
Change-Id: I46820f222c9c5ef8078ac8675d2adc12b06f1253
Signed-off-by: Yaroslav Lastivka <yaroslav.lastivka@pantheon.tech>
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

Refactor OpenApiTestUtils methods

Change getPathParameters into getPathGetParameters and add
getPathPostParameters.

We assume that all paths present in OpenApi should have at least
GET example in the documentation.

Thus we can use GET operation as a base operation to get number of
parameters in path. We just need to drop 'content' parameter.

For actions and RPCs we can use getPathPostParameters method.

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

Skip structural nodes

We cannot just examine the content being changed, as this could lead
to us trying to report the entire MapNode disappearing -- which does not
have a instance-identifier representation.

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

Improve RestconfStrategy API surface

We have a number of methods which are only used internally, which formed
our (ugly) API surface. Hide them so we are free to evolve them.

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

Use NormalizedNodeStreamWriter to emit XML data

We are already instantiating NormalizedNode-based writer, which means we
get help in terms for formatting things. This is especially true of
identityrefs and instance-identifiers, where the XML codec does
absolutely the right thing.

This also allows us to ditch namespace repairing from our
XMLWriterFactory, so we are in explicit control of our namespaces.

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

Use NormalizedNodeStreamWriter to emit JSON data

We are already instantiating NormalizedNode-based writer, which means we
get help in terms for formatting things.

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

Eliminate (ReadData)TransactionUtil

Integrate these utilities directly into RestconfStrategy.

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

Cross-relate WithDefaultsParam

RFC6243 defines with-defaults-mode typedef, tie it together with
WithDefaultsParam. This improves interoperability with NETCONF.

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

Inline readAllData()

The two methods have simpler callers, remove an indirection by inlining
the code in the corresponding lambda.

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

Remove extractReadData()

This utility function is tempting, but inlining will allow us to take on
the transaction callers separetely.

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

Eliminate PatchDataTransactionUtil

This class is hosting a single method, which is strongly related to
RestconfStrategy.

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

Rehost PatchDataTransactionUtil logging

We have simple wrappers that just log stuff and forward execution to
RestconfTransaction. Rehost these to final RestconfTransaction methods,
so we have greater traceability coverage.

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

Clean up PatchDataTransactionUtil

Clean up code dispatch to simplify rehosting these utilities.

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

Hide RestconfTransaction.readList()

We have completed migration of utility methods which refer to this tiny
bit. Hide it from outside world, as it is an implementation detail.

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

Refactor WriteDataParams

The empty holder is useless, as we can model the same thing with a
@Nullable parameter.

Rename WriteDataParams to Insert and make it nullable. This improves
dispatch in users and will allow us to further remodel this contract.

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

Optimize RestconfStrategy.postData()

The implementation is now tightly bound to a RestconfStrategy, which
allows us to check invariants before we embark on modifying the
datastore.

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

Clean up documentation

Do not reference DOMTransactionChain, we are not using it.

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

Eliminate PostDataTransactionUtil

These utilities are tightly coupled to RestconfStrategy, rehost them
there.

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

Restructure putData() implementation

Our new how in RestconfStrategy allows us to restructure how we go about
executing this operation. The new layout checks the paramteres before
allocating a transaction and nicely differentiates the two distinct code
paths based on insert parameter.

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

Eliminate PutDataTransactionUtil

The implementation here is inherently tied to RestconfStrategy. Rehost
methods to RestconfStrategy and PostDataTransactionUtil (for now).

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

Move CreateOrReplaceResult

This is the result of a create-or-replace operation, which we will be
introducing in RestconfStrategy. Move the class there for co-location.

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

Switch netconf-server to transport-api

New transport implementation makes NetconfServerDispatcher outdated.
NetconfServerFactory interface and implementation introduced
as replacement. Netconf subsystem factory added to serve netconf
over ssh.

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

Fix postData() definition

The version which posts to top-level of datastore should not take an
identifier. Fix that, preventing an error.

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

Capture ListenersBroker instances

Rather than having random code call out to
ListenersBroker.getInstance(), make sure we get the reference in outer
scope, mostly in a field.

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

Do not serialize the unchanged for child-nodes-only

Subscribing data change notification with the child-nodes-only mode may
generate notifications with unnecessary data change events for the list
key nodes in the data path.

Serialize only data tree nodes with effective changes. This filters out
unnecessary data change events and prevents notification of data updates
without any real changes.

Also, add missing test cases for the child-nodes-only mode.

JIRA: NETCONF-1153
Change-Id: I2ed388ee2e5c364a96240d8b5a519651cf2f5b47
Signed-off-by: Sangwook Ha <sangwook.ha@verizon.com>

Remove unused InstanceIdentifierContext methods

With NormalizedNodeBodyReaders gone, we have sufficiently trimmed the
variability on InstanceIdentifierContext invocation paths.

This patch removes almost all of the weird callers who do not provide an
actual YangInstanceIdentifier.

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

Remove rpc-{input,output}.json

These resources are not used anywhere. Remove them, but retain the
content of rpc-input.json in the expanded JsonOperationInputBodyTest.

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

Remove rpc-{input,output}.xml

rpc-input.xml is used in only a single place, inline it. rpc-output.xml
is not used, as we are asserting just the NormalizedNode structure,
hence it is removed.

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

Remove xmldata.xml

This is a single-use simple resource, inline it.

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

Remove bug7933.xml

This is a single-use simple resource, inline it.

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

Inline jsondata.json

This is a simple document used in a single place, inline it.

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

Remove AbstractNormalizedNodeBodyReader

We are using this reader only in postData(). Refactor users to
centralized dispatch and move the logic to ChildBody and its
specializations.

This refactor also improves dispatch to datastore resource, as we know
that such POST requests are not action invocations.

Invocation path is using OperationInputBody, so as to share decoding
with the RPC invocation path.

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

Fix stream prefix

The change to tie together the strings has missed a path component, fix
that.

JIRA: NETCONF-1151
Change-Id: I09b3fa5d201c5316d0a8a63fa9c6f1b2d2fbaedc
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 1b88bdb4cde0c2f04ec7a235a887cb8889ffb6c2)

Centralize notification XML namespace

We have a string literal which is not tied to a standard. Replace it
with a NamespaceURN.NOTIFICATION reference.

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

Clean up JSON*Formatter.createText()

Use proper try-with-resources blocks and take advantage of method
returns.

Also improve element ordering, with eventTime being emitted before the
body.

Finally we tie the constants used to model and highlight the problems
that are in the current implementation.

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

Refactor EventFormatter interface

We are passing a ton of parameters around, where they are really state
captured in EventFormatter, but passed to AbstractWebsocketSerializer.

Split the four booleans we dance around into a separate TextParameters
record, which gets stored in each EventFormatter and passed to the
serializer as appropriate.

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

Optimize AbstractWebsocketSerializer.convertPath()

The serialization has a couple of classic duplications. Eliminate them
through simple addition of appendQName() utility method.

Also drop FIXMEs to examine the output, as it seems extremely fishy:
the namespace is a URN, which means ':' and '-' are not the same thing
and namespaces can very much differ by these.

Also key values are straight toString()'d which means QNames and similar
go through ODL-specific encoding.

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

Add Child Nodes Only query parameter to SSE events

For SSE or Websocket, events are reported from the root node of the listen
Data Tree or from the modified leafs with the 'odl-leaf-nodes-only=true'
query parameter. This patch introduce a new 'odl-child-nodes-only' query
parameter to get only modified children in a compact form. This new option
could be considered as something in the middle between getting the whole
Data Tree from the root node and a per leaf node report.

JIRA: NETCONF-1076
Change-Id: Ibeda878a38a52f911dd8fbaa0503c4bfb56ea9b7
Signed-off-by: Olivier Dugeon <olivier.dugeon@orange.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Clarify NotificationQueryParams checks

Using checkArgument() involves logical negation of the condition with
respect the message. This can get quite daunting when trying to
understand.

Switch to using plain if() statements, which make the logic much
clearer.

Also tie the messages to uri names, so users get clearer guidance.

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

Fix stack mismatch in data change event streams

Data change event stream targeting a list instance requires a special
treatment to avoid mismatch between the writer stack and data.
Wrap the map entry in a map node to avoid the stack mismatch and
serialization error.

Also, add test cases to cover the use case to ListenerAdapterTest and
refactor/clean up the test class to reduce duplicate code and make file
& variable names more consistent.

JIRA: NETCONF-1051
Change-Id: I0e266f04fc77db6add4f1c44e6fd47f208c2e2f9
Signed-off-by: Sangwook Ha <sangwook.ha@verizon.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Refactor OperationInputBody

Add utility to extract the ContainerNode rather than streaming them.
Also detect empty body.

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

Fix XmlResourceBody builder failure

The migration patch was not rebased, leading to a mismatch in thrown
exceptions. Fix that up.

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

Specialize RestconfDataServiceImpl.{put,plainPatch}Data()

This peels another two uses of *NormalizedNodeBodyReader, where the operation
is:
- create-or-replace, i.e. PUT
- merge, i.e. plain PATCH

In both cases the body is the resource being acted on. These body types
relate directly to target resource, hence they are represented through
ResourceBody.

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

Require parent topology to have been created

There is no point in re-issuing empty merges for topology, just require
it to have been created beforehand (which we already require in the
constructor).

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

Properly synchronize NetconfDeviceTopologyAdapter

The adapter is using TransactionChain, which should always be guarded.
Make sure we guard that even when callers fail to provide their own
guards.

There is a further bug, where a failed last transaction would not
complete the closeFuture(), leaving callers stack.

Finally we disconnect from AutoCloseable and provide a shutdown()
method, which allows users to decide on how to synchronize the datastore
access. This allows us to concurrently shutdown the mount point and
datastore state and wait for the completion of the latter only after
both actions have been initiated.

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

Bump upstreams

Adopt:
- odlparent-13.0.4
- infrautils-6.0.2
- yangtools-11.0.1
- mdsal-12.0.1
- controller-8.0.1
- aaa-0.18.1

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

Fix checkstyle

Upgraded checkstyle is flushing out new things, fix them up.

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

Add NetconfMessage.of(Document) method

Added a new static NetconfMessage.of(Document) factory method, which
takes a document as a parameter and based on its content returns a
correct NetconfMessage subclass or throws an exception.

JIRA: NETCONF-1014
Change-Id: I55d3c29c01df5c7d34d53f372a80729ec129a22c
Signed-off-by: matus.matok <matus.matok@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix netconf-server-mdsal dependencies

We have used undeclared dependencies, fix that.

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

Rework NetconfStartExiMessage

Renamed NetconfStartExiMessage to NetconfStartExiMessageProvider as
it iss no longer a subclass of NetconfMessage. but instead it provides
a create() method that returns an RpcMessage with specific payload.

JIRA: NETCONF-1014
Change-Id: I9ab387523663854fd40bcd831bd00c0940467780
Signed-off-by: matus.matok <matus.matok@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Make NotificationMessage constructor private

Made NotificationMessage constructor private and added a new static
factory method, ofNotificationContext(), with the same semantics.

This enables adding another static method, where the document will
not be wrapped, if it already represents a valid NotificationMessage.

JIRA: NETCONF-1014
Change-Id: I74ca0bb09be2652d5179956e5b54c17d9c655ab0
Signed-off-by: matus.matok <matus.matok@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add is{Notification,Rpc,RpcReply}Message methods

Added public static methods is{Notification,Rpc,RpcReply}Message(Document) methods
that return true, if the document in the parameter is a NotificationMessage,
RpcMessage or RpcReplyMessage respectively. To be used in the static method
NetconfMessage.of(Document).

JIRA: NETCONF-1014
Change-Id: I61da444487bafcf9bc5e15cf8dbf8b9fa978218e
Signed-off-by: matus.matok <matus.matok@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Introduce RpcMessage and RpcReplyMessage

Introduced RpcMessage and RpcReplyMessage both extending NetconfMessage, with
basic wrapping functionality. Added tests to verify correct wrapping.

JIRA: NETCONF-1014
Change-Id: I72d5d03dde6553201d2fca4db1faf5d0a844d872
Signed-off-by: matus.matok <matus.matok@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Assert reported error

We are asserting that an error is throw, but we are not making it clear
which error should be reported. Check all errors for their
getErrorMessage().

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

Clean up JsonBodyReaderTest

Introduce useful utility methods.

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

Clean up RestconfDataServiceImplTest

Constify test data, so that we have proper connections to the model,
reusing constructs where possible.

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

Inline XML patch payloads

Having external XMLs is not nice, inline them as multiline strings.

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

Make resource access explicit

Rather than providing an InputStream, introduce a method which takes
the resource name and acquires that way.

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

Migrate more JSON patch payloads

Inline test resources as string literals, making tests a lot cleaner.

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

Improve reader mocking

Rather than passing a boolean flag, split POST/PUT method mocking, so as
to identify callers relying on either.

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

Eliminate AbstractBodyReaderTest.mediaType

This field is used only by concrete tests, where its content can be made
constant. Performing the trivial conversion eliminates a constructor
upcall.

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

Introduce AbstractBody

We need to be careful about managing our InputStreams, hence unify
handling between PatchBody and StreamableOperationInputBody.

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

Inline AbstractIdentifierAwareJaxRsProvider

This class has only AbstractNormalizedNodeBodyReader as its subclass,
merge the two together, allowing us to further evolve interactions.

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

Eliminate AbstractPatchBodyReader

This patch refactors RestconfDataServiceImpl to not rely on JAX-RS for
parsing YANG Patch body. Instead of that we introduce PatchBody backed
by an InputStream and two specializations to handle parsing from
JSON/XML.

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

Disconnect JsonPatchBodyReader state

The parser bits are independent of the class, make sure have that
expressed.

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

Peel out AbstractInstanceIdentifierTest

We are parsing the same context in three different tests. Centralize its
creation.

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

Clarify patchData() naming

We have three methods with deal with HTTP PATCH, but they are defined in
different standards. Make this obvious in method names, lowering
reader's mental overload.

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

Inline some JSON patch test files

There is just no point in keeping these files separate, integrate them
into tests.

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

Unify *PatchBodyReader(MountPoint)Tests

We have thorough duplication of tests here, refactor the class hierarchy
to keep us from duplicating things.

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

Introduce AbstractPatchBodyReaderTest

Eliminate the explicit dependency of AbstractBodyReaderTest on
PatchContext by introducing an intermediate superclass.

Also clean up references for resource loading, using the local class
name instead of a superclass.

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

Clean up moduleDataTest()

Make sure this test is as close as possible to its non-mounpoint
counterpart.

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

Add global description

Add global description (info/description) to OpenApi that
states that we are providing full API for configurational
data because only those can be edited (by POST, PUT, PATCH,
DELETE) and for operational we only provide GET API.

In addition, for majority of request user can see only
config data in examples. That’s because we can show only one
example per request.

The exception when user can see operational data in example
is when data are representing operational (config false)
container with no config data in it.

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

Eliminate TestUtils

The utility methods in this class are not used anywhere: eliminate the
entire class.

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

Eliminate TestUtils.loadSchemaContext()

There is no need to have this particular method, as we have a
replacement readily available. Also speed up the sole callers by
unifying them in an abstract superclass.

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

Eliminate TestRestconfUtils

This class is largely unused. Move the single used method to
AbstractBodyReaderTest, adjusting its users.

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

Eliminate schemaContextLoader()

This is a superfluous method, replace it with a proper constructor
defaulting to the shared baseline context.

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

Remove superfluous throws

There is no point in declaring reflection-related exceptions as thrown,
since we never report them. Clean this particular mess up.

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

Reduce TestRestconfUtils proliferation

We are parsing a single resource directory, use the appropriate tool
from yang-test-util.

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

Speed up XmlPatchBodyReader

Use local variable type inference to improve binding to
implementations. Also use instanceof pattern to skip duplicate item
lookups.

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

Remove unused jsonPatchBodyReader.readFrom()

This method is completely unused, remove it.

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

Fix restconf capabilities writer

Using incorrect type of leaf set causes validation failure
on attempt to persist capabilities to datastore. Also fixed
typo in method name.

JIRA: NETCONF-668
Change-Id: Ia87102b069a5a42b69a29c462e912775154c2be7
Signed-off-by: Ruslan Kashapov <ruslan.kashapov@pantheon.tech>