[phpMyAdmin Developers] templates validation

Marc Delisle marc at infomarc.info
Sat Sep 5 23:22:10 CEST 2015


Le 2015-09-05 15:38, Hugues Peccatte a écrit :
> Le lun. 31 août 2015 à 10:06, Michal Čihař <michal at cihar.com
> <mailto:michal at cihar.com>> a écrit :
> 
>     Dne Sun, 30 Aug 2015 13:45:14 +0000
>     Hugues Peccatte <hugues.peccatte at gmail.com
>     <mailto:hugues.peccatte at gmail.com>> napsal(a):
> 
>     > Le dim. 30 août 2015 à 15:11, Marc Delisle <marc at infomarc.info
>     <mailto:marc at infomarc.info>> a écrit :
>     >
>     > > Hi,
>     > > please have a look at
>     > >
>     > >
>     https://github.com/phpmyadmin/phpmyadmin/commit/993134cacd61add9f049eee96a53943a87c99342
>     > >
>     > > Can we have a validation when a template is rendered, to ensure
>     we don't
>     > > forget passing some of the parameters?
>     > >
>     > > Probably too late for 4.5, but at least for 4.6?
>     > >
>     > > --
>     > > Marc Delisle | phpMyAdmin
>     > >
>     >
>     > Hi,
>     >
>     > What do you mean by "validation"? Is it a code review or a message by
>     > Template class that tells you that a variable / parameter is missing?
> 
>     The best would be if we can make the template layer fail on missing
>     variable and test all templates in testsuite.
>
> Hi,
> 
> I agree with this last comment.
> Should we review our position regarding template unit tests?

As a reminder, here it is:
https://wiki.phpmyadmin.net/pma/2015-06_Meeting#Unit_tests_on_HTML

But failing on a missing variable does not necessarily mean that we have
to test in the testsuite.

-- 
Marc Delisle | phpMyAdmin



More information about the Developers mailing list