Repository: sentry Updated Branches: refs/heads/sentry-ha-redesign 3c2a8a2e5 -> ec1e01d45
SENTRY-1721: Unit test failures in TestSentryStore due to changeId miscount (Na Li, reviewed by Alex Kolbasov) Project: http://git-wip-us.apache.org/repos/asf/sentry/repo Commit: http://git-wip-us.apache.org/repos/asf/sentry/commit/ec1e01d4 Tree: http://git-wip-us.apache.org/repos/asf/sentry/tree/ec1e01d4 Diff: http://git-wip-us.apache.org/repos/asf/sentry/diff/ec1e01d4 Branch: refs/heads/sentry-ha-redesign Commit: ec1e01d450422af380c17a8786289deb933a0441 Parents: 3c2a8a2 Author: Alexander Kolbasov <[email protected]> Authored: Thu Apr 27 16:56:44 2017 -0700 Committer: Alexander Kolbasov <[email protected]> Committed: Thu Apr 27 16:56:44 2017 -0700 ---------------------------------------------------------------------- .../sentry/provider/db/service/persistent/TestSentryStore.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/sentry/blob/ec1e01d4/sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java ---------------------------------------------------------------------- diff --git a/sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java b/sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java index d12b0da..acf8732 100644 --- a/sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java +++ b/sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java @@ -2704,7 +2704,7 @@ public class TestSentryStore extends org.junit.Assert { // Verify getMSentryPermChanges will return all MSentryPermChanges up // to the given changeID. List<MSentryPermChange> mSentryPermChanges = sentryStore.getMSentryPermChanges(initialID); - assertEquals(lastChangeID, mSentryPermChanges.size()); + assertEquals(lastChangeID - initialID + 1, mSentryPermChanges.size()); // Verify ifPermChangeExists will return true for persisted MSentryPermChange. assertEquals(true, sentryStore.permChangeExists(1));
