Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update test suite: Suppress output & expected warnings + improve performance #658

Open
wants to merge 169 commits into
base: main
Choose a base branch
from
Open

Update test suite: Suppress output & expected warnings + improve performance #658

wants to merge 169 commits into from

Conversation

ehennestad
Copy link
Collaborator

@ehennestad ehennestad commented Feb 1, 2025 •
edited
Loading

Todo:

Motivation

  • Make tests run faster
  • Minimise unnecessary text output in command window when running tests

How to test the behavior?

nwbtest

Checklist

  • Have you ensured the PR description clearly describes the problem and solutions?
  • Have you checked to ensure that there aren't other open or previously closed Pull Requests for the same change?
  • If this PR fixes an issue, is the first line of the PR description fix #XX where XX is the issue number?

@ehennestad
Coarse fix for #595
fc9ca2a
@ehennestad
Merge branch 'master' into 595-use-empty-indices-for-datastub
1b4dbfb
@ehennestad
Only include namespaces for types that are included in NWB file on ex…
a458d62 
…port (Issue #607)
@ehennestad
Merge branch 'master' into 607-fix-namespace-embedding-in-file
13822a8
@ehennestad
Add functionality for installing extensions
35150e9
@ehennestad
Minor fixes
bd40894
@ehennestad
Update comment
132aa67
@ehennestad
Add comment + print message when extension has been installed
781f303
@ehennestad
Update installExtension.m
eab998e
@ehennestad
Fix changed variable name
be4be3d
@ehennestad
Update matnwb_createNwbInstallExtension.m
650ceec 
Update docstring
@ehennestad
Create listNwbTypeHierarchy.m
8e18448 
Add utility function for listing the type hierarchy of an nwb type
@ehennestad
Add private method for embedding specifications to file on export
7c41ca6
@ehennestad
Fix variable name
a467c46
@ehennestad
Update testCreateParsedType.m
0911141
@ehennestad
Suppress output of nwbfile
744fef9
@ehennestad
Add workflow for updating nwbInstallExtension
ff95e98
@ehennestad
Add option to save extension in custom location
bb3514b
@ehennestad
Create InstallExtensionTest.m
184fa81
@ehennestad
Add .env file to replace env.mat
e0cba2f
@ehennestad
Update .gitignore
df37d10
@ehennestad
Update docstring
ddbe9dc
@ehennestad
Merge branch 'master' into add-nwb-install-extension
44a6a20
@ehennestad
Merge branch 'add-nwb-install-extension' of https://github.com/Neurod…
24c3899 
…ataWithoutBorders/matnwb into add-nwb-install-extension
@ehennestad
Change dispExtensionInfo to return info instead of displaying + add test
da00cea
@ehennestad
Reorganize code into separate functions and add tests
40b7703
@ehennestad
Merge branch 'master' into 607-fix-namespace-embedding-in-file
eeb2006
@ehennestad
Merge branch 'master' into add-nwb-install-extension
e3b4906
@ehennestad
Minor changes to improve test coverage
6faba21
@ehennestad
add nwbInstallExtension to docs
a74a2d2
@ehennestad
Add shared fixture to new test MustBeH5FileTest
1f852f4
@ehennestad
Add abstract NwbTestCase class with utility methods that can be reused
302a25d
@ehennestad
Suppress output from various tests
593ad63
@ehennestad
Fix failing test
b901755
@ehennestad
Merge branch 'main' into update-tests-suppress-expected-warnings
26ab33c
@ehennestad
Merge branch 'main' into update-tests-suppress-expected-warnings
0f78abd
@ehennestad
Merge branch 'main' into update-tests-suppress-expected-warnings
be18268
@ehennestad
Remove commented code block from ObjectView class
b369205
@ehennestad
Merge branch 'update-tests-suppress-expected-warnings' of https://git…
45bdbb1 
…hub.com/NeurodataWithoutBorders/matnwb into update-tests-suppress-expected-warnings
@ehennestad
Add utility method to NwbTestCase class
f35fe9d
@ehennestad
Fix bug and add test
a0baa19
@ehennestad
Add "version" as property to Namespace class
9f66274
@ehennestad
Add routine for writing a function for retrieving version of generate…
ccfd763 
…d namespace
@ehennestad
Regenerate core with version numbers
156e36e
@ehennestad
Update generateRstForNeurodataTypeClasses.m
6282f4a 
Ignore version function when creating list of neurodata types to document
@ehennestad
Add version specificity to cache containing map of required properties
7e87d4c
@ehennestad
Merge branch '668-bug-nwb-to-table' into update-tests-suppress-expect…
ffac64d 
…ed-warnings
@ehennestad
Merge branch 'fix/required-property-cache-must-be-version-specific' i…
2977a2a 
…nto update-tests-suppress-expected-warnings
@ehennestad
Delete GenerateSpecificationMixin.m
cb3571d
@ehennestad
Split test class setup into methods with single responsibilities
257743a
@ehennestad
Add factory function to create NwbFile for unit testing
9433cb8
@ehennestad
Merge branch 'main' into update-tests-suppress-expected-warnings
72ccff3
@ehennestad
Merge branch 'main' into update-tests-suppress-expected-warnings
9387541
@ehennestad
Clean up nwbExport test
923949d
@ehennestad
Clean up a few more test classes
68be31c
@ehennestad
Add fixture for setting environment variables used in test suite and …
e095d4a 
…remove env.mat
@ehennestad
Update PyNWBIOTest.m
bf8f937 
Add shared fixture
@ehennestad
Fix wrong shape of data in PhotonSeriesIOTest
1ea9afc
@ehennestad
Update PyNWBIOTest.py
c72aada
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bendichter bendichter Awaiting requested review from bendichter

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ehennestad @bendichter