Age | Commit message (Collapse) | Author | Files | Lines |
|
- test labels are modified to set standards for writing tests
- labels are not expected to be technical
- labels need to be in simple language that ANYONE can understand
- labels should not have exact method names except for some mocking
- variables are modified to give highlight to ONLY important parameter of test
- wild cards are preferred when possible i.e. for exception cases
Issue-ID: CPS-588
Signed-off-by: emaclee <lee.anjella.macabuhay@est.tech>
Change-Id: I3d42471e7aa4bc61f962ad19cc1f237f6985a9a2
|
|
- Remove this unused method parameter "operation"
(https://sonarcloud.io/project/issues?resolved=false&types=CODE_SMELL&id=onap_cps-ncmp-dmi-plugin&open=AYEgIe-wVOe7T_lfxJBt)
- Change method to private
Issue-ID: CPS-475
Signed-off-by: emaclee <lee.anjella.macabuhay@est.tech>
Change-Id: I07e9b8150da935387f2a63c5b0e622d5157412de
|
|
-Added Async for passthrough running and operational
-Build will fail until cps is merged https://gerrit.onap.org/r/c/cps/+/128685
Issue-ID: CPS-830
Change-Id: Iedbfab109f5cd777a5be8eed7414758d0f5ec05c
Signed-off-by: JosephKeenan <joseph.keenan@est.tech>
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
Signed-off-by: JosephKeenan <joseph.keenan@est.tech>
|
|
- logback encoder and sleuth dependencies are added.
- New appender is added to log JSON format content on console.
Issue-ID: CPS-986
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
Change-Id: Id7e4d836188da54b94144076a8755e4a2643a7ad
|
|
Issue-ID: CPS-987
Change-Id: Ie4c40b91e45ad325658a32eb474840af761e0029
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
|
|
- Producer configuration and Topic related information.
- Accepted topicParameter and request id from NCMP to process async
- Replacing Springfox and using Springdoc instead
Issue-ID: CPS-829
Change-Id: I369b5ec6c16318220bb218701006918a0bf21419
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
|
|
- update to oparent 3.3.0 to allow for checkstyle @SupressWarnings (not used in the end)
- refactor code to minimize unused branches (no more switch-statements :-))
- Added test for neccessary but uncovered branches
Issue-ID: CPS-475
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
Change-Id: I03c7355a7e9d19f57523a65fbff45c9d8f1c9e07
|
|
Get endpoint only accepts application/json.
Issue-ID: CPS-957
Change-Id: I7b9dd00f293f183dc76623c8e32699e8e90fd17c
Signed-off-by: puthuparambil.aditya <aditya.puthuparambil@bell.ca>
|
|
Issue-ID: CPS-475
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
Change-Id: Idc2752dae882dd994892d91b7038ae8ef81e532e
|
|
- Used UriComponentBuilder is populate urls.
- Removed incorrect import of StringUtils
Issue-ID: CPS-569
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
Change-Id: I11c6ad2fec528f9792c05405a7242128c0a6d4e2
|
|
Change-Id: Ie87216fa6cb120bea685286b7f0dacd8e22f603b
Signed-off-by: tragait <rahul.tyagi@est.tech>
Issue-ID: CPS-641
|
|
-Extracted out relevant method to separate class
-Added comprehensive tests around relevant methods
Issue-ID: CPS-788
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
Change-Id: Icb2104ca606b28f1904ead368bcde27f578e7e53
|
|
-Add Update operation for Passthrough Running
-Use POST operatiosn for all methods as agrred
-refactor to simplify and beter reuse of common methods
-add delete operation for passthrough running
Issue-ID: CPS-637
Issue-ID: CPS-638
Change-Id: I441181f977dee2bc0be944662465d868595ea452
Signed-off-by: DylanB95EST <dylan.byrne@est.tech>
|
|
- Used jayway to convert sdnc response to POJO to clean up classes
- Moved SDNC error handling to the SDNCOperation
- Added SdncException
- Added spring-security test Mock user in the controllerSpec
Issue-ID: CPS-571
Signed-off-by: Renu Kumari <renu.kumari@bell.ca>
Change-Id: Ice91e48536dd2e309721440b61d37a49ce43b403
|
|
Issue-ID: CPS-706
Signed-off-by: niamhcore <niamh.core@est.tech>
Change-Id: I92d241c4ccde77aaaf8aa5a3903437016edf65d0
|
|
Issue-ID: CPS-678
Signed-off-by: tragait <rahul.tyagi@est.tech>
Change-Id: I4ac72da512e2c7883920907137b8834ce20d4528
|
|
|
|
-removed invalid jacoco exclude
-removed strange (generated/accidental?) comment in main pom header
-moved manually coded 'models' package (included in coverage now) to separate it
from swagger generated package/classes with same name (exluded in coverage)
-added some missing test scenarios
-increased coverage limit from 70 to 98% to prevent regression
Issue-ID: CPS-475
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
Change-Id: I2f1c276c543926d2d259e33b418f21de4abfea96
|
|
* Updating yaml
* Update jib-maven-plugin version
Issue-ID: CPS-679
Signed-off-by: niamhcore <niamh.core@est.tech>
Change-Id: Ie2f49bd8e22723ca4dcd2c20ed39722f22c281c1
|
|
Issue-ID: CPS-653
Signed-off-by: tragait <rahul.tyagi@est.tech>
Change-Id: I6b5f0fc74ea4e70e65b05ed353e25311ed36135e
|
|
Issue-ID: CPS-657
Signed-off-by: niamhcore <niamh.core@est.tech>
Change-Id: Ifcd22b6e82aaf4993f9071ad56b524fe620afaf7
|
|
Also, make dmi service url configurable,
change name of dmi creds env var
Issue-ID: CPS-675
Issue-ID: CPS-634
Signed-off-by: tragait <rahul.tyagi@est.tech>
Change-Id: I6fe794b3e65b6a3aae2a1698ea64a925238a18d3
|
|
Issue-ID: CPS-659
Signed-off-by: niamhcore <niamh.core@est.tech>
Change-Id: I4ebc4f68604efe78efb951989c2fb021443c36c9
|
|
This commit also have fix in application yaml for
correct ncmp api path for registration url.
Issue-ID: CPS-634
Issue-ID: CPS-617
Signed-off-by: tragait <rahul.tyagi@est.tech>
Change-Id: I465991492a01092e28b97583f84ed959c54ffaa6
|
|
Issue-ID: CPS-634
Signed-off-by: tragait <rahul.tyagi@est.tech>
Change-Id: Ib3d9d7cab12c77332d799b89e73e5a9b934a4473
|
|
|
|
-Modified dmi-base-path from /dmi/api to /dmi in src/main/resources/application.yml &
src/test/resources/application/yml
Issue-ID: CPS-617
Signed-off-by: JosephKeenan <joseph.keenan@est.tech>
Change-Id: Ic4e76731ee44abb6e6f260d7c8adb1023ebe4e34
|
|
|
|
Issue-ID: CPS-589
Signed-off-by: niamhcore <niamh.core@est.tech>
Change-Id: I43f6971f416c6aa3ac1b1a56626930ad16288680
|
|
|
|
Issue-ID: CPS-573
Signed-off-by: tragait <rahul.tyagi@est.tech>
Change-Id: Idb2186e6f3275530513bbd5e788c98b4ceda9d10
|
|
Issue-ID: CPS-565
Signed-off-by: niamhcore <niamh.core@est.tech>
Change-Id: I8cb28cf60622e5a9c0211b4fa7db2f255ede165a
|
|
Issue-ID: CPS-574
Signed-off-by: niamhcore <niamh.core@est.tech>
Change-Id: I15e2ad500d2bbf7a1d408b8a852287e55bddbecb
|
|
Issue-ID: CPS-561
Signed-off-by: tragait <rahul.tyagi@est.tech>
Change-Id: Ia92a97296e2c3e84d0b6f6d9d1d0daf1e8f46b55
|
|
Issue-ID: CPS-531
Signed-off-by: niamhcore <niamh.core@est.tech>
Change-Id: I918be4db5066d92b23e25fc7fbc22d4535fafc8c
|
|
Issue-ID: CPS-486
Signed-off-by: tragait <rahul.tyagi@est.tech>
Change-Id: Icf48fa93ea1f0d8a27d2e7e1ab0cfd6096a765ec
|
|
Updating openapi to add a new rest endpoint
Updating restconf client to support post with json
Adding a ModuleResourceNotFound exception
Adding a test util class
Fixing merge conflict
Refactoring SDNC operations
Issue-ID: CPS-484
Signed-off-by: niamhcore <niamh.core@est.tech>
Change-Id: Id76dfe4cb12053771883e0271153d7bf7cd98548
|
|
Issue-ID: CPS-406
Change-Id: I64e88643221403e117146443287d03788fb71c3e
Signed-off-by: tragait <rahul.tyagi@est.tech>
|
|
Issue-ID: CPS-483
Change-Id: Ib9b730cabeba308f11db31ef1b45bbd92e3a6ed5
Signed-off-by: tragait <rahul.tyagi@est.tech>
|
|
- Refactor ncmp/dmi package
Issue-ID: CPS-405
Issue-ID: CPS-499
Signed-off-by: niamhcore <niamh.core@est.tech>
Change-Id: I0a7ab9bb0ca3861b171388613c44f7bc6e7f9aa9
|
|
Issue-ID: CPS-489
Signed-off-by: lukegleeson <luke.gleeson@est.tech>
Change-Id: I21e088af64576ea0486ac959ba615bd8adebb2f9
|
|
Issue-ID: CPS-432
Issue-ID: CPS-403
Signed-off-by: tragait <rahul.tyagi@est.tech>
Change-Id: I4ada1e4927a3726500396da64c454d9937f95bcf
Signed-off-by: shivasubedi <shiva.subedi@est.tech>
|
|
Spring Application class
Issue-ID: CPS-431
Change-Id: I6fd4379df6133108c1f8f2a5339f990f16773ad6
Signed-off-by: DylanB95EST <dylan.byrne@est.tech>
|