Fix: FileSystemEntry.Attributes property is not correct on Unix#52235

Merged

carlossanlop merged 17 commits intodotnet:mainfrom carlossanlop:FileAttributes

May 8, 2021

+355-182

Commits on May 4, 2021

Hidden and ReadOnly unit tests.
carlossanlop
committed
Unix: Correctly collect ReadOnly and Hidden flags from files and directories.
carlossanlop
committed
Remove the _initialAttributes field from FileStatus. Directly call _status.GetAttributes from FileSYstemEntry.Attributes since the disk hit is necessary anyway.

carlossanlop
authored and
carlossanlop
committed
Address simplication and nullcheck suggestions

carlossanlop
authored and
carlossanlop
committed

Commits on May 5, 2021

Defering some more file checking actions to FileStatus. Addressing accessibility of methods/properties. Addressing suggestions.

carlossanlop
authored and
carlossanlop
committed

Commits on May 7, 2021

Address feedback: Remove unnecessary asserts but document why. Remove struct initialization. Simplify the order in which Initialize checks for unknown and symlink and add a comment.

carlossanlop
authored and
carlossanlop
committed
Cache egid and euid, and refresh should reset them

carlossanlop
authored and
carlossanlop
committed
Check IsNameHidden before ensuring cache initialized and checking HasHiddenFlag.

carlossanlop
authored and
carlossanlop
committed
The ifs inside FileSystemEntry.Initialize has two mutually exclusive conditions: symlink and unknown, so use else if.

carlossanlop
authored and
carlossanlop
committed
Use entry.IsSymbolicLink directly inside Initialize.

carlossanlop
authored and
carlossanlop
committed
Remove extra symlink check in FileSystemEnumerator.MoveNext, not needed and could cause unnecessary disk hit.

carlossanlop
authored and
carlossanlop
committed
Remove unused cache checks.

carlossanlop
authored and
carlossanlop
committed
Use HasHiddenFlag as expression body

carlossanlop
authored and
carlossanlop
committed
No need to soft retrieve hidden and read-only in FileSystemEntry.Initialize if in AttributesToSkip we are going to retrieve it anyway with a disk hit. Only in the case of DT_UNKNOWN, we will pre-re…

carlossanlop
authored and
carlossanlop
committed
Make FileStatus.Initialize inlined in the only place where its called.

carlossanlop
authored and
carlossanlop
committed

Commits on May 8, 2021

EnsureCacheInitialized bring back order of conditions

carlossanlop
authored and
carlossanlop
committed
Syntax sugar for nullable egid and euid lazy retrieval

carlossanlop
authored and
carlossanlop
committed