Skip to content

Conversation

@rodrigopavezi
Copy link
Member

@rodrigopavezi rodrigopavezi commented Feb 11, 2026

Description of the changes

Upgraded CircleCI resource classes from large to xlarge for the test-unit and integration test jobs, and added an xlarge resource class to the third test job to improve test execution performance.

Summary by CodeRabbit

  • Chores

    • Increased CI job resource allocation to speed up and stabilize test runs.
  • Tests

    • Made integration tests resilient to external service outages by detecting availability and skipping affected tests and cleanup when unreachable.
    • Increased test timeouts to allow longer-running asynchronous operations and reduce flaky failures.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 11, 2026

Walkthrough

CircleCI job resource allocations increased to xlarge; Lit Protocol integration tests now detect Lit network availability and skip/adjust cleanup when unreachable; several request-client.js tests had Jest timeouts increased to 180000 ms.

Changes

Cohort / File(s) Summary
CircleCI Configuration
.circleci/config.yml
Updated resource_class values: changed test-unit and test-integration-with-request-node from large to xlarge, and added resource_class: xlarge for test-nightly. No workflow/steps logic changed.
Integration Tests (Lit Protocol)
packages/integration-test/test/lit-protocol.test.ts
Added litNetworkAvailable flag and skipIfNoLitNetwork helper; wrapped litProvider.initializeClient() in try/catch, set flag on failure, log warning, early-return/skip tests when network unreachable, and conditionally attempt cleanup in afterAll. No public API changes.
Request-client.js Tests (Jest timeouts)
packages/request-client.js/test/index.test.ts
Increased global Jest timeout and multiple individual test timeouts from shorter values (15–60s) to 180000 ms to allow longer async operations. No test logic changes.

Sequence Diagram(s)

mermaid
sequenceDiagram
autonumber
participant TestRunner as Test Runner
participant LitProvider as Lit Provider Client
participant LitNetwork as Lit Network
rect rgba(100,149,237,0.5)
TestRunner->>LitProvider: initializeClient()
LitProvider->>LitNetwork: connect / handshake
alt connection succeeds
LitNetwork-->>LitProvider: success
LitProvider-->>TestRunner: initialized
TestRunner->>TestRunner: run gated tests (encrypt/decrypt, requests)
TestRunner->>LitProvider: perform cleanup
LitProvider->>LitNetwork: cleanup calls
else connection fails
LitNetwork-->>LitProvider: error/timeout
LitProvider-->>TestRunner: throw / return failure
TestRunner->>TestRunner: set litNetworkAvailable = false
TestRunner->>TestRunner: skip gated tests
TestRunner->>LitProvider: attempt conditional cleanup (no-op if uninitialized)
end
end

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: upgrading CircleCI resource classes to xlarge, which is the primary focus of the .circleci/config.yml changes.
Description check ✅ Passed The description provides clear explanation of the changes, covering upgraded resource classes for test jobs and the objective of improving test execution performance, matching the template requirement.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/ci

No actionable comments were generated in the recent review. 🎉


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 11, 2026

Greptile Overview

Greptile Summary

This PR adjusts CI capacity by switching the resource_class for test-unit and test-integration-with-request-node from large to xlarge, and by explicitly setting resource_class: xlarge on the smart-contracts integration test job as well.

These changes are limited to .circleci/config.yml and should only affect how CircleCI schedules/allocates compute for the existing jobs in the build-and-test workflow; no build/test commands or workflow wiring were modified.

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk.
  • The change is a small, syntactically correct adjustment to CircleCI resource allocation; xlarge is already used elsewhere in the same config (e.g., build/test-nightly), and no commands, images, or workflow dependencies were altered.
  • No files require special attention

Important Files Changed

Filename Overview
.circleci/config.yml Updates CircleCI job resource classes: test-unit and test-integration-with-request-node set to xlarge; test-integration-with-smart-contracts now explicitly uses xlarge. No YAML/schema issues found; xlarge already used elsewhere in this config.

Sequence Diagram

sequenceDiagram
  participant Dev as Developer
  participant GH as GitHub PR
  participant CC as CircleCI Workflow
  participant J1 as test-unit job
  participant J2 as test-integration-with-smart-contracts job
  participant J3 as test-integration-with-request-node job

  Dev->>GH: Merge PR #1698 (config.yml)
  GH-->>CC: Trigger build-and-test workflow
  CC->>J1: Run with resource_class=xlarge
  CC->>J2: Run with resource_class=xlarge
  CC->>J3: Run with resource_class=xlarge
  J1-->>CC: Report test results
  J2-->>CC: Report test results
  J3-->>CC: Report test results
  CC-->>GH: Status checks updated
Loading

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

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.

1 participant