Skip to content

Conversation

@SeanHersh
Copy link

Summary

Reorganizes test suite into logical groups by functionality (preprocessing vs. MFR processing) and updates author information.

Changes

New test files:

  • test_setup.py - Common setup code for all tests
  • test_preprocessing.py - Tests for prime(), updates(), expand(), involution() functions
  • test_mfr_processing.py - Tests for get_mfrs() and MFR computation

Updated files:

  • test_update_expand.py - Removed "bug" references from test names, improved documentation
  • setup.py - Added Sean Hershkowitz to author field
  • README.md - Added Sean Hershkowitz to authors section

Benefits

  • Tests organized by function rather than bug fix number
  • Easier to find and maintain tests for specific functionality
  • Clearer test naming that describes what is being tested
  • Proper attribution of all contributors

Fix rtmod#1: Composite node initialization (CRITICAL)

Fix rtmod#2: Empty list handling

Fix rtmod#3: Index lookup error prevention

Fix rtmod#4: Missing edge handling

Fix rtmod#5: Missing dependency (tabulate)

Fix rtmod#6: Empty edgelist handling

Added comprehensive unit tests (31 tests, all passing)
…author info

- Created test_setup.py for common test setup code
- Created test_preprocessing.py for prime, updates, expand, involution tests
- Created test_mfr_processing.py for get_mfrs and MFR computation tests
- Updated test_update_expand.py to remove 'bug' references from test names
- Added Sean Hershkowitz as author in setup.py and README.md
- Tests now organized by function rather than by bug fix number
@corybrunson
Copy link
Contributor

Thanks @SeanHersh! I'm having a look. When you have a moment, could you put some standard commands here for running the test scripts? I keep looking them up fresh each time.

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