Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merging to release-5.1: [TT-9586] Prevent panic in setHasMock func (#5361) #5363

Conversation

buger
Copy link
Member

@buger buger commented Jul 26, 2023

TT-9586 Prevent panic in setHasMock func (#5361)

This PR fixes a panic case inside APISpec.setHasMock func.

This PR fixes a panic case inside `APISpec.setHasMock` func.

(cherry picked from commit 8a2b4c1)
@TykTechnologies TykTechnologies deleted a comment from github-actions bot Jul 26, 2023
@furkansenharputlu furkansenharputlu marked this pull request as ready for review July 26, 2023 12:10
@furkansenharputlu furkansenharputlu enabled auto-merge (squash) July 26, 2023 12:10
@furkansenharputlu furkansenharputlu self-assigned this Jul 26, 2023
@Tyk-ITS
Copy link
Contributor

Tyk-ITS commented Jul 26, 2023

API tests result: success
Branch used: refs/pull/5363/merge
Commit:
Triggered by: pull_request (@buger)
Execution page

@sonarcloud
Copy link

sonarcloud bot commented Jul 26, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@furkansenharputlu furkansenharputlu merged commit e4301e3 into release-5.1 Jul 26, 2023
11 checks passed
@furkansenharputlu furkansenharputlu deleted the merge/release-5.1/8a2b4c1488356df9649f4d4a7e008032e6f1e154 branch July 26, 2023 12:27
@Tyk-ITS
Copy link
Contributor

Tyk-ITS commented Jul 26, 2023

API tests result: success
Branch used: refs/pull/5363/merge
Commit: 43c4773
Triggered by: pull_request (@furkansenharputlu)
Execution page

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants