apache / apache/texera

Add behavior-focused spec files for untested dashboard/hub components

Open
#5,167 2 comments 0 reactions 1 assignee Claimed by @Ma77Ball View on GitHub
Dominant language
Scala
Stars
314
Forks
187
Avg merge
1d 21h
Merged PRs (30d)
214

Description

### Task Summary

Several dashboard and hub components ship without unit tests. Add a
behavior-focused `.spec.ts` file alongside each component listed below.
Target observable behavior — event handlers, service calls, navigation,
form submission — and avoid asserting on implementation details like URL
string literals.

Components needing specs:
- dashboard/component/user/share-access/share-access.component.ts
(form validation, email sending, access-grant flow)
- hub/component/hub.component.ts
(navigation and route subscription logic)
- hub/component/landing-page/landing-page.component.ts
- hub/component/about/local-login/local-login.component.ts
- hub/component/workflow/detail/hub-workflow-detail.component.ts
- dashboard/component/user/search-bar/search-bar.component.ts
(search behavior, debouncing)
- dashboard/component/user/user-dataset/user-dataset.component.ts
(list rendering, deletion, pagination)
- dashboard/component/user/user-dataset/user-dataset-list-item/
user-dataset-list-item.component.ts

Conventions: place each spec next to its component, mock injected services
with Jasmine spies, use fakeAsync/tick for async behavior, and assert on
spy calls / emitted outputs rather than URL strings.

### Task Type

- [ ] Refactor / Cleanup
- [ ] DevOps / Deployment / CI
- [x] Testing / QA
- [ ] Documentation
- [ ] Performance
- [ ] Other

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.