[GitHub] [camel-k] squakez commented on issue #3813: TestRunAndLog refactoring

2022-11-28 Thread GitBox
squakez commented on issue #3813: URL: https://github.com/apache/camel-k/issues/3813#issuecomment-1328717083 So, the test you wrote is correct. It proves however that the implementation of `RunAndLog` func is wrong. If you run it, you will see the output: ``` OUT: Executed command: /us

[GitHub] [camel-k] squakez commented on issue #3813: TestRunAndLog refactoring

2022-11-24 Thread GitBox
squakez commented on issue #3813: URL: https://github.com/apache/camel-k/issues/3813#issuecomment-1326479128 I have seen that, and, I think we should rework entirely the test to use `go` command instead of `date` command. Ideally, it should produce the very same output on all platforms. -

[GitHub] [camel-k] squakez commented on issue #3813: TestRunAndLog refactoring

2022-11-24 Thread GitBox
squakez commented on issue #3813: URL: https://github.com/apache/camel-k/issues/3813#issuecomment-1326141521 I think I have a "universal command" that must be present on the machine we're using to test... it's `go`! We can try to test the following on any machine: ``` $ go help

[GitHub] [camel-k] squakez commented on issue #3813: TestRunAndLog refactoring

2022-11-15 Thread GitBox
squakez commented on issue #3813: URL: https://github.com/apache/camel-k/issues/3813#issuecomment-1315373349 Thanks for taking interest in this. Well, it could be a solution. As alternative we may find any command that return the same output error for the most of OS existing (if any exists,