More consistently throw EncodeError

src/alphabet.jl

@@ -79,14 +79,31 @@ iscomplete(A::Alphabet) = Val(length(symbols(A)) === 1 << bits_per_symbol(A))
 ## Encoders & Decoders
 
 """
-    encode(::Alphabet, x::S)
+    encode(::Alphabet, s::BioSymbol)
 
-
-Encode BioSymbol `S` to an internal representation using an `Alphabet`.
+Internal function, do not use in user code.
+Encode BioSymbol `s` to an internal representation using an [`Alphabet`](@ref).
 This decoding is checked to enforce valid data element.
+If `s` cannot be encoded to the given alphabet, throw an `EncodeError`
+
 """
-function encode end
+encode(A::Alphabet, s::BioSymbol) = throw(EncodeError(A, s))
 
+"""
+    EncodeError
+
+Exception thrown when a `BioSymbol` cannot be encoded to a given [`Alphabet`](@ref).
+
+# Examples
+```
+julia> try
+           BioSequences.encode(DNAAlphabet{2}(), DNA_N)
+       catch err
+           println(err isa BioSequences.EncodeError)
+       end
+true
+```
+"""
 struct EncodeError{A<:Alphabet,T} <: Exception
     val::T
 end

test/alphabet.jl

@@ -48,6 +48,16 @@ encode = BioSequences.encode
 EncodeError = BioSequences.EncodeError
 decode = BioSequences.decode
 
+@testset "EncodeError" begin
+    @test_throws EncodeError encode(DNAAlphabet{4}(), RNA_U)
+    @test_throws EncodeError encode(DNAAlphabet{2}(), DNA_M)
+    @test_throws EncodeError encode(DNAAlphabet{4}(), AA_C)
+    @test_throws EncodeError encode(AminoAcidAlphabet(), DNA_C)
+    @test_throws EncodeError encode(AminoAcidAlphabet(), RNA_N)
+    @test_throws EncodeError encode(RNAAlphabet{2}(), DNA_C)
+    @test_throws EncodeError encode(RNAAlphabet{2}(), RNA_K)
+end
+
 # NOTE: See the docs for the interface of Alphabet
 struct ReducedAAAlphabet <: Alphabet end
 
@@ -185,4 +195,4 @@ end
     @test BioSequences.has_interface(Alphabet, RNAAlphabet{2}())
     @test BioSequences.has_interface(Alphabet, RNAAlphabet{4}())
     @test BioSequences.has_interface(Alphabet, AminoAcidAlphabet())
-end
+end

test/runtests.jl

@@ -6,7 +6,7 @@ using Documenter
 using Random
 using StableRNGs
 using LinearAlgebra: normalize
-import BioSymbols
+using BioSymbols
 using BioSequences
 using StatsBase
 using YAML