Age | Commit message (Collapse) | Author | Files | Lines |
|
Refactored overcomplicated methods by splitting
into the more simple.
Remove redundant conditional checks.
Move common checks into the utils class.
Fixed raw types usage where it is possible.
Add some tests.
Fixed typos in test names.
Fixed lost switch branch.
Issue-ID: SDC-2698
Signed-off-by: Dmitry Puzikov <d.puzikov2@partner.samsung.com>
Change-Id: Ifb5887401be4b397267b2f922cc9ad3f01c757f6
|
|
Process NPE safe way.
Add no NPE failures tests.
Add caught exceptions logging.
Issue-ID: SDC-2694
Signed-off-by: Dmitry Puzikov <d.puzikov2@partner.samsung.com>
Change-Id: Ic1d6e7f9b4f210ac0ca289dc6f44d7c6e15ef126
|
|
Added exxception logging,
added tests.
Change-Id: Ia878030d13570b1445e9b077fce77ca387e69f9a
Issue-ID: SDC-2711
Signed-off-by: Dmitry Puzikov <d.puzikov2@partner.samsung.com>
|
|
Added tests to increase coverage.
Fixed test faiulures.
Issue-ID: SDC-2710
Signed-off-by: Dmitry Puzikov <d.puzikov2@partner.samsung.com>
Change-Id: I3fb49f2fc794411347780a904837d8d0c61aa7bf
|
|
Getting rid of nested ifs-fors
Change-Id: I71ef686683b7070614c27aabbf6cd058b5d35ffd
Issue-ID: SDC-2654
Signed-off-by: Dmitry Puzikov <d.puzikov2@partner.samsung.com>
|
|
Simplified method ConfigurationUtils::getProperty.
Added unit tests for method.
Change-Id: I521661b464e7f60101ee51ccbc9d915bf5030f31
Issue-ID: SDC-2654
Signed-off-by: Dmitry Puzikov <d.puzikov2@partner.samsung.com>
|
|
Refactor the ConfigurationRepositoryTest to doesn't affect to other tests.
Removed powermock.
Issue-ID: SDC-2473
Signed-off-by: Krystian Kedron <k.kedron@partner.samsung.com>
Change-Id: Iad67cdce365fef89da8b56499d86139d6927da07
|
|
Removed errors regarding to a missing license and others.
Started locally and executed basic api tests
Change-Id: Iea37613defc97f7b40613d60c10990841cb2a209
Issue-ID: SDC-2419
Signed-off-by: Tomasz Golabek <tomasz.golabek@nokia.com>
|
|
Checkstyle for CliConfigurationImpTest.
Improved the ConfigurationRepositoryTest
to not has an influence on other tests.
Issue-ID: SDC-2327
Signed-off-by: Krystian Kedron <k.kedron@partner.samsung.com>
Change-Id: I74b95617ad1b17c26a8d6237c7e7469ef2213596
|
|
Improved the unit test coverage
in onap-configuration-management-core project
Issue-ID: SDC-2327
Signed-off-by: Piotr Borelowski <p.borelowski@partner.samsung.com>
Change-Id: Ib448d3f318a89d65d4b6ae912e3ca38a31b86146
|
|
Improve the NonConfigResourceTest.
Remove the throws the Exception in CliConfigurationImp constructor.
ConfigurationImpl class:
- Replace the string to chart in indexOf method.
- Extract the assigment out from expression
Issue-ID: SDC-2327
Signed-off-by: Krystian Kedron <k.kedron@partner.samsung.com>
Change-Id: If72a75e6a2d81a9ea74e3ae1af94b16a8489a29a
|
|
Issue-ID: SDC-2327
Signed-off-by: Krystian Kedron <k.kedron@partner.samsung.com>
Change-Id: I96269b0bcdf3c1254052d057b8c4ad2d7a48c89d
|
|
Add more test for ConfigurationUtils.
Fix error in getCompatibleCollectionForAbstractDef method bad return value
for TransferQueue.class.
Sonar fixes:
- use not deprecated method IOUtils.toString.
- remove code duplication.
- return empty collection instead of null.
Issue-ID: SDC-2327
Signed-off-by: Krystian Kedron <k.kedron@partner.samsung.com>
Change-Id: I5abb10bca0a1c409ec20daf6c22066836a0b76a3
|
|
Simplified code, added unit tests.
Change-Id: I0a35daf8a2d4cfd0979e30363c9fad30e7c0570b
Issue-ID: SDC-1867
Signed-off-by: vempo <vitaliy.emporopulo@amdocs.com>
|
|
Removed code duplicates, stabilized test execution via
Maven, re-aranged code, fixed spelling.
Change-Id: I41fc303ea0a8c7d78d89a12bb20850de51cb8c52
Issue-ID: SDC-1867
Signed-off-by: vempo <vitaliy.emporopulo@amdocs.com>
|
|
Replaced not thread-safe fields with synchornized versions,
removed duplicate code, deleted class that was accessing DB,
made surefire plugin to pick up all available unit tests
(instead of hand-picked).
Change-Id: Idff3ac333dc87ebfd3ecf50438ba0179556eb9c9
Issue-ID: SDC-1867
Signed-off-by: vempo <vitaliy.emporopulo@amdocs.com>
|
|
Configuration framework will not poll configuration
for changes, and will not notify client code of them.
Also minor cleanup, added unit tests.
Change-Id: I428b23f7acb13c6610390f46aae6e011d6b0ee80
Issue-ID: SDC-1867
Signed-off-by: vempo <vitaliy.emporopulo@amdocs.com>
|
|
Added slf4j binding for tests.
Updated patch version of commons-configuration.
Change-Id: Ifef09eb3591f01be4f90f16758448018361dcdb7
Issue-ID: SDC-1867
Signed-off-by: vempo <vitaliy.emporopulo@amdocs.com>
|
|
Fixed code formatting, removed meaningless Javadoc comments,
added copyright headers, minor (and safe) static analysis fixes.
Change-Id: I3eda1f242905da5b80e024cf30a69ff59381fc43
Issue-ID: SDC-1867
Signed-off-by: vempo <vitaliy.emporopulo@amdocs.com>
|
|
On Linux, environment variables are case sensitive,
while on Windows they are not.
Change-Id: I49dfd68ab3596ff7ee4a7a53aeba906b606df81b
Issue-ID: SDC-1867
Signed-off-by: vempo <vitaliy.emporopulo@amdocs.com>
|
|
Conditionally enabled unit tests, fixed a few
static analysis violations, added copyright headers.
Change-Id: I46ccafc5f4397f82b86d5c51bf0452f2060f6a70
Issue-ID: SDC-1867
Signed-off-by: vempo <vitaliy.emporopulo@amdocs.com>
|
|
The tests with Thread.sleep() caused a long build time (>7 min).
Aslo fixed formatting, copyright headers, static analysis violations.
Change-Id: I8279478c1e6812facc51730679d2ee4e73e22ec7
Issue-ID: SDC-1867
Signed-off-by: vempo <vitaliy.emporopulo@amdocs.com>
|
|
Change-Id: I7b99fdeeeb66f5de9bcd9283d815976c3d7520b0
Issue-ID: SDC-1867
Signed-off-by: vempo <vitaliy.emporopulo@amdocs.com>
|
|
Fixes of sonar violations sdc and refactor
This reverts commit baf7f0a965d0ffebd5308d44758bfa9ba96c0c76.
except the catalog files.
the onbording files will be resubmited.
Change-Id: I84c00cec41665211e0bd16ff9cc0c87073d6b897
Issue-ID: SDC-1484
Signed-off-by: Michael Lando <ml636r@att.com>
|
|
Issue-ID: SDC-1484
Change-Id: I3cf17454c533d3419c97af63cc6b5412976726fb
Signed-off-by: Sandra Koblosz <sandra.koblosz@nokia.com>
|
|
Change-Id: I12a5ca14a6d8a87e9316b9ff362eb131105f98a5
Issue-ID: SDC-1566
Signed-off-by: Michael Lando <ml636r@att.com>
|
|
This task is all about package name space change also make changes to pom for common module
Change-Id: Ie9bda0f958a9a05826c0374830cc9cb7d6d196b6
Issue-ID: SDC-1272
Signed-off-by: amitjai <amitjai@amdocs.com>
|