//TODO
-Handle errors more ergonimically 

- log integration

-look into memory allocation and ownership to allow explixit control over secret data allocations and zero out after use

-MORE TESTS!!! Use NIST TESTS! Maybe create seperate crate just for heavy testing of project or set flags for testing depth espisally author tests are CPU intensive 

-Look into parralizing some tests for CI 

-change verify32 to boolean return

-there are some error cases that should be handled ergonimically 

-shared verify16 and verify32 tests 
https://github.com/maidsafe/rust_sodium/blob/master/src/crypto/verify.rs

-scalarbase and scalarmult tests are needed!!!
https://github.com/maidsafe/rust_sodium/blob/master/src/crypto/scalarmult/curve25519.rs

// //https://github.com/maidsafe/rust_sodium/blob/master/src/crypto/scalarmult/curve25519.rs
// #[cfg(test)]
// fn test_scalarmult_1() {
//    // corresponding to tests/scalarmult5.c and tests/scalarmult7.cpp from NaCl
//     let alicesk = [0x77, 0x07, 0x6d, 0x0a, 0x73, 0x18, 0xa5, 0x7d, 0x3c, 0x16, 0xc1,
//                               0x72, 0x51, 0xb2, 0x66, 0x45, 0xdf, 0x4c, 0x2f, 0x87, 0xeb, 0xc0,
//                               0x99, 0x2a, 0xb1, 0x77, 0xfb, 0xa5, 0x1d, 0xb9, 0x2c, 0x2a];
//     let bobpk = [0xde, 0x9e, 0xdb, 0x7d, 0x7b, 0x7d, 0xc1, 0xb4, 0xd3, 0x5b,
//                                   0x61, 0xc2, 0xec, 0xe4, 0x35, 0x37, 0x3f, 0x83, 0x43, 0xc8,
//                                   0x5b, 0x78, 0x67, 0x4d, 0xad, 0xfc, 0x7e, 0x14, 0x6f, 0x88,
//                                   0x2b, 0x4f];
//     let k_expected = [0x4a, 0x5d, 0x9d, 0x5b, 0xa4, 0xce, 0x2d, 0xe1, 0x72, 0x8e, 0x3b, 0xf4,
//                           0x80, 0x35, 0x0f, 0x25, 0xe0, 0x7e, 0x21, 0xc9, 0x47, 0xd1, 0x9e, 0x33,
//                           0x76, 0xf0, 0x9b, 0x3c, 0x1e, 0x16, 0x17, 0x42];
                          
//     let mut k = [0;PUBLIC_KEY_BYTES];
//     scalarmult(&mut k, &alicesk, &bobpk);
//     assert!(k == k_expected);
// }



