[GitHub] [camel-k] squakez commented on pull request #2561: fix TestPermissionDenied should use assert.Nil

[GitBox <gi...@apache.org>]

squakez commented on pull request #2561:
URL: https://github.com/apache/camel-k/pull/2561#issuecomment-896015636


   > Originally i opened that PR. But in the end they were arguing on thing i didn't see on our repo..
   
   Yeah, I think the problem is caused by how the CI is building the application. The best approach is indeed to work on improving the test in order to make it OS agnostic by creating a file and setting some privilege to deny access. I created it some time ago and I did not consider the impact on other systems that can build the application.
   
   I am opening a follow up issue to work on such improvement, in the while, we may even remove/comment the test in order to fix the problem described there.


-- 
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: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org