mengw15 opened a new issue, #5456:
URL: https://github.com/apache/texera/issues/5456

   ### Task Summary
   
   `gmail.service.ts` 
([frontend/.../common/service/gmail/gmail.service.ts](https://github.com/apache/texera/blob/a820f6727/frontend/src/app/common/service/gmail/gmail.service.ts))
 is a 62-LOC service that sends `PUT /api/gmail/send` and `GET 
/api/gmail/sender/email` requests. It is consumed by share-access flows and 
admin dashboards. It currently has no `.spec.ts`.
   
   ### Suggested coverage
   
   - `sendEmail(subject, content, receiver)` issues the right HTTP request shape
   - `getSenderEmail()` returns the expected text payload
   - Success notification fires on 2xx response
   - **Error path fires on non-2xx response** — this currently triggers no UI 
feedback because the backend always returns 204, see #5161; the frontend test 
should still assert the behaviour the wired-up `error:` branch is meant to 
deliver, so it remains a regression net once #5161 is fixed.
   
   ### Acceptance
   
   - New `gmail.service.spec.ts` next to the service
   - Use Angular's `HttpClientTestingModule` to control responses
   - Cross-references #5161
   - All new tests green under default jsdom target


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to