Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add npm run stress to run stress tests #720

Merged
merged 1 commit into from
Apr 4, 2016
Merged

Add npm run stress to run stress tests #720

merged 1 commit into from
Apr 4, 2016

Conversation

reconbot
Copy link
Member

@reconbot reconbot commented Apr 3, 2016

  • Fixed the write memory leak test
  • failed attempt at a open close memory leak test

Closes #602

@rwaldron
Copy link
Contributor

rwaldron commented Apr 3, 2016

Tangentially related... all the "test" stuff should go in one place

@reconbot
Copy link
Member Author

reconbot commented Apr 3, 2016

Slowly getting there

On Sat, Apr 2, 2016, 9:26 PM Rick Waldron notifications@github.com wrote:

Tangentially related... all the "test" stuff should go in one place


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#720 (comment)

@reconbot reconbot force-pushed the stress-tests branch 3 times, most recently from f461ebc to cea7328 Compare April 4, 2016 00:38
 - Fixed the write memory leak test
 - failed attempt at a open close memory leak test

Closes #602
@reconbot
Copy link
Member Author

reconbot commented Apr 4, 2016

I had to basically disable this unless the person manually installs memwatch-next, but it's better than what we've currently got as it actually works. If my issue on memwatch next is ever closed (it's mentioned in the file) then maybe we can have the CI run this.

Also I moved the arduinoTests

@reconbot reconbot merged commit 540a960 into master Apr 4, 2016
@reconbot reconbot deleted the stress-tests branch April 4, 2016 01:21
@lock lock bot locked as resolved and limited conversation to collaborators Jan 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

arduinoTest is broken
2 participants