Uploaded image for project: 'Fedora Repository Project'
  1. Fedora Repository Project
  2. FCREPO-457

Remove unnecessary testing dependencies on external hosts

    Details

    • Type: Story
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: Fedora 3.5
    • Component/s: legacy - Fedora
    • Labels:
      None

      Description

      Several of the system tests currently rely, unnecessarily, on access to external hosts.

      When such a dependency exists, it should be for a good reason (e.g. to test something that can't be tested vs localhost)

      Offending tests include:
        fedora.test.api.TestManagedDatastreams#testModifyDatastreamByReference
        fedora.test.api.TestManagementNotifications#testDatastreamMethodNotifications
        fedora.test.api.TestAPIM#testDatastreamMethods

        Attachments

          Issue Links

            Activity

            Hide
            cwilper Chris Wilper added a comment -

            Fixed; Now pointing to a URL that's always locally-resolvable instead

            Show
            cwilper Chris Wilper added a comment - Fixed; Now pointing to a URL that's always locally-resolvable instead
            Hide
            cwilper Chris Wilper added a comment -

            Also removed top-level junit directory from fedora-commons.org/

            Show
            cwilper Chris Wilper added a comment - Also removed top-level junit directory from fedora-commons.org/
            Hide
            cwilper Chris Wilper added a comment -

            This is still a problem. Particularly with testGetDisseminationChained.

            Before closing this again, we need to unplug the Fedora host from the wall and make the tests pass.

            Show
            cwilper Chris Wilper added a comment - This is still a problem. Particularly with testGetDisseminationChained. Before closing this again, we need to unplug the Fedora host from the wall and make the tests pass.
            Hide
            cwilper Chris Wilper added a comment -

            Andrew also noted in FCREPO-502: "not all ConfigB tests pass (fedora.test.api.TestXACMLPolicies) since demo:31 refers to an item not currently available (http://www.fedora-commons.org/demo/misc/batikBatik.svg)."

            Show
            cwilper Chris Wilper added a comment - Andrew also noted in FCREPO-502 : "not all ConfigB tests pass (fedora.test.api.TestXACMLPolicies) since demo:31 refers to an item not currently available ( http://www.fedora-commons.org/demo/misc/batikBatik.svg )."
            Hide
            cwilper Chris Wilper added a comment -

            Suggested fix on May 10th call: Move svg to a locally-accessible location and update object to point to that. Then try unplugging Fedora from the wall and doing a "mvn install -Doffline=true". If that succeeds then this can be closed.

            Show
            cwilper Chris Wilper added a comment - Suggested fix on May 10th call: Move svg to a locally-accessible location and update object to point to that. Then try unplugging Fedora from the wall and doing a "mvn install -Doffline=true". If that succeeds then this can be closed.
            Hide
            cwilper Chris Wilper added a comment -

            Hey, success.

            Show
            cwilper Chris Wilper added a comment - Hey, success.

              People

              • Assignee:
                cwilper Chris Wilper
                Reporter:
                cwilper Chris Wilper
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: