Cover `frostfsid.GetSubjectExtended()` with tests #130

New issue

Open

opened 2024-11-20 10:29:37 +00:00 by fyrchik · 0 comments

fyrchik commented

2024-11-20 10:29:37 +00:00

Owner

We would like it to mirror GetSubject.
I think it should be a single test and checkSubject does both GetSubject and GetSubjectExtended.
I also believe this line is unreachable and should either be removed or have a test covering it.

We would like it to mirror `GetSubject`. I think it should be a single test and `checkSubject` does both `GetSubject` and `GetSubjectExtended`. I also believe [this line](https://git.frostfs.info/TrueCloudLab/frostfs-contract/src/commit/a2c2791146d65887c14fb38a531c01a318fe84cc/frostfsid/frostfsid_contract.go#L445) is unreachable and should either be removed or have a test covering it.