More tests - [merged]

[slrtbtfs]

_Merges morecoverage -> main

[slrtbtfs]

great, i managed to write tests that work on my machine but not in the CI environment. @bjthehun does it work on your machine?

[slrtbtfs]

In GitLab by @bjthehun on Feb 11, 2023, 16:37

@slrtbtfs All tests work for me.

[slrtbtfs]

In GitLab by @bjthehun on Feb 11, 2023, 16:48

I've manually updated our dependencies on my laptop, and nothing has changed. I thought something went wrong with serde/serde_json, but apparently not.

[slrtbtfs]

In GitLab by @bjthehun on Feb 11, 2023, 16:54

added 1 commit

• c41122fc - Locating test error

Compare with previous version

[slrtbtfs]

In GitLab by @bjthehun on Feb 11, 2023, 17:30

Also cleared runner caches. Error still occurs.

[slrtbtfs]

In GitLab by @bjthehun on Feb 11, 2023, 17:45

Reading the Rust library docs tells me that if we set a file to read-only with Permissions, we may still write to it under certain circumstances:

https://doc.rust-lang.org/std/fs/struct.Permissions.html#unix-including-macos

[slrtbtfs]

In GitLab by @bjthehun on Feb 11, 2023, 18:09

I'm assuming that within our Docker runner, we have superuser rights, so we may write to a file even if it is set as write-only. (This would also why we can just install packages without sudo).

On my machine, I can write to a read-only file as root like this:

[slrtbtfs]

In GitLab by @bjthehun on Feb 12, 2023, 16:47

added 1 commit

• fb81a551 - Removing failing test

Compare with previous version

[slrtbtfs]

ah well, i guess it's easiest to just leave out the test then

thanks for the research!

[slrtbtfs]

mentioned in commit eb9420f482a125d4710414093612c00c8bcb3a53