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.

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

test: drop obsolete tests #8149

Merged
merged 1 commit into from
Aug 4, 2023
Merged

test: drop obsolete tests #8149

merged 1 commit into from
Aug 4, 2023

Conversation

Chris-Hibbert
Copy link
Contributor

closes: #7175
refs: #70747

Description

#7047 .skipped some tests that were hard to maintain. @dckc pointed out that most of the functionality is tested elsewhere.

Security Considerations

None

Scaling Considerations

None

Documentation Considerations

None

Testing Considerations

unused, redundant test aren't helpful, and are a maintenance burden.

Upgrade Considerations

None

@Chris-Hibbert Chris-Hibbert added Inter-protocol Overarching Inter Protocol Governance Governance Vaults VaultFactor (née Treasury) labels Aug 4, 2023
@Chris-Hibbert Chris-Hibbert requested a review from dckc August 4, 2023 21:50
@Chris-Hibbert Chris-Hibbert self-assigned this Aug 4, 2023
@Chris-Hibbert Chris-Hibbert added this pull request to the merge queue Aug 4, 2023
Merged via the queue into master with commit ff34bdf Aug 4, 2023
@Chris-Hibbert Chris-Hibbert deleted the 7175-expungeTests branch August 4, 2023 22:49
mhofman pushed a commit that referenced this pull request Aug 7, 2023
mhofman pushed a commit that referenced this pull request Jan 15, 2024
mhofman pushed a commit that referenced this pull request Jan 15, 2024
mhofman pushed a commit that referenced this pull request Jan 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Governance Governance Inter-protocol Overarching Inter Protocol Vaults VaultFactor (née Treasury)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

restore or replace test-gov-collateral
2 participants