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

New capability: a minimal nightwatch scaffold. Zero lint errors. #67

Merged
merged 1 commit into from
Jul 29, 2015

Conversation

martinhbramwell
Copy link
Contributor

Basically this is a stripped version of scaffolds/sample-tests/nightwatch.
tool/generate-autoconfig.js has a new function, called from two places, to replace the anonymous file save function, that was only called once.

NOTE : Changes to documentation still to do.

tool/generate-autoconfig.js is now free of lint errors, but has warnings about use of single quotes.

Will fix those when there are no other changes in the file that would be obscured.

Changes committed:
new file: scaffolds/sample-tests/nightwatch-minimal/globals.json
new file: scaffolds/sample-tests/nightwatch-minimal/logs/selenium-debug.log
new file: scaffolds/sample-tests/nightwatch-minimal/walkthroughs/appLayout.js
modified: tool/generate-autoconfig.js
modified: tool/scaffold.js

 Changes to be committed:
	new file:   scaffolds/sample-tests/nightwatch-minimal/globals.json
	new file:   scaffolds/sample-tests/nightwatch-minimal/logs/selenium-debug.log
	new file:   scaffolds/sample-tests/nightwatch-minimal/walkthroughs/appLayout.js
	modified:   tool/generate-autoconfig.js
	modified:   tool/scaffold.js
log.info('------------------------------------------');
log.info('Removing "custom path" elements .... ');
delete autoConfigObject.custom_commands_path;
delete autoConfigObject.custom_assertions_path;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice use of delete here.

@awatson1978
Copy link
Owner

Dude, this is great.

awatson1978 added a commit that referenced this pull request Jul 29, 2015
New capability: a minimal nightwatch scaffold.  Zero lint errors.
@awatson1978 awatson1978 merged commit 517a0d5 into awatson1978:master Jul 29, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants