Re: RFR: 8293883: Add test/.classpath (Eclipse) [v2]

2022-09-22 Thread Andy Goryachev
> Adding a tests/.classpath file to include tests/manual/controls source folder 
> so Eclipse could see/execute manual tests there.
> 
> Also, I would rather move the sources there to a specific package 
> (test.manual ?) instead of a default package. Any objections?
> edit: won't be done as a part of this PR.

Andy Goryachev has updated the pull request incrementally with one additional 
commit since the last revision:

  8293883: more dirs under manual/ and performance/

-

Changes:
  - all: https://git.openjdk.org/jfx/pull/901/files
  - new: https://git.openjdk.org/jfx/pull/901/files/06877a4b..9949c68b

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jfx&pr=901&range=01
 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=901&range=00-01

  Stats: 7 lines in 1 file changed: 7 ins; 0 del; 0 mod
  Patch: https://git.openjdk.org/jfx/pull/901.diff
  Fetch: git fetch https://git.openjdk.org/jfx pull/901/head:pull/901

PR: https://git.openjdk.org/jfx/pull/901


Re: RFR: 8293883: Add test/.classpath (Eclipse)

2022-09-22 Thread Andy Goryachev
On Thu, 15 Sep 2022 21:55:20 GMT, Andy Goryachev  wrote:

> Adding a tests/.classpath file to include tests/manual/controls source folder 
> so Eclipse could see/execute manual tests there.
> 
> Also, I would rather move the sources there to a specific package 
> (test.manual ?) instead of a default package. Any objections?
> edit: won't be done as a part of this PR.

Added more dirs under manual/ and performance/ - had to skip certain 
directories that looked like projects in their own right, or that required 
additional dependencies.

-

PR: https://git.openjdk.org/jfx/pull/901


Re: RFR: 8293883: Add test/.classpath (Eclipse)

2022-09-22 Thread Nir Lisker
On Thu, 15 Sep 2022 21:55:20 GMT, Andy Goryachev  wrote:

> Adding a tests/.classpath file to include tests/manual/controls source folder 
> so Eclipse could see/execute manual tests there.
> 
> Also, I would rather move the sources there to a specific package 
> (test.manual ?) instead of a default package. Any objections?
> edit: won't be done as a part of this PR.

The default package is pre-existing bad practice, but doesn't pose an issue, I 
tried it myself. It makes little sense to add just one source folder while 
neglecting all the others. If we are editing this classpath to make the tests 
runnable (even if with the incorrect configuration), then we should make the 
tests runnable.

-

PR: https://git.openjdk.org/jfx/pull/901


Re: RFR: 8293883: Add test/.classpath (Eclipse)

2022-09-22 Thread Andy Goryachev
On Thu, 22 Sep 2022 19:54:41 GMT, Nir Lisker  wrote:

> There are sources under `tests/performance` and `tests/manual` that should 
> also be included in the classpath.

I would like to limit the scope of this PR.  

The problem is that one thing leads to another - and the fact that all these 
manual tests are using the default package instead of their own individual 
packages might lead to issues and/or require more time and attention.

-

PR: https://git.openjdk.org/jfx/pull/901


Re: RFR: 8293883: Add test/.classpath (Eclipse)

2022-09-22 Thread Nir Lisker
On Thu, 15 Sep 2022 21:55:20 GMT, Andy Goryachev  wrote:

> Adding a tests/.classpath file to include tests/manual/controls source folder 
> so Eclipse could see/execute manual tests there.
> 
> Also, I would rather move the sources there to a specific package 
> (test.manual ?) instead of a default package. Any objections?
> edit: won't be done as a part of this PR.

There are sources under `tests/performance` and `tests/manual` that should also 
be included in the classpath.

I was actually reviewing it just now.

Same as with apps, this is not the way the tests should be ordered. While it 
allows to run them from Eclipse, the way the tests are currently divided means 
that they should be (sub-) projects. We will need a follow-up on this.

@kevinrushforth Are you planning on continuing the cleanup of apps and tests 
soon? If so, we should wait for it with the Eclipse changes.

-

Changes requested by nlisker (Reviewer).

PR: https://git.openjdk.org/jfx/pull/901


Re: RFR: 8293883: Add test/.classpath (Eclipse)

2022-09-22 Thread Andy Goryachev
On Thu, 15 Sep 2022 21:55:20 GMT, Andy Goryachev  wrote:

> Adding a tests/.classpath file to include tests/manual/controls source folder 
> so Eclipse could see/execute manual tests there.
> 
> Also, I would rather move the sources there to a specific package 
> (test.manual ?) instead of a default package. Any objections?
> edit: won't be done as a part of this PR.

@nlisker :
could you please review this small change, I wanted to publish a demo/manual 
test for the constrained resize policy in 
[JDK-8293119](https://bugs.openjdk.org/browse/JDK-8293119) to the mailing list?
thank you in advance!
-andy

-

PR: https://git.openjdk.org/jfx/pull/901


Re: RFR: 8293883: Add test/.classpath (Eclipse)

2022-09-16 Thread Kevin Rushforth
On Thu, 15 Sep 2022 21:55:20 GMT, Andy Goryachev  wrote:

> Also, I would rather move the sources there to a specific package 
> (test.manual ?) instead of a default package. Any objections?

That should probably be done via a separate cleanup bug, since it is outside 
the scope of this IDE-only fix. It would make sense to at least look at all of 
the manual tests (since they all exist in the unnamed package), even if you 
want to limit the scope of the follow-on bug to the manual controls tests.

-

PR: https://git.openjdk.org/jfx/pull/901


RFR: 8293883: Add test/.classpath (Eclipse)

2022-09-15 Thread Andy Goryachev
Adding a tests/.classpath file to include tests/manual/controls source folder 
so Eclipse could see/execute manual tests there.

Also, I would rather move the sources there to a specific package (test.manual 
?) instead of a default package. Any objections?

-

Commit messages:
 - 8293883: added tests/.classpath

Changes: https://git.openjdk.org/jfx/pull/901/files
 Webrev: https://webrevs.openjdk.org/?repo=jfx&pr=901&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8293883
  Stats: 2 lines in 1 file changed: 1 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jfx/pull/901.diff
  Fetch: git fetch https://git.openjdk.org/jfx pull/901/head:pull/901

PR: https://git.openjdk.org/jfx/pull/901