Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Addition of Unit Testing for all projects #30

Open
kMutagene opened this issue Nov 8, 2018 · 1 comment
Open

Addition of Unit Testing for all projects #30

kMutagene opened this issue Nov 8, 2018 · 1 comment

Comments

@kMutagene
Copy link
Member

kMutagene commented Nov 8, 2018

Description

None of the projects currently contain any unit tests. There are placeholder tests under the BioFSharp.Tests.NetCore project, which use Expecto.

How to design tests:
Basically, you can do what you want, as long as the test is as granular and self-contained as possible. Some inspiration can be taken from the current tests or this blog post:
https://stackify.com/unit-testing-basics-best-practices/ (ignore the C# code and setup instructions and focus on the concepts)

How to add tests:

How to run tests(from project root):

  • Windows: ./build.cmd -t runTestsAll
  • Linux: ./build.sh -t runTestsDotnet

Starting point resources

Expecto docs
This well written blog post

@kMutagene
Copy link
Member Author

Example commit for addition of tests: 456b000

ZimmerD added a commit that referenced this issue Aug 1, 2019
ZimmerD added a commit that referenced this issue Aug 1, 2019
ZimmerD added a commit that referenced this issue Aug 1, 2019
ZimmerD added a commit that referenced this issue Aug 2, 2019
ZimmerD added a commit that referenced this issue Aug 2, 2019
ZimmerD added a commit that referenced this issue Aug 2, 2019
ZimmerD added a commit that referenced this issue Aug 5, 2019
#30: add unit testing IsotopicDistribution.fs
@kMutagene kMutagene added this to the Backlog milestone Feb 19, 2020
ZimmerD added a commit to ZimmerD/BioFSharp that referenced this issue Jun 26, 2020
This was referenced Jul 17, 2020
kMutagene pushed a commit that referenced this issue Jul 21, 2020
* Add testCase "isEqual" to BioList testList

* Add testCase "toString" to testList "BioList"

* Fix "toString" and add "toMonoisotopicMass" test

* Add testCase "toAverageMass"

* Add "toMonoisotopicMassWith"

* Add testCase "toAverageMassWith"

* Add testCase "toCompositionVector"

* Fix error message for "toCompositionVector"

* Change "BioList.ofBioArray" to "List.ofArray"
@kMutagene kMutagene mentioned this issue Jul 29, 2020
1 task
kMutagene pushed a commit that referenced this issue Jul 29, 2020
* Add testCase "isEqual" to BioList testList

* Add testCase "toString" to testList "BioList"

* Fix "toString" and add "toMonoisotopicMass" test

* Add testCase "toAverageMass"

* Add "toMonoisotopicMassWith"

* Add testCase "toAverageMassWith"

* Add testCase "toCompositionVector"

* Fix error message for "toCompositionVector"

* Change "BioList.ofBioArray" to "List.ofArray"

* Change BioSeq.ofArray to Seq.ofArray

* Add "isEqual" to testList "BioSeq"

* Add testCase "toString"

* Add testCase "toMonoisotopicMass"

* Add testCase "toAverageMass"

* Add testCase "toMonoisotopicMassWith"

* Add testCase "toAverageMassWith"

* Add testList "toCompositionVector"
kMutagene pushed a commit that referenced this issue Aug 7, 2020
* Add "teset_rangePpm" to testList "BioFSharp.Mass"

* Add testCase "formula" to testList "AminoAcids"

* Add "formulaModifiedAA" to testList "AminoAcids"

* Fix "formulaModifiedAA" testCase

* Add "isTerminator" to testList "AminoAcids"

* Add "isGap" to testList "AminoAcids"

* Add "setModification" to testList "AminoAcids"

* Clean up tests

* Change error message for test "setModification"

* Add "setModifications" to "AminoAcids" testList

* Add "getModifications" to "AminoAcids" testList
kMutagene added a commit that referenced this issue Aug 21, 2020
* Add "teset_rangePpm" to testList "BioFSharp.Mass"

* Add testCase "formula" to testList "AminoAcids"

* Add "formulaModifiedAA" to testList "AminoAcids"

* Fix "formulaModifiedAA" testCase

* Add "isTerminator" to testList "AminoAcids"

* Add "isGap" to testList "AminoAcids"

* Add "setModification" to testList "AminoAcids"

* Clean up tests

* Change error message for test "setModification"

* Add "setModifications" to "AminoAcids" testList

* Add "getModifications" to "AminoAcids" testList

* Add testCase "getAminoAcidWithoutMod"

* Add test "tryGetModifications"

* Add test "tryGetModifications_withNone"

* Add test "isotopicLabelFunc"

* Add "isotopicLabelFunc_withUnmodifiedAAs"

* Add "charToParsedAminoAcidChar"

* Add test "aminoAcidSetStandard"

* Add test "aminoAcidSetProteinogenic"

* Add test "aminoAcidSetProteinogenicEucaryotes"

* Add test "aminoAcidSetAmbiguity"

* Add test "aminoAcidSetGapTer"

* Add test "aminoAcidSetPosCharged"

* Add test "aminoAcidSetNegCharged"

* Add test "aminoAcidSetPolarUncharged"

* Add test "aminoAcidSetHydrophobic"

* Add test "aminoAcidSetSpecialCases"

* Add test "isPolarUncharged"

* Add test "isModified_withUnmodAAs"

* Add test "isModified_withModAAs"

* Add test "getModifications_withUnmodAAs"

* Delete coverage.xml.637327725072680050L.exn

Seems like these are artifacts from your testing process, i cant reproduce these when starting the tests from your branch, so i would delete these and merge the PR afterwards

* Delete coverage.xml.637330276045185520L.exn

see previous commit

* Fix merge commit for aa tests

Co-authored-by: Kevin Schneider <[email protected]>
Co-authored-by: Kevin Schneider <[email protected]>
kMutagene pushed a commit that referenced this issue Aug 26, 2020
* Add testCase "symbol"

* Add test "formula"

* Add test "isTerminator"

* Add test "isTerminator"

* Add test "isGap"

* Add test "name"

* Add NucleotideTests.fs to Main. Fix errors

* Add test "charToParsedNucleotideChar"

* Add test "complement"

* Add test "replaceTbyU"

* Add test "replaceUbyT"
graemevissers added a commit to graemevissers/BioFSharp that referenced this issue Aug 26, 2020
kMutagene added a commit that referenced this issue Sep 9, 2020
* Add test "countLeafs"

* Add test "map"

* Add test "iter"

* Add test "fold"

* Add test "tryGetNodeBy"

* Delete coverage.xml.637344003428108790L.exn

Co-authored-by: Kevin Schneider <[email protected]>
HLWeil pushed a commit to HLWeil/BioFSharp that referenced this issue Sep 21, 2020
* Add "teset_rangePpm" to testList "BioFSharp.Mass"

* Add testCase "formula" to testList "AminoAcids"

* Add "formulaModifiedAA" to testList "AminoAcids"

* Fix "formulaModifiedAA" testCase

* Add "isTerminator" to testList "AminoAcids"

* Add "isGap" to testList "AminoAcids"

* Add "setModification" to testList "AminoAcids"

* Clean up tests

* Change error message for test "setModification"

* Add "setModifications" to "AminoAcids" testList

* Add "getModifications" to "AminoAcids" testList
HLWeil pushed a commit to HLWeil/BioFSharp that referenced this issue Sep 21, 2020
* Add "teset_rangePpm" to testList "BioFSharp.Mass"

* Add testCase "formula" to testList "AminoAcids"

* Add "formulaModifiedAA" to testList "AminoAcids"

* Fix "formulaModifiedAA" testCase

* Add "isTerminator" to testList "AminoAcids"

* Add "isGap" to testList "AminoAcids"

* Add "setModification" to testList "AminoAcids"

* Clean up tests

* Change error message for test "setModification"

* Add "setModifications" to "AminoAcids" testList

* Add "getModifications" to "AminoAcids" testList

* Add testCase "getAminoAcidWithoutMod"

* Add test "tryGetModifications"

* Add test "tryGetModifications_withNone"

* Add test "isotopicLabelFunc"

* Add "isotopicLabelFunc_withUnmodifiedAAs"

* Add "charToParsedAminoAcidChar"

* Add test "aminoAcidSetStandard"

* Add test "aminoAcidSetProteinogenic"

* Add test "aminoAcidSetProteinogenicEucaryotes"

* Add test "aminoAcidSetAmbiguity"

* Add test "aminoAcidSetGapTer"

* Add test "aminoAcidSetPosCharged"

* Add test "aminoAcidSetNegCharged"

* Add test "aminoAcidSetPolarUncharged"

* Add test "aminoAcidSetHydrophobic"

* Add test "aminoAcidSetSpecialCases"

* Add test "isPolarUncharged"

* Add test "isModified_withUnmodAAs"

* Add test "isModified_withModAAs"

* Add test "getModifications_withUnmodAAs"

* Delete coverage.xml.637327725072680050L.exn

Seems like these are artifacts from your testing process, i cant reproduce these when starting the tests from your branch, so i would delete these and merge the PR afterwards

* Delete coverage.xml.637330276045185520L.exn

see previous commit

* Fix merge commit for aa tests

Co-authored-by: Kevin Schneider <[email protected]>
Co-authored-by: Kevin Schneider <[email protected]>
HLWeil pushed a commit to HLWeil/BioFSharp that referenced this issue Sep 21, 2020
* Add testCase "symbol"

* Add test "formula"

* Add test "isTerminator"

* Add test "isTerminator"

* Add test "isGap"

* Add test "name"

* Add NucleotideTests.fs to Main. Fix errors

* Add test "charToParsedNucleotideChar"

* Add test "complement"

* Add test "replaceTbyU"

* Add test "replaceUbyT"
HLWeil pushed a commit to HLWeil/BioFSharp that referenced this issue Sep 21, 2020
* Add test "countLeafs"

* Add test "map"

* Add test "iter"

* Add test "fold"

* Add test "tryGetNodeBy"

* Delete coverage.xml.637344003428108790L.exn

Co-authored-by: Kevin Schneider <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant