-
Notifications
You must be signed in to change notification settings - Fork 0
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
initial demo of new runner-adapter logic with fs adapter #74
Conversation
Changed the interface of the adapter a bit to keep the eject task name generation out of the actual adapter implementations. Adapters got a new getDomains( callback( error, db_name_list ) ) -call, which should resolve names of the of the separate databases this current adapter is connected to. The database name of each task is also passed to the runner as part of the listenTask callback (as the parameter "domain") so that the eject_function name can be properly added to the task before it is passed on to the controller. This way runner does not need to parse the name from the id (which is good as id implementation can be highly adapter dependent). None of the adapter specific tasks work yet but unit and e2e test should be updated to match the new spec. I'll go and update the original pull request description if I can to match the small changes made. |
The current version does some initial work for #68 and implements #65. DB initialization was also moved inside the constructor which should ease development on #66 and #25. Make test runs now with a limited test coverage BUT if you run the unit tests before the integration tests (and not the other way around like we do now) inside the same mocha runtime, stuff starts to break... So there is still a lot to work with in the current tests. |
Changes include: * whitespace changes per Googles style guide * breaking some long lines * explicit boolean conversions * jsdoc fixes * syntax fix, which prevented running the program
Directory for fs-adapter and dependencies for precise installation
From documentation: "fs.exists() is an anachronism and exists only for historical reasons. There should almost never be a reason to use it in your own code."
This pull request has gone stale. The sqlite branch has been built on top of this branch, and there should be a pull request to merge that instead. |
THIS IS NOT READY TO MERGE.
It is meant to showcase the proposal for new runner-adapter logic and the new ejector logic - and acts as a place to discuss and make them better. The code is not tested through, but should give a good idea of the execution path.
The idea with the new logic would be the following:
A succesfull execution path could look like this: