community
/
device-mgt-core
12
0
Fork 47
Code Issues Pull Requests 15 Packages Projects Releases Wiki Activity
Labels Milestones

Add missing test cases for DevicePersistTests and GroupPersistTests #408

Merged
tcdlpds merged 1 commits from pramilaniroshan/device-mgt-core:add-missing-test-cases into master 8 months ago
Conversation 0 Commits 1 Files Changed 3
pramilaniroshan commented 8 months ago

https://roadmap.entgra.net/issues/11282

https://roadmap.entgra.net/issues/11282
pramilaniroshan added 1 commit 8 months ago
pramilaniroshan added 1 commit 8 months ago
tcdlpds requested changes 8 months ago
components/device-mgt/io.entgra.device.mgt.core.device.mgt.core/src/test/java/io/entgra/device/mgt/core/device/mgt/core/dao/DevicePersistTests.java Outdated
Device device = TestDataHolder.initialTestDevice;
try {
DeviceManagementDAOFactory.beginTransaction();
Device results = deviceDAO.getDevice(device.getDeviceIdentifier(),TestDataHolder.SUPER_TENANT_ID);
tcdlpds commented 8 months ago
Owner

Format the source

Format the source
pramilaniroshan marked this conversation as resolved
components/device-mgt/io.entgra.device.mgt.core.device.mgt.core/src/test/java/io/entgra/device/mgt/core/device/mgt/core/dao/DevicePersistTests.java Outdated
try {
DeviceManagementDAOFactory.beginTransaction();
DeviceIdentifier deviceIdentifier = new DeviceIdentifier(device.getDeviceIdentifier(), device.getType());
Device results = deviceDAO.getDevice(deviceIdentifier,TestDataHolder.SUPER_TENANT_ID);
tcdlpds commented 8 months ago
Owner

Format the source

Format the source
pramilaniroshan marked this conversation as resolved
components/device-mgt/io.entgra.device.mgt.core.device.mgt.core/src/test/java/io/entgra/device/mgt/core/device/mgt/core/dao/DevicePersistTests.java Outdated
DeviceIdentifier deviceIdentifier = new DeviceIdentifier(device.getDeviceIdentifier(), device.getType());
try {
DeviceManagementDAOFactory.beginTransaction();
Device results = deviceDAO.getDevice(deviceIdentifier,TestDataHolder.OWNER, TestDataHolder.SUPER_TENANT_ID);
tcdlpds commented 8 months ago
Owner

Format the source

Format the source
pramilaniroshan marked this conversation as resolved
components/device-mgt/io.entgra.device.mgt.core.device.mgt.core/src/test/java/io/entgra/device/mgt/core/device/mgt/core/dao/DevicePersistTests.java Outdated
pr.setDeviceType(device.getType());
try {
DeviceManagementDAOFactory.beginTransaction();
List<Device> results = deviceDAO.getAppNotInstalledDevices(pr, TestDataHolder.SUPER_TENANT_ID,"com.google.calc" ,"1.0.0");
tcdlpds commented 8 months ago
Owner

Format the source

Format the source
pramilaniroshan marked this conversation as resolved
components/device-mgt/io.entgra.device.mgt.core.device.mgt.core/src/test/java/io/entgra/device/mgt/core/device/mgt/core/dao/DevicePersistTests.java Outdated
pr.setDeviceType(device.getType());
try {
DeviceManagementDAOFactory.beginTransaction();
int results = deviceDAO.getCountOfAppNotInstalledDevices(pr, TestDataHolder.SUPER_TENANT_ID,"com.google.calc" ,"1.0.0");
tcdlpds commented 8 months ago
Owner

Format the source

Format the source
pramilaniroshan marked this conversation as resolved
components/device-mgt/io.entgra.device.mgt.core.device.mgt.core/src/test/java/io/entgra/device/mgt/core/device/mgt/core/dao/DevicePersistTests.java Outdated
PaginationRequest pr = new PaginationRequest(0, 10);
try {
DeviceManagementDAOFactory.beginTransaction();
List<Device> results = deviceDAO.getDevicesByEncryptionStatus(pr, TestDataHolder.SUPER_TENANT_ID,false);
tcdlpds commented 8 months ago
Owner

Format the source

Format the source
pramilaniroshan marked this conversation as resolved
components/device-mgt/io.entgra.device.mgt.core.device.mgt.core/src/test/java/io/entgra/device/mgt/core/device/mgt/core/dao/DevicePersistTests.java Outdated
public void getCountOfDevicesByEncryptionStatus() throws DeviceManagementDAOException, TransactionManagementException {
try {
DeviceManagementDAOFactory.beginTransaction();
int results = deviceDAO.getCountOfDevicesByEncryptionStatus(TestDataHolder.SUPER_TENANT_ID,true);
tcdlpds commented 8 months ago
Owner

Format the source

Format the source
pramilaniroshan marked this conversation as resolved
components/device-mgt/io.entgra.device.mgt.core.device.mgt.core/src/test/java/io/entgra/device/mgt/core/device/mgt/core/dao/GroupPersistTests.java Outdated
GroupManagementDAOFactory.beginTransaction();
groupDAO.getAllDevicesOfGroup(deviceGroup.getName(), deviceStatus, TestDataHolder.SUPER_TENANT_ID);
GroupManagementDAOFactory.commitTransaction();
GroupManagementDAOFactory.closeConnection();
tcdlpds commented 8 months ago
Owner

move the close connection into finally block

move the close connection into finally block
pramilaniroshan marked this conversation as resolved
components/device-mgt/io.entgra.device.mgt.core.device.mgt.core/src/test/java/io/entgra/device/mgt/core/device/mgt/core/dao/GroupPersistTests.java Outdated
GroupManagementDAOFactory.beginTransaction();
groupDAO.getAllDevicesOfGroup(deviceGroup.getName(), TestDataHolder.SUPER_TENANT_ID);
GroupManagementDAOFactory.commitTransaction();
GroupManagementDAOFactory.closeConnection();
tcdlpds commented 8 months ago
Owner

Move the close connection into finally block. Check in catch blocks as well.

Move the close connection into finally block. Check in catch blocks as well.
pramilaniroshan marked this conversation as resolved
components/device-mgt/io.entgra.device.mgt.core.device.mgt.core/src/test/java/io/entgra/device/mgt/core/device/mgt/core/dao/GroupPersistTests.java Outdated
GroupManagementDAOFactory.beginTransaction();
groupDAO.getGroupUnassignedDevices(pr, groupNames);
GroupManagementDAOFactory.commitTransaction();
GroupManagementDAOFactory.closeConnection();
tcdlpds commented 8 months ago
Owner

Move the close connection into finally block.

Move the close connection into finally block.
pramilaniroshan marked this conversation as resolved
components/device-mgt/io.entgra.device.mgt.core.device.mgt.core/src/test/java/io/entgra/device/mgt/core/device/mgt/core/dao/GroupPersistTests.java Outdated
GroupManagementDAOFactory.beginTransaction();
groupDAO.getOwnGroups(TestDataHolder.OWNER, TestDataHolder.SUPER_TENANT_ID);
GroupManagementDAOFactory.commitTransaction();
GroupManagementDAOFactory.closeConnection();
tcdlpds commented 8 months ago
Owner

Move the close connection into finally block.

Move the close connection into finally block.
pramilaniroshan marked this conversation as resolved
components/device-mgt/io.entgra.device.mgt.core.device.mgt.core/src/test/java/io/entgra/device/mgt/core/device/mgt/core/dao/GroupPersistTests.java Outdated
GroupManagementDAOFactory.beginTransaction();
groupDAO.getOwnGroupIds(TestDataHolder.OWNER, TestDataHolder.SUPER_TENANT_ID);
GroupManagementDAOFactory.commitTransaction();
GroupManagementDAOFactory.closeConnection();
tcdlpds commented 8 months ago
Owner

Move the close connection into finally block

Move the close connection into finally block
pramilaniroshan marked this conversation as resolved
components/device-mgt/io.entgra.device.mgt.core.device.mgt.core/src/test/java/io/entgra/device/mgt/core/device/mgt/core/dao/GroupPersistTests.java Outdated
GroupManagementDAOFactory.beginTransaction();
groupDAO.getDeviceCount(deviceGroup.getGroupId(), TestDataHolder.SUPER_TENANT_ID);
GroupManagementDAOFactory.commitTransaction();
GroupManagementDAOFactory.closeConnection();
tcdlpds commented 8 months ago
Owner

Move the close connection into finally block and check all the methods in this file and do the modifications.

Move the close connection into finally block and check all the methods in this file and do the modifications.
pramilaniroshan marked this conversation as resolved
components/device-mgt/io.entgra.device.mgt.core.device.mgt.core/src/test/java/io/entgra/device/mgt/core/device/mgt/core/dao/GroupPersistTests.java Outdated
} catch (GroupManagementDAOException e) {
GroupManagementDAOFactory.rollbackTransaction();
GroupManagementDAOFactory.closeConnection();
String msg = "Error occurred while getting group count for '" +TestDataHolder.SUPER_TENANT_ID+ "'.";
tcdlpds commented 8 months ago
Owner

Format the source

Format the source
pramilaniroshan marked this conversation as resolved
components/device-mgt/io.entgra.device.mgt.core.device.mgt.core/src/test/java/io/entgra/device/mgt/core/device/mgt/core/dao/GroupPersistTests.java Outdated
} catch (GroupManagementDAOException e) {
GroupManagementDAOFactory.rollbackTransaction();
GroupManagementDAOFactory.closeConnection();
String msg = "Error occurred while getting group count for '" +TestDataHolder.SUPER_TENANT_ID+ "'.";
tcdlpds commented 8 months ago
Owner

Format the source

Format the source
pramilaniroshan marked this conversation as resolved
components/device-mgt/io.entgra.device.mgt.core.device.mgt.core/src/test/java/io/entgra/device/mgt/core/device/mgt/core/dao/GroupPersistTests.java Outdated
} catch (GroupManagementDAOException e) {
GroupManagementDAOFactory.rollbackTransaction();
GroupManagementDAOFactory.closeConnection();
String msg = "Error occurred while getting groups for '" +TestDataHolder.SUPER_TENANT_ID+ "'.";
tcdlpds commented 8 months ago
Owner

Format the source

Format the source
pramilaniroshan marked this conversation as resolved
pramilaniroshan force-pushed add-missing-test-cases from 178b249c04 to d443fef6e0 8 months ago
tcdlpds merged commit e84a479c23 into master 8 months ago

Reviewers

tcdlpds requested changes 8 months ago
The pull request has been merged as e84a479c23.
Sign in to join this conversation.
Reviewers
Request review
No reviewers
tcdlpds
Labels
Apply labels
Clear labels
pending upstream
PR depends on pending upstream dependency
No Label pending upstream
Milestone
Set milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Assign users
Clear assignees
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: community/device-mgt-core#408
Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch 'pramilaniroshan/device-mgt-core:add-missing-test-cases'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes