-
Notifications
You must be signed in to change notification settings - Fork 16
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
Tests for DA job #61
Tests for DA job #61
Conversation
Co-authored-by: Apoorv Sadana <95699312+apoorvsadana@users.noreply.github.com>
Co-authored-by: Apoorv Sadana <95699312+apoorvsadana@users.noreply.github.com>
Co-authored-by: Apoorv Sadana <95699312+apoorvsadana@users.noreply.github.com>
* update: added thiserror JobError * update: introducing DaError * update: add ProverJobError * update: Register proof JobError * update: SNOS proof JobError * update: JobError on Queue * update: added fix for borrow in state update process job * update: cleaning rework for JobError * updaet: wrap_err_with -> wrap_err wherever used with String * update: moved all JobErrors to top of file * update: Errors for comsume_job_from_queue * update: linting fixes * update: optimised consume_job_from_queue match statement * update: code optimisation based on PR reviews
#[rstest] | ||
#[tokio::test] | ||
async fn test_put_and_get_data_s3() -> color_eyre::Result<()> { | ||
TestConfigBuilder::new().build().await; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can be moved into a fixture
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TestConfigBuilder
allows for customisation over any external client
of our choice,
moving it to a global fixture is not feasible since all test cases have different customisation requirements.
We can make fixture for tests under same scope if they require same customised external clients.
for eg :
All tests under da_job
if require same config customisation, can implement a fixture
just for themselves.
similarly for other scopes .
We can create a separate issue for this and resolve there.
IMPORTANT
[This branch is a fork from increasing coverage ]
[This comment refers to changed already merged to main, they are shown here since #52 is merged and the base of this PR automatically shifted to main
]
Closing in favor of #74 |
This PR introduces tests for DA Job.
Pre-Requisites :
Integration Tests :
(No tests needed for
create_job
andverify_job
)test_da_job_process_job_failure_on_impossible_blob_length
test_da_job_process_job_failure_on_pending_block
test_da_job_process_job_success
Unit Tests :
test_da_word
test_state_update_to_blob_data
test_fft_transformation
test_bincode
Build_Config :
Link to File
S3
initiation in init storage section.madara_rpc_url
tolocalhost:{server.port}
since ServerMock doesn't work with predefined url.