Merge pull request #97 from alex/signing-coverage

Improved coverage with signing

tests/test_signing.py

@@ -47,10 +47,17 @@ def ed25519_known_answers():
 
 
 class TestSigningKey:
-
     def test_initialize_with_generate(self):
         nacl.signing.SigningKey.generate()
 
+    def test_wrong_length(self):
+        with pytest.raises(ValueError):
+            nacl.signing.SigningKey(b"")
+
+    def test_bytes(self):
+        k = nacl.signing.SigningKey(b"\x00" * nacl.c.crypto_sign_SEEDBYTES)
+        assert bytes(k) == b"\x00" * nacl.c.crypto_sign_SEEDBYTES
+
     @pytest.mark.parametrize("seed", [
         b"77076d0a7318a57d3c16c17251b26645df4c2f87ebc0992ab177fba51db92c2a",
     ])
@@ -80,6 +87,13 @@ def test_message_signing(self, seed, message, signature, expected):
 
 
 class TestVerifyKey:
+    def test_wrong_length(self):
+        with pytest.raises(ValueError):
+            nacl.signing.VerifyKey(b"")
+
+    def test_bytes(self):
+        k = nacl.signing.VerifyKey(b"\x00" * nacl.c.crypto_sign_PUBLICKEYBYTES)
+        assert bytes(k) == b"\x00" * nacl.c.crypto_sign_PUBLICKEYBYTES
 
     @pytest.mark.parametrize(
         ("public_key", "signed", "message", "signature"),