This is an automated email from the ASF dual-hosted git repository.

dill0wn pushed a commit to branch dw/8565
in repository https://gitbox.apache.org/repos/asf/allura.git

commit 8bb18c4536bb2fde8606f1dc1515cb991212cbd9
Author: Dillon Walls <dillon.wa...@slashdotmedia.com>
AuthorDate: Tue Jul 2 13:29:30 2024 -0400

    fixup! replace uses of Collection.count() with 
Collection.estimated_document_count()
---
 Allura/allura/tests/model/test_filesystem.py | 60 ++++++++++++++--------------
 1 file changed, 30 insertions(+), 30 deletions(-)

diff --git a/Allura/allura/tests/model/test_filesystem.py 
b/Allura/allura/tests/model/test_filesystem.py
index 8d2b46de8..3b2e54db1 100644
--- a/Allura/allura/tests/model/test_filesystem.py
+++ b/Allura/allura/tests/model/test_filesystem.py
@@ -56,9 +56,9 @@ class TestFile(TestCase):
     def test_from_stream(self):
         f = File.from_stream('test1.txt', BytesIO(b'test1'))
         self.session.flush()
-        assert self.db.fs.estimated_document_count() == 1
-        assert self.db.fs.files.estimated_document_count() == 1
-        assert self.db.fs.chunks.estimated_document_count() == 1
+        assert self.db.fs.count_documents({}) == 1
+        assert self.db.fs.files.count_documents({}) == 1
+        assert self.db.fs.chunks.count_documents({}) == 1
         assert f.filename == 'test1.txt'
         assert f.content_type == 'text/plain'
         self._assert_content(f, b'test1')
@@ -66,9 +66,9 @@ class TestFile(TestCase):
     def test_from_data(self):
         f = File.from_data('test2.txt', b'test2')
         self.session.flush(f)
-        assert self.db.fs.estimated_document_count() == 1
-        assert self.db.fs.files.estimated_document_count() == 1
-        assert self.db.fs.chunks.estimated_document_count() == 1
+        assert self.db.fs.count_documents({}) == 1
+        assert self.db.fs.files.count_documents({}) == 1
+        assert self.db.fs.chunks.count_documents({}) == 1
         assert f.filename == 'test2.txt'
         assert f.content_type == 'text/plain'
         self._assert_content(f, b'test2')
@@ -77,50 +77,50 @@ class TestFile(TestCase):
         path = __file__.rstrip('c')
         f = File.from_path(path)
         self.session.flush()
-        assert self.db.fs.estimated_document_count() == 1
-        assert self.db.fs.files.estimated_document_count() == 1
-        assert self.db.fs.chunks.estimated_document_count() >= 1
+        assert self.db.fs.count_documents({}) == 1
+        assert self.db.fs.files.count_documents({}) == 1
+        assert self.db.fs.chunks.count_documents({}) >= 1
         assert f.filename == os.path.basename(path)
         text = f.rfile().read()
 
     def test_delete(self):
         f = File.from_data('test1.txt', b'test1')
         self.session.flush()
-        assert self.db.fs.estimated_document_count() == 1
-        assert self.db.fs.files.estimated_document_count() == 1
-        assert self.db.fs.chunks.estimated_document_count() == 1
+        assert self.db.fs.count_documents({}) == 1
+        assert self.db.fs.files.count_documents({}) == 1
+        assert self.db.fs.chunks.count_documents({}) == 1
         f.delete()
         self.session.flush()
-        assert self.db.fs.estimated_document_count() == 0
-        assert self.db.fs.files.estimated_document_count() == 0
-        assert self.db.fs.chunks.estimated_document_count() == 0
+        assert self.db.fs.count_documents({}) == 0
+        assert self.db.fs.files.count_documents({}) == 0
+        assert self.db.fs.chunks.count_documents({}) == 0
 
     def test_remove(self):
         File.from_data('test1.txt', b'test1')
         File.from_data('test2.txt', b'test2')
         self.session.flush()
-        assert self.db.fs.estimated_document_count() == 2
-        assert self.db.fs.files.estimated_document_count() == 2
-        assert self.db.fs.chunks.estimated_document_count() == 2
+        assert self.db.fs.count_documents({}) == 2
+        assert self.db.fs.files.count_documents({}) == 2
+        assert self.db.fs.chunks.count_documents({}) == 2
         File.remove(dict(filename='test1.txt'))
         self.session.flush()
-        assert self.db.fs.estimated_document_count() == 1
-        assert self.db.fs.files.estimated_document_count() == 1
-        assert self.db.fs.chunks.estimated_document_count() == 1
+        assert self.db.fs.count_documents({}) == 1
+        assert self.db.fs.files.count_documents({}) == 1
+        assert self.db.fs.chunks.count_documents({}) == 1
 
     def test_overwrite(self):
         f = File.from_data('test1.txt', b'test1')
         self.session.flush()
-        assert self.db.fs.estimated_document_count() == 1
-        assert self.db.fs.files.estimated_document_count() == 1
-        assert self.db.fs.chunks.estimated_document_count() == 1
+        assert self.db.fs.count_documents({}) == 1
+        assert self.db.fs.files.count_documents({}) == 1
+        assert self.db.fs.chunks.count_documents({}) == 1
         self._assert_content(f, b'test1')
         with f.wfile() as fp:
             fp.write(b'test2')
         self.session.flush()
-        assert self.db.fs.estimated_document_count() == 1
-        assert self.db.fs.files.estimated_document_count() == 2
-        assert self.db.fs.chunks.estimated_document_count() == 2
+        assert self.db.fs.count_documents({}) == 1
+        assert self.db.fs.files.count_documents({}) == 2
+        assert self.db.fs.chunks.count_documents({}) == 2
         self._assert_content(f, b'test2')
 
     def test_serve_embed(self):
@@ -167,9 +167,9 @@ class TestFile(TestCase):
         assert t.content_type == 'image/png'
         assert t.is_image()
         assert f.filename == t.filename
-        assert self.db.fs.estimated_document_count() == 2
-        assert self.db.fs.files.estimated_document_count() == 2
-        assert self.db.fs.chunks.estimated_document_count() == 2
+        assert self.db.fs.count_documents({}) == 2
+        assert self.db.fs.files.count_documents({}) == 2
+        assert self.db.fs.chunks.count_documents({}) == 2
 
     def test_not_image(self):
         f, t = File.save_image(

Reply via email to