Skip to content

Conversation

@MiguelLZPF
Copy link
Contributor

@MiguelLZPF MiguelLZPF commented Jan 29, 2026

Description

Reorganized contract test suite structure to better reflect test categorization. The test directory test/contracts/unit has been renamed to test/contracts/integration to accurately represent the nature of these tests.

This improves code organization and makes the test structure clearer for developers.

Fixes #BBND-1347

Type of change

  • Refactor 🔧

Testing

The existing test suite has been executed to verify that all tests pass after the reorganization.

  • All existing tests pass
  • No functionality changes, only directory structure reorganization

Test Results

All contract tests pass with the new directory structure.

Node version:

  • 24

Checklist

  • Style Guidelines followed ✅
  • Documentation Updated 📚
  • Linters - No New Warnings ⚠️
  • Local Tests Pass ✅
  • No reduction of Coverage

Rename test/contracts/unit/ to test/contracts/integration/ to better reflect
test classification. These tests deploy real contracts to Hardhat's EVM and test
contract interactions, making them integration tests rather than true unit tests.

Updated all related package.json test scripts to reference the new integration
path for consistency across the codebase.

Changes:
- Moved 52 test files from unit/ to integration/ directory
- Updated 8 npm test script references in package.json
- Maintains all test functionality and import paths

Signed-off-by: Miguel_LZPF <[email protected]>
@MiguelLZPF MiguelLZPF requested review from a team as code owners January 29, 2026 15:06
@MiguelLZPF MiguelLZPF requested a review from kpachhai January 29, 2026 15:06
@hedera-eng-infrastructure
Copy link

hedera-eng-infrastructure commented Jan 29, 2026

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@MiguelLZPF MiguelLZPF self-assigned this Jan 29, 2026
@MiguelLZPF MiguelLZPF added the Contracts Changes related to the “contracts” module label Jan 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Contracts Changes related to the “contracts” module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants