Move a few more tests out of electron-browser
#178305
Conversation
There was a problem hiding this comment.
@andreamah @roblourens fyi this test was skipped so I decided to remove it. if you feel strong about it, then please look into restoring it on a layer that is not electron-browser which will eventually disappear. See #171508 for context.
There was a problem hiding this comment.
This was skipped because it was a "run on demand" test. I haven't looked at it in a long time, up to you @andreamah whether it's worth doing the work to keep. We haven't done any work in a long time that impacts backend search perf, you might just consider bringing it back if you do that kind of work in the future.
There was a problem hiding this comment.
If you want to bring it back then maybe consider to turn it into a node.js only test without electron dependencies if possible 👍
No description provided.