Martin Adámek
Martin Adámek
it does not store just screenshots, but also html snapshots, right? edit: in fact it does not store any screenshots when used with non-browser crawlers
I tried to introduce new option for this, but it's far away from trivial, so in the end, I surrendered and only improved the type of the `launchOptions`.
Closing as we have the proxy tiers support, feel free to reopen if you think there is still something missing.
> @eric-deeporigin I'd suggest you to report this to MikroORM repo as well. This has been reported on our end many times, no need for yet another report I can't...
so what about this one, merge or not? i would keep what we have, but no strong opinions really
I will close this since it's no longer surfacing in the current version and I haven't been able to confirm my hunch from above either (also the PR in question...
this apparently breaks the tests pretty badly
i'd guess its gonna be this change https://github.com/mikro-orm/nestjs-realworld-example-app/pull/102/files?w=1#diff-8d9225643f0fb2cba6a02345b85424c6ce325e67c6512326c659b83e85ff57e5R46
> I ran the tests locally on this branch What tests? The failure is from e2e test suite, not from the unit tests.
Did you try reverting that change I noted? It feels like the only thing you changed that can have runtime effect.