Unverified Commit 47940d64 authored by samz406's avatar samz406 Committed by GitHub
Browse files

Merge pull request #9 from apache/dev

update
parents deb215ec 99fc87f9
Loading
Loading
Loading
Loading
+7 −0
Original line number Diff line number Diff line
@@ -16,9 +16,16 @@
 */
package org.apache.dolphinscheduler.alert.utils;

import org.apache.commons.lang.StringUtils;

public class FuncUtils {

    static public String mkString(Iterable<String> list, String split) {

        if (null == list || StringUtils.isEmpty(split)){
            return null;
        }

        StringBuilder sb = new StringBuilder();
        boolean first = true;
        for (String item : list) {
+60 −9
Original line number Diff line number Diff line
@@ -17,15 +17,16 @@
package org.apache.dolphinscheduler.alert.utils;

import org.apache.commons.io.IOUtils;
import org.apache.commons.lang.StringUtils;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

import java.io.IOException;
import java.io.InputStream;
import java.util.Properties;
import java.util.regex.PatternSyntaxException;

import static org.apache.dolphinscheduler.alert.utils.Constants.ALERT_PROPERTIES_PATH;
import static org.apache.dolphinscheduler.alert.utils.Constants.DATA_SOURCE_PROPERTIES_PATH;

/**
 * property utils
@@ -72,7 +73,10 @@ public class PropertyUtils {
     * @return the value
     */
    public static String getString(String key) {
        return properties.getProperty(key);
        if (StringUtils.isEmpty(key)) {
            return null;
        }
        return properties.getProperty(key.trim());
    }

    /**
@@ -82,6 +86,7 @@ public class PropertyUtils {
     * @return  get property int value , if key == null, then return -1
     */
    public static int getInt(String key) {

        return getInt(key, -1);
    }

@@ -111,6 +116,11 @@ public class PropertyUtils {
     * @return  the boolean result value
     */
    public static Boolean getBoolean(String key) {

        if (StringUtils.isEmpty(key)) {
            return false;
        }

        String value = properties.getProperty(key.trim());
        if(null != value){
            return Boolean.parseBoolean(value);
@@ -135,10 +145,30 @@ public class PropertyUtils {
     * @return the value related the key or the default value if the key not existed
     */
    public static long getLong(String key, long defaultVal) {

        String val = getString(key);
        return val == null ? defaultVal : Long.parseLong(val);
        if (val == null) {
            return defaultVal;
        }

        try {
            return Long.parseLong(val);
        } catch (NumberFormatException e) {
            logger.info(e.getMessage(),e);
        }

        return defaultVal;
    }

    /**
     * get double value
     * @param key the key
     * @return if the value not existed, return -1.0, or will return the related value
     */
    public static double getDouble(String key) {
        String val = getString(key);
        return getDouble(key,-1.0);
    }

    /**
     * get double value
@@ -146,9 +176,20 @@ public class PropertyUtils {
     * @param defaultVal the default value
     * @return the value related the key or the default value if the key not existed
     */
    public double getDouble(String key, double defaultVal) {
    public static double getDouble(String key, double defaultVal) {

        String val = getString(key);
        return val == null ? defaultVal : Double.parseDouble(val);
        if (val == null) {
            return defaultVal;
        }

        try {
            return Double.parseDouble(val);
        } catch (NumberFormatException e) {
            logger.info(e.getMessage(),e);
        }

        return defaultVal;
    }


@@ -160,13 +201,13 @@ public class PropertyUtils {
     */
    public static String[] getArray(String key, String splitStr) {
        String value = getString(key);
        if (value == null) {
        if (value == null || StringUtils.isEmpty(splitStr)) {
            return null;
        }
        try {
            String[] propertyArray = value.split(splitStr);
            return propertyArray;
        } catch (NumberFormatException e) {
        } catch (PatternSyntaxException e) {
            logger.info(e.getMessage(),e);
        }
        return null;
@@ -180,9 +221,19 @@ public class PropertyUtils {
     * @param <T> the generic class type
     * @return  get enum value
     */
    public <T extends Enum<T>> T getEnum(String key, Class<T> type,
    public static <T extends Enum<T>> T getEnum(String key, Class<T> type,
                                         T defaultValue) {
        String val = getString(key);
        return val == null ? defaultValue : Enum.valueOf(type, val);
        if (val == null) {
            return defaultValue;
        }

        try {
            return Enum.valueOf(type, val);
        } catch (IllegalArgumentException e) {
            logger.info(e.getMessage(),e);
        }

        return defaultValue;
    }
}
+60 −0
Original line number Diff line number Diff line
/*
 * Licensed to the Apache Software Foundation (ASF) under one or more
 * contributor license agreements.  See the NOTICE file distributed with
 * this work for additional information regarding copyright ownership.
 * The ASF 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.apache.dolphinscheduler.alert.utils;

import org.junit.Test;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

import java.util.Arrays;

import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertNull;

public class FuncUtilsTest {

    private static final Logger logger = LoggerFactory.getLogger(FuncUtilsTest.class);

    /**
     * Test mkString
     */
    @Test
    public void testMKString() {

        //Define users list
        Iterable<String> users = Arrays.asList("user1", "user2", "user3");
        //Define split
        String split = "|";

        //Invoke mkString with correctParams
        String result = FuncUtils.mkString(users, split);
        logger.info(result);

        //Expected result string
        assertEquals(result, "user1|user2|user3");

        //Null list expected return null
        result = FuncUtils.mkString(null, split);
        assertNull(result);

        //Null split expected return null
        result = FuncUtils.mkString(users, null);
        assertNull(result);

    }
}
 No newline at end of file
+112 −0
Original line number Diff line number Diff line
/*
 * Licensed to the Apache Software Foundation (ASF) under one or more
 * contributor license agreements.  See the NOTICE file distributed with
 * this work for additional information regarding copyright ownership.
 * The ASF 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.apache.dolphinscheduler.alert.utils;

import org.junit.After;
import org.junit.Before;
import org.junit.Test;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

import java.util.ArrayList;
import java.util.LinkedHashMap;
import java.util.List;

import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertNull;

public class JSONUtilsTest {

    private static final Logger logger = LoggerFactory.getLogger(JSONUtilsTest.class);

    public List<LinkedHashMap<String, Object>> list = new ArrayList<>();

    public String expected = null;

    @Before
    public void setUp() throws Exception {

        //Define expected json string
        expected = "[{\"mysql service name\":\"mysql200\",\"mysql address\":\"192.168.xx.xx\",\"port\":\"3306\",\"no index of number\":\"80\",\"database client connections\":\"190\"}]";

        //Initial map
        LinkedHashMap<String, Object> map = new LinkedHashMap<>();
        map.put("mysql service name","mysql200");
        map.put("mysql address","192.168.xx.xx");
        map.put("port","3306");
        map.put("no index of number","80");
        map.put("database client connections","190");

        //Add map into list
        list.add(map);
    }


    /**
     * Test toJsonString
     */
    @Test
    public void testToJsonString() {

        //Invoke toJsonString
        String result = JSONUtils.toJsonString(list);
        logger.info(result);

        //Equal result with expected string
        assertEquals(result,expected);

        //If param is null, then return null string
        result = JSONUtils.toJsonString(null);
        logger.info(result);

        assertEquals(result,"null");

    }

    /**
     * Test toList
     */
    @Test
    public void testToList() {

        //Invoke toList
        List<LinkedHashMap> result = JSONUtils.toList(expected ,LinkedHashMap.class);
        //Equal list size=1
        assertEquals(result.size(),1);

        //Transform entity to LinkedHashMap<String, Object>
        LinkedHashMap<String, Object> entity = result.get(0);

        //Equal expected values
        assertEquals(entity.get("mysql service name"),"mysql200");
        assertEquals(entity.get("mysql address"),"192.168.xx.xx");
        assertEquals(entity.get("port"),"3306");
        assertEquals(entity.get("no index of number"),"80");
        assertEquals(entity.get("database client connections"),"190");

        //If param is null, then return null
        result = JSONUtils.toList(null ,LinkedHashMap.class);
        assertNull(result);

        //If param is incorrect, then return null and log error message
        result = JSONUtils.toList("}{" ,LinkedHashMap.class);
        assertNull(result);

    }

}
+215 −0
Original line number Diff line number Diff line
/*
 * Licensed to the Apache Software Foundation (ASF) under one or more
 * contributor license agreements.  See the NOTICE file distributed with
 * this work for additional information regarding copyright ownership.
 * The ASF 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.apache.dolphinscheduler.alert.utils;

import org.apache.dolphinscheduler.common.enums.ZKNodeType;
import org.junit.Test;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

import static org.junit.Assert.*;

/**
 * Test PropertyUtils
 * and the resource path is src/test/resources/alert.properties.
 */
public class PropertyUtilsTest {

    private static final Logger logger = LoggerFactory.getLogger(PropertyUtilsTest.class);

    /**
     * Test getString
     */
    @Test
    public void testGetString() {

        //Expected "EMAIL"
        String result = PropertyUtils.getString("alert.type");
        logger.info(result);
        assertEquals(result, "EMAIL");

        //Expected "xxx.xxx.test"
        result = PropertyUtils.getString("mail.server.host");
        assertEquals(result, "xxx.xxx.test");

        //If key is undefine in alert.properties, then return null
        result = PropertyUtils.getString("abc");
        assertNull(result);

        //If key is null, then return null
        result = PropertyUtils.getString(null);
        assertNull(result);
    }


    /**
     * Test getBoolean
     */
    @Test
    public void testGetBoolean() {

        //Expected true
        Boolean result = PropertyUtils.getBoolean("mail.smtp.starttls.enable");
        assertTrue(result);

        //Expected false
        result = PropertyUtils.getBoolean("mail.smtp.ssl.enable");
        assertFalse(result);

        //If key is undefine in alert.properties, then return null
        result = PropertyUtils.getBoolean("abc");
        assertFalse(result);

        //If key is null, then return false
        result = PropertyUtils.getBoolean(null);
        assertFalse(result);
    }

    /**
     * Test getLong
     */
    @Test
    public void testGetLong() {

        //Expected 25
        long result = PropertyUtils.getLong("mail.server.port");
        assertSame(result, 25L);

        //If key is null, then return -1
        result = PropertyUtils.getLong(null);
        assertSame(result, -1L);

        //If key is undefine in alert.properties, then return -1
        result = PropertyUtils.getLong("abc");
        assertSame(result, -1L);

        //If key is undefine in alert.properties, and there is a defaultval, then return defaultval
        result = PropertyUtils.getLong("abc", 200);
        assertEquals(result, 200L);

        //If the value can not parse to long ,it will log the error and return -1L
        result = PropertyUtils.getLong("test.server.testnumber");
        assertSame(result, -1L);
    }

    /**
     * Test getDouble
     */
    @Test
    public void testGetDouble() {

        //Expected 3.0
        double result = PropertyUtils.getDouble("test.server.factor");
        assertEquals(result, 3.0, 0);

        //If key is null, then return -1.0
        result = PropertyUtils.getDouble(null);
        assertEquals(result, -1.0, 0);

        //If key is undefine in alert.properties, then return -1
        result = PropertyUtils.getDouble("abc");
        assertEquals(result, -1.0, 0);

        //If key is undefine in alert.properties, and there is a defaultval, then return defaultval
        result = PropertyUtils.getDouble("abc", 5.0);
        assertEquals(result, 5.0, 0);

        //If the value can not parse to double ,it will log the error and return -1.0
        result = PropertyUtils.getDouble("test.server.testnumber");
        assertEquals(result, -1.0, 0);
    }

    /**
     * Test getArray
     */
    @Test
    public void testGetArray() {

        //Expected length 3
        String[] result = PropertyUtils.getArray("test.server.list", ",");
        assertEquals(result.length, 3);

        //Equal array values
        assertEquals(result[0], "xxx.xxx.test1");
        assertEquals(result[1], "xxx.xxx.test2");
        assertEquals(result[2], "xxx.xxx.test3");

        //If key is null, then return -1
        result = PropertyUtils.getArray(null, ",");
        assertNull(result);

        //If key is undefine in alert.properties, then return null
        result = PropertyUtils.getArray("abc", ",");
        assertNull(result);

        //If splitStr is null, then return null
        result = PropertyUtils.getArray("test.server.list", null);
        assertNull(result);
    }

    /**
     * test getInt
     */
    @Test
    public void testGetInt() {

        //Expected 25
        int result = PropertyUtils.getInt("mail.server.port");
        assertSame(result, 25);

        //If key is null, then return -1
        result = PropertyUtils.getInt(null);
        assertSame(result, -1);

        //If key is undefine in alert.properties, then return -1
        result = PropertyUtils.getInt("abc");
        assertSame(result, -1);

        //If key is undefine in alert.properties, and there is a defaultval, then return defaultval
        result = PropertyUtils.getInt("abc", 300);
        assertEquals(result, 300);

        //If the value can not parse to int ,it will log the error and return -1
        result = PropertyUtils.getInt("test.server.testnumber");
        assertSame(result, -1);
    }

    /**
     * Test getEnum
     */
    @Test
    public void testGetEnum() {

        //Expected MASTER
        ZKNodeType zkNodeType = PropertyUtils.getEnum("test.server.enum1", ZKNodeType.class,ZKNodeType.WORKER);
        assertEquals(zkNodeType, ZKNodeType.MASTER);

        //Expected DEAD_SERVER
        zkNodeType = PropertyUtils.getEnum("test.server.enum2", ZKNodeType.class,ZKNodeType.WORKER);
        assertEquals(zkNodeType, ZKNodeType.DEAD_SERVER);

        //If key is null, then return defaultval
        zkNodeType = PropertyUtils.getEnum(null, ZKNodeType.class,ZKNodeType.WORKER);
        assertEquals(zkNodeType, ZKNodeType.WORKER);

        //If the value doesn't define in enum ,it will log the error and return -1
        zkNodeType = PropertyUtils.getEnum("test.server.enum3", ZKNodeType.class,ZKNodeType.WORKER);
        assertEquals(zkNodeType, ZKNodeType.WORKER);
    }

}
 No newline at end of file
Loading