This is a side quest of #8558. The motivation is to be able to do dependency injection in the tests without overwriting global data. I saw the first merge conflict from #8551 and voila: It seems @Mogge could have used this already.
refactor: follow @Mogge's review
See: https://github.com/Ocelot-Social-Community/Ocelot-Social/pull/8603#pullrequestreview-2880714796
refactor: better test helper methods
wip: continue refactoring
wip: continue posts
continue
wip: continue groups
continue registration
registration
continue messages
continue observeposts
continue categories
continue posts in groups
continue invite codes
refactor: continue notificationsMiddleware
continue statistics spec
followed-users
online-status
mentions-in-groups
posts-in-groups
email spec
finish all tests
improve typescript
missed one test
remove one more reference of CONFIG
eliminate one more global import of CONFIG
fix language spec test
fix two more test suites
refactor: completely mock out 3rd part API request
refactor test
fixed user_management spec
fixed more locatoin specs
install types for jsonwebtoken
one more fetchmock
fixed one more suite
fix one more spec
yet another spec
fix spec
delete whitespaces
remove beforeAll that the same as the default
fix merge conflict
fix e2e test
refactor: use single callback function for `context` setup
refactor: display logs from backend during CI
Because why not?
fix seeds
fix login
refactor: one unnecessary naming
refactor: better editor support
refactor: fail early
Interestingly, I've had to destructure `context.user` in order to make
typescript happy. Weird.
refactor: undo changes to workflows - no effect
We're running in `--detached` mode on CI, so I guess we won't be able to
see the logs anyways.
refactor: remove fetch from context after review
See:
refactor: found an easier way for required props
Co-authored-by: Max <maxharz@gmail.com>
Co-authored-by: Ulf Gebhardt <ulf.gebhardt@webcraft-media.de>
* type for neo4j and neode
* fix build
* remove flakyness
* wait for neode to install schema
* remove flakyness
* explain why we wait for a non-promise
* lint @typescript-eslint/recommended
* lint @typescript-eslint/recommended-requiring-type-checking
fix type not detected locally due to wierd uuid typings
missing save
error not reported locally
---------
Co-authored-by: Wolfgang Huß <wolle.huss@pjannto.com>
* refactor(graphql): Introduce image type
* Undo changes to .travis.yml
* chore: Upgrade travis to node LTS
- URL is available since v10
* chore: use lts
Co-authored-by: mattwr18 <mattwr18@gmail.com>
- start refactoring
- locations does not have any automated tests, which makes it more
difficult to refactor and have confidence that functionality will not be
broken
- notificationsMiddleware in progress
We had this error in our neo4j pod recently:
```
2019-12-02 08:29:42.680+0000 ERROR Unable to schedule bolt session 'bolt-1018230' for execution since there are no available threads to serve it at the moment. You can retry at a later time or consider increasing max thread pool size for bolt connector(s).
2019-12-02 08:29:42.680+0000 ERROR Unable to schedule bolt session 'bolt-1018224' for execution since there are no available threads to serve it at the moment. You can retry at a later time or consider increasing max thread pool size for bolt connector(s).
2019-12-02 08:29:42.681+0000 ERROR Unable to schedule bolt session 'bolt-1018352' for execution since there are no available threads to serve it at the moment. You can retry at a later time or consider increasing max thread pool size for bolt connector(s).
2019-12-02 08:29:42.682+0000 ERROR Unable to schedule bolt session 'bolt-1018243' for execution since there are no available threads to serve it at the moment. You can retry at a later time or consider increasing max thread pool size for bolt connector(s).
```
Apparently the default is 400 threads. So we must have a leak somewhere.
I have seen this PR here: https://github.com/Human-Connection/Human-Connection/pull/774 and I thought: Never saw that one before, where is it used and what is the purpose? I could find this line only and thought this is not a justification for having the package installed.
