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

Fix AppendVec test breakage... #7693

Merged
merged 1 commit into from
Jan 7, 2020

Conversation

ryoqun
Copy link
Contributor

@ryoqun ryoqun commented Jan 6, 2020

#7373 started to break master branch's CI.. This is caused by incompatible changes introduced by my parallel PR #7464. They are met with each other for the first time on the master branch..

@@ -680,18 +681,6 @@ pub mod tests {
);
}

#[test]
fn test_set_file_empty_data() {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@@ -567,6 +567,7 @@ pub mod tests {
let file = get_append_vec_path("test_append_vec_set_file_bad_size");
let path = &file.path;
let mut av = AppendVec::new_empty_map(0);
assert_eq!(av.accounts(0).len(), 0);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ryoqun
Copy link
Contributor Author

ryoqun commented Jan 7, 2020

What... never seen ci error...:

   Compiling solana-gossip v0.23.0 (/solana/gossip)
   Compiling solana-validator v0.23.0 (/solana/validator)
   Compiling solana-bench-streamer v0.23.0 (/solana/bench-streamer)
error: Failed to delete invalidated or incompatible incremental compilation session directory contents `/solana/target/debug/incremental/solana_bench_tps-yafnuyc7rtvm/s-fjhaf3fvuq-1md9dn4-working/dep-graph.bin`: No such file or directory (os error 2).
 
error: aborting due to previous error
 
error: could not compile `solana-bench-tps`.
warning: build failed, waiting for other jobs to finish...
error: build failed
🚨 Error: The command exited with status 101





@ryoqun
Copy link
Contributor Author

ryoqun commented Jan 7, 2020

Although stable failed for mysterious and unrelated reason, coverage build succeeded. :) I'm merging this.

@ryoqun ryoqun merged commit e5425d4 into solana-labs:master Jan 7, 2020
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.

None yet

1 participant