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

Add unit tests for pod/pod_spec/pod_temp_spec #328

Merged
merged 3 commits into from
Oct 4, 2023

Conversation

ZichengMa
Copy link
Contributor

In the PodSpec's tests, the Volume API is used. The unit tests for Volume will be in next PR. There are many different APIs should be tested for Volume, so I split those tests to next PR.

#[verifier(external_body)]
pub fn clone(&self) -> (volume: Volume)
ensures
volume@ == volume@,
Copy link
Collaborator

Choose a reason for hiding this comment

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

postcondition is wrong

@marshtompsxd marshtompsxd changed the title Finish unit-tests for pod/pod_spec/pod_temp_spec Add unit-tests for pod/pod_spec/pod_temp_spec Oct 2, 2023
@marshtompsxd marshtompsxd changed the title Add unit-tests for pod/pod_spec/pod_temp_spec Add unit tests for pod/pod_spec/pod_temp_spec Oct 2, 2023
@@ -13,3 +13,6 @@ pub mod probe;
pub mod resource_requirements;
pub mod tcp_socket_action;
pub mod volume_mount;
pub mod pod;
pub mod pod_spec;
pub mod pod_template_spec;
Copy link
Collaborator

Choose a reason for hiding this comment

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

You can add a new empty line at the end of file

Signed-off-by: ZichengMa <[email protected]>
@marshtompsxd marshtompsxd added this pull request to the merge queue Oct 4, 2023
Merged via the queue into main with commit 906d44b Oct 4, 2023
@marshtompsxd marshtompsxd deleted the unit-tests-with-cargo-test-pod branch October 11, 2023 21:04
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.

2 participants