Incorporate server and cli into new code #13

Merged
Jaculabilis merged 16 commits from tvb/server into develop 2021-06-16 03:53:07 +00:00
Owner

This refactor brings the server and the command line interface into the new code. The server is runnable via amanuensis-server and the cli is runnable via amanuensis-cli.

A significant amount of old code has been removed as part of this change:

  • All of the old CLI code, which is either obsolete or would require a total rewrite, is gone. Much of the removed code was for the bespoke command definition and discovery process, which has been replaced with a more streamlined subparser scheme.
  • The old "config directory context" code has been removed completely, since the database supersedes it. This also removes the only explicit Windows incompatibilty in the project.
  • The old log submodule has been removed in favor of some hard-coded configs in the cli submodule. This may see another rewrite when server logging becomes an issue.
  • Some old server code has been commented out in anticipation of some yet-unimplemented subsystems, such as the auth system.

A toy unit test has been added to demonstrate that server unit tests work. The sqlite memory database doesn't work with multiple connections, though, so the server tests will need to create tempfile databases in the future.

This refactor brings the server and the command line interface into the new code. The server is runnable via `amanuensis-server` and the cli is runnable via `amanuensis-cli`. A significant amount of old code has been removed as part of this change: * All of the old CLI code, which is either obsolete or would require a total rewrite, is gone. Much of the removed code was for the bespoke command definition and discovery process, which has been replaced with a more streamlined subparser scheme. * The old "config directory context" code has been removed completely, since the database supersedes it. This also removes the only explicit Windows incompatibilty in the project. * The old log submodule has been removed in favor of some hard-coded configs in the cli submodule. This may see another rewrite when server logging becomes an issue. * Some old server code has been commented out in anticipation of some yet-unimplemented subsystems, such as the auth system. A toy unit test has been added to demonstrate that server unit tests work. The sqlite memory database doesn't work with multiple connections, though, so the server tests will need to create tempfile databases in the future.
Jaculabilis added 16 commits 2021-06-16 03:48:35 +00:00
Jaculabilis merged commit aaa7982f67 into develop 2021-06-16 03:53:07 +00:00
Jaculabilis deleted branch tvb/server 2021-06-16 03:53:13 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Jaculabilis/amanuensis#13
No description provided.