Fwd: [fyi][icedtea-web]Refactoring of reproducers as agreed in April

Jiri Vanek jvanek at redhat.com
Fri Jun 22 11:02:59 PDT 2012


hmhm .. First version did not reach the list for its size, resending with compressed patches to keep you in loop..
J.

-------- Original Message --------
Subject: [fyi][icedtea-web]Refactoring of reproducers  as agreed in April
Date: Fri, 22 Jun 2012 13:16:38 +0200
From: Jiri Vanek <jvanek at redhat.com>
To: Omair Majid <omajid at redhat.com>
CC: IcedTea Distro List <distro-pkg-dev at openjdk.java.net>

Hi!

This is effort to make reproducers structure little bit more sense-full and to *get rid of all the initialization errors* in tests runs.
The refactoring is done as we have agreed in "Re: [rfc] [icedtea-web] blacklist for reproducers" from 13.4.2012 11:54PM (cet?)

so sumamry :)
	1. hg move tests/jnlp_tests/ tests/netx/reproducers
	2. hg move tests/netx/jnlp_testsengine/ tests/test-extensions/
          3. extracted inner classes and tests from test-extensions
          4. adapted makefile

Those are four changesets. I have included them together as first two are little bit "nothing to do" and each separated have no sense.

At the end I have discovered I have forgot to add header files to new (extracted inner classes) files. Please, if you can survive them, I will add them in next changeset.

Best regards
    J.


2012-06-21  Jiri Vanek  <jvanek at redhat.com>

	Makefile adapted to recent (three changesets) refactoring
	* Makefile.am: (JNLP_TESTS_ENGINE_SRCDIR) now points correctly to
	test-extensions. (JNLP_TESTS_ENGINE_TESTS_SRCDIR) new variable for
	test-extensions-tests. (JNLP_TESTS_SRCDIR) now points to reproducers.
	(JNLP_TESTS_ENGINE_TESTS_DIR) new variable for built
	JNLP_TESTS_ENGINE_TESTS_SRCDIR
	(netx-dist-tests-tests-source-files.txt) new target for list of
	content of JNLP_TESTS_ENGINE_TESTS_SRCDIR.
	(stamps/netx-dist-tests-tests-compile.stamp) new target for compiling
	netx-dist-tests-tests-source-files.txt
	(netx-dist-tests-source-files.tx) now depends on
	stamps/netx-dist-tests-tests-compile.stamp
	($(REPRODUCERS_CLASS_NAMES)) target is now working in JNLP_TESTS_ENGINE_TESTS_DIR
	instead of JNLP_TESTS_ENGINE_DIR
	(stamps/run-netx-dist-tests.stamp): added JNLP_TESTS_ENGINE_TESTS_DIR to classpath
	(stamps/run-unit-test-code-coverage.stamp), (stamps/run-reproducers-test-code-coverage.stamp)
	added JNLP_TESTS_ENGINE_TESTS_DIR to classpath and
	JNLP_TESTS_ENGINE_TESTS_SRCDIR to sources path

2012-06-21  Jiri Vanek  <jvanek at redhat.com>

	All tests from test-extensions extracted to test-extensions-tests.
	All inner classes in test-extensions extracted as outer classes
	* tests/test-extensions/net/sourceforge/jnlp/ResourcesTest.java:
	moved to test-extensions-tests
	* tests/test-extensions-tests/net/sourceforge/jnlp/ResourcesTest.java:
	new file, copied from test-extensions
	* tests/test-extensions-tests/net/sourceforge/jnlp/ServerAccessTest.java:
	all tests from original ServerAccess.java
	* tests/test-extensions/net/sourceforge/jnlp/ContentReader.java:
	* tests/test-extensions/net/sourceforge/jnlp/LogItem.java:
	* tests/test-extensions/net/sourceforge/jnlp/LoggingBottleneck.java:
	* tests/test-extensions/net/sourceforge/jnlp/ProcessAssasin.java:
	* tests/test-extensions/net/sourceforge/jnlp/ProcessResult.java:
	* tests/test-extensions/net/sourceforge/jnlp/ServerLauncher.java:
	* tests/test-extensions/net/sourceforge/jnlp/TestsLogs.java:
	* tests/test-extensions/net/sourceforge/jnlp/ThreadedProcess.java:
	* tests/test-extensions/net/sourceforge/jnlp/TinyHttpdImpl.java:
	new files, extracted classes from ServerAccess
	* tests/test-extensions/net/sourceforge/jnlp/ServerAccess.java:
	extracted tests and inner classes
	

2012-06-21  Jiri Vanek  <jvanek at redhat.com>

	hg move tests/netx/jnlp_testsengine/ tests/test-extensions/

2012-06-21  Jiri Vanek  <jvanek at redhat.com>

	hg move tests/jnlp_tests/ tests/netx/reproducers

-------------- next part --------------
A non-text attachment was scrubbed...
Name: icedtea-web-commitRefactoringOnReview.tar.gz
Type: application/x-gzip
Size: 120669 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20120622/620e4b58/icedtea-web-commitRefactoringOnReview.tar.gz 


More information about the distro-pkg-dev mailing list