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

Initial testing framework. #2

Closed
wants to merge 1 commit into from
Closed

Initial testing framework. #2

wants to merge 1 commit into from

Conversation

KevinGrandon
Copy link
Contributor

WIP - not ready to merge. Just trying to get some pieces in place so we can start bringing the tests over. I haven't used selenium in a while, so it might take a bit for me to come up to speed on it.

The current problem I'm having is just getting to the right 'frame' so we can click on things and make assertions.

Also need to try to use webdriverio for the test syntax, and I think I'd like to use await instead of promises or chaining.

@KevinGrandon KevinGrandon added enhancement help wanted The PR/issue opener needs help to complete/report the task. labels Dec 1, 2015
@bbondy
Copy link
Member

bbondy commented Dec 1, 2015

oh great! Electron has some tests btw you can get ideas from in coffeescript. I had an asana task to do tests this sprint but I haven't started on it yet, so I'll just assign it to you.

@KevinGrandon
Copy link
Contributor Author

@KevinGrandon
Copy link
Contributor Author

@bbondy - I haven't had time to get this working yet, and won't finish this sprint so I've gone ahead and re-assigned the asana task to you.

There's really not much to use as a reference here, so I'll just close this - but you can look at the libraries I was trying to pull in. Probably better to use the webdriverio library here: https://github.com/atom/electron/blob/master/docs/tutorial/using-selenium-and-webdriver.md

@KevinGrandon KevinGrandon deleted the tests branch December 3, 2015 01:35
@bbondy
Copy link
Member

bbondy commented Dec 3, 2015

No problem, thanks for the dump.

@bridiver bridiver mentioned this pull request Feb 11, 2016
@bbondy bbondy mentioned this pull request May 6, 2016
2 tasks
@bsclifton bsclifton mentioned this pull request Sep 14, 2016
30 tasks
@jkup jkup mentioned this pull request Dec 16, 2016
4 tasks
@luixxiul luixxiul removed the help wanted The PR/issue opener needs help to complete/report the task. label Jan 21, 2017
@rwehe rwehe mentioned this pull request Sep 26, 2017
47 tasks
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