Add some tests for the Qhull optional package

[sagetrac-mhampton]

At our dear leader's request I am adding a test file for the Qhull optional package.

This will be quite minimal, since what I'd really like is to add a Qhull backend for the Polyhedron class, which would come with its own tests.

If anyone is interested in adding more elaborate tests it wouldn't be very hard since the documentation at the Qhull site is pretty good.

CC: @videlec

Component: packages: optional

Keywords: Qhull, polyhedra, polyhedron

Author: Frédéric Chapoton

Branch/Commit: public/ticket/12352 @ 547c508

Reviewer: Frédéric Chapoton

Issue created by migration from https://trac.sagemath.org/ticket/12352

[sagetrac-mhampton]

Attachment: trac_12352_Qhull_optional_package_testfile.patch.gz

adds a minimal test for the Qhull optional package

[loefflerd]

comment:2

The tests need to be marked with "# optional qhull" or something, since otherwise they fail when this optional package isn't installed.

[fchapoton]

Branch: public/ticket/12352

[fchapoton]

New commits:

24a3059	trac #12352: adding some tests for the Qhull optional package
547c508	trac #12352 refreshed

[fchapoton]

Commit: 547c508

[jdemeyer]

comment:8

Since this is just tests and nothing else, why not add this to a spkg-check file inside the qhull package?

[videlec]

comment:9

Replying to @jdemeyer:

Since this is just tests and nothing else, why not add this to a spkg-check file inside the qhull package?

+1. There is no interface being tested or so.

Doesn't qhull have a testsuite?

[mkoeppe]

comment:10

Marking this as "wontfix". The spkg-check of the qhull package (#21143) seems enough.

[fchapoton]

Changed author from Marshall Hampton to Frédéric Chapoton

[fchapoton]

Reviewer: Frédéric Chapoton

[embray]

comment:12

Determined to be invalid/duplicate/wontfix (closing as "wontfix" as a catch-all resolution).