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
fix tempfile handling #183
Conversation
requre/helpers/tempfile.py
Outdated
|
||
|
||
class TempFile(ObjectStorage): | ||
class mktemp(Simple): |
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.
Is this name needed? Can we use Python's UperCamelCase
class naming style?
Build failed.
|
b0c9690
to
1a7379d
Compare
Build failed.
|
642b7ce
to
c2ed8bb
Compare
Build failed.
|
Build succeeded.
|
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.
Thanks!
def cassette_teardown(self, cassette: Cassette): | ||
self.assertIn( | ||
"tests.test_modules_decorate_all_methods.ApplyCommonCase.test_git.yaml", | ||
str(cassette.storage_file), | ||
) | ||
|
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 you are removing it?
Isn't there a problem with the new name?
def cassette_teardown(self, cassette: Cassette):
self.assertIn(
"ApplyCommonCase.test_git.yaml",
str(cassette.storage_file),
)
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.
Ah yes, I've removed it, because it failed :-) but I can let check there for sure
Build succeeded (gate pipeline).
|
No description provided.