personal-projects/personal-portfolio
Template
1
1
Fork 0
Code Issues 3 Pull Requests Actions Packages Projects 1 Releases Wiki Activity

Audit and remove TRREB-related tests #50

New Issue
Closed
opened 2026-01-16 14:54:11 +00:00 by lmiranda · 0 comments
lmiranda commented 2026-01-16 14:54:11 +00:00
Owner
Copy Link

Description

Find and remove all test files and test cases that reference TRREB parsers, schemas, loaders, and models.

Branch: feature/9-cleanup

Reference: docs/changes/Change-Toronto-Analysis-Reviewed.md - Phase 1

Scope

Search the tests/ directory for any tests that:

  • Import from trreb.py modules
  • Test TRREB PDF parsing functionality
  • Test FactPurchases or DimTRREBDistrict models
  • Use TRREB-specific fixtures or test data

Acceptance Criteria

  • All TRREB-specific test files identified and deleted
  • All TRREB test cases within shared test files removed
  • TRREB-related fixtures in conftest.py removed
  • make test passes with no import errors
  • pytest --collect-only shows no TRREB-related test names
  • Test coverage report excludes deleted TRREB modules

Investigation Steps

  1. grep -r "trreb" tests/ to find all references
  2. grep -r "FactPurchases\|DimTRREBDistrict" tests/ for model tests
  3. Review tests/conftest.py for TRREB fixtures
  4. Check for test data files in tests/fixtures/ or tests/data/

Technical Notes

  • Preserve all CMHC-related tests
  • Preserve all general utility tests
  • Preserve test infrastructure (conftest.py base setup)
  • Document what was removed in commit message

Sprint

Sprint 9 - Phase 1 (Cleanup)

Labels: Type/Test, Priority/Medium, Efforts/S, Component/Testing, Tech/Python

## Description Find and remove all test files and test cases that reference TRREB parsers, schemas, loaders, and models. **Branch:** `feature/9-cleanup` **Reference:** `docs/changes/Change-Toronto-Analysis-Reviewed.md` - Phase 1 ## Scope Search the `tests/` directory for any tests that: - Import from `trreb.py` modules - Test TRREB PDF parsing functionality - Test `FactPurchases` or `DimTRREBDistrict` models - Use TRREB-specific fixtures or test data ## Acceptance Criteria - [ ] All TRREB-specific test files identified and deleted - [ ] All TRREB test cases within shared test files removed - [ ] TRREB-related fixtures in `conftest.py` removed - [ ] `make test` passes with no import errors - [ ] `pytest --collect-only` shows no TRREB-related test names - [ ] Test coverage report excludes deleted TRREB modules ## Investigation Steps 1. `grep -r "trreb" tests/` to find all references 2. `grep -r "FactPurchases\|DimTRREBDistrict" tests/` for model tests 3. Review `tests/conftest.py` for TRREB fixtures 4. Check for test data files in `tests/fixtures/` or `tests/data/` ## Technical Notes - Preserve all CMHC-related tests - Preserve all general utility tests - Preserve test infrastructure (conftest.py base setup) - Document what was removed in commit message ## Sprint Sprint 9 - Phase 1 (Cleanup) **Labels:** Type/Test, Priority/Medium, Efforts/S, Component/Testing, Tech/Python
lmiranda added this to the Launch: Host, Bio and Toronto House Market Analysis project 2026-01-16 14:55:49 +00:00
lmiranda self-assigned this 2026-01-16 14:55:54 +00:00
lmiranda moved this to To Do in Launch: Host, Bio and Toronto House Market Analysis on 2026-01-16 14:56:03 +00:00
lmiranda moved this to Done in Launch: Host, Bio and Toronto House Market Analysis on 2026-01-16 15:33:53 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
Complexity/Complex
High uncertainty or many dependencies
 Complexity/Medium
Some unknowns or dependencies
 Complexity/Simple
Straightforward, well-understood
 Effort/L
3-5 days
 Effort/M
1-2 days
 Effort/S
2-4 hours
 Effort/XL
> 1 week
 Effort/XS
< 2 hours
 Priority/Critical
Immediate attention required
 Priority/High
Important, needs attention soon
 Priority/Low
Nice to have, can wait
 Priority/Medium
Normal priority
 Type/Bug
Something isn't working
 Type/Chore
Maintenance tasks
 Type/Documentation
Documentation improvements
 Type/Feature
New feature or request
 Type/Refactor
Code refactoring without changing behavior
 Type/Test
Testing related
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project Launch: Host, Bio and Toronto House Market Analysis
Assignees
Clear assignees
lmiranda
No Assignees lmiranda
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: personal-projects/personal-portfolio#50
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?