- Issue created by @bbrala
- Assigned to Indranil Roy
- Merge request !8196Issue #3449890: Remove extra line in MediaTest since referenced issue has been resolved. → (Open) created by Indranil Roy
- Issue was unassigned.
- Status changed to Needs review
8 months ago 7:55am 27 May 2024 - Status changed to Needs work
8 months ago 2:35pm 27 May 2024 - 🇺🇸United States smustgrave
Seems to be another reference to https://www.drupal.org/node/2824851 →
- 🇳🇱Netherlands bbrala Netherlands
Ow, guess i missed one, awesome.
Also, please remove the relevant code, not just the comment. Those comments are about code paths that need removing.
- First commit to issue fork.
Addressed the request changes, however it seems still "access content" permission require otherwise changes leads to test failure
Any suggestion for the further precedence?
- First commit to issue fork.
- 🇳🇱Netherlands bbrala Netherlands
Removing novice tag since it is not as simple as initially hoped.
@bbrala can you please confirm , as per description mentioned need to remove the "access content" content permission, however still test fails, that means need to work in the code level to fix this issue , is there any followup that can be check what changes occurs that code level that not lead to pass the pass yet ?
- 🇳🇱Netherlands bbrala Netherlands
I would assume no tests should need to change. But it will require diving into the issue references to see what we are looking for. Can't currently give a real complete anwser.
Concluded that once reference issue fixed(i.e part of another issue), then we can resume on it for "Remove extra line in MediaTest" and can be sure after fixing issue whether it is safe to "Remove extra line in MediaTest" for now