Skip to content

Saved Jobs: navigation, placeholder screen, and unit tests #35

Merged
AchsahJojo merged 1 commit intomainfrom
JobListingPage-Setup
Sep 11, 2025
Merged

Saved Jobs: navigation, placeholder screen, and unit tests #35
AchsahJojo merged 1 commit intomainfrom
JobListingPage-Setup

Conversation

@kassandrabeas5
Copy link
Copy Markdown
Collaborator

@kassandrabeas5 kassandrabeas5 commented Sep 10, 2025

Issue addressed: Implements Job View Full Detail #24 and Save Job Feature #23 (sub-issues of Job Listing page #13).

Tests written & passed:

  • jobs.test.tsx → verifies “Saved Jobs” button navigates to /savedJobs.
  • jobs.save.test.tsx → verifies pressing “Save Job” calls the add function.
  • savedJobs.test.tsx (smoke) → verifies Saved Jobs screen renders the title/placeholders.
    All tests pass (npm test → screenshot included for coverage requirement).

Changes made:
jobs.tsx

  • Added navigation: “Saved Jobs” button routes to /savedJobs.
  • Hooked up Save Job button to add jobs into a simple store/context.

savedJobs.tsx
-Implemented dark-themed placeholder UI.

  • Displays saved jobs (placeholder for now)

tests

  • Added unit tests for navigation, saving, and placeholders.

_layout.tsx
-Wrapped app in SavedJobsProvider to share state across screens.

Copy link
Copy Markdown
Collaborator

@AchsahJojo AchsahJojo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to go, the style is very clear and modern. But start adding in API implementation asap

@AchsahJojo AchsahJojo merged commit 163eb5d into main Sep 11, 2025
@kassandrabeas5 kassandrabeas5 self-assigned this Sep 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants