Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2487)

Unified Diff: appengine/findit/model/test/versioned_config_test.py

Issue 2488113005: [Findit] Re-org code. (Closed)
Patch Set: Rebase. Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: appengine/findit/model/test/versioned_config_test.py
diff --git a/appengine/findit/model/test/versioned_config_test.py b/appengine/findit/model/test/versioned_config_test.py
deleted file mode 100644
index 1584e61eea5d8538395a4ba54214402bacfc69c1..0000000000000000000000000000000000000000
--- a/appengine/findit/model/test/versioned_config_test.py
+++ /dev/null
@@ -1,54 +0,0 @@
-# Copyright 2015 The Chromium Authors. All rights reserved.
-# Use of this source code is governed by a BSD-style license that can be
-# found in the LICENSE file.
-
-from google.appengine.ext import ndb
-from google.appengine.api import users
-from testing_utils import testing
-
-from model.versioned_config import VersionedConfig
-
-
-class _Config(VersionedConfig):
- a = ndb.IntegerProperty(indexed=False, default=0)
-
-
-class VersionedConfigTest(testing.AppengineTestCase):
-
- def _CreateFirstVersion(self):
- config = _Config.Get()
- config.Update(users.User(email='admin@chromium.org'), True, a=1)
-
- def testGetWhenNoConfigCreatedYet(self):
- config = _Config.Get()
- self.assertIsNotNone(config)
- self.assertEqual(0, config.a)
-
- def testNonAdminCanNotUpdate(self):
- config = _Config.Get()
- with self.assertRaises(Exception):
- config.Update(users.User(email='admin@chromium.org'), False, a=1)
-
- def testUpdateWhenChanged(self):
- self._CreateFirstVersion()
- config = _Config.Get()
- self.assertIsNotNone(config)
- self.assertTrue(config.Update(users.User(email='admin@chromium.org'), True,
- a=2))
-
- config = _Config.Get()
- self.assertIsNotNone(config)
- self.assertEqual(2, config.version_number)
- self.assertEqual(2, config.a)
-
- def testNotUpdateWhenNotChanged(self):
- self._CreateFirstVersion()
- config = _Config.Get()
- self.assertIsNotNone(config)
- self.assertFalse(config.Update(users.User(email='admin@chromium.org'), True,
- a=1))
-
- config = _Config.Get()
- self.assertIsNotNone(config)
- self.assertEqual(1, config.version_number)
- self.assertEqual(1, config.a)
« no previous file with comments | « appengine/findit/model/suspected_cl_confidence.py ('k') | appengine/findit/model/test/versioned_model_test.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698