-
Notifications
You must be signed in to change notification settings - Fork 0
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
137 create a function for container name creation logic #194
base: main
Are you sure you want to change the base?
137 create a function for container name creation logic #194
Conversation
I haven't write
I will write unittest to make sure the 2 functions behave as expected soon (current tests workflow are broken atm). Previous tests should tell us if the changes are working for the refactoring. |
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.
There might still be manual building of those strings in:
tests/test_azure_storage.py
tests/fertiscan/test_datastore.py
I will rebase on main and check if any tests are having issues atm. I will then add new tests where it is necessary and update the old tests with the new functions! |
7a507e3
to
0f8bddb
Compare
There seems to be issues with certain tests. I will look into it first thing next week! 🚀 |
@@ -50,6 +50,37 @@ async def generate_hash(image): | |||
print(error.__str__()) | |||
raise Exception("Unhandeled Datastore.blob.azure_storage Error") | |||
|
|||
def build_container_name(name:str,tier:str="user"): |
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.
The spacing on the types does not look right. Are we running ruff on these files?
ref #137