2 Replies Latest reply on Sep 12, 2009 10:32 PM by johnbailey

    Exploded Exporter Test Case

    alrubinger
    alrubinger Sep 12, 2009 6:11 PM

      While doing some refactoring I came across:

      Asset assetOne = new ClassLoaderAsset("org/jboss/declarchive/impl/base/asset/Test.properties");
 Path pathOne = new BasicPath("test.properties");


      Couple things for clarification?

      1) Please externalize these test paths. Makes refactoring easier. (I'll do that now)
      2) Why the mixed upper/lower case in the path?

      S
      ALR


      • 2001 Views
      • Tags:
        • 1. Re: Exploded Exporter Test Case
          alrubinger
          alrubinger Sep 12, 2009 6:21 PM (in response to alrubinger)

          Also I'd like to centralize a bunch of the setup logic for the Zip and Exploded exporter test cases. See some copy/paste in there.

          S,
          ALR

            • Actions
          • 2. Re: Exploded Exporter Test Case
            johnbailey
            johnbailey Sep 12, 2009 10:32 PM (in response to alrubinger)

             


            Also I'd like to centralize a bunch of the setup logic for the Zip and Exploded exporter test cases. See some copy/paste in there.


            Sure I can go through an centralize some of the exporter test case pieces if you haven't already.


            2) Why the mixed upper/lower case in the path?


            As far as the mixed casein the paths, I am not sure how that came about. Is started in one of the early patches, and should have been cleaned up.

              • Actions