Uploaded image for project: 'JBoss Developer Materials'
  1. JBoss Developer Materials
  2. JDF-736

kitchensink-html5-mobile quickstart profiles

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Obsolete
    • Icon: Major Major
    • None
    • None
    • None

      There is written in the README.md file in the paragraph "Minification":

      NOTE: You must either specify the default profile for no tests or the arquillian test profile to run tests when minifying to avoid test errors. For example:

      #No Tests
      mvn clean package jboss-as:deploy -Pminify,default

      However, the tests are skipped even if the "default" profile isn't specified. That's because the "minify" profile contains a configuration of the maven-surefire-plugin with the skip configuration set to true.
      https://github.com/jboss-developer/jboss-wfk-quickstarts/blob/2.6.x/kitchensink-html5-mobile/pom.xml#L276-L282

      The README.md file should be definitely changed and I would recommend to change the configuration of the test skipping in the pom file as well.

      I would suggest to move the configuration of the maven-surefire-plugin

      <plugin>
          <artifactId>maven-surefire-plugin</artifactId>
          <version>${version.surefire.plugin}</version>
          <configuration>
              <skip>true</skip>
          </configuration>
      </plugin>
      

      from the "default" profile (this profile remove at all) to the main build part of the pom and remove it from the "minife" profile. The tests will be always skipped, so in the both profiles "arq-jbossas-managed" and "arq-jbossas-remote" should be the configuration rewritten:

      <plugin>
          <artifactId>maven-surefire-plugin</artifactId>
          <version>${version.surefire.plugin}</version>
          <configuration>
              <skip>false</skip>
          </configuration>
      </plugin>
      

      What will be the benefits of these changes?
      1. The profile will IMHO more understandable for the beginners (test are launched only when they are needed - in the two profiles, otherwise they are skipped)
      2. And, MAINLY, there won't be needed so crazy configuration in the archetype as it is now:
      https://github.com/jboss-developer/jboss-wfk-archetypes/blob/2.6.x-develop/jboss-html5-mobile-archetype/src/main/resources/archetype-resources/pom.xml

              maschmid@redhat.com Marek Schmidt
              mjobanek Matous Jobanek
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated:
                Resolved: