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

Backend infrastructure #65

Merged
merged 12 commits into from
Feb 16, 2022
Merged

Backend infrastructure #65

merged 12 commits into from
Feb 16, 2022

Conversation

jankapunkt
Copy link
Member

@jankapunkt jankapunkt commented Feb 11, 2022

implements #64 and partially #62

This pull requests adds a complete code-infrastructure to the backend-server.

This includes:

  • standardized setup of contexts and methods using factories
  • decoupled declaration from instantiation
  • decouple API-specific logic from general method calls
  • use environment variables to configure API internals
  • validate arguments using schema
  • lint using standardx
  • added initial test infra using mocha/chai + coverage

It does not include:

  • Implemented sync of content, which should be done in saparate PR (see Load data from content server #57)
  • Implement full tests, since we may have many changes in the next two weeks, so let's first get the whole infra ready and once this is all fixed we can add the tests
  • running jsdoc, which should be done in a separate pr (see Document backend code #62)

How to run:

  • the server has an own run.sh script which runs the server as expected
  • it also has a test.sh script which makes it easier to define all those test parameters

This was linked to issues Feb 11, 2022
@jankapunkt jankapunkt merged commit 30252bf into main Feb 16, 2022
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.

Backend infrastructure
2 participants