Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Remoun Metyas <[email protected]>
  • Loading branch information
nick-mobilecoin and remoun authored May 19, 2022
1 parent 856b248 commit 3d93705
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions untrusted/urts/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ use mc_sgx_urts_sys::{
use std::ops::Deref;
use std::{os::raw::c_int, ptr};

#[derive(PartialEq, Debug)]
#[derive(Debug, PartialEq)]
pub enum Error {
// An error provided from the SGX SDK
SgxStatus(sgx_status_t),
Expand Down Expand Up @@ -160,6 +160,12 @@ mod tests {
assert!(builder.create().is_ok());
}

#[test]
fn create_enclave_builder_from_vector() {
let vector = ENCLAVE.to_vec();
assert!(EnclaveBuilder::from(vector).create().is_ok());
}

#[test]
fn calling_into_an_enclave_function_provides_valid_results() {
// Note: the `debug()` was added to ensure proper builder behavior of
Expand All @@ -173,12 +179,6 @@ mod tests {
assert_eq!(sum, 3 + 2);
}

#[test]
fn create_enclave_builder_from_vector() {
let vector = ENCLAVE.to_vec();
assert!(EnclaveBuilder::from(vector).create().is_ok());
}

#[test]
fn default_debug_flag_is_0() {
// For the debug flag it's not easy, in a unit test, to test it was
Expand Down

0 comments on commit 3d93705

Please sign in to comment.