After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 738000 - Current "tests" directory should be removed
Current "tests" directory should be removed
Status: RESOLVED FIXED
Product: at-spi
Classification: Platform
Component: at-spi2-atk
unspecified
Other Linux
: Normal normal
: ---
Assigned To: At-spi maintainer(s)
At-spi maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2014-10-06 12:32 UTC by Alejandro Piñeiro Iglesias (IRC: infapi00)
Modified: 2014-10-21 00:24 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Removed tests directory (52.32 KB, patch)
2014-10-06 12:32 UTC, Alejandro Piñeiro Iglesias (IRC: infapi00)
committed Details | Review

Description Alejandro Piñeiro Iglesias (IRC: infapi00) 2014-10-06 12:32:51 UTC
Created attachment 287846 [details] [review]
Removed tests directory

There is a directory currently named "tests", and in my opinion they should be removed. Reasons:

 * They are not tests: they are just basic examples
 * They don't belong to at-spi2-atk: those examples doesn't depend at all of ATK. So they would belong to at-spi2-core, but ...
 * They are totally outdated. They are based on the old cspi. If that functionality is worthy as examples, those should be rewritten using atspi.
 * Real tests are coming (bug 730505), so it would be good to have the directory name "tests" available for it.
Comment 1 Mike Gorse 2014-10-07 09:28:20 UTC
Comment on attachment 287846 [details] [review]
Removed tests directory

I agree--I didn't even realize that that directory existed! Thanks for the report.
Comment 2 Mike Gorse 2014-10-21 00:24:29 UTC
Comment on attachment 287846 [details] [review]
Removed tests directory

Pushed to master as commit cdc593.