You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Test utilities should ideally contain helper classes to build other tests. Any testing should be done in the tests that invoke these util classes.
Example: velox/dwio/dwrf/test/utils/E2EWriterTestUtil.cpp does some test checks, which introduces a dependency on googletest. These checks must ideally be performed in the tests where the methods in utils/E2EWriterTestUtil.cpp are invoked. This helps with test readability as well as helps maintain clean dependencies.
The text was updated successfully, but these errors were encountered:
Test utilities should ideally contain helper classes to build other tests. Any testing should be done in the tests that invoke these util classes.
Example:
velox/dwio/dwrf/test/utils/E2EWriterTestUtil.cpp
does some test checks, which introduces a dependency on googletest. These checks must ideally be performed in the tests where the methods inutils/E2EWriterTestUtil.cpp
are invoked. This helps with test readability as well as helps maintain clean dependencies.The text was updated successfully, but these errors were encountered: