Skip to content

Commit

Permalink
Added todos for issues in getters
Browse files Browse the repository at this point in the history
  • Loading branch information
TheCedarPrince committed Aug 10, 2022
1 parent 1ed20e1 commit 6a22ec3
Showing 1 changed file with 16 additions and 0 deletions.
16 changes: 16 additions & 0 deletions test/sqlite/getters.jl
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,13 @@ end
@test isequal(minuend_2_test, GetPatientAgeGroup(test_ids, sqlite_conn; minuend = minuend_2, age_groupings = test_age_grouping_2))
end

#= TODO: Create tests for GetPatientVisits
Reference the test set for GetPatientRace. There should be at least one test for a person who has only 1 visit and another that has multiple visits
labels: tests
assignees: malinahy, jomoanime, VarshC
=#
# @testset "GetPatientVisits Tests" begin

# end

# @testset "GetMostRecentConditions Tests" begin
Expand All @@ -76,13 +82,23 @@ end
# @test # Multiple ids
# end

#= TODO: Create tests for GetMostRecentVisit
Reference the test set for GetPatientVisits when done. There should be at least one test for a person who has only 1 visit and another that has multiple visits. Confirm using SQL or FunSQL to determine accuracy
labels: tests, good first issue
assignees: malinahy, jomoanime, VarshC
=#
# @testset "GetMostRecentVisit Tests" begin
# test_ids =
#
#
# @test # Multiple ids
# end

#= TODO: Create tests for GetVisitCondition
Reference the test set for GetPatientVisits when done. Two tests for about three different codes works well and one test for multiple visit codes is great
labels: tests, good first issue
assignees: malinahy, jomoanime, VarshC
=#
# @testset "GetVisitCondition Tests" begin
# test_ids =
#
Expand Down

0 comments on commit 6a22ec3

Please sign in to comment.