Skip to content

Feature/date filtering and tests#1

Open
jxtorres wants to merge 4 commits into
JamesStuder:mainfrom
jxtorres:feature/date-filtering-and-tests
Open

Feature/date filtering and tests#1
jxtorres wants to merge 4 commits into
JamesStuder:mainfrom
jxtorres:feature/date-filtering-and-tests

Conversation

@jxtorres

@jxtorres jxtorres commented Apr 3, 2025

Copy link
Copy Markdown

PR Summary: Add Date Filtering and Unit Tests

Overview

This PR adds date filtering functionality to the Plaud Bulk Downloader and includes comprehensive unit tests for the new feature. It also updates the project to use .NET 9.0 and improves documentation.

Changes

  1. Date Filtering Feature

    • Added StartDate parameter to UserInput model
    • Created FilterRecordingsByDate helper method
    • Updated ProcessFiles to filter recordings by date
    • Added date information to console output
    • Made Program class public for testing
  2. Unit Tests

    • Created test project with xUnit
    • Added comprehensive test cases for FilterRecordingsByDate method
    • Tests cover null inputs, empty inputs, invalid dates, and valid filtering
    • Added test data helper method
  3. Project Updates

    • Updated global.json to use .NET 9.0
    • Updated project file to target .NET 9.0
    • Added test project to solution
  4. Documentation

    • Updated README with .NET SDK requirements and installation instructions
    • Added development section with test running instructions
    • Updated examples to include date filtering

Testing

  • All 7 unit tests pass successfully
  • Manual testing confirms date filtering works as expected

Branch Information

  • Branch name: feature/date-filtering-and-tests

Commits

  1. "Add date filtering functionality to bulk downloader"
  2. "Add unit tests for date filtering functionality"
  3. "Update project files for .NET 9.0 and test project"
  4. "Update README with test instructions and requirements"

This PR enhances the Plaud Bulk Downloader with the ability to filter recordings by date, making it more flexible for users who only need recent recordings. The addition of unit tests ensures the reliability of this feature, and the updated documentation makes it easier for new users to get started with the tool.

jxtorres added 4 commits April 2, 2025 23:09
- Add StartDate parameter to UserInput model

- Create FilterRecordingsByDate helper method

- Update ProcessFiles to filter recordings by date

- Add date information to console output

- Make Program class public for testing
- Create test project with xUnit

- Add comprehensive test cases for FilterRecordingsByDate method

- Test null inputs, empty inputs, invalid dates, and valid filtering

- Add test data helper method
- Update global.json to use .NET 9.0

- Update project file to target .NET 9.0

- Add test project to solution
- Add .NET SDK requirements and installation instructions

- Add development section with test running instructions

- Update examples to include date filtering
@jxtorres

jxtorres commented Apr 3, 2025

Copy link
Copy Markdown
Author

Hi there! I wanted to download only the recent transcripts, so I've added an optional date filter. Let me know if you have issues running the tests in the readme.

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.

1 participant