Age | Commit message (Collapse) | Author | Files | Lines |
|
Fixed multiple sonar issues across this file
Issue-ID: APPC-1715
Change-Id: I5a7df4bde164b8ec3c8f91f07646d0630d52fa59
Signed-off-by: Arundathi Patil <arundpil@in.ibm.com>
|
|
runtime exception.
remove the thrown exception
Issue-ID: APPC-1681
Change-Id: I940a5dd42bfe7a52b39be81a016cf4cf47805cae
Signed-off-by: Thugutla Sailakshmi <tsaila10@in.ibm.com>
|
|
Sonar Fix - Removed unused Variable Utils.java
Issue-ID: APPC-1629
Change-Id: I557c065ff835834f5eed5eb80bd7136e182217f5
Signed-off-by: Sudarshan Kumar <sudars19@in.ibm.com>
|
|
Sonar fix
Issue-ID: APPC-1562
Change-Id: Ic9d14b32f4deac975b46154a2ac8e4664ec11c52
Signed-off-by: Jegadeesh Babu <jegabab1@in.ibm.com>
|
|
Sonar fix
Issue-ID: APPC-1563
Change-Id: I9a4b82ca67b5c7f5184457b95c347089de728658
Signed-off-by: Jegadeesh Babu <jegabab1@in.ibm.com>
|
|
Sonar fix
Issue-ID: APPC-1557
Change-Id: I4b52b9febabd0fae8aa098004aeba0ef9a085710
Signed-off-by: Jegadeesh Babu <jegabab1@in.ibm.com>
|
|
Sonar fix
Issue-ID: APPC-1539
Change-Id: Id3891e50bdf89799d1310b37c80ca920d7bdc02c
Signed-off-by: Jegadeesh Babu <jegabab1@in.ibm.com>
|
|
Sonar fix
Issue-ID: APPC-1527
Change-Id: Ic2708d67b6f2740ddabe834e56b09c11389b8894
Signed-off-by: Jegadeesh Babu <jegabab1@in.ibm.com>
|
|
Changes required to update to latest opendaylight
version. Some powermock tests have been temporarily
removed due to an powermock incompatibility. They
will be fixed and re-added.
Change-Id: I418d44913fcd42423d2d9bd37b189f6301eaab81
Signed-off-by: Patrick Brady <patrick.brady@att.com>
Issue-ID: APPC-1370
|
|
Change-Id: I5edf48443c10f08200b0ea376194a9a8fc6dc29a
Issue-ID: APPC-1302
Signed-off-by: ezhil <ezhrajam@in.ibm.com>
|
|
Fixed sonar issues/code-smells across this file.
Issue-ID: APPC-1279
Change-Id: I50d5de26d69928037882686452aecd3deb6edbec
Signed-off-by: Arundathi Patil <arundpil@in.ibm.com>
|
|
Fixed sonar issues/code-smells across this file
Issue-ID: APPC-1253
Change-Id: Id1060c34f0b99e906dba618d9ea0aebca0fc1beb
Signed-off-by: Arundathi Patil <arundpil@in.ibm.com>
|
|
Issue-ID: APPC-1012
Change-Id: Iac03825b092f62a8d3747a62303f7869bc8280e8
Signed-off-by: Modaboina, Kusumakumari (km583p) <km583p@us.att.com>
|
|
License header fixes to reflect 2018 changes in various appc-adapters, appc-client, appc-common, appc-config, appc-dg and appc-directed-graph java and properties files.
Change-Id: Idbe000f175e11ba42f6e83689a2b2d3d353be008
Issue-ID: APPC-996
Signed-off-by: John McClung <jm1364@att.com>
|
|
-Add new JUnits to ChefApiClientImplTest to check if body is properly
set for put/post http requests.
Change-Id: Ie9ea2de7a1c68fa12815586a4f85646b06ce69a9
Issue-ID: APPC-879
Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
|
|
Remove ECOMP trademark license header in miscellaneous files.
Change-Id: I76e386462402cb959de377b56997ecf7986ffd6b
Issue-ID: APPC-931
Signed-off-by: John McClung <jm1364@att.com>
|
|
Programatically configure values for truststore and password when
initializing the HttpClient.
Update post and put methods to set entity
Add organization into request URI
Restore log messages to original values
Issue-ID: APPC-868
Change-Id: Ia0fdf13fcb1e3ad91a4d2c5ce704523562191379
Signed-off-by: Wonnell, Skip (kw5258) <kw5258@att.com>
|
|
-Added junit tests for fetchResults method
-Support for Michal Kabaj as part of APPC-437
Change-Id: I3219a5ec0ffd530338ce364f608a9706d5a92c03
Issue-ID: APPC-858
Signed-off-by: Tomasz Gwozdecki <tomasz.gwozdecki@nokia.com>
|
|
-Add new JUnits to ChefAdapterImplVNFCOperationsTest
Change-Id: Idd4cd456d6409a499e569c7175d97642fc3607d2
Issue-ID: APPC-437
Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
|
|
-Add new JUnits to ChefAdapterImplVNFCOperationsTest
-remove unused JSONException catch clause from ChefAdapterImpl,
when there is no jackson lib used
Change-Id: I96b8d6155ca13087b697a19e1fcbba2ee82a8291
Issue-ID: APPC-437
Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
|
|
- Switched usage of RequestContext class for wrapping the SvcLogicContext
to using the SvcLogicContext directly
- Removed unused RequestContext class
Change-Id: I7376e3d6f5602887cf5a25f1f90ff45805c620b2
Issue-ID: APPC-437
Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
|
|
Add new JUnits to ChefAdapterImplVNFCOperationsTest
Change-Id: I243b2f46fdb75ad3c689b7875361fb9a654eaf60
Issue-ID: APPC-437
Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
|
|
- Added new Junit class ChefAdapterImplVNFCOperationsTest
- Added unit tests for method vnfcEnvironment
- Small refactor to for exception handling in vnfcEnvironment method to
improve readability and avoid catching and rethrowing same exception
Change-Id: I58a43390ff45598f6d17da48eed3620667008cb7
Issue-ID: APPC-437
Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
|
|
- Added new Unit tests for ChefAdapterImpl checkPushJob method
to class ChefAdapterImplJobPusherTest
- Added param check in implementation method
- Small refactor of nested conditions for improved readability
Change-Id: Id65b897535874eea6aafda66b7ac227832446cf4
Issue-ID: APPC-437
Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
|
|
-Added new Unit tests for ChefAdapterImpl pushJob method
-Added new Junit class ChefAdapterImplJobPusher
-Some minor construction refactor in other junits
Change-Id: Id6fbcba2c5f5e11bfad2c7dccf6c06f777756a47
Issue-ID: APPC-437
Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
|
|
-Added junit tests for checkInfo method to verify if input params are present
-Support for Michal Kabaj as part of APPC-437
Change-Id: I29758f29f32354439ea0471e1afa3df6bb8184b9
Issue-ID: APPC-437
Signed-off-by: Tomasz Gwozdecki <tomasz.gwozdecki@nokia.com>
|
|
-Added new Unit tests for ChefAdapterImpl#trigger method
-Refactored ChefAdapterFactory to allow mock injection
-Added new create() method to ChefApiClientFactory which does not set
any Chef specific headers for the purpose of trigger method
-Cleaned up the trigger method - simplified usage, removed useless doFailure
method and inlined its logic and unified HttClient usage -> HttpClient is not
used directly but through existing ChefApiClient
-removed redundant TestChefAdapterImpl tests which have already been rewritten
Change-Id: I830d203877224aa2c5e1b0dff39d1171f1f2f0ef
Issue-ID: APPC-437
Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
|
|
Introduce new FunctionalInterface HttpHeaderFactory to allow customization of
http header providers when creating ChefApiClientImpl using ChefApiClientFactory
This allows for better encapsulation of header specific values such as:
userId,organizations,pemPath.
Change-Id: Iec2c7cff8e664a18e15d7e973920152fdb158c98
Issue-ID: APPC-437
Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
|
|
-Added unit tests for chefXXX methods - extracted new test class
-Moved privateKeyCheck method to new class PrivateKeyChecker + JUnits
-New Factory class ChefAdapterFactory for ChefAdapterImpl construction
-Fixed potential bug with wrong String being assigned in iniSvcLogic method
Change-Id: I27badfd01bfaa807b8ecb9b4a4c13e7f026e34af
Issue-ID: APPC-437
Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
|
|
-Added unit testcases for retrieveData method - extracted new test class
-Added unit test for combineStrings method
Change-Id: I8adcc9d2dc17e4ebf305fc2a8139095d5841a445
Issue-ID: APPC-437
Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
|
|
-Added new ChefAdapterImplTest class for new ChefAdapterImpl JUnits
-Added test for nodeObjectBuilder method - JSON Format is now verified using JSONObject
-Unified ChefAdapterImpl construction - removed unused constructors
Change-Id: I63f96bac0129b5f13d34efd3a173784e0b1d8a5d
Issue-ID: APPC-437
Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
|
|
-Reorganize ChefApiClient classes according to their usage, construction
and implementation.
-Introduce api and impl packages.
-Extract interface from ChefApiClient impl
-Remove unused old chefapi.* classes
Change-Id: Ic1ddbdcfd0e1bbc0830b923da5aac5b68ac44a2f
Issue-ID: APPC-437
Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
|
|
Change-Id: I551f9e9033de2f13028f28ad434003d61102d54f
Issue-ID: APPC-637
Signed-off-by: Skip Wonnell <skip@att.com>
|
|
Add(fix) JUnits for ChefApiClient + refactor.
Junits work on mocked HttpClient instead of attempting to make real calls.
All ChefApiClient methods are now tested.
Improve readability and refactored the ChefApiClient flow (simplified).
Heavily improved SRP in both ChefApiClient and Api request creation.
Change-Id: I997490bf269e7d78b741baf7f8d5d7bf64f6b99a
Issue-ID: APPC-437
Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
|
|
Change-Id: I3e1a496e18a4b9e6e9dc954bc16fb4af991fd5d3
Issue-ID: APPC-579
Signed-off-by: Jakub Dudycz <jakub.dudycz@nokia.com>
|
|
The domain that was being used for the chef test was a real url.
The test failed most of the time. Setting it to localhost.
Change-Id: I7c162b313b78e174ca3845920dfa88a7004983d5
Signed-off-by: Patrick Brady <pb071s@att.com>
Issue-ID: APPC-556
|
|
Removing references to bvc, att
Change-Id: I794861fe40b96c922f6e5c011b2fe5d2c4bc515b
Signed-off-by: Patrick Brady <pb071s@att.com>
Issue-ID: APPC-292
|
|
Issue-ID: APPC-411
Change-Id: I4936559bfa0cf8dbe89c2a81d464fecc0a49afa0
Signed-off-by: Skip Wonnell <skip@att.com>
|
|
Add new JUnits for ChefActivator class, minor refactor + cleanup.
- Remove redundant getAdapterName() method from ChefAdapter to unify
adapter name constant definition
Change-Id: I483d34aaa0f4e76a4360b179f1a60cc1263ec9b7
Issue-ID: APPC-437
Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>
|
|
Actually did some other refactor too
Change-Id: I4b622018baa2ed130a5904dbc824fc01e82c8a60
Issue-ID: APPC-508
Signed-off-by: Jakub Dudycz <jakub.dudycz@nokia.com>
|
|
Change-Id: Iddff91573fa5f7cc99171ca0b782415e77560154
Issue-ID: APPC-514
Signed-off-by: Jakub Dudycz <jakub.dudycz@nokia.com>
|
|
Change-Id: Ibe5250e426bfc3387cbe886be4048f18dbe45c13
Issue-ID: APPC-491
Signed-off-by: burdziak <olaf.burdziakowski@nokia.com>
|
|
Fix some Sonar issues
Update code to ONAP java style
Issue-ID: APPC-419
Change-Id: I0c6b7b684c10b3874c3ef7de40066eb6c3b6345e
Signed-off-by: Skip Wonnell <skip@att.com>
|
|
Issue-ID: APPC-420
Change-Id: Ia55c85cec3024f4199920200e614d60c7822bbbb
Signed-off-by: Skip Wonnell <skip@att.com>
|
|
Change-Id: I9dfe73a45b43919ff7b01ab3217d957a40ac2018
Issue-ID: APPC-408
Signed-off-by: Taka <tc012c@att.com>
|
|
The check of the timestamp header fails when rolling over to a new
minute. Change the code to compare the length of the interval
instead.
Issue-ID: APPC-411
Change-Id: I3a98aab2de878335a2bc320d261fee3101fd0652
Signed-off-by: Skip Wonnell <skip@att.com>
|
|
Issue-ID: APPC-305
Change-Id: Ice60ca77ffccc409d50ef11318e841c3b1f82ce1
Signed-off-by: kusuma kumari M <km583p@att.com>
|
|
Some paths within the docker image need to be updated from
opt/openecomp to opt/onap
Change-Id: I74b32ba6906a0b7c44764485dccef6140c21afa3
Signed-off-by: Patrick Brady <pb071s@att.com>
Issue-ID: APPC-13
|
|
This is the second commit of the rename. The folder structure
is renamed for appc-adapters and appc-config in this commit.
Change-Id: Iaa2b8c937ff1ca1b5d1178128961fb115ee65d9b
Signed-off-by: Patrick Brady <pb071s@att.com>
Issue-ID: APPC-13
|
|
This is the first commit of the rename. This commit changes
refereces to org.openecomp.appc > org.onap.appc that occur within
files. The package folder structure is not changed in this commit.
Change-Id: Ic95d749eb99d8a6f4f2b9ee9b06eb41c5cfa7e1c
Signed-off-by: Patrick Brady <pb071s@att.com>
Issue-ID: APPC-13
|