Re: PGweb: Patches and tests

Поиск
Список
Период
Сортировка
От Rodrigo Ramírez Norambuena
Тема Re: PGweb: Patches and tests
Дата
Msg-id CAE9kuxNFeVzkbM0V3Th=0uqgJUcuw302L8Fu2Rag7Ftogk1AZA@mail.gmail.com
обсуждение исходный текст
Ответ на Re: PGweb: Patches and tests  (Rodrigo Ramírez Norambuena <decipher.hk@gmail.com>)
Список pgsql-www
On Sun, Sep 15, 2019 at 2:34 PM Rodrigo Ramírez Norambuena
<decipher.hk@gmail.com> wrote:
>
> On Thu, Sep 12, 2019 at 12:25 PM Rodrigo Ramírez Norambuena
> <decipher.hk@gmail.com> wrote:
> >
> > On Wed, Sep 11, 2019 at 4:05 PM Jonathan S. Katz <jkatz@postgresql.org> wrote:
> > >
>
> >
> >
> > > For 0005, I would not put that in a migration. I would put that in a
> > > test runner or whatever script we will use to build the test
> > > environment, to mock varnish (if we need to -- not sure if we do?).
> >
> > Mock the varnish in first step could be more obfuscate because a
> > purge_urls is present in the almost all models. The kick start is a
> > add sql into db as varnish_local, after that could add a helper to
> > test varnish function to load these functions in db.
> >
> > About not put in migration seams good choice but this migration run
> > only in test model. I'll could find a other way to add for the runner
> > or more clean. If you have any idea about this let me know.
>
>
> For this topic, I attach a new patch with a runner. Inside the runner
> is execute SQL sentences for Varnish.
>
> This is better approach against to previous inside of migration model.


Hey Guys!,

There someone can take a look to my previous patches?

I want to introduce some changes to build a test suite but I need
address with the before patches.

Regards!



--
Rodrigo Ramírez Norambuena
http://www.rodrigoramirez.com/



В списке pgsql-www по дате отправления:

Предыдущее
От: Magnus Hagander
Дата:
Сообщение: Re: [PATCH] 32x32 and 48x48 favicons
Следующее
От: Peter Geoghegan
Дата:
Сообщение: Re: [PATCH] 32x32 and 48x48 favicons