From 330f059a09f524ca55753001067b0a945c89c4b4 Mon Sep 17 00:00:00 2001 From: harshanl Date: Sat, 5 Sep 2015 01:26:52 +0530 Subject: [PATCH 1/4] Added App-Mgt config testcases --- ...AppManagementConfigurationManagerTest.java | 96 +++++++++++++++++++ ...management-config-no-client-key-secret.xml | 23 +++++ ...ormed-app-management-config-no-enabled.xml | 24 +++++ .../config/schema/app-mgt-config-schema.xsd | 31 ++++++ .../src/test/resources/testng.xml | 1 + .../impl/ConfigurationServiceImpl.java | 2 +- 6 files changed, 176 insertions(+), 1 deletion(-) create mode 100644 components/device-mgt/org.wso2.carbon.device.mgt.core/src/test/java/org/wso2/carbon/device/mgt/core/app/mgt/AppManagementConfigurationManagerTest.java create mode 100644 components/device-mgt/org.wso2.carbon.device.mgt.core/src/test/resources/config/malformed-app-management-config-no-client-key-secret.xml create mode 100644 components/device-mgt/org.wso2.carbon.device.mgt.core/src/test/resources/config/malformed-app-management-config-no-enabled.xml create mode 100644 components/device-mgt/org.wso2.carbon.device.mgt.core/src/test/resources/config/schema/app-mgt-config-schema.xsd diff --git a/components/device-mgt/org.wso2.carbon.device.mgt.core/src/test/java/org/wso2/carbon/device/mgt/core/app/mgt/AppManagementConfigurationManagerTest.java b/components/device-mgt/org.wso2.carbon.device.mgt.core/src/test/java/org/wso2/carbon/device/mgt/core/app/mgt/AppManagementConfigurationManagerTest.java new file mode 100644 index 0000000000..5ff931837f --- /dev/null +++ b/components/device-mgt/org.wso2.carbon.device.mgt.core/src/test/java/org/wso2/carbon/device/mgt/core/app/mgt/AppManagementConfigurationManagerTest.java @@ -0,0 +1,96 @@ +/* + * Copyright (c) 2015, WSO2 Inc. (http://www.wso2.org) All Rights Reserved. + * + * WSO2 Inc. licenses this file to you under the Apache License, + * Version 2.0 (the "License"); you may not use this file except + * in compliance with the License. + * you may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, + * software distributed under the License is distributed on an + * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY + * KIND, either express or implied. See the License for the + * specific language governing permissions and limitations + * under the License. + */ + +package org.wso2.carbon.device.mgt.core.app.mgt; + +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; +import org.testng.Assert; +import org.testng.annotations.BeforeClass; +import org.testng.annotations.Test; +import org.wso2.carbon.device.mgt.core.app.mgt.config.AppManagementConfig; +import org.xml.sax.SAXException; +import org.xml.sax.SAXParseException; + +import javax.xml.XMLConstants; +import javax.xml.bind.JAXBContext; +import javax.xml.bind.JAXBException; +import javax.xml.bind.Unmarshaller; +import javax.xml.validation.Schema; +import javax.xml.validation.SchemaFactory; +import java.io.File; + +public class AppManagementConfigurationManagerTest { + + private static final Log log = LogFactory.getLog(AppManagementConfigurationManagerTest.class); + private static final String MALFORMED_TEST_CONFIG_LOCATION_NO_ENABLED = + "./src/test/resources/config/malformed-app-management-config-no-enabled.xml"; + private static final String MALFORMED_TEST_CONFIG_LOCATION_NO_CLIENT_KEY = + "./src/test/resources/config/malformed-app-management-config-no-client-key-secret.xml"; + private static final String TEST_CONFIG_SCHEMA_LOCATION = + "./src/test/resources/config/schema/app-mgt-config-schema.xsd"; + + private Schema schema; + + @BeforeClass + private void initSchema() { + File deviceManagementSchemaConfig = new File(AppManagementConfigurationManagerTest.TEST_CONFIG_SCHEMA_LOCATION); + SchemaFactory factory = SchemaFactory.newInstance(XMLConstants.W3C_XML_SCHEMA_NS_URI); + try { + schema = factory.newSchema(deviceManagementSchemaConfig); + } catch (SAXException e) { + Assert.fail("Invalid schema found", e); + } + } + + @Test() + public void testMandateEnabledElement() { + File malformedConfig = + new File(AppManagementConfigurationManagerTest.MALFORMED_TEST_CONFIG_LOCATION_NO_ENABLED); + this.validateMalformedConfig(malformedConfig); + } + + @Test() + public void testMandateClientKeySecretElement() { + File malformedConfig = + new File(AppManagementConfigurationManagerTest.MALFORMED_TEST_CONFIG_LOCATION_NO_CLIENT_KEY); + this.validateMalformedConfig(malformedConfig); + } + + private void validateMalformedConfig(File malformedConfig) { + try { + JAXBContext ctx = JAXBContext.newInstance(AppManagementConfig.class); + Unmarshaller um = ctx.createUnmarshaller(); + um.setSchema(this.getSchema()); + um.unmarshal(malformedConfig); + Assert.assertTrue(false); + } catch (JAXBException e) { + Throwable linkedException = e.getLinkedException(); + if (!(linkedException instanceof SAXParseException)) { + log.error("Unexpected error occurred while unmarshalling app management config", e); + Assert.assertTrue(false); + } + log.error("JAXB parser occurred while unmarsharlling app management config", e); + Assert.assertTrue(true); + } + } + + private Schema getSchema() { + return schema; + } +} diff --git a/components/device-mgt/org.wso2.carbon.device.mgt.core/src/test/resources/config/malformed-app-management-config-no-client-key-secret.xml b/components/device-mgt/org.wso2.carbon.device.mgt.core/src/test/resources/config/malformed-app-management-config-no-client-key-secret.xml new file mode 100644 index 0000000000..ef7a146cdd --- /dev/null +++ b/components/device-mgt/org.wso2.carbon.device.mgt.core/src/test/resources/config/malformed-app-management-config-no-client-key-secret.xml @@ -0,0 +1,23 @@ + + + + + true + http:/www.google.com + diff --git a/components/device-mgt/org.wso2.carbon.device.mgt.core/src/test/resources/config/malformed-app-management-config-no-enabled.xml b/components/device-mgt/org.wso2.carbon.device.mgt.core/src/test/resources/config/malformed-app-management-config-no-enabled.xml new file mode 100644 index 0000000000..2087b37891 --- /dev/null +++ b/components/device-mgt/org.wso2.carbon.device.mgt.core/src/test/resources/config/malformed-app-management-config-no-enabled.xml @@ -0,0 +1,24 @@ + + + + + http:/www.google.com + ConsumerKey + ConsumerSecret + diff --git a/components/device-mgt/org.wso2.carbon.device.mgt.core/src/test/resources/config/schema/app-mgt-config-schema.xsd b/components/device-mgt/org.wso2.carbon.device.mgt.core/src/test/resources/config/schema/app-mgt-config-schema.xsd new file mode 100644 index 0000000000..2b0ec374aa --- /dev/null +++ b/components/device-mgt/org.wso2.carbon.device.mgt.core/src/test/resources/config/schema/app-mgt-config-schema.xsd @@ -0,0 +1,31 @@ + + + + + + + + + + + + + + \ No newline at end of file diff --git a/components/device-mgt/org.wso2.carbon.device.mgt.core/src/test/resources/testng.xml b/components/device-mgt/org.wso2.carbon.device.mgt.core/src/test/resources/testng.xml index 4622a58873..4da53d1247 100644 --- a/components/device-mgt/org.wso2.carbon.device.mgt.core/src/test/resources/testng.xml +++ b/components/device-mgt/org.wso2.carbon.device.mgt.core/src/test/resources/testng.xml @@ -28,6 +28,7 @@ + diff --git a/components/oauth-extensions/dynamic-client-manager/src/main/java/org/wso2/carbon/identity/oauth/extension/impl/ConfigurationServiceImpl.java b/components/oauth-extensions/dynamic-client-manager/src/main/java/org/wso2/carbon/identity/oauth/extension/impl/ConfigurationServiceImpl.java index 87f36b6fbf..840e0ee935 100644 --- a/components/oauth-extensions/dynamic-client-manager/src/main/java/org/wso2/carbon/identity/oauth/extension/impl/ConfigurationServiceImpl.java +++ b/components/oauth-extensions/dynamic-client-manager/src/main/java/org/wso2/carbon/identity/oauth/extension/impl/ConfigurationServiceImpl.java @@ -30,4 +30,4 @@ public class ConfigurationServiceImpl implements ConfigurationService { return null; } -} +} \ No newline at end of file From ccffdb225246d1507f972969ac73d4118b0932fd Mon Sep 17 00:00:00 2001 From: Dilshan Edirisuriya Date: Mon, 7 Sep 2015 15:15:59 +0530 Subject: [PATCH 2/4] Get attribute implementation --- .../org.wso2.carbon.certificate.mgt.core/pom.xml | 1 + .../carbon/certificate/mgt/core/impl/CertificateGenerator.java | 2 ++ 2 files changed, 3 insertions(+) diff --git a/components/certificate-mgt/org.wso2.carbon.certificate.mgt.core/pom.xml b/components/certificate-mgt/org.wso2.carbon.certificate.mgt.core/pom.xml index 81d6be9ba4..f22bffeab8 100644 --- a/components/certificate-mgt/org.wso2.carbon.certificate.mgt.core/pom.xml +++ b/components/certificate-mgt/org.wso2.carbon.certificate.mgt.core/pom.xml @@ -62,6 +62,7 @@ org.bouncycastle.asn1, org.bouncycastle.asn1.x500, org.bouncycastle.asn1.x509, + org.bouncycastle.asn1.pkcs, org.bouncycastle.cert, org.bouncycastle.cert.jcajce, org.bouncycastle.cms, diff --git a/components/certificate-mgt/org.wso2.carbon.certificate.mgt.core/src/main/java/org/wso2/carbon/certificate/mgt/core/impl/CertificateGenerator.java b/components/certificate-mgt/org.wso2.carbon.certificate.mgt.core/src/main/java/org/wso2/carbon/certificate/mgt/core/impl/CertificateGenerator.java index 7a2538af22..3aed3f4004 100755 --- a/components/certificate-mgt/org.wso2.carbon.certificate.mgt.core/src/main/java/org/wso2/carbon/certificate/mgt/core/impl/CertificateGenerator.java +++ b/components/certificate-mgt/org.wso2.carbon.certificate.mgt.core/src/main/java/org/wso2/carbon/certificate/mgt/core/impl/CertificateGenerator.java @@ -20,6 +20,7 @@ package org.wso2.carbon.certificate.mgt.core.impl; import org.apache.commons.codec.binary.Base64; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; +import org.bouncycastle.asn1.pkcs.Attribute; import org.bouncycastle.asn1.x500.X500Name; import org.bouncycastle.asn1.x509.KeyUsage; import org.bouncycastle.asn1.x509.X509Extension; @@ -344,6 +345,7 @@ public class CertificateGenerator { Date validityEndDate = commonUtil.getValidityEndDate(); X500Name certSubject = request.getSubject(); + Attribute attributes[] = request.getAttributes(); if (certSubject == null) { certSubject = new X500Name(ConfigurationUtil.DEFAULT_PRINCIPAL); From bf0e95fa6c15a141a20667282ec85a47fae59b07 Mon Sep 17 00:00:00 2001 From: harshanl Date: Mon, 7 Sep 2015 18:14:20 +0530 Subject: [PATCH 3/4] Fixed notification service issues --- .../DeviceManagementServiceComponent.java | 2 + .../mgt/dao/impl/NotificationDAOImpl.java | 21 +++------- .../src/test/resources/log4j.properties | 32 +++++++++++++++ .../src/test/resources/testng.xml | 41 +++++++++++++++++++ 4 files changed, 81 insertions(+), 15 deletions(-) create mode 100644 components/oauth-extensions/org.wso2.carbon.device.mgt.oauth.extension/src/test/resources/log4j.properties create mode 100644 components/oauth-extensions/org.wso2.carbon.device.mgt.oauth.extension/src/test/resources/testng.xml diff --git a/components/device-mgt/org.wso2.carbon.device.mgt.core/src/main/java/org/wso2/carbon/device/mgt/core/internal/DeviceManagementServiceComponent.java b/components/device-mgt/org.wso2.carbon.device.mgt.core/src/main/java/org/wso2/carbon/device/mgt/core/internal/DeviceManagementServiceComponent.java index 7b34760c80..bb916522ed 100644 --- a/components/device-mgt/org.wso2.carbon.device.mgt.core/src/main/java/org/wso2/carbon/device/mgt/core/internal/DeviceManagementServiceComponent.java +++ b/components/device-mgt/org.wso2.carbon.device.mgt.core/src/main/java/org/wso2/carbon/device/mgt/core/internal/DeviceManagementServiceComponent.java @@ -42,6 +42,7 @@ import org.wso2.carbon.device.mgt.core.config.datasource.DataSourceConfig; import org.wso2.carbon.device.mgt.core.config.tenant.TenantConfigurationManagementServiceImpl; import org.wso2.carbon.device.mgt.core.dao.DeviceManagementDAOFactory; import org.wso2.carbon.device.mgt.core.notification.mgt.NotificationManagementServiceImpl; +import org.wso2.carbon.device.mgt.core.notification.mgt.dao.NotificationManagementDAOFactory; import org.wso2.carbon.device.mgt.core.operation.mgt.OperationManagerImpl; import org.wso2.carbon.device.mgt.core.operation.mgt.dao.OperationManagementDAOFactory; import org.wso2.carbon.device.mgt.core.service.DeviceManagementProviderService; @@ -117,6 +118,7 @@ public class DeviceManagementServiceComponent { DataSourceConfig dsConfig = config.getDeviceManagementConfigRepository().getDataSourceConfig(); DeviceManagementDAOFactory.init(dsConfig); + NotificationManagementDAOFactory.init(dsConfig); /*Initialize Operation Manager*/ this.initOperationsManager(); diff --git a/components/device-mgt/org.wso2.carbon.device.mgt.core/src/main/java/org/wso2/carbon/device/mgt/core/notification/mgt/dao/impl/NotificationDAOImpl.java b/components/device-mgt/org.wso2.carbon.device.mgt.core/src/main/java/org/wso2/carbon/device/mgt/core/notification/mgt/dao/impl/NotificationDAOImpl.java index ab1152dc92..2a1208ff12 100644 --- a/components/device-mgt/org.wso2.carbon.device.mgt.core/src/main/java/org/wso2/carbon/device/mgt/core/notification/mgt/dao/impl/NotificationDAOImpl.java +++ b/components/device-mgt/org.wso2.carbon.device.mgt.core/src/main/java/org/wso2/carbon/device/mgt/core/notification/mgt/dao/impl/NotificationDAOImpl.java @@ -42,11 +42,10 @@ public class NotificationDAOImpl implements NotificationDAO { public int addNotification(int deviceId, int tenantId, Notification notification) throws NotificationManagementException { Connection conn; - PreparedStatement stmt; + PreparedStatement stmt = null; ResultSet rs; int notificationId = -1; try { - NotificationManagementDAOFactory.beginTransaction(); conn = NotificationManagementDAOFactory.getConnection(); String sql = "INSERT INTO DM_NOTIFICATION(DEVICE_ID, OPERATION_ID, STATUS, DESCRIPTION, TENANT_ID) " + @@ -62,14 +61,12 @@ public class NotificationDAOImpl implements NotificationDAO { if (rs.next()) { notificationId = rs.getInt(1); } - NotificationManagementDAOFactory.commitTransaction(); } catch (Exception e) { - NotificationManagementDAOFactory.rollbackTransaction(); throw new NotificationManagementException("Error occurred while adding the " + "Notification for device id : " + deviceId, e); } finally { - NotificationManagementDAOFactory.closeConnection(); + NotificationDAOUtil.cleanupResources(stmt, null); } return notificationId; } @@ -78,10 +75,9 @@ public class NotificationDAOImpl implements NotificationDAO { public int updateNotification(Notification notification) throws NotificationManagementException { Connection conn; - PreparedStatement stmt; + PreparedStatement stmt = null; int rows; try { - NotificationManagementDAOFactory.beginTransaction(); conn = NotificationManagementDAOFactory.getConnection(); String sql = "UPDATE DM_NOTIFICATION SET OPERATION_ID = ?, STATUS = ?, DESCRIPTION = ? " + "WHERE NOTIFICATION_ID = ?"; @@ -91,13 +87,11 @@ public class NotificationDAOImpl implements NotificationDAO { stmt.setString(3, notification.getDescription()); stmt.setInt(4, notification.getNotificationId()); rows = stmt.executeUpdate(); - NotificationManagementDAOFactory.commitTransaction(); } catch (Exception e) { - NotificationManagementDAOFactory.rollbackTransaction(); throw new NotificationManagementException("Error occurred while updating the " + "Notification id : " + notification.getNotificationId(), e); } finally { - NotificationManagementDAOFactory.closeConnection(); + NotificationDAOUtil.cleanupResources(stmt, null); } return rows; } @@ -106,23 +100,20 @@ public class NotificationDAOImpl implements NotificationDAO { public int updateNotificationStatus(int notificationId, Notification.Status status) throws NotificationManagementException { Connection conn; - PreparedStatement stmt; + PreparedStatement stmt = null; int rows; try { - NotificationManagementDAOFactory.beginTransaction(); conn = NotificationManagementDAOFactory.getConnection(); String sql = "UPDATE DM_NOTIFICATION SET STATUS = ? WHERE NOTIFICATION_ID = ?"; stmt = conn.prepareStatement(sql); stmt.setString(1, status.toString()); stmt.setInt(2, notificationId); rows = stmt.executeUpdate(); - NotificationManagementDAOFactory.commitTransaction(); } catch (Exception e) { - NotificationManagementDAOFactory.rollbackTransaction(); throw new NotificationManagementException("Error occurred while updating the status of " + "Notification id : " + notificationId, e); } finally { - NotificationManagementDAOFactory.closeConnection(); + NotificationDAOUtil.cleanupResources(stmt, null); } return rows; } diff --git a/components/oauth-extensions/org.wso2.carbon.device.mgt.oauth.extension/src/test/resources/log4j.properties b/components/oauth-extensions/org.wso2.carbon.device.mgt.oauth.extension/src/test/resources/log4j.properties new file mode 100644 index 0000000000..7da6d6c9e1 --- /dev/null +++ b/components/oauth-extensions/org.wso2.carbon.device.mgt.oauth.extension/src/test/resources/log4j.properties @@ -0,0 +1,32 @@ +# +# Copyright 2009 WSO2, Inc. (http://wso2.com) +# +# Licensed under the Apache License, Version 2.0 (the "License"); +# you may not use this file except in compliance with the License. +# You may obtain a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, software +# distributed under the License is distributed on an "AS IS" BASIS, +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +# See the License for the specific language governing permissions and +# limitations under the License. +# + +# +# This is the log4j configuration file used by WSO2 Carbon +# +# IMPORTANT : Please do not remove or change the names of any +# of the Appenders defined here. The layout pattern & log file +# can be changed using the WSO2 Carbon Management Console, and those +# settings will override the settings in this file. +# + +log4j.rootLogger=DEBUG, STD_OUT + +# Redirect log messages to console +log4j.appender.STD_OUT=org.apache.log4j.ConsoleAppender +log4j.appender.STD_OUT.Target=System.out +log4j.appender.STD_OUT.layout=org.apache.log4j.PatternLayout +log4j.appender.STD_OUT.layout.ConversionPattern=%d{yyyy-MM-dd HH:mm:ss} %-5p %c{1}:%L - %m%n diff --git a/components/oauth-extensions/org.wso2.carbon.device.mgt.oauth.extension/src/test/resources/testng.xml b/components/oauth-extensions/org.wso2.carbon.device.mgt.oauth.extension/src/test/resources/testng.xml new file mode 100644 index 0000000000..4da53d1247 --- /dev/null +++ b/components/oauth-extensions/org.wso2.carbon.device.mgt.oauth.extension/src/test/resources/testng.xml @@ -0,0 +1,41 @@ + + + + + + + + + + + + + + + + + + + + + + + + + From 9ae6099d2bd10d26212b20337bbdb184784370c7 Mon Sep 17 00:00:00 2001 From: harshanl Date: Mon, 7 Sep 2015 18:47:11 +0530 Subject: [PATCH 4/4] Fixed nitification table --- .../src/main/resources/dbscripts/cdm/h2.sql | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/features/device-mgt/org.wso2.carbon.device.mgt.server.feature/src/main/resources/dbscripts/cdm/h2.sql b/features/device-mgt/org.wso2.carbon.device.mgt.server.feature/src/main/resources/dbscripts/cdm/h2.sql index 8ce7795373..4e337ef6a3 100644 --- a/features/device-mgt/org.wso2.carbon.device.mgt.server.feature/src/main/resources/dbscripts/cdm/h2.sql +++ b/features/device-mgt/org.wso2.carbon.device.mgt.server.feature/src/main/resources/dbscripts/cdm/h2.sql @@ -392,7 +392,7 @@ CREATE TABLE IF NOT EXISTS DM_NOTIFICATION ( TENANT_ID INTEGER NOT NULL, STATUS VARCHAR(10) NULL, DESCRIPTION VARCHAR(100) NULL, - PRIMARY KEY (ID), + PRIMARY KEY (NOTIFICATION_ID), CONSTRAINT fk_dm_device_notification FOREIGN KEY (DEVICE_ID) REFERENCES DM_DEVICE (ID) ON DELETE NO ACTION ON UPDATE NO ACTION, CONSTRAINT fk_dm_operation_notification FOREIGN KEY (OPERATION_ID) REFERENCES