Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added tests for Tag service #2815

Merged
merged 1 commit into from
Mar 11, 2020

Conversation

zanpizmoht
Copy link
Contributor

Added tests for creating, deleting and editing tags, and assigning tags to devices.

Signed-off-by: zanpizmoht zan.pizmoht@comtrade.com

Test scenarios for Tags are added in this PR.

Related Issue
None

Description of the solution adopted
Test scenarios that are added in this PR are checking functionalities regarding Tags. All scenarios are added to TagService.feature. Scenarios are testing if Tags are correctly created, edited and deleted.

Screenshots
/

@Coduz Coduz added the Test Test related stuff. It's a dirty job, but someone needs to do that! label Jan 10, 2020
@Coduz Coduz self-requested a review January 10, 2020 16:09
@Coduz
Copy link
Contributor

Coduz commented Jan 10, 2020

CQ submitted: CQ 21397

@Coduz Coduz added the CQ pending This PR needs a CQ to be approved from Eclipse before merging. label Jan 10, 2020
@codecov
Copy link

codecov bot commented Jan 10, 2020

Codecov Report

Merging #2815 into develop will increase coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##             develop    #2815      +/-   ##
=============================================
+ Coverage      55.15%   55.19%   +0.03%     
- Complexity      2368     2373       +5     
=============================================
  Files           1021     1021              
  Lines          21871    21871              
  Branches        1977     1977              
=============================================
+ Hits           12063    12071       +8     
+ Misses          8897     8890       -7     
+ Partials         911      910       -1     
Impacted Files Coverage Δ Complexity Δ
...pse/kapua/broker/core/pool/JmsProducerWrapper.java 50.00% <0.00%> (-15.00%) 0.00% <0.00%> (ø%)
...rtz/persistence/KapuaQuartzConnectionProvider.java 87.50% <0.00%> (-12.50%) 4.00% <0.00%> (-1.00%)
...e/kapua/broker/core/BrokerJAXBContextProvider.java 70.00% <0.00%> (-10.00%) 0.00% <0.00%> (ø%)
.../DeviceManagementOperationRegistryServiceImpl.java 69.64% <0.00%> (+3.57%) 14.00% <0.00%> (+2.00%)
...rvice/device/registry/common/DeviceValidation.java 73.75% <0.00%> (+3.75%) 15.00% <0.00%> (+1.00%)
...a/service/device/registry/internal/DeviceImpl.java 71.85% <0.00%> (+5.92%) 62.00% <0.00%> (+3.00%)

Copy link
Contributor

@LeoNerdoG LeoNerdoG left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @zanpizmoht,
I have reviewed this PR and there are some changes to be made. Can you please fix the commented stuff? Thanks!

Copy link
Contributor

@LeoNerdoG LeoNerdoG left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @zanpizmoht,
I have reviewed this PR and there are some changes to be made. Can you please fix the commented stuff? Thanks!

@zanpizmoht zanpizmoht force-pushed the tagIntegrationTests branch 2 times, most recently from 8f87145 to 215637c Compare January 31, 2020 07:30
@Coduz
Copy link
Contributor

Coduz commented Mar 10, 2020

CQ Approved! 🎉

@Coduz Coduz added CQ approved The PR has passed CQ approvation and removed CQ pending This PR needs a CQ to be approved from Eclipse before merging. labels Mar 10, 2020
@zanpizmoht zanpizmoht force-pushed the tagIntegrationTests branch 2 times, most recently from 8a3660a to 358c9f4 Compare March 11, 2020 08:24
Added tests for creating, deleting and editing tags, and assigning tags to devices.

Signed-off-by: zanpizmoht <zan.pizmoht@comtrade.com>
@zanpizmoht zanpizmoht force-pushed the tagIntegrationTests branch from 358c9f4 to dec492f Compare March 11, 2020 11:55
@Coduz Coduz merged commit c28dadb into eclipse-kapua:develop Mar 11, 2020
@zanpizmoht zanpizmoht deleted the tagIntegrationTests branch March 26, 2020 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CQ approved The PR has passed CQ approvation Test Test related stuff. It's a dirty job, but someone needs to do that!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants