| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Only a handful of critical errors are present in the codebase.
They all occur during server startup (in `app.StartServer()`).
Currently, when one of these critical error occurs, it is simpled
mentionned in the logs – then the error is discarded, and the app
attempts to continue the execution (and probably fails pretty quickly in
a weird way).
Rather than continuing operations in an unknow state, these errors should
trigger a clean exit.
This commit rewrites critical startup errors to be correctly
propagated, logged, and then terminate the command execution.
Additionnaly, it makes the server return a proper error code to the
shell.
|
|
|
|
|
|
|
|
| |
* role refactor
* add missing file
* fix web test
|
| |
|
| |
|
|
|
|
|
|
| |
* refactoring cfg refs and load / save functions
* improve error output
|
|
|
|
|
|
|
|
|
|
|
|
| |
* app cleanup
* whoops, forgot a file
* some minor cleanup
* longer container deadline
* defensive checks
|
|
|
|
|
|
| |
* goroutine wranglin
* synchronize WebConn.WritePump
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
* remove global app references
* test fix
* fix api4 test compilation
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
* PLT-6471 Properly panic when translations can't be loaded
* Print usage messages when errors occur during CLI initialization
* Reverted behaviour of FindDir and added second return value to it
* Fixed merge conflict
|
|
|
|
|
|
|
|
|
|
|
|
| |
* PLT-5860 Updated copyright date in about modal
* PLT-5860 Updated copyright notice in JSX files
* PLT-5860 Updated copyright notice in go files
* Fixed misc copyright dates
* Fixed component snapshots
|
| |
|
|
|
|
|
|
|
|
|
|
| |
This migrates the existing webapp tests to using Jest and Enzyme. The
infrastructure is put in place for React component testing, and a few
simple example component tests are implemented.
This also adds snapshot testing of components, coverage checking for the
webapp (although that is not yet integrated to Coveralls), and the
ability to run npm run test:watch to automatically re-run affected tests
when working on the webapp codebase.
|
|
|
|
| |
(#5168)
|
|
|
|
|
|
|
|
|
|
| |
* Add app package and move logic over from api package
* Change app package functions to return errors
* Move non-api tests into app package
* Fix merge
|
|
|