Uploaded image for project: 'OODT'
  1. OODT
  2. OODT-605 Upgrade the CAS-Product web application to use JAX-RS
  3. OODT-642

Move the CAS-Product test classes to src/test/java and the resources to src/test/resources

    XMLWordPrintableJSON

    Details

    • Type: Sub-task
    • Status: Resolved
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: 0.6
    • Fix Version/s: 0.7
    • Component/s: product server
    • Labels:
    • Skill Level:
      Newcomer (Easy) - Everyone can do this

      Description

      This update moves two test classes for the CAS-Product web application from 'src/test' to 'src/test/java', and it moves one resource file from 'src/testdata' to 'src/test/resources'.

      These changes are an attempt to bring the project structure closer to the standard directory layout for a Maven project.

      This is a trivial change, it makes no Java code changes and simply moves three items (two Java classes and one properties file). But to support the move, there are a couple of minor changes to the POM as shown below:

      pom.xml
      <build>
        <testSourceDirectory>${basedir}/src/test/java</testSourceDirectory>
        ...
        <plugins>
          <plugin>
            <artifactId>maven-assembly-plugin</artifactId>
            <configuration>
              <systemProperties>
                <property>
                  <name>java.util.logging.config.file</name>
                  <value>
                    ${basedir}/src/test/resources/test.logging.properties
                  </value>
                </property>
              </systemProperties>
            </configuration>
          </plugin>
        </plugins>
      </build>
      

      I'm not sure if the above moves can be captured by a patch. Nevertheless, I'll attach a patch made by running svn diff after all of the moves and POM updates had been carried out locally.

        Attachments

          Activity

            People

            • Assignee:
              rlaidlaw Ross Laidlaw
              Reporter:
              rlaidlaw Ross Laidlaw
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: