- Issue created by @xrobp
- Merge request !21Issue #3471105: Use correct group type id prefix in GroupContentRespectUnpublished views filter plugin query. → (Open) created by xrobp
- Status changed to Needs review
7 months ago 11:28pm 29 August 2024 - Status changed to RTBC
4 months ago 2:19pm 5 December 2024 - First commit to issue fork.
- 🇧🇪Belgium msnassar
For now we should ignore the failing tests in D11 builds. This is going to be fixed in another issue.
The tests using D10 are passing, this demonstrate that our tests are not good. So in order to demonstrate this issue, lets write a test that fails .
BTW, from group 2, I think we might not need this views plugin anymore! But lets demonstrate this later or in another ticket. - 🇧🇪Belgium msnassar
Now we have a failing tests... Lets have them passing by fixing the reported issue.
- 🇧🇪Belgium msnassar
I have cherry picked the fix from MR21. Now the tests are green again...
I think the exiting code in this test can be improved... Anyway, the next step is to demonstrate that we might not need this views plugin anymore!