Skip to content

Add test to enforce keyword arguments#1664

Merged
h-mayorquin merged 1 commit intodeprecate_postional_only_on_nwbfilefrom
test_for_enforcing_nwbfile_position_only
Feb 13, 2026
Merged

Add test to enforce keyword arguments#1664
h-mayorquin merged 1 commit intodeprecate_postional_only_on_nwbfilefrom
test_for_enforcing_nwbfile_position_only

Conversation

@h-mayorquin
Copy link
Collaborator

@h-mayorquin h-mayorquin commented Feb 13, 2026

This add testing for #1663. It catched an error that I did not fix there. This has #1663 as base.

@h-mayorquin h-mayorquin self-assigned this Feb 13, 2026
@h-mayorquin h-mayorquin marked this pull request as ready for review February 13, 2026 17:18
@h-mayorquin h-mayorquin merged commit 80ba461 into deprecate_postional_only_on_nwbfile Feb 13, 2026
27 of 32 checks passed
@h-mayorquin h-mayorquin deleted the test_for_enforcing_nwbfile_position_only branch February 13, 2026 18:36
@codecov
Copy link

codecov bot commented Feb 13, 2026

Codecov Report

❌ Patch coverage is 10.00000% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.18%. Comparing base (a6281e2) to head (3344cc1).
⚠️ Report is 1 commits behind head on deprecate_postional_only_on_nwbfile.

Files with missing lines Patch % Lines
...s/ophys/miniscope/miniscopeimagingdatainterface.py 10.00% 9 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@                           Coverage Diff                           @@
##           deprecate_postional_only_on_nwbfile    #1664      +/-   ##
=======================================================================
- Coverage                                80.24%   80.18%   -0.06%     
=======================================================================
  Files                                      159      159              
  Lines                                    12614    12624      +10     
=======================================================================
+ Hits                                     10122    10123       +1     
- Misses                                    2492     2501       +9     
Flag Coverage Δ
unittests 80.18% <10.00%> (-0.06%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...s/ophys/miniscope/miniscopeimagingdatainterface.py 78.04% <10.00%> (-4.42%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants