Skip to content

Allow Cloud Storage usage without S3 features#121

Merged
agritheory merged 1 commit intoversion-15from
local-storage-test
Mar 19, 2026
Merged

Allow Cloud Storage usage without S3 features#121
agritheory merged 1 commit intoversion-15from
local-storage-test

Conversation

@IshwaryaM1030
Copy link
Copy Markdown
Collaborator

@IshwaryaM1030 IshwaryaM1030 commented Mar 13, 2026

Closes #70

@github-actions
Copy link
Copy Markdown

📝 Draft Changelog Entry

Changelog Entry

Testing coverage for local storage functionality has been expanded with the addition of comprehensive test cases for storage operations without S3 integration.

This changelog entry was automatically generated by the Changelog Generator Action.

@IshwaryaM1030 IshwaryaM1030 changed the title Adding without s3 storage test Allow Cloud Storage usage without S3 features Mar 13, 2026
@github-actions
Copy link
Copy Markdown

Coverage

Coverage Report
FileStmtsMissCoverMissing
__init__.py11554%13–14, 20–22
commands.py35350%4–6, 9–11, 17–20, 23, 26–27, 49–51, 53–54, 56, 65, 68–72, 75–76, 85–87, 89–90, 92, 94, 97
customize.py25250%4–5, 7, 10–12, 15–22, 27–35, 47–48
hooks.py150100% 
install.py36360%4–7, 10–11, 14–18, 21–27, 30–34, 36–39, 41–49
migration.py19611541%25, 46, 68–69, 88–89, 96, 115, 123–125, 158–159, 161–162, 166–168, 171–172, 187, 196–197, 199–203, 205–210, 212, 219–221, 223, 225–226, 228, 235–236, 238–242, 244–247, 249, 254–257, 259–268, 270–279, 281–282, 284–285, 287–288, 290–294, 298–301, 305, 307–308, 310, 312–314, 317–318, 320–321, 323–329, 331–332, 334, 336
cloud_storage
   __init__.py00100% 
cloud_storage/doctype
   __init__.py00100% 
cloud_storage/doctype/file_association
   __init__.py00100% 
   file_association.py30100% 
cloud_storage/doctype/file_version
   __init__.py00100% 
   file_version.py30100% 
cloud_storage/overrides
   file.py41013267%66, 76, 96, 119, 121, 136, 138, 145–147, 155–156, 171, 176, 183, 198, 231, 241–250, 252–260, 271, 276–277, 285–287, 289, 294, 300, 302, 318, 322, 327, 335–336, 339, 342, 371, 416, 419, 458, 464, 470, 476, 482, 488, 498, 500–502, 506, 514–516, 518, 533, 536, 547–551, 557–558, 560, 583–587, 599–600, 649, 652–661, 668–669, 671–673, 676–677, 682–684, 686, 691–694, 699, 701, 713–714, 716, 721–725, 727, 738–744
config
   __init__.py00100% 
   desktop.py330%4, 7–8
   docs.py660%4, 6–8, 12–13
tests
   conftest.py43197%54
   test_file.py173696%89, 174, 208–210, 223
   test_file_preview.py630100% 
   test_local_storage.py610100% 
TOTAL108336466% 

@agritheory agritheory merged commit daa7269 into version-15 Mar 19, 2026
6 checks passed
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.

Allow Cloud Storage usage without S3 features

2 participants