Ma77Ball opened a new pull request, #5238: URL: https://github.com/apache/texera/pull/5238
### What changes were proposed in this PR? - Adds `user-dataset.component.spec.ts` covering `UserDatasetComponent` (previously uncovered apart from its file-renderer child). - Verifies user-state tracking via `UserService.userChanged()`, `ngAfterViewInit` re-search behavior, `search()` filterScope variants (`private` / `public` / `all`), the full 8-argument `SearchService.executeSearch` call shape, the LakeFS mismatch warning (text + 4000ms duration), the `UserDatasetVersionCreatorComponent` modal config and post-close navigation, and `deleteDataset` (undefined-`did` no-op plus filtering of the deleted entry). - Uses direct component instantiation with `vi.fn()` mocks rather than the heavier TestBed pattern, so assertions can target exact positional argument shapes without bootstrapping the modal content component or the ng-zorro module graph. ### Any related issues, documentation, or discussions? Closes: #5229 ### How was this PR tested? - `yarn test --include='**/user-dataset/user-dataset.component.spec.ts' --watch=false` (14 tests passed, ~12ms). ### Was this PR authored or co-authored using generative AI tooling? Co-authored with Claude Opus 4.7 in compliance with ASF -- 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]
