Christoph Völker
Christoph Völker
Thanks for the detailed feedback! That makes sense. I'll try to fix my local compatibility issues with Node.js and my Ubuntu (24) sytem first ...
Hi! I tried to follow your suggestion and base the test on `CopierMessages`, but noticed that it also imports `@keymanapp/developer-test-helpers`, which seems to be internal and not part of the...
> Hi @cvosoft, just following up on this stalled PR. Do you think you'll be able to do any more on it? Note: I am now living in the same...
Updated the PR: - Rebased on latest master - Added a working unit test for Warn_PreviousMapFileCouldNotBeLoaded, verified in Docker - Increased coverage threshold to 75% - Tested successfully with resources/docker-images/run.sh...
> Thanks, this looks good. One small change requested: > > Rather than poking into the internal method by casting to `any`, can you use the `unitTestEndPoints` pattern as shown...
> Looks good to me! (small suggestion for checking return value of `loadPreviousMap` but no biggie) > > I think we should duplicate this pattern for other warnings generated in...