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
test: migrate old tests to the new test framework #806
Conversation
Rename new integration test project
increate timeout on disposition_at test
…-windows-sdk-v2 into sdk-1976-remove-appveyor
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.
I think that maybe we would like and adopt more BDD style testing?
When removing elements we should check that method is calling API, I think we might get deleted object and verify it was deleted?
|
||
var result = await UserClient.FoldersManager.RemoveWatermarkAsync(folder.Id); | ||
|
||
Assert.IsTrue(result); |
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 not do this in a BDD way - which for me is way better in Integration Tests:
Add watermark to folder,
check information,
get watermark
then remove
I this test if the ApplyWatermarkAsync fails would the test still pass as remove would succeed?
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.
👍
Migrate all tests that were previously running on AppVeyor. Fixes #399.