Uploaded image for project: 'IGB'
  1. IGB
  2. IGBF-2252

Implement TwentyThreeAndMeSymLoader using test-first programming

    Details

    • Type: Task
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None
    • Story Points:
      3
    • Sprint:
      Spring 3 : 17 Feb to 28 Feb, Spring 4 : 2 Mar to 13 Mar, Spring 5 : 16 Mar to 27 Mar, Spring 6 : 30 Mar to Apr 10

      Description

      Implement methods in genotyping SymLoader.
      Write javadoc comments and check format by compiling javadocs.

      • First, create unit tests for methods (may require some re-factoring)
      • Start implementing methods, updating and adding unit tests as needed
      • Create test files as needed using example file as a start.
      • Note: example file is a "real" file typical of what the SymLoader will encounter in the wild. Create simpler examples files to test specific things, e.g., format flaws.
      • Note: will likely need to mock some of the classes, notably GenomeVersion. Look through other testing code for ideas on how to do this.

        Attachments

          Issue Links

            Activity

            Hide
            stiwari8 Srishti Tiwari (Inactive) added a comment - - edited

            Hi Dr. Loraine,

            As the changes were related, the changes in branch IGBF-2253 contain changes for IGBF-2252 as well. Kindly review:

            https://bitbucket.org/srishtitiwari/integrated-genome-browser/commits/36a86308d9f7e21036a2a50b42383f7538e54a52

            Show
            stiwari8 Srishti Tiwari (Inactive) added a comment - - edited Hi Dr. Loraine, As the changes were related, the changes in branch IGBF-2253 contain changes for IGBF-2252 as well. Kindly review: https://bitbucket.org/srishtitiwari/integrated-genome-browser/commits/36a86308d9f7e21036a2a50b42383f7538e54a52
            Hide
            ann.loraine Ann Loraine added a comment -

            PR is merged. Moving to Done.

            Show
            ann.loraine Ann Loraine added a comment - PR is merged. Moving to Done.

              People

              • Assignee:
                Unassigned
                Reporter:
                ann.loraine Ann Loraine
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: