Sure - I keep that requirement in mind.
What's hardcoded now will be default value, yet configurable.

Ondra



Jaikiran Pai píše v Po 07. 11. 2011 v 13:58 +0530:
On Monday 07 November 2011 01:54 PM, Ondřej Žižka wrote:
> Please don't hardcode paths in tests.
> Remember that tests may be shuffled around.
> I'll introduce a new system properties, pointing to the testsuite dir, 
> evt. to project's root dir.
Just a FYI - Some of the developers use the IDE to run the tests (I 
don't). So if you introduce these system properties, this should not 
force the IDE users to have to set the system properties for these tests 
to be able to run them, IMO.

-Jaikiran
>
> Also please remember that the testsuite will run agains productized 
> build -
> perhaps the following tests could use XSD's from src/main/resources 
> instead?
> (QA will run against provided EAP, let's not force them to run whole 
> build.)
>
> Thanks,
> Ondra
>
>
>
> public class AbstractValidationUnitTest {
>     private static final Map<String, File> JBOSS_SCHEMAS = new 
> HashMap<String, File>();
>     private static Map<String, String> NAMESPACE_MAP = new 
> HashMap<String, String>();
>     static final File MOD_DIR = new File(System.getProperty("user.dir"));
>     static final File TARGET_DIR = new File(MOD_DIR, 
> "../../build/target/");
>     private static File BASE_DIR = null;
>
>     static {
>         
> NAMESPACE_MAP.put("http://java.sun.com/xml/ns/javaee/javaee_6.xsd", 
> "schema/javaee_6.xsd");
>         NAMESPACE_MAP.put("http://www.w3.org/2001/xml.xsd", 
> "schema/xml.xsd");
>         final File[] children = TARGET_DIR.listFiles();
>         if (children != null) {
>             for (File child : children) {
>                 if (child.getName().startsWith("jboss-")) {
>                     BASE_DIR = child;
>                     break;
>                 }
>             }
>             if (BASE_DIR != null) {
>                 final File schemaDir = new File(BASE_DIR, "docs/schema");
>                 final File[] xsds = 
> schemaDir.listFiles(SchemaFilter.FILTER);
>                 for (File xsd : xsds) {
>                     JBOSS_SCHEMAS.put(xsd.getName(), xsd);
>                 }
>             }
>         }
>     }
>
>
>
> _______________________________________________
> jboss-as7-dev mailing list
> jboss-as7-dev@lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev

_______________________________________________
jboss-as7-dev mailing list
jboss-as7-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev