Skip to content

Conversation

@BruceBlanchard
Copy link
Contributor

API call returns "downloaded_file_count" not "download_file_count".. As such object is not representing the correct download count.

API call returns "downloaded_file_count" not "download_file_count"..  As such object is not representing the correct download count.
@CLAassistant
Copy link

CLAassistant commented May 12, 2025

CLA assistant check
All committers have signed the CLA.

@lukaszsocha2 lukaszsocha2 changed the title Update BoxZipDownloadStatus.java fix: Fix parsing downloadFileCount property for BoxZipDownloadStatus May 13, 2025
@lukaszsocha2
Copy link
Contributor

@BruceBlanchard thanks for creating this PR. Please accept Contributor License Agreement so that we can merge your pr.
Best,
@lukaszsocha2

@coveralls
Copy link

coveralls commented May 13, 2025

Pull Request Test Coverage Report for Build #4599

Details

  • 0 of 1 (0.0%) changed or added relevant line in 1 file are covered.
  • 3 unchanged lines in 3 files lost coverage.
  • Overall coverage increased (+0.004%) to 72.062%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/main/java/com/box/sdk/BoxZipDownloadStatus.java 0 1 0.0%
Files with Coverage Reduction New Missed Lines %
src/main/java/com/box/sdk/BoxFileUploadSession.java 1 64.67%
src/main/java/com/box/sdk/EventStream.java 1 86.46%
src/main/java/com/box/sdk/MetadataQuery.java 1 92.59%
Totals Coverage Status
Change from base Build #4597: 0.004%
Covered Lines: 8223
Relevant Lines: 11411

💛 - Coveralls

@coveralls
Copy link

coveralls commented May 13, 2025

Pull Request Test Coverage Report for Build #4601

Details

  • 0 of 1 (0.0%) changed or added relevant line in 1 file are covered.
  • 3 unchanged lines in 3 files lost coverage.
  • Overall coverage increased (+0.004%) to 72.062%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/main/java/com/box/sdk/BoxZipDownloadStatus.java 0 1 0.0%
Files with Coverage Reduction New Missed Lines %
src/main/java/com/box/sdk/BoxFileUploadSession.java 1 64.67%
src/main/java/com/box/sdk/EventStream.java 1 86.46%
src/main/java/com/box/sdk/MetadataQuery.java 1 92.59%
Totals Coverage Status
Change from base Build #4597: 0.004%
Covered Lines: 8223
Relevant Lines: 11411

💛 - Coveralls

@lukaszsocha2 lukaszsocha2 merged commit 50c2249 into box:main May 14, 2025
9 of 13 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.

4 participants