Remove org.jboss.resteasy/jaxrs-api dependency

This dependency is superseded by javax.ws.rs-api, use that instead
of this relic.

JIRA: ODLPARENT-196
Change-Id: Ie34e98bd2e9a93a4242cd32e89082d1e0528849d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove use of Throwables.propagate()

We can throw a checked IOException, do not use a runtime exception
to report errors.

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

Merge "Do not check whether two nodes can be merged with reference comparison"

Do not check whether two nodes can be merged with reference comparison

These need to be compared with .equals().
Also fixup the wording slightly.

Change-Id: Ib723a9a2de9810234bdbd725326d6ebd7c4c4151
JIRA: NETCONF-600
Signed-off-by: Tomas Cere <tomas.cere@pantheon.tech>

Cleanup BaseSchema

There is no need to throw ExceptionInInitializerError, as that
will be propagated by the class initializer.

Also let's not use Lists.newArrayList(), but rather use variadic
arguments and wrap them with Arrays.asList().

Finally make BaseSchema implement SchemaContextProvider, so that
its getSchemaContext() method is properly documented.

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

Optimize NetconfMessageTransformer.toRpcRequest()

YangConstants provides a utility input QName factory method,
which is slighty more efficient than QName.create(), hence we use
that. Since the resulting QName is only used for lookups, we do
not intern it (it is short-lived anyway).

When we are determining the RPC definition, we unnecessarily
consult mappedRpcs twice -- store the result in a local variable
to make it a bit faster.

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

Merge "Include namespace in output element of json rpc responses"

Fix preparing of action request

When the YangInstanceIdentifier used for creation of action request
contained mixin path argument (e.g. list as whole), the list ID was in
request twice. Once for list as whole and once for the list entry.

Change-Id: Ic6507c0137abedb6c87718c4a536c983033488f6
JIRA: NETCONF-566
Signed-off-by: Anna Bencurova <Anna.Bencurova@pantheon.tech>
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>

Include namespace in output element of json rpc responses

According to the rfc these should be present.

JIRA: NETCONF-592
Change-Id: I461a04726ce8ab5bd4963902070000f5f78d5d45
Signed-off-by: Tomas Cere <tomas.cere@pantheon.tech>

Add shading checker script

This simple script can be used to cheeck whether the shaded jar
does not contain any references which should also be shaded.

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

Do not suppress module name for top-level elements

RFC8040 states that top-level elements should start off fully
qualified -- hence we should disregard initial namespace when
for them.

JIRA: NETCONF-497
Change-Id: I1e638957510fe2efa8a42a430fb4c2ded48082ec
Signed-off-by: Tomas Cere <tomas.cere@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Merge "Fix distribution generation"

Merge "GET on /rests/operations with json always fails with 500"

Merge "Fix preparing result of action"

GET on /rests/operations with json always fails with 500

Stop throwing UnsupportedOperationException in FakeContainerSchemaNode
isPresence() method, since this container is only used for get on /rests/operations
we can assume this container is not a presence container.

JIRA: NETCONF-593
Change-Id: I06692da9d77e4230f32aa19aa25989ab72dd114a
Signed-off-by: Tomas Cere <tomas.cere@pantheon.tech>

Shade exificient

Shade exificient and xerces, so they do not end up polluting
the container with a full XML stack.

JIRA: CONTROLLER-1867
JIRA: NETCONF-591
Change-Id: If9e5d468de08dd820636a5eeb8ecdb2871fe1a3f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix distribution generation

This is a hot-fix for creation of distro, just to get things
rolling.

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

Fix preparing result of action

NullPointerException was thrown when action with no output
was defined and used and OK reply returned.

Change-Id: I2bcf846c501a650d754c943d2e1dc69e73a8dd37
JIRA: NETCONF-567
Signed-off-by: Anna Bencurova <Anna.Bencurova@pantheon.tech>

Bump mdsal to 3.0.4

This realigns odlparent/yangtools/mdsal and brings in the latest
improvements.

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

Bump yangtools to 2.1.7

This realigns odlparent/yangtools and brings in a few minor
improvements.

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

Bump to odlparent 4.0.8

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

Merge "Fix testtool on input modules without revision"

Merge "Remove netconf-monitoring"

Fix testtool on input modules without revision

- According to RFC 7950 revision number in module is not
  mandatory (only recommended) - currently Netconf Testtool fails
  on such modules with NullPointerException because it cannot find
  revision line pattern.
- Revision number doesn't have to be placed in input YANG schema -
  the filename is kept unmodified.
- Regex pattern that matches date in revision is also supplemented
  by revision keyword and optional double-apostrophes - there could
  be a collision with date in include statement.
- YANG module capability model is modified so the revision can be
  null - reurning of Optional.absent().
- DummyMonitoringService sets empty string in place of null
  revision numbers.

Change-Id: I868d39c111d788a0de86251da10f76a448f47de4
Signed-off-by: Jaroslav Tóth <jtoth@frinx.io>

Remove netconf-monitoring

netconf-monitoring is a forgotten remark of CSS times. It used to be
netconf monitoring implementation for CSS netconf endpoint. CSS is gone
so we do not need to support netconf monitoring for it.

Move GetSchema class to mdsal-netconf-monitoring (only used class
here) and move the parts needed by testtool into testtool monitoring
while we decide what we want to do with them.

JIRA: NETCONF-589
Change-Id: Ic734ddab4e9a94547a8547f61b173f8eed1111ea
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>

Install both restconf northbounds with odl-restconf

Now that we have separate packaging for both northbounds, make
sure we pull them into odl-restconf.

This eliminates the need for odl-restconf-nb-rfc8040-all, as that
is a subset of odl-restconf-all.

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

Separate out odl-restconf-nb-bierman02

odl-restconf is our user-installable feature which we want to use
to install both northbounds. Separate out odl-restconf-nb-bierman02,
which is authenticated version of bierman02 and pull it in.

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

Separate out odl-restconf-nb-bierman02-noauth

odl-restconf-noauth is the top-level, default feature, which
we want to use to install both northbounds.

Separate out bierman02 into its own feature and make sure
odl-restconf-noauth depends on it.

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

Rename odl-restconf-base to odl-restconf-nb-bierman02-base

This feature is hosting the base bierman02 bundle and configuration,
hence it should be named accordingly.

This reduces confusion a bit and allows us to install both
northbounds simultaneously.

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

Remove odl-restconf dependency on jackson

Jackson is part of odl-restconf-common, it should not be in
the user-installable feature.

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

Prevent duplicate netty packaging

With protocol-framework gone, odl-netty-api is missing
a dependency on odl-netty-4, preventing duplicates. Make sure
we have that in place.

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

Merge "Migrate NetconfHelloMessage's use of Optional"

Migrate NetconfHelloMessage's use of Optional

Use java.util.Optional, cascading that to all users.

Change-Id: I3ba97d060c8ea564bec378285bf6d8dbdb6a59d6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>

Merge "Replace GsonProvider to JacksonJaxbJsonProvider"

Replace GsonProvider to JacksonJaxbJsonProvider

Problem: GsonProvider uses com.google.gson.Gson to convert a javabean to a json string
and is not compatitble with ApiDelaration which has a defined field:
com.fasterxml.jackson.databind.node.ObjectNode models for the swagger model object.
So it should use ObjectMapper to convert swagger api object instead of google Gson.
Obviously, com.fasterxml.jackson.jaxrs.json.JacksonJaxbJsonProvider can achieve this conversion
by using the org.opendaylight.netconf.sal.rest.doc.jaxrs.JaxbContextResolver.

JIRA: NETCONF-588
Change-Id: Iee8a0528a29601a8737ed0247f183fddc9c5ebe5
Signed-off-by: wsd <wusandi@163.com>

Merge "Prevent ssh hang during shutdown"

Prevent ssh hang during shutdown

Add a reference listener for netty executor service provided by blueprint
so we can close ssh servers while the bundle still exists. This prevents
the hang seen during shutdown since the executor was being shutdown before
the mdsal-netconf-ssh.

JIRA: NETCONF-577
Change-Id: I062c9e8afb349817435ea0b1dde796a54ebb22d2
Signed-off-by: Tomas Cere <tomas.cere@pantheon.tech>

Specialize protocol-framework to netconf

As a first step towards sanity, we eliminate protocol-framwork
by moving it into netconf-api, but specialized to deal with
NETCONF protocol only.

This eliminates one level of abstraction, allowing us to reduce
the number of classes and artifacts involved. We also sacrifice
tests, as most of the mechanics should be covered by netconf-level
tests.

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

Deprecate protocol-framework tests

Since the framework is deprecated, deprecate its tests, too, so
we lower the compile-time noise.

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

Remove duplicate TimedReconnectStrategyFactory

This code is duplicated between the two netconf-topology implementations
and is tightly coupled with TimedReconnectStrategy.

Move the class into protocol-framework and deprecate it, eliminating
duplication.

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

Refactor client builder logic

We acquire protocol multiple times, leading to it being flagged
for null violations. Refactor the code a bit to make it easier
to follow what is going on.

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

Factor out SslHandlerFactoryImpl

SslHandlerFactoryImpl is a separate piece of logic independent
of AbstractNetconfTopology. Move it to its own package-protected
class and cleanup its use of Optional.

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

Modernize collection allocation

Eliminate Maps.newHashMap(), Lists.newArrayList(), Sets.newHashSet()
and related calls in favor of explicit instantiation.

Also eliminate use of deprecated ConcurrentSet.

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

Cache netconf operations

NetconfMonitoringOperationService is creating operations on demand,
where these are immutable and can be freely reused.

Instantiate them exactly once and keep them around for reuse.

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

Ignore NETCONF message document whitespace

The transformer we use is specifically allowed to add whitespace,
which does not offer us with enough control as to what it is going
to be -- making it implementation-dependent.

Checking whitespace match actually breaks with Java11, so disable
that checking.

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

Add JAXB dependencies

Java 11 is shipping without JAXB, make sure we declare them where
they are used.

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

Bump odlparent to 4.0.7

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

Switch to SpotBugs

findbugs does not work JDK9+, while spotbugs does. Switch to
SpotBugs.

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

Fix NamespaceContext test implementation

Java 10 changed the getPrefixes() to return Iterator<String>, making
this class fail to compile with JDK10+.

Fix this by adjusting the return type to match.

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

Fix HardcodedNamespaceResolver

Java 10 changed the getPrefixes() to return Iterator<String>, making
this class fail to compile with JDK10+.

Fix this by adjusting the return type to match.

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

Remove unneeded aaa versions

AAA is advertizing proper versions in artifacts, address FIXMEs
to remove versions.

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

Bump mdsal to 3.0.3

This brings in latest fixes and alignes with odlparent-4.0.5.

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

Bump to yangtools-2.1.6

This brings in latest fixes and odlparent-4.0.5 alignment.

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

Bump to odlparent 4.0.5

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

ApiDocServiceImpl.getListOfMounts should return an Object instead of JSON string

Problem: when swagger access mounted resources via /apidoc/explorer/index.html，
the explorer can't show mounted resources, only some undefined.

Reason: The JSON string in response entity is escaped by GsonProvider.

JIRA: NETCONF-587
Change-Id: Ida30a50fd7168d96904ba59fb275cf4d24817bfa
Signed-off-by: wsd <wusandi@163.com>

Merge "Remove test-tool CSS references"

Do not deploy/install netconf-tools POM

netconf-tools is aggregate POM, do not deploy/install its artifact to
repo/nexus.

Change-Id: Ieef8627b66939287fa507cb8c1cc5cbe05a0d4cc
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>

Remove test-tool CSS references

Change-Id: Ia9e4be7c9ff8c01ed45c1d8121144abc2f2817ac
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>

Merge "Get rid of checked futures in tests"

Merge "Use FluentFuture in DOMDataTransactionValidator"

Merge "Refactor bad local variable name in KeepaliveSalFacade"

Merge changes Ie852df32,I731a0066,I2e8558f1

* changes:
  Fix some of Java 8 IDE migration warnings
  Fix Java 7 IDE migration warnings
  Fix Java 5 IDE migration warnings

Merge "Remove references to controller.git"

Use FluentFuture in DOMDataTransactionValidator

Instead of CheckedFuture, use FluentFuture in
DOMDataTransactionValidator interface. This way it will get unified with
MD-SAL APIs since they are relying on FluentFutures.

Change-Id: I17c80c76a8c90c6bc1ae9091978038d8e97db839
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>

Get rid of checked futures in tests

We switched netconf components to MD-SAL APIs. MD-SAL APIs rely on
FluentFuture not on CheckedFuture. Use FluentFuture when mocking MD-SAL
APIs.

Change-Id: I1e4f71a0e1f57d16c7074c46fb42b6bcd836202a
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>

Refactor bad local variable name in KeepaliveSalFacade

domRpcResultDOMRpcExceptionCheckedFuture -> rpcResultFuture

Change-Id: I62624d593dd1562a427656efba5af5db8da3aaec
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>

Fix NetconfDocumentedExceptionTest iterator

Iterator should return Strings, fix that.

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

Fix some of Java 8 IDE migration warnings

- anonymous type can be replaced with lambda
- anonymous type can be replaced with method reference
- statement lambda can be replaced with expression lambda

Change-Id: Ie852df32bf6590e322655e8c983672914a139028
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>

Fix Java 7 IDE migration warnings

- explicit type argument can be replaces with <>
- identical 'catch' branches in 'try' statement

Change-Id: I731a00664a50fae9d4fa5f3fe9a908779948adfb
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>

Fix Java 5 IDE migration warnings

- 'StringBuffer' can be replaced with 'StringBuilder'
- get rid of unnecessary boxing and unboxing
- 'while' loop replaceable with 'foreach'

Change-Id: I2e8558f1aef8a2598cfc022e614b2f4f6e212785
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>

Remove references to controller.git

This removes references to controller.git in maven scm plugin
configuration.

Change-Id: I939fc8be9f57d3867c4a8ddef117e0f3819c52af
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>

Do not deploy/install features aggragator POMs

These POMs are for build itself only, do not produce their artifacts in
repo/nexus.

This is a follow-up patch to https://git.opendaylight.org/gerrit/#/c/78234/

Change-Id: If55df5f98e21285018870ccaf9125d80847f3460
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>

Aggregator POMs shouldn't have dependencies

features-netconf-connector-aggregator does not need any dependencies, it
is aggregator POM. Get rid of the declared dependencies.

Change-Id: I28413f336e7fca2f01746caed79bfc92aaa839b0
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>

Migrate netconf to MD-SAL APIs

This patch mass-migrates netconf components to use MD-SAL APIs.

Change-Id: I2ddd0004ef0e72a7fa1fc7a9d7df26515326ca4c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>

Merge "Bump to yangtools-2.1.4 and mdsal-3.0.2"

Support modules w/o revision in restconf/modules

Modules do not have to have revision. We should be able to handle such
modules in RescontfImpl getModules methods (GET resconf/modules invokes
these). When building modules container (as a response to such request)
use empty string for revision leaf to indicate that model does not have
revision.

Change-Id: Ie639c419dd27a4531194ee92a7d2d0bde4e89f70
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>

Merge "Log the address and port on bind errors"

Log the address and port on bind errors

This ensures the address and port are available in the logs when we
fail to bind.

Change-Id: I100f68112d90573333340b4dc3e0a6f73c8ad90a
JIRA: NETCONF-585
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Bump to yangtools-2.1.4 and mdsal-3.0.2

This bumps the dependencies to 2.1.4 and 3.0.2, respectively.

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

Do not deploy/install aggregator poms

These poms are for build system only, do not not produce their
artifacts in repo/nexus.

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

Add handling for aggregater sonar reports

This adds the maven-side of things for having UT coverage
aggregated for the entire project.

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

Bump exificient to 1.0.4

The new release improves memory overhead for idle sessions, bump
to it.

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

Merge "Migrate restconf to MD-SAL APIs"

Merge "Support consecutive slashes in RFC 8040 URIs"

Migrate restconf to MD-SAL APIs

This performs a mass-migration to MD-SAL APIs, switching Optional
to Java 8 and eliminating all references to CheckedFuture.

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

Use odl hello message capabilities

Fill the parameter of odl hello message capabilities
into NetconfReconnectingClientConfiguration to be used,
when connecting to the device.

JIRA: NETCONF-580
Change-Id: I9a4b2b97f335c2db330f0c1c7e86b76c4e2cffae
Signed-off-by: Anna Puchyova <Anna.Puchyova@pantheon.tech>

Support consecutive slashes in RFC 8040 URIs

Users often assume restconf URLs behave just as HTTP does by squashing
multiple slashes into a single one. Make sure we do not trigger an error
about an empty element in this case.

This deals with URIs in RFC 8040 restconf implementation.

NETCONF-24
Change-Id: I1c3a2085528e4177b09a2d330a6b05bb1b100147
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>

Fix continuation indentation in restconf UTs

According to our java style guide, when line-wrapping, each line after
the first (each continuation line) is indented at least +4 from the
original line.

Format RestconfSchemaServiceTest and ParserIdentifierTest according to
that.

Change-Id: Iac8ad245624c6d4a2bc2df635517d8c7d5d80fef
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>

Do not use deprecated mountpoint registration

Use proper builder pattern to talk to the mount point.

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

Merge "Migrate netconf to mockito ArgumentMatchers"

Merge "Migrate restconf to mockito ArgumentMatchers"

Migrate netconf to mockito ArgumentMatchers

org.mockito.Matchers is deprecated in Mockito 2. We should use
org.mockito.ArgumentMatchers instead.

Change-Id: Ib6a17cb55c95635435c14aa5f885d67ab2b02410
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>

Use mockito ArgumentMatchers in protocol-framework

org.mockito.Matchers class is deprecated so use
org.mockito.ArgumentMatchers class instead in protocol-framework
ServerTest.

Change-Id: Ia7e2d0daf40bb8ef1f3b616638e1a91f0c3ba30a
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>

Migrate restconf to mockito ArgumentMatchers

org.mockito.Matchers is deprecated in Mockito 2. We should use
org.mockito.ArgumentMatchers instead. Use this class in restconf UTs.

Change-Id: I3bca3a847a00f9ee5885683c9aa4c6d253c49938
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>

Migrate sal-rest-docgen to MD-SAL APIs

This performs a straight-forward migration of sal-rest-docgen.

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

Migrate from JSONCodecFactory.getShared()

This method is deprecated, move to using its replacement.

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

Simplify MultivaluedHashMap.getFirst()

Rather than using an if/else block, use a trigraph, fixing a minor
eclipse warning about an unnecessary 'else'.

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

Simplify MultivaluedHashMap.addAll(Object, Object...)

This method is a copy of addAll(Object, List) and it induces
a varargs warnings. Make it final, annotate it with @SafeVarArgs
and use Arrays.asList() to forward to addAll(Object, List).

This saves some code and fixes the warning.

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

Do not allocate a short-lived ByteBuf

ByteBuf.writeCharSequence() allows us to bypass manual buffer
management, use that.

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

Do not hard-code OK status code

HttpStatusResponse.OK gives us everything we need, simplify
response handling using its equals() method.

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