Changeset 3 for profile


Ignore:
Timestamp:
08/29/19 17:09:32 (5 years ago)
Author:
bart
Message:

Small fixes

Location:
profile/src
Files:
3 edited

Legend:

Unmodified
Added
Removed

  • profile/src/main/java/geniusweb/profile/utilityspace/NumberValueSetUtilities.java

    r1 r3  
    66import com.fasterxml.jackson.annotation.JsonAutoDetect;
    77import com.fasterxml.jackson.annotation.JsonAutoDetect.Visibility;
     8import com.fasterxml.jackson.annotation.JsonCreator;
     9import com.fasterxml.jackson.annotation.JsonProperty;
     10import com.fasterxml.jackson.annotation.JsonTypeName;
    811
    912import geniusweb.issuevalue.NumberValue;
    … …  
    1114import geniusweb.issuevalue.Value;
    1215import geniusweb.issuevalue.ValueSet;
    13 
    14 import com.fasterxml.jackson.annotation.JsonCreator;
    15 import com.fasterxml.jackson.annotation.JsonProperty;
    16 import com.fasterxml.jackson.annotation.JsonTypeName;
    17 
    1816import tudelft.utilities.immutablelist.Range;
    1917
    … …  
    10098                }
    10199                if (numvalset.getRange().getHigh().compareTo(highValue) != 0) {
    102                         return "the utilities are specified down to " + highValue
    103                                         + " but the valueset starts at "
     100                        return "the utilities are specified up to " + highValue
     101                                        + " but the valueset ends at "
    104102                                        + numvalset.getRange().getHigh();
    105103                }

  • profile/src/test/java/geniusweb/profile/utilityspace/DiscreteValueSetUtilitiesTest.java

    r1 r3  
    11package geniusweb.profile.utilityspace;
     2
     3import static org.junit.Assert.assertEquals;
    24
    35import java.math.BigDecimal;
    … …  
    1012
    1113import geniusweb.issuevalue.DiscreteValue;
    12 import geniusweb.profile.utilityspace.DiscreteValueSetUtilities;
     14import geniusweb.issuevalue.DiscreteValueSet;
     15import geniusweb.issuevalue.NumberValue;
     16import geniusweb.issuevalue.NumberValueSet;
     17import geniusweb.issuevalue.ValueSet;
    1318import tudelft.utilities.junit.GeneralTests;
    1419
    15 public class DiscreteValueSetUtilitiesTest extends GeneralTests<DiscreteValueSetUtilities> {
     20public class DiscreteValueSetUtilitiesTest
     21                extends GeneralTests<DiscreteValueSetUtilities> {
    1622
    1723        private final static DiscreteValue vala = new DiscreteValue("a");
    1824        private final static DiscreteValue vala1 = new DiscreteValue("a");
    1925        private final static DiscreteValue valb = new DiscreteValue("b");
     26        private final static NumberValue valn = new NumberValue("99");
    2027
    2128        private final static BigDecimal util = new BigDecimal("0.14");
    … …  
    2330        private final static BigDecimal utilb = new BigDecimal("0.37");
    2431
    25         private final static DiscreteValueSetUtilities utils, utils1, utilsb, utils1b, utilsc1, utilsc2;
     32        private final static BigDecimal badutil = new BigDecimal("1.37");
     33        private final static BigDecimal badutil2 = new BigDecimal("-0.37");
     34
     35        private final static DiscreteValueSetUtilities utils, utils1, utilsb,
     36                        utils1b, utilsc1, utilsc2;
    2637
    2738        static {
    … …  
    5667        @Override
    5768        public List<List<DiscreteValueSetUtilities>> getGeneralTestData() {
    58                 return Arrays.asList(Arrays.asList(utils, utils1), Arrays.asList(utilsb), Arrays.asList(utils1b),
     69                return Arrays.asList(Arrays.asList(utils, utils1),
     70                                Arrays.asList(utilsb), Arrays.asList(utils1b),
    5971                                Arrays.asList(utilsc1, utilsc2));
    6072        }
    … …  
    6274        @Override
    6375        public List<String> getGeneralTestStrings() {
    64                 return Arrays.asList("ValueSetUtilities\\{\"a\"=0.14\\}", "ValueSetUtilities\\{\"b\"=0.37\\}",
    65                                 "ValueSetUtilities\\{\"a\"=0.37\\}", "ValueSetUtilities\\{\"a\"=0.14, \"b\"=0.37\\}");
     76                return Arrays.asList("ValueSetUtilities\\{\"a\"=0.14\\}",
     77                                "ValueSetUtilities\\{\"b\"=0.37\\}",
     78                                "ValueSetUtilities\\{\"a\"=0.37\\}",
     79                                "ValueSetUtilities\\{\"a\"=0.14, \"b\"=0.37\\}");
    6680        }
    6781
    … …  
    7286        }
    7387
     88        @Test(expected = IllegalArgumentException.class)
     89        public void testTooLargeUtility() {
     90                Map<DiscreteValue, BigDecimal> valueUtils = new HashMap<>();
     91                valueUtils.put(vala, util);
     92                valueUtils.put(valb, badutil);
     93                new DiscreteValueSetUtilities(valueUtils);
     94        }
     95
     96        @Test(expected = IllegalArgumentException.class)
     97        public void testNegativeUtility() {
     98                Map<DiscreteValue, BigDecimal> valueUtils = new HashMap<>();
     99                valueUtils.put(vala, util);
     100                valueUtils.put(valb, badutil2);
     101                new DiscreteValueSetUtilities(valueUtils);
     102        }
     103
     104        @Test(expected = IllegalArgumentException.class)
     105        public void testNullUtility() {
     106                Map<DiscreteValue, BigDecimal> valueUtils = new HashMap<>();
     107                valueUtils.put(vala, null);
     108                valueUtils.put(valb, util);
     109                new DiscreteValueSetUtilities(valueUtils);
     110        }
     111
     112        @Test(expected = NullPointerException.class)
     113        public void testNullValueUtils() {
     114                new DiscreteValueSetUtilities(null);
     115
     116        }
     117
     118        @Test(expected = NullPointerException.class)
     119        public void testNullIssue() {
     120                Map<DiscreteValue, BigDecimal> valueUtils = new HashMap<>();
     121                valueUtils.put(vala, util);
     122                valueUtils.put(null, utilb);
     123                new DiscreteValueSetUtilities(valueUtils);
     124        }
     125
     126        @Test
     127        public void testGetUtility() {
     128                Map<DiscreteValue, BigDecimal> valueUtils = new HashMap<>();
     129                valueUtils.put(vala, util);
     130                valueUtils.put(valb, utilb);
     131                DiscreteValueSetUtilities utilset = new DiscreteValueSetUtilities(
     132                                valueUtils);
     133                assertEquals(util, utilset.getUtility(vala));
     134                assertEquals(utilb, utilset.getUtility(valb));
     135                assertEquals(BigDecimal.ZERO,
     136                                utilset.getUtility(new DiscreteValue("unknownIssue")));
     137
     138        }
     139
     140        @Test
     141        public void testIsFitting() {
     142                ValueSet valueset = new DiscreteValueSet(Arrays.asList(vala, valb));
     143
     144                Map<DiscreteValue, BigDecimal> valueUtils = new HashMap<>();
     145                valueUtils.put(vala, util);
     146                DiscreteValueSetUtilities utilset = new DiscreteValueSetUtilities(
     147                                valueUtils);
     148
     149                assertEquals(
     150                                "The values in the set DiscreteValueSet[\"a\", \"b\"] do not match the values mapped to utilities [\"a\"]",
     151                                utilset.isFitting(valueset));
     152
     153                valueUtils.put(valb, utilb);
     154                utilset = new DiscreteValueSetUtilities(valueUtils);
     155                assertEquals(null, utilset.isFitting(valueset));
     156
     157                valueUtils.put(new DiscreteValue("unknownIssue"), util1);
     158                utilset = new DiscreteValueSetUtilities(valueUtils);
     159                assertEquals(
     160                                "The values in the set DiscreteValueSet[\"a\", \"b\"] do not match the values mapped to utilities [\"a\", \"b\", \"unknownIssue\"]",
     161                                utilset.isFitting(valueset));
     162
     163        }
     164
     165        @Test
     166        public void testIsFittingWrongType() {
     167                ValueSet valueset = new NumberValueSet(BigDecimal.ZERO, BigDecimal.ONE,
     168                                new BigDecimal("0.3"));
     169
     170                Map<DiscreteValue, BigDecimal> valueUtils = new HashMap<>();
     171                valueUtils.put(vala, util);
     172                valueUtils.put(valb, utilb);
     173                DiscreteValueSetUtilities utilset = new DiscreteValueSetUtilities(
     174                                valueUtils);
     175
     176                assertEquals(
     177                                "The utilities are for a discrete valueset but the given values are numberValueSet[0,1,0.3]",
     178                                utilset.isFitting(valueset));
     179
     180        }
    74181}

  • profile/src/test/java/geniusweb/profile/utilityspace/NumberValueSetUtilTest.java

    r1 r3  
    99import org.junit.Test;
    1010
     11import geniusweb.issuevalue.DiscreteValue;
     12import geniusweb.issuevalue.DiscreteValueSet;
    1113import geniusweb.issuevalue.NumberValue;
    12 import geniusweb.profile.utilityspace.NumberValueSetUtilities;
     14import geniusweb.issuevalue.NumberValueSet;
     15import geniusweb.issuevalue.ValueSet;
     16import tudelft.utilities.immutablelist.Range;
    1317import tudelft.utilities.junit.GeneralTests;
    1418
    15 public class NumberValueSetUtilTest extends GeneralTests<NumberValueSetUtilities> {
     19public class NumberValueSetUtilTest
     20                extends GeneralTests<NumberValueSetUtilities> {
    1621
    1722        private static final BigDecimal lowval1 = new BigDecimal("12.5");
    … …  
    2126        private static final BigDecimal highvalb = new BigDecimal("18.4");
    2227        private static final BigDecimal highutil = new BigDecimal("0.9");
    23         private static final NumberValueSetUtilities valueset1 = new NumberValueSetUtilities(lowval1, low1util, highval,
    24                         highutil);
    25         private static final NumberValueSetUtilities valueset1b = new NumberValueSetUtilities(lowval1, low1util, highval,
    26                         highutil);
    27         private static final NumberValueSetUtilities valueset2 = new NumberValueSetUtilities(lowval1, low1utilb, highval,
    28                         highutil);
    29         private static final NumberValueSetUtilities valueset3 = new NumberValueSetUtilities(lowval1, low1util, highvalb,
    30                         highutil);
     28        private static final NumberValueSetUtilities valueset1 = new NumberValueSetUtilities(
     29                        lowval1, low1util, highval, highutil);
     30        private static final NumberValueSetUtilities valueset1b = new NumberValueSetUtilities(
     31                        lowval1, low1util, highval, highutil);
     32        private static final NumberValueSetUtilities valueset2 = new NumberValueSetUtilities(
     33                        lowval1, low1utilb, highval, highutil);
     34        private static final NumberValueSetUtilities valueset3 = new NumberValueSetUtilities(
     35                        lowval1, low1util, highvalb, highutil);
     36        private static final BigDecimal ZERO_1 = new BigDecimal("0.1");
    3137
    3238        @Override
    3339        public List<List<NumberValueSetUtilities>> getGeneralTestData() {
    34                 return Arrays.asList(Arrays.asList(valueset1, valueset1b), Arrays.asList(valueset2), Arrays.asList(valueset3));
     40                return Arrays.asList(Arrays.asList(valueset1, valueset1b),
     41                                Arrays.asList(valueset2), Arrays.asList(valueset3));
    3542        }
    3643
    3744        @Override
    3845        public List<String> getGeneralTestStrings() {
    39                 return Arrays.asList("NumberValUtility\\(12.5->0.5,18.5->0.9\\)", "NumberValUtility\\(12.5->0.6,18.5->0.9\\)",
     46                return Arrays.asList("NumberValUtility\\(12.5->0.5,18.5->0.9\\)",
     47                                "NumberValUtility\\(12.5->0.6,18.5->0.9\\)",
    4048                                "NumberValUtility\\(12.5->0.5,18.4->0.9\\)");
    4149        }
    … …  
    4957        public void utilAscentingTest() {
    5058                NumberValueSetUtilities utils = valueset1;
    51                 assertEquals(0, low1util.compareTo(utils.getUtility(new NumberValue(lowval1))));
     59                assertEquals(0,
     60                                low1util.compareTo(utils.getUtility(new NumberValue(lowval1))));
    5261        }
    5362
    5463        @Test
    5564        public void utilDescendingTest() {
    56                 NumberValueSetUtilities utils = new NumberValueSetUtilities(lowval1, highutil, highval, low1util);
    57                 assertEquals(0, highutil.compareTo(utils.getUtility(new NumberValue(lowval1))));
     65                NumberValueSetUtilities utils = new NumberValueSetUtilities(lowval1,
     66                                highutil, highval, low1util);
     67                assertEquals(0,
     68                                highutil.compareTo(utils.getUtility(new NumberValue(lowval1))));
    5869        }
    5970
    6071        @Test(expected = IllegalArgumentException.class)
    6172        public void reverserangetest() {
    62                 NumberValueSetUtilities utils = new NumberValueSetUtilities(highval, highutil, lowval1, low1util);
     73                NumberValueSetUtilities utils = new NumberValueSetUtilities(highval,
     74                                low1util, highval, highutil);
     75        }
     76
     77        @Test(expected = NullPointerException.class)
     78        public void testNullHigh() {
     79                NumberValueSetUtilities utils = new NumberValueSetUtilities(lowval1,
     80                                low1util, null, highutil);
     81        }
     82
     83        @Test(expected = NullPointerException.class)
     84        public void testNullHighUtil() {
     85                NumberValueSetUtilities utils = new NumberValueSetUtilities(lowval1,
     86                                low1util, highval, null);
     87        }
     88
     89        @Test(expected = NullPointerException.class)
     90        public void testNullLow() {
     91                NumberValueSetUtilities utils = new NumberValueSetUtilities(null,
     92                                low1util, highval, highutil);
     93        }
     94
     95        @Test(expected = NullPointerException.class)
     96        public void testNullLowUtil() {
     97                NumberValueSetUtilities utils = new NumberValueSetUtilities(lowval1,
     98                                null, highval, highutil);
     99        }
     100
     101        @Test(expected = IllegalArgumentException.class)
     102        public void testLowUtilNegative() {
     103                NumberValueSetUtilities utils = new NumberValueSetUtilities(lowval1,
     104                                new BigDecimal("-0.2"), highval, highutil);
     105        }
     106
     107        @Test(expected = IllegalArgumentException.class)
     108        public void testHighUtilNegative() {
     109                NumberValueSetUtilities utils = new NumberValueSetUtilities(lowval1,
     110                                low1util, highval, new BigDecimal("-0.000003"));
     111        }
     112
     113        @Test
     114        public void getUtilTest() {
     115                NumberValueSetUtilities utils = new NumberValueSetUtilities(lowval1,
     116                                low1util, highval, highutil);
     117                assertEquals(0,
     118                                highutil.compareTo(utils.getUtility(new NumberValue(highval))));
     119                assertEquals(0,
     120                                low1util.compareTo(utils.getUtility(new NumberValue(lowval1))));
     121                // linear interpolated , check center
     122                BigDecimal middleutil = low1util.add(highutil)
     123                                .divide(new BigDecimal("2"));
     124                NumberValue middleval = new NumberValue(
     125                                lowval1.add(highval).divide(new BigDecimal("2")));
     126                assertEquals(0, middleutil.compareTo(utils.getUtility(middleval)));
     127
     128        }
     129
     130        @Test
     131        public void testIsNotFittingRange() {
     132                ValueSet valueset = new NumberValueSet(
     133                                new Range(low1util, highutil, new BigDecimal("0.02")));
     134                ValueSetUtilities utilset = new NumberValueSetUtilities(lowval1,
     135                                low1util, highval, highutil);
     136                assertEquals(
     137                                "the utilities are specified down to 12.5 but the valueset starts at 0.5",
     138                                utilset.isFitting(valueset));
     139        }
     140
     141        @Test
     142        public void testTopOfRangeTooLarge() {
     143                ValueSet valueset = new NumberValueSet(
     144                                new Range(lowval1, highval.subtract(ZERO_1), ZERO_1));
     145                ValueSetUtilities utilset = new NumberValueSetUtilities(lowval1,
     146                                low1util, highval, highutil);
     147                assertEquals(
     148                                "the utilities are specified up to 18.5 but the valueset ends at 18.4",
     149                                utilset.isFitting(valueset));
     150        }
     151
     152        @Test
     153        public void testIsFitting() {
     154                ValueSet valueset = new NumberValueSet(
     155                                new Range(lowval1, highval, BigDecimal.ONE));
     156                ValueSetUtilities utilset = new NumberValueSetUtilities(lowval1,
     157                                low1util, highval, highutil);
     158                assertEquals(null, utilset.isFitting(valueset));
     159        }
     160
     161        @Test
     162        public void testIsFittingWrongType() {
     163                DiscreteValue vala = new DiscreteValue("a");
     164                DiscreteValue valb = new DiscreteValue("b");
     165                ValueSet valueset = new DiscreteValueSet(Arrays.asList(vala, valb));
     166
     167                NumberValueSetUtilities utilset = new NumberValueSetUtilities(lowval1,
     168                                low1util, highval, highutil);
     169                assertEquals(
     170                                "The utilities are for a number valueset but the given values are DiscreteValueSet[\"a\", \"b\"]",
     171                                utilset.isFitting(valueset));
     172
    63173        }
    64174
Note: See TracChangeset for help on using the changeset viewer.