Uploaded image for project: 'camunda BPM'
  1. camunda BPM
  2. CAM-13193

Java External Task Client has needless test dependency to webapp

    • Icon: Task Task
    • Resolution: Fixed
    • Icon: L3 - Default L3 - Default
    • 7.15.0-alpha5, 7.15.0
    • None
    • None
    • None

      Context

      • The test release job 7.15-TEST-RELEASE-build-camunda-bpm-EE-tags failed since the dependency org.camunda.bpm.webapp:camunda-webapp-tomcat could not be found by the Java External Task Client module
      • Building the Tomcat artifact of the CE Webapps is not part of this particular release job
        • The job runs with the profile distro which prevents building some of the community edition artifacts (distro, webapps, SA webapps)
      • There is no technical reason for the Java External Task Client to have a test dependency to the aforementioned artifact
        • It was added during the development phase of the Java External Task Client to allow easier debugging of the integration tests by checking the process state in the Webapps

      Solution

      Remove the dependency to org.camunda.bpm.webapp:camunda-webapp-tomcat

      Acceptance Criteria

      • Test dependency to org.camunda.bpm.webapp:camunda-webapp-tomcat is removed
      • Cargo plugin is adjusted not to deploy the removed dependency

        This is the controller panel for Smart Panels app

            [CAM-13193] Java External Task Client has needless test dependency to webapp

            Tassilo Weidner created issue -
            Tassilo Weidner made changes -
            Status Original: Open [ 1 ] New: In Progress [ 3 ]
            Tassilo Weidner made changes -
            Summary Original: Java External Task Client has needless dependency to webapp New: Java External Task Client has needless test dependency to webapp
            Tassilo Weidner made changes -
            Issue Type Original: Bug Report [ 1 ] New: Task [ 3 ]
            Tassilo Weidner made changes -
            Description Original: h4. Environment (Required on creation):


            h4. Description (Required on creation; please attach any relevant screenshots, stacktraces, log files, etc. to the ticket):


            h4. Steps to reproduce (Required on creation):


            h4. Observed Behavior (Required on creation):


            h4. Expected behavior (Required on creation):


            h4. Root Cause (Required on prioritization):


            h4. Solution Ideas (Optional):


            h4. Hints (Optional):


            New: h4. Acceptance Criteria (Required on creation):


            h4. Hints (Optional):


            Tassilo Weidner made changes -
            Mentioned Roles
            Tassilo Weidner made changes -
            Mentioned Groups
            Tassilo Weidner made changes -
            Description Original: h4. Acceptance Criteria (Required on creation):


            h4. Hints (Optional):


            New: h4. Context

            * The test release job {{7.15-TEST-RELEASE-build-camunda-bpm-EE-tags}} failed since the dependency {{org.camunda.bpm.webapp:camunda-webapp-tomcat}} could not be found
            * Building the Tomcat artifact of the CE Webapps is not part of this particular release job
            ** The job runs with the profile {{distro}} which prevents building some of the community edition artifacts (distro, webapps, SA webapps)
            * There is no technical reason for the Java External Task Client to have a test dependency to the aforementioned artifact
            ** It was added during the development phase of the Java External Task Client to allow easier debugging of the integration tests by checking the process state in the Webapps

            h4. Solution

            Remove the dependency to {org.camunda.bpm.webapp:camunda-webapp-tomcat}}

            h4. Acceptance Criteria

            * Test dependency to {{org.camunda.bpm.webapp:camunda-webapp-tomcat}} is removed
            * Cargo plugin is adjusted not to deploy the removed dependency

            Tassilo Weidner made changes -
            Mentioned Roles
            Tassilo Weidner made changes -
            Mentioned Groups

              tassilo.weidner Tassilo Weidner
              tassilo.weidner Tassilo Weidner
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated:
                Resolved: