* fix bug where savesettings did not honor local settings variables
* don't create lib/flows on read; It's already created in localfilesystem.init and in saveLibraryEntry -
so removed call to promiseDir, and added a return of [] if accessing a folder which did not exist.
This is important because else when settings.readOnly is true, it still creates folders.
* Fix a CI failure where path passed to getLibraryEntry is empty;
treat this case as meaning it was wanting a folder, and return empty if the folder dioes not exist
* Add a test for getLibraryEntry( type, '/' ) as called by node-red
* change newsettings to camelCase newSettings
* Flow lib now adheres to pretty flow file, ui also adheres to this as well
* added settings mock object in flows_spec get api test
* reverted api changes, fixed parse of flow only
* try spell flows correctly
Closes#819
If a flow file is found to be zero-bytes:
If there is a non-empty backup, restore the backup and resolve
If there is no backup or it is also empty, resolve empty flow
If a flow file is found to be invalid json:
Log and resolve empty flow
The tests still use it in places - particular localfilesystem tests,
but those tests need to be redone with sinon stubbing in place and
not rely on real fs operations.
Nick O'Leary
btsimonh
Josh