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.

Sign up for GitHub

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

Jump to bottom

[Test] Implementing unit tests for SyncedEnforcer #190

Closed
EmperorYP7 opened this issue Mar 5, 2022 · 2 comments · Fixed by #191
Closed

[Test] Implementing unit tests for SyncedEnforcer #190

EmperorYP7 opened this issue Mar 5, 2022 · 2 comments · Fixed by #191
Assignees
@hsluoyz
Labels
bug Something isn't working help wanted Extra attention is needed released

Comments

@EmperorYP7
Copy link
Member

EmperorYP7 commented Mar 5, 2022 •
edited
Loading

Description

Functional tests for casbin::SyncedEnforcer. The problem arose due to ambiguity in handling multi-threaded tests in GoogleTest.

We need to inspect ways to get around this and implement it in our current test setup.

Your System Info

NA

Screenshots/Testimonials

NA
@EmperorYP7 EmperorYP7 added bug Something isn't working help wanted Extra attention is needed labels Mar 5, 2022
@casbin-bot
Copy link
Member

@EmperorYP7 @divy9881 @noob20000405 @sheny1xuan

@casbin-bot casbin-bot added the enhancement New feature or request label Mar 5, 2022
@hsluoyz hsluoyz removed the enhancement New feature or request label Mar 6, 2022
@sheny1xuan sheny1xuan mentioned this issue Mar 10, 2022
Merged
@github-actions
Copy link

🎉 This issue has been resolved in version 1.49.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hsluoyz hsluoyz

Labels
bug Something isn't working help wanted Extra attention is needed released
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: fix sync test by use base enforcer's ClearPolicy and BuildRoleLink. sheny1xuan/casbin-cpp
3 participants
@hsluoyz @EmperorYP7 @casbin-bot