Skip to content
This repository was archived by the owner on Jan 22, 2025. It is now read-only.

Fix check failure in test_process_instruction_compute_budget#22202

Merged
yhchiang-sol merged 1 commit intosolana-labs:masterfrom
yhchiang-sol:fix-test
Dec 31, 2021
Merged

Fix check failure in test_process_instruction_compute_budget#22202
yhchiang-sol merged 1 commit intosolana-labs:masterfrom
yhchiang-sol:fix-test

Conversation

@yhchiang-sol
Copy link
Copy Markdown
Contributor

Problem

Looks like the master is failing the follow check in test_process_instruction_compute_budget:

error[E0560]: struct `solana_sdk::transaction_context::InstructionAccount` has no field named `index`
--
  | --> program-runtime/src/invoke_context.rs:1677:17
  | \|
  | 1677 \|                 index: account_index,
  | \|                 ^^^^^ `solana_sdk::transaction_context::InstructionAccount` does not have this field
  | \|
  | = note: available fields are: `index_in_transaction`, `index_in_caller`, `is_signer`, `is_writable`

Summary of Changes

This PR fixes the check failure by assigning account_index to the right field.

Copy link
Copy Markdown
Contributor

@carllin carllin left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!

@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 31, 2021

Codecov Report

Merging #22202 (c80c851) into master (e1a0660) will decrease coverage by 0.1%.
The diff coverage is 61.1%.

@@            Coverage Diff            @@
##           master   #22202     +/-   ##
=========================================
- Coverage    81.2%    81.1%   -0.2%     
=========================================
  Files         520      523      +3     
  Lines      145949   146609    +660     
=========================================
+ Hits       118534   118910    +376     
- Misses      27415    27699    +284     

@yhchiang-sol yhchiang-sol merged commit 94a9b71 into solana-labs:master Dec 31, 2021
@Lichtso Lichtso mentioned this pull request Dec 31, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants