Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add more testing to @autonomys/auto-dag-data #177

Merged
merged 9 commits into from
Nov 19, 2024
Merged

Conversation

clostao
Copy link
Contributor

@clostao clostao commented Nov 19, 2024

User description

Adding more testing for:

  • Ensuring file retrievability
  • Cover processChunksToIPLDFormat method

PR Type

Tests, Enhancement


Description

  • Added comprehensive tests for file retrievability, including single and chunked file retrieval, ensuring file integrity.
  • Updated versions across multiple packages to maintain consistency and compatibility.
  • Adjusted dependencies to align with the new package versions.

Changes walkthrough 📝

Relevant files
Tests
fileRetrievability.spec.ts
Add tests for file retrievability and chunk processing     

packages/auto-dag-data/tests/fileRetrievability.spec.ts

  • Added new test cases for file retrievability.
  • Tested single and chunked file retrieval.
  • Verified file integrity after retrieval.
  • +133/-0 
    Configuration changes
    lerna.json
    Update project version in Lerna configuration                       

    lerna.json

    • Updated version from 1.0.6 to 1.0.8.
    +1/-1     
    Dependencies
    package.json
    Update package and dependency versions                                     

    packages/auto-consensus/package.json

  • Updated package version to 1.0.7.
  • Updated dependency version for @autonomys/auto-utils.
  • +2/-2     
    package.json
    Update package version                                                                     

    packages/auto-dag-data/package.json

    • Updated package version to 1.0.8.
    +1/-1     
    package.json
    Update package and dependency versions                                     

    packages/auto-drive/package.json

  • Updated package version to 1.0.8.
  • Updated dependency version for @autonomys/auto-dag-data.
  • +2/-2     
    package.json
    Update package and dependency versions                                     

    packages/auto-id/package.json

  • Updated package version to 1.0.7.
  • Updated dependency version for @autonomys/auto-utils.
  • +2/-2     
    package.json
    Update package version                                                                     

    packages/auto-utils/package.json

    • Updated package version to 1.0.7.
    +1/-1     
    package.json
    Update package and dependency versions                                     

    packages/auto-xdm/package.json

  • Updated package version to 1.0.7.
  • Updated dependency version for @autonomys/auto-utils.
  • +2/-2     

    💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

    Copy link

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 3 🔵🔵🔵⚪⚪
    🧪 PR contains tests
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    Test Completeness
    Ensure that the tests cover all edge cases and failure scenarios for file retrievability, especially under network or hardware failures.

    Randomness in Tests
    The use of randomness in tests may lead to non-deterministic outcomes. Consider using fixed inputs to ensure consistent test results.

    Copy link

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    @clostao clostao changed the base branch from main to chore/publish-v1.0.7 November 19, 2024 17:28
    Copy link
    Collaborator

    @marc-aurele-besner marc-aurele-besner left a comment

    Choose a reason for hiding this comment

    The reason will be displayed to describe this comment to others. Learn more.

    Awesome 🚀

    Base automatically changed from chore/publish-v1.0.7 to main November 19, 2024 19:01
    @clostao clostao dismissed marc-aurele-besner’s stale review November 19, 2024 19:01

    The base branch was changed.

    @clostao
    Copy link
    Contributor Author

    clostao commented Nov 19, 2024

    I updated the version to v1.0.8 of all packages that was missed to be updated in prior PR

    @clostao clostao merged commit 4b2dde9 into main Nov 19, 2024
    1 check passed
    @clostao clostao deleted the test/retrievability branch November 19, 2024 19:53
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    2 participants