Skip to content
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

Run DistributedLock tests #35

Open
1 task
anna-mazhar opened this issue Jun 12, 2023 · 3 comments
Open
1 task

Run DistributedLock tests #35

anna-mazhar opened this issue Jun 12, 2023 · 3 comments
Assignees

Comments

@anna-mazhar
Copy link
Collaborator

Run tests for the following app:

@tianyin
Copy link
Member

tianyin commented Jun 12, 2023

@anna-mazhar please create priority tags instead of putting them onto the titles.

@anna-mazhar anna-mazhar changed the title Low priority -- Run DistributedLock tests Run DistributedLock tests Jun 12, 2023
@saadsheralam
Copy link
Member

Added results

@anna-mazhar
Copy link
Collaborator Author

// Copied from Slack

"Hmm, that's strange. We know for sure that this app does have unit tests for Azure Blob. I suggest you do the following:

  • Run all the tests instead of just the Azure directory ones and observe the traffic on the emulator. If you see traffic, it means the tests using Azure Blob are in other directories. If you don't see any traffic for all the tests then they probably have removed them for some reason. In this case, we can use the version that the rainmaker folks used. Follow the rest of instructions.
  • Reset to the following hash: 509f0c146418b52a0eecf4bdeb5de766f2454270
  • Apply this patch to the repo.
  • Here are some instructions to help you on how to do it.

If you don't see traffic after resetting it to this older version then its likely that something is missing in the emulator configuration of this app"

@anna-mazhar anna-mazhar reopened this Jul 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants