graphite-web icon indicating copy to clipboard operation
graphite-web copied to clipboard

[BUG] Please include test suite in source release

Open niol opened this issue 2 years ago • 3 comments

Describe the bug Source release does not include webapp/tests/.

Expected behavior This makes is not possible to run the test suite when building from the source tarball.

Environment (please complete the following information):

  • OS flavor: all
  • Graphite-web version 1.1.10
  • Django/Python version all
  • Setup type sources and OS packages

Additional context It would be handy to run the test suite in distributions CI infrastructures.

I'm not sure how releases are generated, but the following may be enough.

--- a/MANIFEST.in
+++ b/MANIFEST.in
@@ -8,5 +8,6 @@ include webapp/graphite/local_settings.py.example
 recursive-include distro/ *
 recursive-include webapp/graphite/ *.html
 recursive-include webapp/content/ *
+recursive-include webapp/tests/ *
 exclude webapp/graphite/local_settings.py
 exclude conf/*.conf

niol avatar Feb 02 '24 08:02 niol

Make sense, will do

deniszh avatar Aug 24 '24 08:08 deniszh

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Apr 26 '25 15:04 stale[bot]

Fix included in the master

deniszh avatar Apr 27 '25 05:04 deniszh

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Jun 27 '25 04:06 stale[bot]