I know - very few people would like that solution.  I would welcome other 
suggestions.

The only thing I can say in my defense is that this is only a test, so as long 
as the intent is clear we should be fine.

-andy

From: openjfx-dev <openjfx-dev-r...@openjdk.org> on behalf of Kevin Rushforth 
<kevin.rushfo...@oracle.com>
Date: Tuesday, July 9, 2024 at 09:03
To: openjfx-dev@openjdk.org <openjfx-dev@openjdk.org>
Subject: Re: [External] : Re: consistent naming for tests


https://github.com/andy-goryachev-oracle/jfx/pull/9

Hmm. I don't really like the abbreviations. They would change something that is 
self-explanatory to something completely opaque, although I see you did leave 
the descriptive name as a comment.

-- Kevin

On 7/9/2024 8:37 AM, Andy Goryachev wrote:
Have you tried building in Eclipse on the latest Linux Mint?  Or building on an 
EncFS mount?

I don't know why Mint decided to use EncFS knowing its issues, and I suppose I 
can try fixing my setup (it's a default Mint installation), but I was quite 
surprised myself and thought that it might be just as easy to fix the tests... 
here is how the fix might look:

https://github.com/andy-goryachev-oracle/jfx/pull/9

-andy

From: John Hendrikx <john.hendr...@gmail.com><mailto:john.hendr...@gmail.com>
Date: Tuesday, July 9, 2024 at 08:22
To: Andy Goryachev 
<andy.goryac...@oracle.com><mailto:andy.goryac...@oracle.com>, Johan Vos 
<johan....@gluonhq.com><mailto:johan....@gluonhq.com>, openjfx-dev 
<openjfx-dev@openjdk.org><mailto:openjfx-dev@openjdk.org>
Subject: [External] : Re: consistent naming for tests


On 09/07/2024 16:52, Andy Goryachev wrote:

Two test files consistently generate an error in Eclipse
- ObservableValueFluentBindingsTest
- LazyObjectBindingTest

I admit I have a weird setup (EncFS on Linux Mint running on MacBook Pro), and 
it only manifests itself in Eclipse and not in the gradle build - perhaps 
Eclipse actually verifies the removal of files?

Anyway, a suggestion - if you use @Nested, please keep the class names short.

This is not an Eclipse bug as I never encounter such issues.  143 characters is 
rather short these days, but I suppose we could limit the nesting a bit.  
Still, I'd look into a way to alleviate this problem in your setup, sooner or 
later this is going to be a problem.
--John

Reply via email to