Remove study-specific cache key logic because it is taking a horrible toll on caching behavior

* Delete long cache key test for CustomKeyGeneratorTest

* Get rid of unnecessary stubbing

---------

Co-authored-by: alisman <[email protected]>

…nt in test (#11693)

* Update Architecture-Overview.md with corrected links

* Fix validator link to current path

* Update API reference link

* Add missing frontend link to match other sections

* Update API reference link in Architecture-Overview.md

* Update API reference link

…om IDs (#11611)

* Create/drop published virtual studies with custom IDs

To support RFC96 Virtual Study Authorisation Model we need static human
readable ids to assign permissions to them

Fix REST template delete call

The delete method on RestTemplate does not accept an HttpEntity parameter for headers.
We should use exchange(url, HttpMethod.DELETE, new HttpEntity<>(getHttpHeaders()), Void.class) to correctly include headers in the DELETE request.

Improve readability of publishVirtualStudy method

The parameter virtualStudyData is reassigned inside the method, shadowing the method parameter.
We started using a separate local variable to avoid confusion and improve readability.

Change createVirtualStudy method to return nothing

SonarQube suggested this change

Make study id duplication errors clearer to user

Add e2e for testing vs publishing/unpublishing

Select better names for virtual study methods

* Clean drop public VS method code

- make it more cleare that we remove VS with the same id we pass to the
  method
- improve method name. Make it more explicit that removal happens by id

* Test creating public VS with custom id and droping it

* Specify json content type for VS custom id e2e test

Co-authored-by: Bryan Lai <[email protected]>

…of profiling status for genes which are covered by the same panels