On 9 March 2017 at 12:11, Philippe Mouawad <[email protected]> wrote: > On Thu, Mar 9, 2017 at 11:59 AM, sebb <[email protected]> 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.
Are you sure? Since variables can be used to replace any text in a script, there's no way of knowing if it will affect compilation or not. >> 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. This is still true. >> >> >> On 9 March 2017 at 10:04, Antonio Gomes Rodrigues <[email protected]> >> 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 < >> [email protected]>: >> > >> >> 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 <[email protected] >> >: >> >> >> >> > 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.
