stevedlawrence commented on issue #301: URL: https://github.com/apache/daffodil-vscode/issues/301#issuecomment-1337991563
An alternative is to create a small library of test utility functions that contain the os specific logic. For example, in the Daffodil CLI integration tests we have a `devNull` function that returns `/dev/null` on linux and `NUL` on windows. Any time a test needs to get the path to the null device, we can call this function and the test itself doesn't have to contain the os.platform logic. You could create similar functions for getting the daffodil path with .bat vs no .bat, etc. It looks like you already have a `killProcess` that does this. Adding more functions for os specific logic should help to avoid the issues we had with the Daffodil CLI tests. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
