Should we add a regression test testing the case of calling canister_status on the station for a canister with log visibility restricted to a set of principals?

I updated the PR to display the disabled "Allowed viewers (unsupported)" option for canisters (disregard screenshot copy) with this value and the former 2-option select box for other canisters.

I also added the regression test to cover the behavior with these two canister_status response variants.

Another PR will come later to support allowed viewers in the UI fully.

I also added the regression test to cover the behavior with these two canister_status response variants.

I don't see any new test: did you push the test?

@mraszyk https://github.com/dfinity/orbit/pull/538/files#diff-e825cffb84cec6b7f569564617db4e3bbcd0265eff65e9137e4308a3274e30d4R30-R49

Edit: But maybe you also meant a test for the external canister page backend itself, to see that the right panel is working properly. The lines above only cover the UI for modifications of LogVisibility, even though it's using the same source of data.

@mraszyk https://github.com/dfinity/orbit/pull/538/files#diff-e825cffb84cec6b7f569564617db4e3bbcd0265eff65e9137e4308a3274e30d4R30-R49

Edit: But maybe you also meant a test for the external canister page backend itself, to see that the right panel is working properly. The lines above only cover the UI for modifications of LogVisibility, even though it's using the same source of data.

What I meant is an integration test for the station's canister_status endpoint. This was the first thing to fail in the reported issue.