Skip to content
This repository has been archived by the owner on Aug 22, 2019. It is now read-only.

move all tests into a subdirectory #73

Merged
merged 1 commit into from
Sep 27, 2016
Merged

Conversation

pwnage101
Copy link
Member

@pwnage101 pwnage101 commented Sep 26, 2016

This should just be the "reorganize tests" part of #71

Changes:

  • load test folders are now organized under loadtests/
  • unfortunately had to add a line to every locustfile to make the helpers importable. Link to discussion which calls to question the way locust loads the locustfile. TL;DR, when you specify "loadtests/testname" on the locust CLI, locust adds the loadtests/ directory to sys.path then imports the test module as a top-level module; the proposed solution involves getting locustio/locust to merge a simple PR.
  • README.rst is updated to reflect the change in structure.

Testing:

I tested a few of the load tests and everything imports fine.

@pwnage101
Copy link
Member Author

FYI @rlucioni @tobz

@tobz tobz self-assigned this Sep 27, 2016
@tobz
Copy link

tobz commented Sep 27, 2016

👍

Not sure how to do the native "approved" action. :)

@tobz
Copy link

tobz commented Sep 27, 2016

There we go!

@pwnage101
Copy link
Member Author

thanks, will merge. FYI: locustio/locust#484

@pwnage101 pwnage101 merged commit b5c1eb8 into master Sep 27, 2016
@pwnage101 pwnage101 deleted the pwnage101/PERF-357-subdirs branch September 27, 2016 18:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants