-
Notifications
You must be signed in to change notification settings - Fork 13
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
Scheduled automated testing & notification emails #410
Comments
Probably using headless webkit, potentially on mendeleev (where phettest.colorado.edu lives) |
@jonathanolson I'll take the emails if this ever happens. Maybe nightly is overkill though? We should give time for things to be fixed, or else we would be getting repetitive errors every night (assuming errors aren't fixed the same day). |
Nightly seems like it would work fine if it reports only new bugs (maybe a weekly email about current bugs). |
Well certainly if it could manage to only report new ones, I'm all for nightly. |
@andrewadare mentioned an interest in continuous integration services for automated testing. |
@samreid I looked into two continuous integration options earlier this summer: Jenkins (self-hosted) and Travis-CI (remote). Jenkins turned out to be more complicated to configure than I anticipated. If people consider this a high priority, the topic can be revisited, but I have not gotten that impression. Since @jonathanolson has some experience in the area of automated testing, it may be valuable to confer with him. |
@andrewadare might be good to bring up again at an upcoming developer meeting. I would not say a "high priority" but I do want us to continue to work on it. |
I'd recommend the following steps:
|
By the way, I'd like to see the type checking tests into our normal tests, but it could take some work to get there (mainly telling it how our types are defined across files): #483 |
@samreid What are "the type checking tests"? |
Ah, never mind. You're referring to flow-jsdoc (or something similar) as mentioned in #410. |
Figaro sounds potentially appropriate? |
I tried running phantomjs on figaro by the following steps:
wget https://bitbucket.org/ariya/phantomjs/downloads/phantomjs-2.1.1-linux-x86_64.tar.bz2
tar -vxjf phantomjs-2.1.1-linux-x86_64.tar.bz2?Signature=pay7OdwEahqHoiYHg1JHp%2FZ6z4w=&Expires=1473898339&AWSAccessKeyId=AKIAIWY5XSVPZPDQYRQQ&versionId=null&response-content-disposition=attachment;\ filename="phantomjs-2.1.1-linux-x86_64.tar.bz2"
create hello.js with this content (from the Quick Start) http://phantomjs.org/quick-start.html console.log('Hello, world!');
phantom.exit(); phantomjs-2.1.1-linux-x86_64/bin/phantomjs hello.js
In the download docs: http://phantomjs.org/download.html It says:
I wonder if we are missing some things? |
We could build our own phantom but it seems somewhat invasive: http://phantomjs.org/build.html |
Just checked, but I can't 'sudo yum' to install packages globally. If we want more full control, we could go with something like phettest.colorado.edu, however it could slow down the normal operation (it's not terribly beefy). |
I'm not sure what the status of figaro is, are we using it as a backup system? Will it still be managed by OIT? If not, perhaps we can have sudo. If it will be managed by OIT, then perhaps we just ask if they can install phantomjs (and other utilities as necessary). |
A bit more background on my research from this summer: Travis CI and CircleCI are remotely-hosted options. For open-source repos, Travis is free, and I'm pretty sure that's also the case for CircleCI. But PhET-iO poses a problem, which is what put me off. |
@ariel-phet says figaro is currently a failover machine. We would have to ask @kathy-phet when we might retire it. |
Can you elaborate? Oh, is it because it is not open source? |
It's not open source, so the licensing is different. We'd have to pay for CI services from Travis or CircleCI. |
@ariel-phet said figaro may be retired, and we may use simian for testing? Let's talk with @kathy-phet about this. |
We could run tests on simian, and just pause the tests if simian has to be used as a failover. |
@mattpen is going to ping Doug Johnson about whether we can use simian for this. |
OIT asked what we will need and I replied:
|
I created repo "aqua" for Automatic QUality Assurance, and moved issues to that repo, closing this one and we will continue in phetsims/aqua#4 |
It would be great to have testing run nightly, and send out an email if there are any failures?
Who should get sent emails?
The text was updated successfully, but these errors were encountered: