[PATCH 2/6] tests: Remove Selenium tests

Stephen Finucane stephen at that.guru
Sat Apr 28 01:29:14 AEST 2018

On Sat, 2018-04-28 at 01:16 +1000, Daniel Axtens wrote:
> It looks like you might have merged this already - sorry for the delay
> as I've been on holiday - but, I still use these, still test with these
> and have (had?) plans to extend these as I update jQuery and Bootstrap
> to more recent versions.

Sure have. We can easily revert these if you do think they're useful,
though I would like to modify things so they don't run by default with
'tox' (possibly by adding an additional target). We can discuss in more
detail when you're back.

Enjoy your holidays :)


> Stephen Finucane <stephen at that.guru> writes:
> > These were added quite some time ago in order to allow some level of UI
> > testing. However, I've personally never used them, they're not used by
> > the CI, and no one has shown any desire in extending them in their time
> > here. It is time to bid these tests adieu.
> > 
> > Removing these allows us to remove a whole load of wiring that existed
> > just to enable these. Some of this, like the '--quick-tox' option for
> > the Dockerfile, is retained so we don't need to use different commands
> > for various versions of Patchwork, but the majority is just stripped
> > out.
> > 
> > Signed-off-by: Stephen Finucane <stephen at that.guru>
> > Cc: Daniel Axtens <dja at axtens.net>

More information about the Patchwork mailing list