aglinxinyuan opened a new pull request, #7437:
URL: https://github.com/apache/texera/pull/7437

   ### What changes were proposed in this PR?
   
   The quota page's Result Cache tab is never rendered by its spec, which 
drives the quota data and its charts only.
   
   Adds 5 tests. The reported cache size is the centrepiece:
   
   ```html
   {{ formatSize(execution.resultBytes + execution.logBytes + 
execution.runTimeStatsBytes) }}
   ```
   
   Three separate byte counts added together. A dropped or double-counted term 
still produces a plausible-looking size and nothing else in the suite would 
notice, so the fixture uses distinct powers of two — any such slip lands on a 
different total rather than coincidentally matching.
   
   Also covered: one collapse panel per workflow headed by its name, the 
executions listed under the opened workflow, the delete button carrying the 
row's **execution** id rather than the workflow id, and the table paging rather 
than listing everything.
   
   **Verified by mutation**, all reverted (template diff empty):
   
   | Mutation | Result |
   |---|---|
   | cache size drops `logBytes` | red |
   | cache size drops runtime statistics | red |
   | cache size double-counts `resultBytes` | red |
   | delete passes the workflow id | red |
   | panel header shows the id | red |
   | execution-id column shows the name | red |
   | pagination widened | red |
   | rows read the raw list instead of the page | red |
   
   `ngOnInit` resets `workflows`, so the first change-detection cycle runs 
before the fixture data is assigned; that is commented in the spec.
   
   No production file is touched.
   
   ### Any related issues, documentation, discussions?
   
   Closes #7434
   
   ### How was this PR tested?
   
   ```
   npx ng test --watch=false --include="**/user-quota.component.spec.ts"
   ```
   
   ```
    Test Files  1 passed (1)
         Tests  16 passed (16)
   ```
   
   5 new on top of the existing 11. `yarn format:ci` passes.
   
   ### Was this PR authored or co-authored using generative AI tooling?
   
   Generated-by: Claude Code (Opus 5)
   


-- 
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