feat(ui): add tests tab indicator

[kobenguyent]

No tests

All passed

Failed tests

[netlify]

✅ Deploy Preview for chimerical-kitsune-a0bfa0 ready!

Name	Link
🔨 Latest commit	dd9712c
🔍 Latest deploy log	https://app.netlify.com/sites/chimerical-kitsune-a0bfa0/deploys/66fc12a615cbdb0008a1e62a
😎 Deploy Preview	https://deploy-preview-265--chimerical-kitsune-a0bfa0.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[flawiddsouza]

No indicator is shown if all tests fail

this is because of v-if triggering only when passedTestCases value is greater than 0.

Indicator should actually be shown if there are any test cases > 0, regardless of fail or pass.

This should be a computed property.

Also this if guard is needed:

Because there are some old responses where testResults property is missing. So when you open those requests, the response panel will error out without this condition.

Restore the spacing before and after flower brackets here:

[flawiddsouza]

isTestResultsAvailable() {
    return this.response && 'testResults' in this.response || false
}

|| false is not required - as the result of that expression will be true or false

Both of these methods can be moved to computed properties:

[kobenguyent]

isTestResultsAvailable() {
    return this.response && 'testResults' in this.response || false
}

|| false is not required - as the result of that expression will be true or false

Both of these methods can be moved to computed properties:

just curious if there is any case this.response would be null? That's why I added || false

[flawiddsouza]

just curious if there is any case this.response would be null? That's why I added || false

this.response can be null. I actually assumed the result of a statement with && would be a boolean but guess not:

Your code was correct.