Skip to content

Tests to demo model/policy getters/setters.#60

Merged
hsluoyz merged 1 commit intoapache:masterfrom
cdennison:testcov-getsetfixed
Dec 19, 2017
Merged

Tests to demo model/policy getters/setters.#60
hsluoyz merged 1 commit intoapache:masterfrom
cdennison:testcov-getsetfixed

Conversation

@cdennison
Copy link

Showing that getters/setters work correctly.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.4%) to 89.024% when pulling 8ebd496 on cdennison:testcov-getsetfixed into 536a0fa on casbin:master.

@hsluoyz hsluoyz merged commit 9566491 into apache:master Dec 19, 2017
@cdennison
Copy link
Author

Code coverage went down because this test is not "deterministic" and will sometime fail to finish because it needs a sleep at the end. Fixing in a follow up PR.
https://github.com/casbin/casbin/blob/master/enforcer_test.go#L322

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants