Heorhii

Results 12 comments of Heorhii

@tshemsedinov Perhaps partially. Maybe it will be better if @leonpolak looks at whether everything is present, what he expected. I did not look at PR #204, but just corrected what...

> Looks like redis can't be accessed in docker: `unhandledRejection: Error: connect ECONNREFUSED 127.0.0.1:6379` @MarhiievHE You are right, I have forgotten to add optional support running API under a container...

@tshemsedinov Do you have any other suggestions to improve or add some more tests?

> > @tshemsedinov Do you have any other suggestions to improve or add some more tests? > > Tests are still failing https://github.com/metarhia/Example/pull/225/checks It's because the test.yml haven't any instance...

> Sorry for such a massive delay in the review. Everything works and all required tests pass (Huge respect to Heorhii and Timur). Just 3 questions: > > 1. Where...

@leonpolak @tshemsedinov I rewrote the test to use a node test runner and fixed a subscription check. And I also corrected the GitHub Actions script. Now, all tests are completed...

@leonpolak Did you talk about this error before? I caught it too now with node version 18. ``` 19:59:09 W2 error warning: MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 Symbol()...

Alternatively, you can do this without the .sh file. To do this, you can simply mount the .sql files in the desired order by naming them inside the container 1.sql,...

> Some tests failed These tests don't check the code I wrote.

I slightly changed the error catching code, and here is the result: ```bash 12:54:05 W6 log destroy chanel { idClient: 193n, accountId: 4577, closeCount: 2 } 12:54:29 W6 log Emmiting...