Fix oppia#2394: Add docstrings to gae_models.py (oppia#2633)

* Update gae_models.py * Update gae_models.py * Update gae_models.py and fixed calling get_all with include_dleted parameter * Update gae_models.py * Update gae_models.py
itaditya · Nov 1, 2016 · 7135c94 · 7135c94
1 parent 5a88f7a
commit 7135c94
Show file tree

Hide file tree

Showing 4 changed files with 232 additions and 52 deletions.
diff --git a/core/domain/collection_services_test.py b/core/domain/collection_services_test.py
@@ -525,13 +525,13 @@ def test_soft_deletion_of_collections(self):
         self.assertIn(self.COLLECTION_ID, [
             collection.id
             for collection in collection_models.CollectionModel.get_all(
-                include_deleted_entities=True)])
+                include_deleted=True)])
 
         # The collection summary is deleted, however.
         self.assertNotIn(self.COLLECTION_ID, [
             collection.id
             for collection in collection_models.CollectionSummaryModel.get_all(
-                include_deleted_entities=True)])
+                include_deleted=True)])
 
     def test_hard_deletion_of_collections(self):
         """Test that hard deletion of collections works correctly."""
@@ -553,7 +553,7 @@ def test_hard_deletion_of_collections(self):
         self.assertNotIn(self.COLLECTION_ID, [
             collection.id
             for collection in collection_models.CollectionModel.get_all(
-                include_deleted_entities=True)])
+                include_deleted=True)])
 
     def test_summaries_of_hard_deleted_collections(self):
         """Test that summaries of hard deleted collections are
@@ -574,7 +574,7 @@ def test_summaries_of_hard_deleted_collections(self):
         self.assertNotIn(self.COLLECTION_ID, [
             collection.id
             for collection in collection_models.CollectionSummaryModel.get_all(
-                include_deleted_entities=True)])
+                include_deleted=True)])
 
     def test_collections_are_removed_from_index_when_deleted(self):
         """Tests that deleted collections are removed from the search index."""

diff --git a/core/domain/exp_services_test.py b/core/domain/exp_services_test.py
@@ -376,14 +376,14 @@ def test_soft_deletion_of_explorations(self):
         self.assertIn(
             self.EXP_ID,
             [exp.id for exp in exp_models.ExplorationModel.get_all(
-                include_deleted_entities=True)]
+                include_deleted=True)]
         )
 
         # The exploration summary is deleted however
         self.assertNotIn(
             self.EXP_ID,
             [exp.id for exp in exp_models.ExpSummaryModel.get_all(
-                include_deleted_entities=True)]
+                include_deleted=True)]
         )
 
     def test_hard_deletion_of_explorations(self):
@@ -406,7 +406,7 @@ def test_hard_deletion_of_explorations(self):
         self.assertNotIn(
             self.EXP_ID,
             [exp.id for exp in exp_models.ExplorationModel.get_all(
-                include_deleted_entities=True)]
+                include_deleted=True)]
         )
 
     def test_summaries_of_hard_deleted_explorations(self):
@@ -427,7 +427,7 @@ def test_summaries_of_hard_deleted_explorations(self):
         self.assertNotIn(
             self.EXP_ID,
             [exp.id for exp in exp_models.ExpSummaryModel.get_all(
-                include_deleted_entities=True)]
+                include_deleted=True)]
         )
 
     def test_explorations_are_removed_from_index_when_deleted(self):