-
Notifications
You must be signed in to change notification settings - Fork 21
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
Improve execution time of unit tests #1018
Conversation
bde7372
to
56fd3ed
Compare
…mes were passed incorrectly
@@ -178,25 +188,53 @@ def get_contract_path(self, *args: str) -> str: | |||
raise FileNotFoundError(path) | |||
return path | |||
|
|||
def get_deploy_file_paths_without_compiling(self, contract_path: str) -> Tuple[str, str]: | |||
file_path_without_ext, _ = os.path.splitext(contract_path) | |||
if USE_UNIQUE_NAME: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why use USE_UNIQUE_NAME
instead of self._use_custom_name
?
Also, other files are always using engine.use_contract_custom_name = self._use_custom_name
. Does USE_UNIQUE_NAME
only exist to be more explicit?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
self._use_custom_name
was included in BoaTest to pass its value to the tests using TestEngine without having to import the USE_UNIQUE_NAME
constant across all the test files, it'll be removed when we remove TestEngine as well
No description provided.