Replies: 1 comment
-
|
+1 for this, |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
When reviewing the files in our codebase, I noticed a lot of duplicated tests between the PAX storage tests
contrib/pax_storage/src/testand the coresrc/testdirectory.Would it be possible to remove the basic, non-PAX-specific tests from the PAX folder and only retain tests unique to PAX functionality there? For the common test cases, could we reuse the existing tests in src/test instead?
This would reduce duplication and help us maintain a single source of truth for shared test logic. Thoughts?
Beta Was this translation helpful? Give feedback.
All reactions