On Thu, Mar 9, 2017 at 11:59 AM, sebb <seb...@gmail.com> wrote: > What happens if the source contains values that are not yet defined? > These may affect the compilation result. > No, there will not be an error for that.
> It would be annoying to have the test element flagged as failed just > because the variable is not defined. > > Equally, if compilation succeeds, it's possible for it to fail at > run-time because of the actual variable value. > As explained in my initial message, compilation will not detect those. It's a first check but unfortunately it will not fail in this case. > > If the feature is added, the docs need to make such restrictions clear. > > > On 9 March 2017 at 10:04, Antonio Gomes Rodrigues <ra0...@gmail.com> > wrote: > > Hi, > > > > +1 > > > > But I think we need to rename it "Validate" and add it to icon menu + > > somewhere in the menu but not in Help. > > > > We can add more verification (check anti pattern...) later > > > > Antonio > > > > 2017-03-09 9:39 GMT+01:00 Maxime Chassagneux < > maxime.chassagn...@gmail.com>: > > > >> Hello, > >> > >> For me this option is really interesting as I use a lot of JSR233 > script. > >> > >> +1 for add it ! > >> > >> -- > >> Max > >> > >> 2017-03-09 8:41 GMT+01:00 Philippe Mouawad <philippe.moua...@gmail.com > >: > >> > >> > Hello, > >> > I have implemented a menu option (in Help that can be moved somewhere > >> else) > >> > that allows compilation of JSR223 Test Elements. > >> > > >> > It visits the test plan tree and compiles any Test Element. > >> > If compilation fails it set the node in red and logs the compilation > >> > failure. > >> > > >> > This allows detecting some errors without running test plan but not > all > >> > errors like invalid variable usage. > >> > > >> > Do you think it should be added to core (It changes JSR223TestElement > >> > (compile method) and add 1 class for menu) ? > >> > Thanks > >> > Regards > >> > > >> > -- Cordialement. Philippe Mouawad.
