Merge "Remove unused code from Immutables"

Merge "Parsing and serialization of composite and simple node to normalized nodes"

Parsing and serialization of composite and simple node to normalized nodes

Added implementation of base classes from yang-data-impl artifact for
serializer and for parser
+ simple test

Change-Id: Ic7353e1be7ef6e00546595304c23b205f7580cee
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

Use a random port in test server

This patch updates the test server to bind to a random port number and expose
this port number via a Future can be fetched by the test client so that it
knows to which port to connect.

Version 2:
Modified the code to use the guava SettableFuture as opposed to a custom built
future, as well as cleaned up a few other things, including makding sure that
the message was received from the server before considering the test passed.
was seeing different behavior on different platforms without this (i.e. some
times the message was display, sometimes not). Also, failed the test case wh
exceptions were caught.

Change-Id: Ia11c374bfbf206a6214d2afe3ba328b1231d6fad
Signed-off-by: David K. Bainbridge <dbainbri@ciena.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

Remove unused code from Immutables

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

Merge "Global cleanup of restconf client dependencies"

Merge "Bump javax.ws.rs-api version to 2.0"

Merge "Remove needless references to xtend library"

Merge "Do not depend on jersey-client"

Merge "Update interface to hide Dispatcher"

Merge "Introduce the concept of ProductAwareBuilder"

Merge "Introduce common directory"

Global cleanup of restconf client dependencies

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

Bump javax.ws.rs-api version to 2.0

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

Remove needless references to xtend library

There is simply no need to use the xtend runtime library. Let's simply
write clean code.

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

Do not depend on jersey-client

JAX-RS annotations do not require full-blown client, so import only the
API bundle.

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

Introduce the concept of ProductAwareBuilder

A ProductAwareBuilder is an builder which can act as a proxy
for its product in collections (obviously provided it is not
mutated while its used this way).

This functionality is useful for implemeting caches, where
the toInstance() call can be skipped as long as there is an
existing equal object.

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

Introduce common directory

This directory will host code common to the various submodules, some of
which are candidates from moving into an upstream repository.

Populate this directory with concepts and mockito-configuration, thus
making sure the top-level POM has no direct code children.

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

Another set of warning fixes

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

Update interface to hide Dispatcher

Dispatcher is really an internal implementation thing. Expose a parser
factory instead.

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

Add attributes to nodes that can take attributes

The following types of nodes seem to be able to accept attributes,
- LeafNode
- ContainerNode
- LeafSetEntryNode
- MapEntryNode

Also added parsing of Attributes

Change-Id: I524efde134e06d5b30a68379c8abdb65bba60cb2
Signed-off-by: Moiz Raja <moraja@cisco.com>
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

Abstract infrastructure for normalized nodes translation.

With implementation for dom <-> normalized nodes.

Change-Id: Ic2c08d2955177bc335d6493d64076a77d2cc7d62
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>
Signed-off-by: Ladislav Borak <lborak@cisco.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Fixed possible NPE in CodecMapping."

Added NormalizedNodeContainerBuilder and helpers.

   - NormalizedNodeContainerBuilder is common root
     to builders which produces NormalizedNodeContainer,
     this allows to write more generic code using builders.

   - NormalizedNodeUtils - provides helper functions to process
     normalized date tree, such as findNode which look up
     subtree based on provided InstanceIdentifier.

   - Added Immutable marker interface to existing builders.

Change-Id: Id842742432c48d76cdf78dab40352af27e10360f
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Fixed possible NPE in CodecMapping.

Signed-off-by: Martin Vitez <mvitez@cisco.com>

Merge "Fix for Bug 511 (model + integration-test)"

Merge "Fix for Bug 511 (concepts)"

Merge "Fix for Bug 511 (yang)"

Merge "Fix for Bug 511 (code-generator)"

Merge "Fix for Bug 495."

Merge "Added documentation for web socket client"

Fix for Bug 511 (model + integration-test)

Removed \r characters from yang-ext.yang and from integration-test project.

Signed-off-by: Martin Vitez <mvitez@cisco.com>

Fix for Bug 511 (concepts)

Removed \r characters from concepts.

Signed-off-by: Martin Vitez <mvitez@cisco.com>

Fix for Bug 511 (yang)

Removed \r characters from all projects under yang directory.

Change-Id: I46a47d523449bc159ea508143d9a06726f8259ff
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Fix for Bug 511 (code-generator)

Removed \r characters from all projects under code-generator directory.

Change-Id: Ic6af8cc1b686c62caeaab9e5f2595a597750cafe
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Merge "Immutable implementation of new yang-data-api"

Immutable implementation of new yang-data-api

Signed-off-by: Maros Marsalek <mmarsale@cisco.com>
Change-Id: If42a5a3fe0d371e9e0d2664705350065b2448473

Add ability to get yang sources from SchemaContext.

SchemaContext should provide yang source code of parsed module files.
To identify modules, move YANGModuleIdentifier from binding-generator-spi
to yang-model-api: ModuleIdentifier. This interface can represent
both modules and submodules in SchemaContext. Add methods to SchemaContext
to iterate ove ModuleIdentifiers and get their source.
In order to maintain backwards compatibility no new parsing method was created,
so submodules cannot be supported now in new methods.
Adding this functionality removes need for most methods defined in
org.opendaylight.yangtools.yang.model.parser.api.YangModelParser .
It is advised that after this change parser api would be refactored to
have a method taking collection of input streams as input
and returning SchemaContext.
Other small changes:
Externalize SimpleDateFormat wrapped in ThreadLocal
Add toString to AbstractObjectRegistration
Add guava dependency to yang-model-api for Optional goodness
Add ModuleIdentifierImpl convenience class allowing users to seach source by
 module name.

Change-Id: I8a46fa699bbd829e5d4a488ebc99012a66fc196d
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>

Merge "Fix for Bug 497."

Merge "Better logging for codegen's generateModuleInfos"

Fix for Bug 495.

Fixed bug in parsing augments defined under uses: target node did not get reference to its augmentations.
Added test.

Change-Id: I4e7e32a6f2eb960d9e6727043e6b4dc4ba26429b
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Merge "Fix for Bug 496."

Merge "Migrate RuntimeGeneratedMappingServiceImpl to use SchemaContextListener"

Fix for Bug 497.

Added generation of toString in YangModuleInfoTemplate.

Change-Id: I4a4781618dc58baa4764be378e92c26c7d15f5a2
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Better logging for codegen's generateModuleInfos

Change-Id: I14798f5f7caf9324344351141751be0001ef3a83
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>

Merge "Added initial draft of Normalized Yang Data Tree model."

Migrate RuntimeGeneratedMappingServiceImpl to use SchemaContextListener

Moves the implemented interface higher up the chain, allowing its users
to start relying on SchemaContextListener -- fixing deprecations
warnings in the process.

Drive-by:
- organized imports to remove unused import
- remove unused field

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

Merge "Create onGlobalContextUpdated() in SchemaContextListener"

Fix for Bug 496.

If leafref type contains invalid path, no warning was reported and no methods were generated from this leaf/leaf-list.
Added exception throwing if leafref target is not found.
Updated tests.

Signed-off-by: Martin Vitez <mvitez@cisco.com>

Introduce ClassLoadingStrategy,ModuleInfoRegistry interface, copy SchemaStoreProvider from md-sal.

To decouple implementation from interface introduce ClassLoadingStrategy and ModuleInfoRegistry interfaces.This allows intercepting
read/write operations.
Copy SchemaContextProvider from org.opendaylight.controller.sal.dom.broker.impl .

Change-Id: Ifebc0f0adbb265f4a4e492c0a9b54f07cc17fe01
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>

Create onGlobalContextUpdated() in SchemaContextListener

With deprecating SchemaServiceListener, we need an explicit mark that
the method itself is not deprecated. Add an overridden method
declaration so the users do not get a useless warning.

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

Fix for Bug 489.

Current code generator version was unable to generate sources for augmentation defined under uses statement
if augment points to nested node (not top level node added by uses).

Failing example:

container c {
    uses route-subobjects {
        augment "links/link" {
            leaf id {
                type string;
            }
        }
    }
}

AugmentationSchemaBuilderImpl: fixed bug in parsing augment target path for augmentations defined under uses.
BindingGeneratorImpl: fixed resolving of augment package name.
BindingGeneratorUtil: added new method to generate package name.

Added tests.

Change-Id: Ie48b2fffc0e6449bf5c13b5a6b636cbd837ffb54
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Added documentation for web socket client

Change-Id: I88cce1f657b4d7c7a6ba6cc99e8da63541d7d66f
Signed-off-by: Ladislav Borak <lborak@cisco.com>

Merge "ModifAction enum added NONE operation + additional checks Added Modify Action to SimpleNode and CompositeNode"

Merge "Fix for Bug 295."

Fix for Bug 295.

Changes in GeneratorJavaFile:
 - produced warning when more than one file with same name is going to be generated
 - constructor parameter type changed from Set to Collection
 - removed unused method generateStringToJavaFile

Fixed implementation of isAcceptable(Type type) in InterfaceGenerator.
Added test.

Change-Id: If2cc5ccf27b15f3749628a6c7059de6703dda213
Signed-off-by: Martin Vitez <mvitez@cisco.com>

ModifAction enum added NONE operation + additional checks
Added Modify Action to SimpleNode and CompositeNode

Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Merge "Organize imports to get rid of warnings"

Organize imports to get rid of warnings

Duplicate/unused imports: get rid of them.

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

Merge "restconf client expects stream discovery to be implemented"

Merge "Fixed for Bug 468."

Fixed for Bug 468.

Fixed prefix resolution of augment target path parts on parser level.

Signed-off-by: Martin Vitez <mvitez@cisco.com>

restconf client expects stream discovery to be implemented

Change-Id: I139807ee67bea72927c52e47fc13d129c576191e
Signed-off-by: Martin Bobak <mbobak@cisco.com>

added missing payload extraction in proxy method
added request type to event stream context calls

Change-Id: Ibecf4ec59f53aa1bf0dcbf1295533b23a0c5b3c4
Signed-off-by: Martin Bobak <mbobak@cisco.com>

added isInterface test for listener registration
AuthenticationHolder for rest authentication

Change-Id: Idf0dfe2ce40d862490eb52b2344e7974d14da148
Signed-off-by: Martin Bobak <mbobak@cisco.com>

Fixed missing predicate in package name regex. Improved error reporting.

Change-Id: If76fe681e37a5421621754d088de99b14eeae05d
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Bug 466:Fixed namespace and revision resolution for empty augmentations.

Change-Id: I85af4d15d7f4a6b604461084e9ff4f73b103cca1
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Fixed possible NPE, improved logging.

Signed-off-by: Martin Vitez <mvitez@cisco.com>

Fixed NPE caused by tryToLoadAugmentations in LazyGeneratedCodecRegistry.

Change-Id: I3142a629b9bd366a960c79447134d87f36f47e25
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Fixed generation of toString method in builder template.

Signed-off-by: Martin Vitez <mvitez@cisco.com>

Fix for Bug-427.

Added test for mapping from dom (with nested augmenation nodes) to data object.
Added javadocs.

Change-Id: I0d0c08092c6485afb820d9bfbb352cf934f282f3
Signed-off-by: Martin Vitez <mvitez@cisco.com>
Signed-off-by: Milos Fabian <milfabia@cisco.com>

Added initial draft of Normalized Yang Data Tree model.

Change-Id: Ic50335f8a7bfe0fa4d7c239c997f6ca090b9b506
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

BUG-432: Start fixing the Registration contract

Registration should be a generic construct, not tied to a particular
object. Unfortunately that requires API-level breakage which needs to be
orchestrated with the users.

As a first step introduce ObjectRegistration concept which is equivalent
with the current Registration concept and gently nudge the users towards
it.

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

Document and change the ListenerRegistration contract

Add text clarifying the role of this class. Also change the API contract
such that the close() method does not throw checked exceptions. Also
change the API contract such that the thread-safety is ensured via an
AtomicBoolean rather than a synchronized block.

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

Fix various warnings

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

Merge "Add registration interface contract definition"

Merge "Add injectable LoadingStrategy to RuntimeGeneratedMappingServiceImpl."

Merge "Use AbstractListenerRegistration"

Add injectable LoadingStrategy to RuntimeGeneratedMappingServiceImpl.

This allows config-managed to inject ModuleInfoBackedContext as LoadingStrategy in order to resolve Identity classes.

Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Fixed getClassName method.

If string contained point character, it wasn't escaped and cause compile errors.

Change-Id: I3d2179133a160adb8e1c79649e880aee124e40f3
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Add registration interface contract definition

This clarifies the contractual details of Registration interface.
Notably it clarifies that the close() method is non-blocking.

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

Use AbstractListenerRegistration

This gets rid of home-grown class and documents that the close()
contract is not properly implemented.

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

Merge "Added more javadocs."

Added more javadocs.

Signed-off-by: Martin Vitez <mvitez@cisco.com>

Fixed incorrect path construction in JUnit tests.

Change-Id: I0a9d73876ea0dd6263d414e91070d405fef9069b
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Fix of $YangModuleInfoImpl for ext. yang models

Fixed bug in writeJavaSource method in CodeGeneratorImpl. If package does not exist the dir structure is created.
Without this fix yang models containing only extension definitions could not be generated.

Signed-off-by: Lukas Sedlak <lsedlak@cisco.com>

Merge changes Id68de4e3,Id4b3c97f

* changes:
  Fixed resolving rpc service classes by name and revision insted of namespace and revision
  - fix of getConfigurationDatastore returning null - basic tests added

Merge "Fixed DataObjectReadingUtil to resolve multiple items from lists if InstanceIdentifier is wildcarded."

Fixed DataObjectReadingUtil to resolve multiple items from lists if InstanceIdentifier is wildcarded.

Change-Id: Iac3d30c548435d1f3a8575ec6f11ac09cd72bb3d
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Merge "Fixed bug in code generator when handling case defined in grouping as augment target."

Revert "Added ietf-restconf yang module"

This reverts commit 9003fb310abfa2773e570424dd4b38953beb0673.

Change-Id: Ifd9255b63b76fa1e697361a6b16826921999695a
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Merge "Temporarily reverted "Update ietf-inet-types and ietf-yang-types with RFC6991""

Temporarily reverted "Update ietf-inet-types and ietf-yang-types with RFC6991"

Upstream dependencies did not honor warnings which required them to specify model revision.

Change-Id: I7ac5d5820bf47fb2e429eb8b8c6ef060b6eab652
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Merge "Added ietf-restconf yang module"

Merge "Update ietf-inet-types and ietf-yang-types with RFC6991"

Fixed resolving rpc service classes by name and revision insted of namespace and revision

Change-Id: Id68de4e3c19883509f28b8e42d05f1db65827aa6
Signed-off-by: Martin Bobak <mbobak@cisco.com>

Update ietf-inet-types and ietf-yang-types with RFC6991

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

Added ietf-restconf yang module

Change-Id: Ifd8012f2ba2b436b68d590caaed512900733fe17
Signed-off-by: Martin Sunal <msunal@cisco.com>

- fix of getConfigurationDatastore returning null
- basic tests added

Change-Id: Id4b3c97fa3c438b1daf822d223066fa4e427a299
Signed-off-by: Martin Bobak <mbobak@cisco.com>

Fixed concurency bug when multiple threads tried to compute package name.

Change-Id: Ifed593dc386124a596cc85e6aea512360a9d942b
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>