Skip to content

Conversation

@devin-ai-integration
Copy link

@devin-ai-integration devin-ai-integration bot commented Feb 7, 2025

Real API Integration Tests for Users

This PR adds integration tests for the Users manager in BoxCLI and fixes environment setup issues that were causing timeouts in CI.

Changes

  • Added integration test directory structure
  • Created test helper with JWT auth support
  • Added user integration tests covering:
    • Get user info
    • List group memberships
  • Added integration test npm script
  • Fixed environment setup reliability:
    • Added shorter timeouts (10-15s) for individual operations
    • Added detailed timing logs for debugging
    • Improved error handling and cleanup
    • Fixed promise handling issues
  • Updated GitHub Actions workflow to properly handle integration test failures

Testing

Integration tests require:

  • Box Platform application with JWT authentication
  • Admin user access
  • Environment variables setup:
    • BOX_JWT_CONFIG: Base64 encoded JWT configuration
    • BOX_ADMIN_USER_ID: ID of admin user account

Link to Devin run: https://app.devin.ai/sessions/530d7ec43e8b497b86e60175a865a55a
Requested by: mcong@box.com

devin-ai-integration bot and others added 4 commits February 7, 2025 11:32
Co-Authored-By: mcong@box.com <mcong@box.com>
Co-Authored-By: mcong@box.com <mcong@box.com>
Co-Authored-By: mcong@box.com <mcong@box.com>
Co-Authored-By: mcong@box.com <mcong@box.com>
@devin-ai-integration
Copy link
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add "(aside)" to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@devin-ai-integration devin-ai-integration bot changed the title Real API Integration Tests for Users feat: add real API integration tests for users Feb 7, 2025
devin-ai-integration bot and others added 4 commits February 7, 2025 21:53
Co-Authored-By: mcong@box.com <mcong@box.com>
Co-Authored-By: mcong@box.com <mcong@box.com>
…ecrets are configured

Co-Authored-By: mcong@box.com <mcong@box.com>
Co-Authored-By: mcong@box.com <mcong@box.com>
@coveralls
Copy link

coveralls commented Feb 7, 2025

Pull Request Test Coverage Report for Build 13208912350

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 25 unchanged lines in 4 files lost coverage.
  • Overall coverage decreased (-0.7%) to 84.603%

Files with Coverage Reduction New Missed Lines %
src/commands/files/zip.js 1 81.03%
src/pagination-utils.js 1 88.24%
src/util.js 6 74.64%
src/box-command.js 17 72.15%
Totals Coverage Status
Change from base Build 13052088771: -0.7%
Covered Lines: 4374
Relevant Lines: 4972

💛 - Coveralls

devin-ai-integration bot and others added 13 commits February 7, 2025 22:03
Co-Authored-By: mcong@box.com <mcong@box.com>
Co-Authored-By: mcong@box.com <mcong@box.com>
Co-Authored-By: mcong@box.com <mcong@box.com>
Co-Authored-By: mcong@box.com <mcong@box.com>
Co-Authored-By: mcong@box.com <mcong@box.com>
Co-Authored-By: mcong@box.com <mcong@box.com>
Co-Authored-By: mcong@box.com <mcong@box.com>
Co-Authored-By: mcong@box.com <mcong@box.com>
Co-Authored-By: mcong@box.com <mcong@box.com>
Co-Authored-By: mcong@box.com <mcong@box.com>
Co-Authored-By: mcong@box.com <mcong@box.com>
Co-Authored-By: mcong@box.com <mcong@box.com>
…issues

Co-Authored-By: mcong@box.com <mcong@box.com>
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.

3 participants