Activity
From 09/05/2022 to 10/04/2022
10/04/2022
- 06:51 AM Feature #6237: OEUnit support
- To pass test descriptors over network, we need to make them @Serializable@ or @Externalizable@. Original POS test des...
10/03/2022
- 12:59 PM Feature #6237: OEUnit support
- I have no objection to the bootstrap cfg approach.
- 12:09 PM Feature #6237: OEUnit support
- Vladimir Tsichevski wrote:
> I think, we can use the @BootstrapConfig@ to pass the test engine discover request data... - 10:20 AM Feature #6237: OEUnit support
- I think, we can use the @BootstrapConfig@ to pass the test engine discover request data from test runner (e.g. Eclips...
- 09:13 AM Feature #6237: OEUnit support
- One more change required for the JUnin5 framework would able to discover the custom test engine.
See @6237-addon.d...
09/30/2022
- 02:05 PM Feature #6237: OEUnit support
- Greg Shah wrote:
> As list of pending changes as noted by Vladimir via email:
>
> [...]
>
> For changes 1 thro...
09/29/2022
- 02:03 PM Feature #6237: OEUnit support
- As list of pending changes as noted by Vladimir via email:...
- 11:46 AM Feature #6237: OEUnit support
- Greg Shah wrote:
> > So we need to either disable the Jupiter engine somehow (which I failed to find out how), or us... - 08:10 AM Feature #6237: OEUnit support
- > So we need to either disable the Jupiter engine somehow (which I failed to find out how), or use our own markup for...
- 05:09 AM Feature #6237: OEUnit support
- Vladimir Tsichevski wrote:
> Vladimir Tsichevski wrote:
>
> > The little problem with Eclipse is that it always u...
09/28/2022
- 12:23 PM Feature #6237: OEUnit support
- Vladimir Tsichevski wrote:
> The little problem with Eclipse is that it always uses some version of the "Jupiter" ... - 12:16 PM Feature #6237: OEUnit support
- Greg Shah wrote:
> Code Review uast-hints-unit-test-type.diff
>
> I don't see any purpose in having @ELEM_UNITTES...
09/27/2022
- 02:14 PM Feature #6237: OEUnit support
- Greg Shah wrote:
> Please look in your directory and search on @RemoteErrorData@ to see the ACL you must have to ens... - 02:08 PM Feature #6237: OEUnit support
- Greg Shah wrote:
> No, not yet. I'm hoping to open this up on Thursday.
>
> Before any commit happens, I will ne... - 02:07 PM Feature #6237: OEUnit support
- No, not yet. I'm hoping to open this up on Thursday.
Before any commit happens, I will need to see the proposed d... - 02:02 PM Feature #6237: OEUnit support
- Greg Shah wrote:
> Code Review strict-junit5.diff
>
> I have no objections except to note that history entries ar... - 01:45 PM Feature #6237: OEUnit support
- Greg Shah wrote:
> Code Review junit5-custom-engine.zip
>
> Overall, I don't have an objection to the code so far... - 01:42 PM Feature #6237: OEUnit support
- Code Review uast-hints-unit-test-type.diff
I don't see any purpose in having @ELEM_UNITTESTTYPE@ and also the @UNI... - 01:37 PM Feature #6237: OEUnit support
- Code Review strict-junit5.diff
I have no objections except to note that history entries are needed. - 01:36 PM Feature #6237: OEUnit support
- Code Review junit5-custom-engine.zip
Overall, I don't have an objection to the code so far. I understand it is ea... - 01:12 PM Feature #6237: OEUnit support
- Please look in your directory and search on @RemoteErrorData@ to see the ACL you must have to ensure access to the ne...
- 12:58 PM Feature #6237: OEUnit support
- Greg Shah wrote:
> Please post your code here.
Done:
@6237-remote-object-problem.diff@ - 12:38 PM Feature #6237: OEUnit support
- Please post your code here.
- 12:19 PM Feature #6237: OEUnit support
- Greg Shah wrote:
> > And the implementation class @UnitTestServer@
>
> @UnitTestServer@ should not be in packag... - 09:38 AM Feature #6237: OEUnit support
- > I mean, if we replace the execution of a method of main by execution of an arbitrary lambda function, the access to...
- 09:11 AM Feature #6237: OEUnit support
- Vladimir Tsichevski wrote:
> and I got the error:
>
> [...]
From experiments I learned that some existing re...
09/26/2022
- 06:14 PM Feature #6237: OEUnit support
- Greg Shah wrote:
> > If we just replace the call @main.standardEntry()@ by some lambda passed to the method as a par... - 01:25 PM Feature #6237: OEUnit support
- > If we just replace the call @main.standardEntry()@ by some lambda passed to the method as a parameter, we at least ...
- 10:57 AM Feature #6237: OEUnit support
- Question: currently a remote instance of @MainEntry@ is not only obtained inside the @ClientCore.start()@, but some m...
- 08:09 AM Feature #6237: OEUnit support
- Not exactly. The conversation thread starts when the session is created with the server. Everything else (steps 1, ...
- 07:45 AM Feature #6237: OEUnit support
- Greg Shah wrote:
> Vladimir Tsichevski wrote:
> > Greg Shah wrote:
> >
> > > > Why do we need another thread?
>... - 07:16 AM Feature #6237: OEUnit support
- Vladimir Tsichevski wrote:
> Greg Shah wrote:
>
> > > Why do we need another thread?
> >
> > It seems incorrec... - 07:10 AM Feature #6237: OEUnit support
- Greg Shah wrote:
>
> I've been assuming that the JUnit stuff is implicitly batch mode, but perhaps it is possibl... - 06:49 AM Feature #6237: OEUnit support
- Greg Shah wrote:
> > Why do we need another thread?
>
> It seems incorrect to "take over" the calling thread to... - 05:47 AM Feature #6237: OEUnit support
- > Does that mean we will not use the StandardServer at all?
Maybe. At least @StandardServer.standardEntry()@ is n...
09/23/2022
- 03:17 PM Feature #6237: OEUnit support
- Greg Shah wrote:
> I think the way to go is to create a FWD client in-process from the JUnit @TestEngine@ code:
>... - 08:55 AM Feature #6237: OEUnit support
- Greg Shah wrote:
> There is no patch posted for the hints changes.
Strangely enough :-( - 08:28 AM Feature #6237: OEUnit support
- There is no patch posted for the hints changes.
- 05:57 AM Feature #6237: OEUnit support
- Vladimir Tsichevski wrote:
> Greg Shah wrote:
>
> > For now, just implement the multilevel configuration approach...
09/21/2022
- 12:25 PM Feature #6237: OEUnit support
- Greg Shah wrote:
> For now, just implement the multilevel configuration approach. If it gets to the point where w... - 12:24 PM Feature #6237: OEUnit support
- I've created a custom JUnit5 testing engine (see @junit5-custom-engine.zip@).
This is the first version, kind of P... - 12:11 PM Feature #6237: OEUnit support
- > If we can define this configuration parameter at any level, using also file and directory hints, then we do not nee...
- 11:55 AM Feature #6237: OEUnit support
- Greg Shah wrote:
> I think we need to know on a per-file level which unit testing framework is used. This is not ha... - 11:28 AM Feature #6237: OEUnit support
- I think we need to know on a per-file level which unit testing framework is used. This is not hard to do. As noted ...
- 11:15 AM Feature #6237: OEUnit support
- Marian Edu wrote:
> Vladimir Tsichevski wrote:
> > So, you propose to pre-scan *all* classes in conversion project ... - 08:59 AM Feature #6237: OEUnit support
- Vladimir Tsichevski wrote:
> So, you propose to pre-scan *all* classes in conversion project to find any @assertXXX@... - 08:44 AM Feature #6237: OEUnit support
- > you propose to pre-scan all classes in conversion project to find any assertXXX statement
This is pretty easy. ... - 08:19 AM Feature #6237: OEUnit support
- Marian Edu wrote:
> Vladimir Tsichevski wrote:
>
> > So, since neither of these methods is reliable
>
> I real... - 08:09 AM Feature #6237: OEUnit support
- > adding implementation for OEUnit Assert in FWD would be the easier path.
We will provide the OEUnit Assert supp... - 07:21 AM Feature #6237: OEUnit support
- Vladimir Tsichevski wrote:
> So, since neither of these methods is reliable
I really fail to see the issue here...
09/20/2022
- 07:35 AM Feature #6237: OEUnit support
- > emit a warning and ignore the unit test annotations.
This one. - 05:54 AM Feature #6237: OEUnit support
- Marian Edu wrote:
> Vladimir Tsichevski wrote:
>
> > I think, our goal is to have neither original @OEUnit@ nor @... - 02:56 AM Feature #6237: OEUnit support
- Vladimir Tsichevski wrote:
> I think, our goal is to have neither original @OEUnit@ nor @ABLUnit@ in the @PROPATH@...
09/19/2022
- 02:52 PM Bug #6768 (Rejected): End of comment "butted" against double-quote fails to parse
- It is already fixed in the latest version of FWD.
- 01:44 PM Bug #6768: End of comment "butted" against double-quote fails to parse
- I updated @uast/comment_test0.p@ to include some additional tests:...
- 12:27 PM Bug #6768: End of comment "butted" against double-quote fails to parse
- If this isn't an issue in FWD v4, then we can close this.
- 11:17 AM Bug #6768: End of comment "butted" against double-quote fails to parse
- It was reported that this yields a conversion error (with an older version of FWD):...
- 10:28 AM Bug #6768 (Rejected): End of comment "butted" against double-quote fails to parse
- 02:22 PM Feature #6237: OEUnit support
- Marian Edu wrote:
> Greg Shah wrote:
> > Define a global parameter (in @cfg/p2j.cfg.xml@) for @OEUnit@ vs @ABLUnit@... - 01:36 PM Feature #6237: OEUnit support
- Greg Shah wrote:
> Define a global parameter (in @cfg/p2j.cfg.xml@) for @OEUnit@ vs @ABLUnit@. Then add an override... - 11:35 AM Feature #6237: OEUnit support
- Greg Shah wrote:
> The code is going to be very specific to converted 4GL code so it should be @com.goldencode.p2j.t... - 09:26 AM Feature #6237: OEUnit support
- The code is going to be very specific to converted 4GL code so it should be @com.goldencode.p2j.testengine@.
- 09:20 AM Feature #6237: OEUnit support
- I am implementing a specialized JUnit5 test engine for FWD. The code works, and now I need some permanent Java packag...
09/16/2022
- 01:59 PM Feature #6237: OEUnit support
- > How global parameters in @cfg/p2j.cfg.xml@ do relate to uast hints?
They are read from @UastHints@ as the defau... - 01:16 PM Feature #6237: OEUnit support
- Greg Shah wrote:
> Define a global parameter (in @cfg/p2j.cfg.xml@) for @OEUnit@ vs @ABLUnit@. Then add an override... - 11:58 AM Feature #6237: OEUnit support
- Define a global parameter (in @cfg/p2j.cfg.xml@) for @OEUnit@ vs @ABLUnit@. Then add an override in the @UastHints@ ...
- 11:49 AM Feature #6237: OEUnit support
- We have some difficulty if we are going to implement both @OEUnit@ and @ABLUnit@.
There is a difference in the usa... - 09:14 AM Feature #6237: OEUnit support
- Marian Edu wrote:
> this is the only OEUnit I'm aware of - [[https://github.com/CameronWills/OEUnit]]
Yes, I m... - 01:55 AM Feature #6237: OEUnit support
- Vladimir Tsichevski wrote:
> So we need to answer the following questions:
>
> # Whether or not and how class-l...
09/15/2022
- 05:06 PM Feature #6237: OEUnit support
- > # Whether or not and how class-level annotations (@@BeforeClass@, @@AfterClass@) are used in 4gl procedures?
Ass... - 04:41 PM Feature #6237: OEUnit support
- The latest problem reveals that we need to convert 4gl procedure annotation either.
So we need to answer the follo...
09/14/2022
- 12:26 PM Feature #6237: OEUnit support
- To add JUnit support to converted projects, the following patch has to be applied to @build.gradle@:...
09/13/2022
- 02:22 PM Feature #6237: OEUnit support
- Constantin Asofiei wrote:
> The code previously meant to allow @@LegacyService@ annotation in the 4GL code. Now i... - 01:43 PM Feature #6237: OEUnit support
- Vladimir Tsichevski wrote:
> Constantin Asofiei wrote:
> > legacy_services.rules is built to allow @LegacyService@ ... - 01:40 PM Feature #6237: OEUnit support
- Constantin Asofiei wrote:
> legacy_services.rules is built to allow @LegacyService@ annotations written directly in ... - 01:16 PM Feature #6237: OEUnit support
- legacy_services.rules is built to allow @LegacyService@ annotations written directly in FWD code - these must be kept...
- 12:59 PM Feature #6237: OEUnit support
- Constantin Asofiei wrote:
> Regarding 3821c/14239:
> * please format @annotations/unit_test_support.rules@ by inden... - 06:10 AM Feature #6237: OEUnit support
- Regarding 3821c/14239:
* please format @annotations/unit_test_support.rules@ by indenting the nested @rules@ tags - ...
09/12/2022
- 05:42 PM Feature #6237: OEUnit support
- Conversion of 4gl annotations to JUnit5 Java annotations implemented in 3821c rev. 14239.
09/08/2022
- 03:37 AM Feature #6237: OEUnit support
- I always use the @String@ methods directly (@equals()@ or @equalsIgnoreCase()@). Even if we did implement string con...
09/07/2022
- 02:02 PM Feature #6237: OEUnit support
- Greg Shah wrote:
> In TRPL, are we mapping @annoName == "Before"@ to @String.equalsIgnoreCase()@?
I did not know th... - 01:40 PM Feature #6237: OEUnit support
- Vladimir Tsichevski wrote:
> Greg Shah wrote:
>
> >
> > Does this make sense?
>
> If I get it right, then:
... - 01:39 PM Feature #6237: OEUnit support
- In TRPL, are we mapping @annoName == "Before"@ to @String.equalsIgnoreCase()@? If not then shouldn't we do that expl...
- 01:38 PM Feature #6237: OEUnit support
- Greg Shah wrote:
>
> Does this make sense?
If I get it right, then:
# We can execute on the server side an... - 12:52 PM Feature #6237: OEUnit support
- Greg Shah wrote:
> > The question is in which module this javaName is assigned? It would be better to calculate the ... - 12:48 PM Feature #6237: OEUnit support
- Greg Shah wrote:
> Please rename both @oeunit.rules@ files to @unit_test_support.rules@. Remember that we are imple... - 12:35 PM Feature #6237: OEUnit support
- Vladimir: Please post questions or comments here. As a next step, I'd like to see the proposed design of the @Remote...
- 12:29 PM Feature #6237: OEUnit support
- Please rename both @oeunit.rules@ files to @unit_test_support.rules@. Remember that we are implementing support for ...
- 12:27 PM Feature #6237: OEUnit support
- Vladimir Tsichevski wrote:
> I suppose, the Java method, produced by conversion of a 4gl class, must be called in so... - 11:56 AM Feature #6237: OEUnit support
- > The question is in which module this javaName is assigned? It would be better to calculate the correct value for th...
- 09:37 AM Bug #6738: incremental conversion when deleting program or class files
- Allow incremental conversion to detect program or class files which have been deleted, and cleanup after it (includin...
- 09:36 AM Bug #6738 (Closed): incremental conversion when deleting program or class files
09/06/2022
- 11:55 AM Feature #6237: OEUnit support
- I suppose, the Java method, produced by conversion of a 4gl class, must be called in some special way?
- 11:35 AM Feature #6237: OEUnit support
- Now that I have all Junit5 annotations in place, I have another problem when I try to run the @SimpleTest@ class as J...
09/05/2022
- 06:27 PM Feature #6237: OEUnit support
- I've added support for JUnit 5 annotations generation from OEUnit annotations.
3 things changed:
# Code which d... - 02:25 PM Feature #6237: OEUnit support
- The question is in which module this @javaName@ is assigned? It would be better to calculate the correct value for th...
- 02:22 PM Feature #6237: OEUnit support
- Greg Shah wrote:
> > What am I doing wrong here? I know setting text works on the copy:
>
> The problem is subtle... - 02:28 AM Feature #6237: OEUnit support
- > The first will be called from "annotations.xml", and do Java annotations preparation, the second will be called fro...
- 02:27 AM Feature #6237: OEUnit support
- > What am I doing wrong here? I know setting text works on the copy:
The problem is subtle. The @setText()@ you a...
Also available in: Atom