[tests-only][full-ci] removing the setresponse in given/then step in TrashbinContext and TagsContext

[KarunAtreya]

Description

We have used setResponse() and $this->response in the Given/Then steps and some helper functions (maybe to reuse existing available methods). But storing responses from Given/Then steps and helper functions is not a good idea because it can lead to a false positive assertion in the Then steps.
So, check the use of setResponse() and $this->response in

• Given steps
• Then steps (Then steps can use $this->response but must prevent saving to it)
• Helper functions

So this pr make the above changes in TrashbinContext.php and TagsContext.php

Related Issue

#7082

Motivation and Context

• To remove setResponse() and $this->response in the Given/Then steps and some helper functions
• To avoid false positive assertions

How Has This Been Tested?

• test environment:
• locally
• CI

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Technical debt
• Tests only (no source changes)

Checklist:

• Code changes
• Unit tests added
• Acceptance tests added
• Documentation ticket raised:

[SwikritiT]

LGTM

[saw-jan]

👍 ✨

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information