fix and test forgotten objective_coefficient conversion

Project.toml

@@ -1,7 +1,7 @@
 name = "AbstractFBCModels"
 uuid = "5a4f3dfa-1789-40f8-8221-69268c29937c"
 authors = ["Authors of AbstractFBCModels.jl"]
-version = "0.2.0"
+version = "0.2.1"
 
 [deps]
 DocStringExtensions = "ffbed154-4ef7-542d-bbb7-c09d3a79fcae"

docs/src/canonical.jl

@@ -75,6 +75,7 @@ m.reactions["forward"] = Reaction(
     name = "import",
     stoichiometry = Dict("m1" => -1.0, "m2" => 1.0),
     gene_association_dnf = [["g1"], ["g2"]],
+    objective_coefficient = 1.0,
 )
 m.reactions["and_back"] =
     Reaction(name = "export", stoichiometry = Dict("m2" => -1.0, "m1" => 1.0))
@@ -98,10 +99,15 @@ show_contains(x, y) = contains(sprint(show, MIME"text/plain"(), x), y) #src
 @test show_contains(m.reactions["forward"], "\"g1\"") #src
 @test show_contains(m.metabolites["m1"], "\"inside\"") #src
 @test show_contains(m.genes["g1"], "name = nothing") #src
+@test show_contains(m.genes["g1"], "name = nothing") #src
 
 # We should immediately find the basic accessors working:
 A.stoichiometry(m)
 
+#
+
+A.objective(m)
+
 # We can check various side things, such as which reactions would and would not work given all gene products disappear:
 products_available = [
     A.reaction_gene_products_available(m, rid, _ -> false) for

src/canonical.jl

@@ -139,7 +139,7 @@ A.metabolite_formula(m::Model, id::String) = m.metabolites[id].formula
 A.metabolite_charge(m::Model, id::String) = m.metabolites[id].charge
 A.metabolite_compartment(m::Model, id::String) = m.metabolites[id].compartment
 
-A.load(::Type{Model}, path::String)::Model = S.deserialize(path)
+A.load(::Type{Model}, path::String) = S.deserialize(path)
 A.save(m::Model, path::String) = S.serialize(path, m)
 A.filename_extensions(::Type{Model}) = ["canonical-serialized-fbc"]
 
@@ -155,10 +155,11 @@ function Base.convert(::Type{Model}, x::A.AbstractFBCModel)
                 lower_bound = lb,
                 upper_bound = ub,
                 stoichiometry = A.reaction_stoichiometry(x, r),
+                objective_coefficient = o,
                 gene_association_dnf = A.reaction_gene_association_dnf(x, r),
                 annotations = A.reaction_annotations(x, r),
                 notes = A.reaction_notes(x, r),
-            ) for (r, lb, ub) in zip(A.reactions(x), lbs, ubs)
+            ) for (r, o, lb, ub) in zip(A.reactions(x), os, lbs, ubs)
         ),
         metabolites = Dict(
             m => Metabolite(

src/testing.jl

@@ -116,6 +116,9 @@ function run_fbcmodel_file_tests(
             @test issetequal(mets, metabolites(m))
             @test issetequal(gens, genes(m))
 
+            @test Dict(rxns .=> collect(obj)) ==
+                  Dict(reactions(m) .=> collect(objective(m)))
+
             # TODO eventually add stricter equality tests
         end