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

common: skip tests that make rpc calls #1059

Merged
merged 1 commit into from
Dec 12, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -175,7 +175,9 @@ const setup = () => {
}
}

describe('TAP Pagination', () => {
// Skipped because it hits real RPC providers and uses up the API key.
// Skipping it works around this issue for now but we should turn it back on once we have a better solution.
describe.skip('TAP Pagination', () => {
beforeAll(setup, timeout)
test(
'test `getAllocationsfromAllocationIds` pagination',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -505,7 +505,8 @@ describe('TAP', () => {
timeout,
)

test('test `submitRAVs` with escrow account lower on balance', async () => {
// Skipped until we can run with local-network in CI
test.skip('test `submitRAVs` with escrow account lower on balance', async () => {
// mock redeemRav to not call the blockchain
const redeemRavFunc = jest
.spyOn(tapCollector, 'redeemRav')
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,10 @@ const setup = async () => {
}

jest.spyOn(TapCollector.prototype, 'startRAVProcessing').mockImplementation()
describe('Validate TAP queries', () => {

// Skipped because this hits real RPC endpoints.
// This test should be re-enabled when we have a test environment that this can hit instead.
describe.skip('Validate TAP queries', () => {
beforeAll(setup, timeout)

test(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,9 @@ const teardownAll = async () => {
await sequelize.drop({})
}

describe('Allocation Manager', () => {
// These tests are skipped because they hit real RPC providers and use up the API key.
// Skipping them works around this issue for now but we should turn them back on once we have a better solution.
describe.skip('Allocation Manager', () => {
beforeAll(setup)
beforeEach(setupEach)
afterEach(teardownEach)
Expand Down Expand Up @@ -126,7 +128,7 @@ describe('Allocation Manager', () => {
// @ts-ignore: Mocking the Action type for this test
const actions = [queuedAllocateAction, unallocateAction, reallocateAction] as Action[]

test.skip('stakeUsageSummary() correctly calculates token balances for array of actions', async () => {
test('stakeUsageSummary() correctly calculates token balances for array of actions', async () => {
const balances = await Promise.all(
actions.map((action: Action) => allocationManager.stakeUsageSummary(action)),
)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -348,7 +348,8 @@ describe.skip('Monitor: local', () => {
})
})

describe('Monitor: CI', () => {
// Skipped until we can run on a local-network based stack.
describe.skip('Monitor: CI', () => {
beforeAll(setupMonitor)

test('Fetch subgraphs', async () => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -224,7 +224,8 @@ const teardownAll = async () => {
await sequelize.drop({})
}

describe('Actions', () => {
// Skipped until we can run on local-network based stack.
describe.skip('Actions', () => {
jest.setTimeout(60_000)
beforeAll(setup)
beforeEach(setupEach)
Expand Down
Loading