search

MetroCS / redistricting

Experimentation with geopolitical redistricting
GNU Affero General Public License v3.0
5 stars 75 forks source link

Rename package name from swdmt to metrocs #172

Closed jody closed 2 years ago

jody commented 2 years ago
jody commented 2 years ago

Transcript: ant test

% ant test
Buildfile: /Users/jody/Dropbox/msuod/cs3250/redistricting/build.xml

jacocoClean:

testClean:
   [delete] Deleting directory /Users/jody/Dropbox/msuod/cs3250/redistricting/reports/test

init:
    [mkdir] Created dir: /Users/jody/Dropbox/msuod/cs3250/redistricting/reports/test

compile:

test.junit.launcher:
[junitlauncher] Running metrocs.redistricting.DistrictTest
[junitlauncher] Tests run: 6, Failures: 0, Aborted: 0, Skipped: 0, Time elapsed: 0.114 sec
[junitlauncher] Running metrocs.redistricting.LocationTest
[junitlauncher] Tests run: 13, Failures: 0, Aborted: 0, Skipped: 0, Time elapsed: 0.02 sec
[junitlauncher] Running metrocs.redistricting.PartyTest
[junitlauncher] Tests run: 2, Failures: 0, Aborted: 0, Skipped: 0, Time elapsed: 0.006 sec
[junitlauncher] Running metrocs.redistricting.RedistrictorTest
[junitlauncher] Tests run: 6, Failures: 0, Aborted: 0, Skipped: 0, Time elapsed: 0.191 sec
[junitlauncher] Running metrocs.redistricting.RegionTest
[junitlauncher] Tests run: 11, Failures: 0, Aborted: 0, Skipped: 0, Time elapsed: 0.027 sec
[junitlauncher] Running metrocs.redistricting.RendererTest
[junitlauncher] Tests run: 14, Failures: 0, Aborted: 0, Skipped: 0, Time elapsed: 0.08 sec
[junitlauncher] Running metrocs.redistricting.VoterTest
[junitlauncher] Tests run: 4, Failures: 0, Aborted: 0, Skipped: 0, Time elapsed: 0.003 sec

test.console.launcher:
     [java] 
     [java] Thanks for using JUnit! Support its development at https://junit.org/sponsoring
     [java] 
     [java] .
     [java] +-- JUnit Jupiter [OK]
     [java] | +-- LocationTest [OK]
     [java] | | +-- verifyCompareToEqualsTest() [OK]
     [java] | | +-- adjacencySelfTest() [OK]
     [java] | | +-- equalsNullTest() [OK]
     [java] | | +-- adjacencyNullTest() [OK]
     [java] | | +-- verifyCompareToSelfTest() [OK]
     [java] | | +-- equalsSelfTest() [OK]
     [java] | | +-- hashCodeTest() [OK]
     [java] | | +-- adjacencyTest() [OK]
     [java] | | +-- verifyCompareToLessThanXTest() [OK]
     [java] | | +-- verifyCompareToLessThanYTest() [OK]
     [java] | | +-- verifyCompareToGreaterThanXTest() [OK]
     [java] | | +-- verifyCompareToGreaterThanYTest() [OK]
     [java] | | '-- coordinateValuesTest() [OK]
     [java] | +-- RegionTest [OK]
     [java] | | +-- squareRegionNonSquareSizeTestNum3() [OK]
     [java] | | +-- squareRegionNonSquareSizeTest() [OK]
     [java] | | +-- emptyRegionNumberOfVotersTest() [OK]
     [java] | | +-- squareRegionNumberOfVotersTest() [OK]
     [java] | | +-- squareRegionNegativeSizeTest() [OK]
     [java] | | +-- regionWithAllVotersInLocationSetTest() [OK]
     [java] | | +-- squareRegionValidSizeTest() [OK]
     [java] | | +-- regionWithVoterNotInLocationSetTest() [OK]
     [java] | | +-- emptyRegionSizeTest() [OK]
     [java] | | +-- voterRegionInputConstructorTest() [OK]
     [java] | | '-- squareRegionExtremeSizeTest() [OK]
     [java] | +-- DistrictTest [OK]
     [java] | | +-- contiguityValidEmptyDistrictTest() [OK]
     [java] | | +-- contiguityInvalidNonemptyDistrictTest() [OK]
     [java] | | +-- contiguityValidNonemptyDistrictTest() [OK]
     [java] | | +-- contiguityValidDistrictsNotContiguous() [OK]
     [java] | | +-- nonzeroSizeDistrictTest() [OK]
     [java] | | '-- zeroSizeEmptyDistrictTest() [OK]
     [java] | +-- RedistrictorTest [OK]
     [java] | | +-- generateDistrictsSquareContiguityTest() [OK]
     [java] | | +-- generateDistrictsSquareSingleDistrictTest() [OK]
     [java] | | +-- invalidRegionParameterConstructorShouldRaiseAnException() [OK]
     [java] | | +-- allDistrictsOfSpecificSizeTest() [OK]
     [java] | | +-- allDistrictsOfSpecificSizeSingleDistrictTest() [OK]
     [java] | | '-- generateDistrictsSquareAppropriateNumberAndSizeTest() [OK]
     [java] | +-- VoterTest [OK]
     [java] | | +-- affiliationAttributeNullTest() [OK]
     [java] | | +-- locationAttributeTest() [OK]
     [java] | | +-- affiliationAttributeSpecifiedTest() [OK]
     [java] | | '-- invalidLocationTest() [OK]
     [java] | +-- RendererTest [OK]
     [java] | | +-- renderAsASCIIRegionLocations3GridSize3x5AffiliationTest() [OK]
     [java] | | +-- renderAsASCIIRegionGridSize3x5District3AffiliationTest() [OK]
     [java] | | +-- renderAsASCIIRegionLocations3GridSize15Test() [OK]
     [java] | | +-- renderAsASCIIRegionSize1District1AffiliationTest() [OK]
     [java] | | +-- renderAsASCIIRegionLocations3GridSize36Test() [OK]
     [java] | | +-- renderAsASCIIRegionGridSize1x2District1AffiliationTest() [OK]
     [java] | | +-- renderAsASCIIRegionLocations3GridSize36AffiliationTest() [OK]
     [java] | | +-- renderAsASCIIRegionLocations3GridSize3x5Test() [OK]
     [java] | | +-- renderAsASCIIRegionSize1Test() [OK]
     [java] | | +-- renderAsASCIIRegionSize16Test() [OK]
     [java] | | +-- renderAsASCIIRegionSize1AffiliationTest() [OK]
     [java] | | +-- renderAsASCIIRegionLocations3GridSize15AffiliationTest() [OK]
     [java] | | +-- renderAsASCIIRegionSize1District1Test() [OK]
     [java] | | '-- renderAsASCIIregionSize16AffiliationTest() [OK]
     [java] | '-- PartyTest [OK]
     [java] |   +-- verifyPartyID() [OK]
     [java] |   '-- verifyPartyBasics() [OK]
     [java] '-- JUnit Vintage [OK]
     [java] 
     [java] Test run finished after 340 ms
     [java] [         9 containers found      ]
     [java] [         0 containers skipped    ]
     [java] [         9 containers started    ]
     [java] [         0 containers aborted    ]
     [java] [         9 containers successful ]
     [java] [         0 containers failed     ]
     [java] [        56 tests found           ]
     [java] [         0 tests skipped         ]
     [java] [        56 tests started         ]
     [java] [         0 tests aborted         ]
     [java] [        56 tests successful      ]
     [java] [         0 tests failed          ]
     [java] 
[junitreport] Processing /Users/jody/Dropbox/msuod/cs3250/redistricting/reports/test/TESTS-TestSuites.xml to /var/folders/qd/z_j9v_tx2cg7s69y2982ysvc0000gn/T/null2083396375
[junitreport] Loading stylesheet jar:file:/opt/local/share/java/apache-ant/lib/ant-junit.jar!/org/apache/tools/ant/taskdefs/optional/junit/xsl/junit-frames.xsl
[junitreport] Transform time: 425ms
[junitreport] Deleting: /var/folders/qd/z_j9v_tx2cg7s69y2982ysvc0000gn/T/null2083396375

unitTest:

test:

BUILD SUCCESSFUL
Total time: 3 seconds
jody commented 2 years ago

Transcript: ant checkstyle

% ant checkstyle
Buildfile: /Users/jody/Dropbox/msuod/cs3250/redistricting/build.xml

init:

checkstyle:
[checkstyle] Running Checkstyle 7.5 on 18 files
[checkstyle] [ERROR] /Users/jody/Dropbox/msuod/cs3250/redistricting/src/metrocs/redistricting/Redistrictor.java:86: Comment matches to-do format 'TODO:'. [TodoComment]
     [xslt] Processing /Users/jody/Dropbox/msuod/cs3250/redistricting/reports/checkstyle_report.xml to /Users/jody/Dropbox/msuod/cs3250/redistricting/reports/checkstyle_report.html
     [xslt] Loading stylesheet /Users/jody/Dropbox/msuod/cs3250/redistricting/lib/checkstyle.xsl

BUILD SUCCESSFUL
Total time: 1 second
jody commented 2 years ago

Warnings reported by SpotBugs

jody commented 2 years ago

Coverage testing inoperative

jody commented 2 years ago

PMD Report

Problems found

</center>
<table align="center" cellspacing="0" cellpadding="3">
    <tr>
        <th>#</th>
        <th>File</th>
        <th>Line</th>
        <th>Problem</th>
    </tr>
    <tr bgcolor="lightgrey">
        <td align="center">1</td>
        <td width="*%">metrocs/redistricting/Location.java</td>
        <td align="center" width="5%">13</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/design.html#ImmutableField">Private field 'xCoordinate' could be made final; it is only initialized in the declaration or constructor.</a>
        </td>
    </tr>
    <tr>
        <td align="center">2</td>
        <td width="*%">metrocs/redistricting/Location.java</td>
        <td align="center" width="5%">15</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/design.html#ImmutableField">Private field 'yCoordinate' could be made final; it is only initialized in the declaration or constructor.</a>
        </td>
    </tr>
    <tr bgcolor="lightgrey">
        <td align="center">3</td>
        <td width="*%">metrocs/redistricting/Location.java</td>
        <td align="center" width="5%">138</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/design.html#FieldDeclarationsShouldBeAtStartOfClass">Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.</a>
        </td>
    </tr>
    <tr>
        <td align="center">4</td>
        <td width="*%">metrocs/redistricting/Location.java</td>
        <td align="center" width="5%">140</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/design.html#FieldDeclarationsShouldBeAtStartOfClass">Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.</a>
        </td>
    </tr>
    <tr bgcolor="lightgrey">
        <td align="center">5</td>
        <td width="*%">metrocs/redistricting/LocationTest.java</td>
        <td align="center" width="5%">1</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/imports.html#TooManyStaticImports">Too many static imports may lead to messy code</a>
        </td>
    </tr>
    <tr>
        <td align="center">6</td>
        <td width="*%">metrocs/redistricting/LocationTest.java</td>
        <td align="center" width="5%">16</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/codesize.html#TooManyMethods">This class has too many methods, consider refactoring it.</a>
        </td>
    </tr>
    <tr bgcolor="lightgrey">
        <td align="center">7</td>
        <td width="*%">metrocs/redistricting/LocationTest.java</td>
        <td align="center" width="5%">43</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/design.html#EqualsNull">Avoid using equals() to compare against null</a>
        </td>
    </tr>
    <tr>
        <td align="center">8</td>
        <td width="*%">metrocs/redistricting/Redistrictor.java</td>
        <td align="center" width="5%">29</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/design.html#ImmutableField">Private field 'region' could be made final; it is only initialized in the declaration or constructor.</a>
        </td>
    </tr>
    <tr bgcolor="lightgrey">
        <td align="center">9</td>
        <td width="*%">metrocs/redistricting/RedistrictorTest.java</td>
        <td align="center" width="5%">1</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/imports.html#TooManyStaticImports">Too many static imports may lead to messy code</a>
        </td>
    </tr>
    <tr>
        <td align="center">10</td>
        <td width="*%">metrocs/redistricting/RedistrictorTest.java</td>
        <td align="center" width="5%">22</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/design.html#UncommentedEmptyConstructor">Document empty constructor</a>
        </td>
    </tr>
    <tr bgcolor="lightgrey">
        <td align="center">11</td>
        <td width="*%">metrocs/redistricting/RedistrictorTest.java</td>
        <td align="center" width="5%">31</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/design.html#UncommentedEmptyMethodBody">Document empty method body</a>
        </td>
    </tr>
    <tr>
        <td align="center">12</td>
        <td width="*%">metrocs/redistricting/RedistrictorTest.java</td>
        <td align="center" width="5%">40</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/design.html#UncommentedEmptyMethodBody">Document empty method body</a>
        </td>
    </tr>
    <tr bgcolor="lightgrey">
        <td align="center">13</td>
        <td width="*%">metrocs/redistricting/RedistrictorTest.java</td>
        <td align="center" width="5%">46</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/unusedcode.html#UnusedLocalVariable">Avoid unused local variables such as 'r'.</a>
        </td>
    </tr>
    <tr>
        <td align="center">14</td>
        <td width="*%">metrocs/redistricting/RedistrictorTest.java</td>
        <td align="center" width="5%">169</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/design.html#CompareObjectsWithEquals">Use equals() to compare object references.</a>
        </td>
    </tr>
    <tr bgcolor="lightgrey">
        <td align="center">15</td>
        <td width="*%">metrocs/redistricting/RedistrictorTest.java</td>
        <td align="center" width="5%">185</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/unusedcode.html#UnusedLocalVariable">Avoid unused local variables such as 'districtSet'.</a>
        </td>
    </tr>
    <tr>
        <td align="center">16</td>
        <td width="*%">metrocs/redistricting/RedistrictorTest.java</td>
        <td align="center" width="5%">200</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/unusedcode.html#UnusedLocalVariable">Avoid unused local variables such as 'districtSet'.</a>
        </td>
    </tr>
    <tr bgcolor="lightgrey">
        <td align="center">17</td>
        <td width="*%">metrocs/redistricting/RedistrictorTest.java</td>
        <td align="center" width="5%">227</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/codesize.html#NPathComplexity">The method generateDistrictsSquareContiguityTest() has an NPath complexity of 256</a>
        </td>
    </tr>
    <tr>
        <td align="center">18</td>
        <td width="*%">metrocs/redistricting/Region.java</td>
        <td align="center" width="5%">39</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/design.html#UnnecessaryLocalBeforeReturn">Consider simply returning the value vs storing it in local variable 'sideSize'</a>
        </td>
    </tr>
    <tr bgcolor="lightgrey">
        <td align="center">19</td>
        <td width="*%">metrocs/redistricting/RegionTest.java</td>
        <td align="center" width="5%">1</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/imports.html#TooManyStaticImports">Too many static imports may lead to messy code</a>
        </td>
    </tr>
    <tr>
        <td align="center">20</td>
        <td width="*%">metrocs/redistricting/RegionTest.java</td>
        <td align="center" width="5%">19</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/codesize.html#TooManyMethods">This class has too many methods, consider refactoring it.</a>
        </td>
    </tr>
    <tr bgcolor="lightgrey">
        <td align="center">21</td>
        <td width="*%">metrocs/redistricting/RegionTest.java</td>
        <td align="center" width="5%">28</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/design.html#UncommentedEmptyConstructor">Document empty constructor</a>
        </td>
    </tr>
    <tr>
        <td align="center">22</td>
        <td width="*%">metrocs/redistricting/RegionTest.java</td>
        <td align="center" width="5%">80</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/unusedcode.html#UnusedLocalVariable">Avoid unused local variables such as 'r'.</a>
        </td>
    </tr>
    <tr bgcolor="lightgrey">
        <td align="center">23</td>
        <td width="*%">metrocs/redistricting/RegionTest.java</td>
        <td align="center" width="5%">92</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/unusedcode.html#UnusedLocalVariable">Avoid unused local variables such as 'r'.</a>
        </td>
    </tr>
    <tr>
        <td align="center">24</td>
        <td width="*%">metrocs/redistricting/RegionTest.java</td>
        <td align="center" width="5%">104</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/unusedcode.html#UnusedLocalVariable">Avoid unused local variables such as 'r'.</a>
        </td>
    </tr>
    <tr bgcolor="lightgrey">
        <td align="center">25</td>
        <td width="*%">metrocs/redistricting/RegionTest.java</td>
        <td align="center" width="5%">116</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/unusedcode.html#UnusedLocalVariable">Avoid unused local variables such as 'r'.</a>
        </td>
    </tr>
    <tr>
        <td align="center">26</td>
        <td width="*%">metrocs/redistricting/Renderer.java</td>
        <td align="center" width="5%">18</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/codesize.html#CyclomaticComplexity">The class 'Renderer' has a Cyclomatic Complexity of 3 (Highest = 10).</a>
        </td>
    </tr>
    <tr bgcolor="lightgrey">
        <td align="center">27</td>
        <td width="*%">metrocs/redistricting/Renderer.java</td>
        <td align="center" width="5%">47</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/codesize.html#CyclomaticComplexity">The method 'renderAsASCII' has a Cyclomatic Complexity of 10.</a>
        </td>
    </tr>
    <tr>
        <td align="center">28</td>
        <td width="*%">metrocs/redistricting/RendererTest.java</td>
        <td align="center" width="5%">14</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/codesize.html#TooManyMethods">This class has too many methods, consider refactoring it.</a>
        </td>
    </tr>
    <tr bgcolor="lightgrey">
        <td align="center">29</td>
        <td width="*%">metrocs/redistricting/RendererTest.java</td>
        <td align="center" width="5%">26</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/unusedcode.html#UnusedPrivateField">Avoid unused private fields such as 'R_TP'.</a>
        </td>
    </tr>
    <tr>
        <td align="center">30</td>
        <td width="*%">metrocs/redistricting/StarGenerator.java</td>
        <td align="center" width="5%">23</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/unusedcode.html#UnusedLocalVariable">Avoid unused local variables such as 'visited'.</a>
        </td>
    </tr>
    <tr bgcolor="lightgrey">
        <td align="center">31</td>
        <td width="*%">metrocs/redistricting/VoterTest.java</td>
        <td align="center" width="5%">43</td>
        <td width="*">
            <a href="https://pmd.github.io/pmd-5.5.2/pmd-java/rules/java/unusedcode.html#UnusedLocalVariable">Avoid unused local variables such as 'voter1'.</a>
        </td>
    </tr>
</table>