jakemoves / cohort-server

A websocket & HTTP server for coordinating media playback and notifications across many clients (i.e. phones)

https://cohort.rocks

MIT License

1 stars 2 forks source link

refactor Cohort to separate from app #72

Open jakemoves opened 5 years ago

jakemoves commented 5 years ago

cohort-app.js should require (i.e.) cohort.js, which gets stored as app.set("cohort"). Its initializer returns a Promise and loads active events from the DB into itself. It should be accessible from HTTP endpoints and from WebSocket code. It should handle stripping fields for response objects to all destinations.

12, #66, #73, #69

jakemoves commented 5 years ago

refactored CHEvent and CHDevice to both be classes with static methods that take a result from a Knex DB query and return a new CH object
now we are tearing down server state between tests
fixed broken tests