Build with -D RDK_BUILD_COORDGEN_SUPPORT=OFF includes a test case that depends on MaeMolSupplier

[rvianello]

Configuration:

• RDKit Version: current tip of the master branch
• Operating system: Linux (might affect other OSs too)
• Are you using conda? no

Description:
In file Code/GraphMol/FileParsers/testMolSupplier.cpp, testGitHub2881 I think is missing some conditional compilation directives, so that it's not built when RDK_BUILD_COORDGEN_SUPPORT is disabled.

[ 76%] Building CXX object Code/GraphMol/FileParsers/CMakeFiles/testMolSupplier.dir/testMolSupplier.cpp.o
/opt/RDKit-build/rdkit/Code/GraphMol/FileParsers/testMolSupplier.cpp: In function 'void testGitHub2881()':
/opt/RDKit-build/rdkit/Code/GraphMol/FileParsers/testMolSupplier.cpp:2806:5: error: 'MaeMolSupplier' was not declared in this scope

[greglandrum]

Yep, that was my fault. Thanks for pointing it out.

[rvianello]

no problem, sorry I didn't just create a PR, I can do so later today I think (although the fix should be trivial enough to be included into any other changeset).

[greglandrum]

Na, I'll take care of it. I'm working on an "Ooops" PR anyway; I will do this one at the same time

[rvianello]

I see, great, thanks a lot then.