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

Investigate why tests are failing.

    Details

    • Story Points:
      2
    • Sprint:
      Fall 2017

      Description

      When developers build the master branch in Netbeans, multiple tests fail.

      IGB still seems to run just fine, but tests failing is a danger sign.

      We need to determine what's wrong, and fix it.

        Attachments

          Issue Links

            Activity

            Hide
            ieclabau Ivory Blakley (Inactive) added a comment -

            I am still curious... why was this broken?

            I know IGB supports synonyms for chromosome names and I think it is supposed to support "Chr1" and "chr1" for the A. thaliana genome. Might this test have been designed to test that the synonym system is working? And the test failing is indicating that the synonym system is NOT working...? I doesn't say that, but this file is pretty void of comments, so we have to make assumptions about what each test is designed to test for.

            In an older version (8.2.3) the chromosome names that display in the A. thaliana genome are lower cased, "chr1".
            This is odd, given that IGB doesn't store data. So both versions that I have running next to each other (v8.2 and v9.0) should be looking at the same data, because they are looking at the same remote quickload site. But in v8.2 shows "chr1" and v9.0 shows "Chr1".
            The "Chr1" is a more accurate reflection of what is in the genome file. So how does v8.2 have "chr1" ?

            The user's guide talks about making a chromosomes.txt file with synonyms for chromosome names.
            https://wiki.transvar.org/display/igbman/Personal+Synonyms

            I think IGB has a built-in chromosomes.txt file.

            I think there must have been a change in that file or the code that uses it.

            We should test the synonyms functionality.

            The current solution is good. It checks that the expected chromosome names are present (based on the names they have in the sequence file). And making the test pass allows all of the subsequent tests to run.

            Show
            ieclabau Ivory Blakley (Inactive) added a comment - I am still curious... why was this broken? I know IGB supports synonyms for chromosome names and I think it is supposed to support "Chr1" and "chr1" for the A. thaliana genome. Might this test have been designed to test that the synonym system is working? And the test failing is indicating that the synonym system is NOT working...? I doesn't say that, but this file is pretty void of comments, so we have to make assumptions about what each test is designed to test for. In an older version (8.2.3) the chromosome names that display in the A. thaliana genome are lower cased, "chr1". This is odd, given that IGB doesn't store data. So both versions that I have running next to each other (v8.2 and v9.0) should be looking at the same data, because they are looking at the same remote quickload site. But in v8.2 shows "chr1" and v9.0 shows "Chr1". The "Chr1" is a more accurate reflection of what is in the genome file. So how does v8.2 have "chr1" ? The user's guide talks about making a chromosomes.txt file with synonyms for chromosome names. https://wiki.transvar.org/display/igbman/Personal+Synonyms I think IGB has a built-in chromosomes.txt file. I think there must have been a change in that file or the code that uses it. We should test the synonyms functionality. The current solution is good. It checks that the expected chromosome names are present (based on the names they have in the sequence file). And making the test pass allows all of the subsequent tests to run.
            Hide
            ieclabau Ivory Blakley (Inactive) added a comment -

            First level review #1:
            The code looks good, simple, logical.
            The outcome (tests passing) is good.

            I'm kicking this back to To Do for minor changes before going to the second "1st level review"
            -There are no comments around the change. I think we are currently operating on a comment-trigger-happy MO.
            -I've requested changes in the git history (details in email).

            Jenn, consider those changes, talk with Ann if needed. Then move to "1st level review" to get the second review.

            Show
            ieclabau Ivory Blakley (Inactive) added a comment - First level review #1: The code looks good, simple, logical. The outcome (tests passing) is good. I'm kicking this back to To Do for minor changes before going to the second "1st level review" -There are no comments around the change. I think we are currently operating on a comment-trigger-happy MO. -I've requested changes in the git history (details in email). Jenn, consider those changes, talk with Ann if needed. Then move to "1st level review" to get the second review.
            Hide
            sneha Sneha Ramesh Watharkar (Inactive) added a comment -

            Re-worked this branch to exclude a stashed commit. Also added a comment above the changes made to QuickloadDataProviderTest to mark the changes made per this issue.

            This branch is now 2 commits ahead of master. Moving to 'needs review' for the second review.

            Show
            sneha Sneha Ramesh Watharkar (Inactive) added a comment - Re-worked this branch to exclude a stashed commit. Also added a comment above the changes made to QuickloadDataProviderTest to mark the changes made per this issue. This branch is now 2 commits ahead of master. Moving to 'needs review' for the second review.
            Hide
            djoshi4 Deepti Joshi (Inactive) added a comment -

            Done with Second level review.

            Code changes are precise and good. I am now having a successful build.
            ------------------------------------------------------------------------------------------------------------------
            Reactor Summary:

            IGB Project ....................................... SUCCESS [0.252s]
            Heatmap Editor .................................... SUCCESS [8.739s]
            Affymetrix Common ................................. SUCCESS [14.532s]
            external-sort ..................................... SUCCESS [9.999s]
            cache-api ......................................... SUCCESS [7.937s]
            SynonymLookup ..................................... SUCCESS [6.259s]
            genometry ......................................... SUCCESS [13.987s]
            IGB Preferences ................................... SUCCESS [5.502s]
            igbSwingExt ....................................... SUCCESS [2.751s]
            IGB Genoviz Extensions ............................ SUCCESS [3.075s]
            IGB Services ...................................... SUCCESS [1.827s]
            frameManager ...................................... SUCCESS [1.032s]
            Window Service .................................... SUCCESS [1.823s]
            igb-javafx-util ................................... SUCCESS [0.864s]
            Menu API .......................................... SUCCESS [2.804s]
            Context Menu Service .............................. SUCCESS [0.818s]
            igb ............................................... SUCCESS [23.155s]
            shared-lib-wrapper ................................ SUCCESS [14.759s]
            Logging Config .................................... SUCCESS [0.764s]
            WebLinks .......................................... SUCCESS [3.376s]
            cache ............................................. SUCCESS [4.635s]
            Quickload ......................................... SUCCESS [7.306s]
            Das ............................................... SUCCESS [6.862s]
            Das 2 ............................................. SUCCESS [3.751s]
            Apollo Library .................................... SUCCESS [1.813s]
            Track Operations .................................. SUCCESS [1.908s]
            Annotation Tab .................................... SUCCESS [1.175s]
            BigBedHandler ..................................... SUCCESS [0.748s]
            BigWigHandler ..................................... SUCCESS [0.843s]
            Bookmark .......................................... SUCCESS [2.162s]
            External View ..................................... SUCCESS [4.000s]
            Graph Tab ......................................... SUCCESS [1.495s]
            image-exporter .................................... SUCCESS [3.748s]
            IGB Script ........................................ SUCCESS [1.030s]
            KeyWord Search .................................... SUCCESS [1.114s]
            NCBI Blast ........................................ SUCCESS [9.439s]
            Property View ..................................... SUCCESS [1.810s]
            Restriction Sites ................................. SUCCESS [1.228s]
            searchModelIdOrProps .............................. SUCCESS [0.784s]
            searchView ........................................ SUCCESS [1.979s]
            sequenceViewer .................................... SUCCESS [0.926s]
            survey ............................................ SUCCESS [1.163s]
            thresholding ...................................... SUCCESS [1.375s]
            update ............................................ SUCCESS [0.941s]
            Plugin Manager .................................... SUCCESS [3.749s]
            Paired End Data Preprocessor ...................... SUCCESS [0.621s]
            Console Gui ....................................... SUCCESS [1.530s]
            Session Manager ................................... SUCCESS [0.667s]
            Keybinding-registry ............................... SUCCESS [2.119s]
            org.lorainelab.igb.error.reporter.manager ......... SUCCESS [2.009s]
            feedback .......................................... SUCCESS [0.795s]
            narrowpeak ........................................ SUCCESS [1.692s]
            bed ............................................... SUCCESS [6.489s]
            bam ............................................... SUCCESS [5.026s]
            main .............................................. SUCCESS [13.935s]
            ------------------------------------------------------------------------
            BUILD SUCCESS
            ------------------------------------------------------------------------
            Total time: 1:34.606s (Wall Clock)
            Finished at: Wed May 31 12:39:08 EDT 2017
            Final Memory: 54M/796M
            ------------------------------------------------------------------------

            Issue is ready for a pull request.

            Show
            djoshi4 Deepti Joshi (Inactive) added a comment - Done with Second level review. Code changes are precise and good. I am now having a successful build. ------------------------------------------------------------------------------------------------------------------ Reactor Summary: IGB Project ....................................... SUCCESS [0.252s] Heatmap Editor .................................... SUCCESS [8.739s] Affymetrix Common ................................. SUCCESS [14.532s] external-sort ..................................... SUCCESS [9.999s] cache-api ......................................... SUCCESS [7.937s] SynonymLookup ..................................... SUCCESS [6.259s] genometry ......................................... SUCCESS [13.987s] IGB Preferences ................................... SUCCESS [5.502s] igbSwingExt ....................................... SUCCESS [2.751s] IGB Genoviz Extensions ............................ SUCCESS [3.075s] IGB Services ...................................... SUCCESS [1.827s] frameManager ...................................... SUCCESS [1.032s] Window Service .................................... SUCCESS [1.823s] igb-javafx-util ................................... SUCCESS [0.864s] Menu API .......................................... SUCCESS [2.804s] Context Menu Service .............................. SUCCESS [0.818s] igb ............................................... SUCCESS [23.155s] shared-lib-wrapper ................................ SUCCESS [14.759s] Logging Config .................................... SUCCESS [0.764s] WebLinks .......................................... SUCCESS [3.376s] cache ............................................. SUCCESS [4.635s] Quickload ......................................... SUCCESS [7.306s] Das ............................................... SUCCESS [6.862s] Das 2 ............................................. SUCCESS [3.751s] Apollo Library .................................... SUCCESS [1.813s] Track Operations .................................. SUCCESS [1.908s] Annotation Tab .................................... SUCCESS [1.175s] BigBedHandler ..................................... SUCCESS [0.748s] BigWigHandler ..................................... SUCCESS [0.843s] Bookmark .......................................... SUCCESS [2.162s] External View ..................................... SUCCESS [4.000s] Graph Tab ......................................... SUCCESS [1.495s] image-exporter .................................... SUCCESS [3.748s] IGB Script ........................................ SUCCESS [1.030s] KeyWord Search .................................... SUCCESS [1.114s] NCBI Blast ........................................ SUCCESS [9.439s] Property View ..................................... SUCCESS [1.810s] Restriction Sites ................................. SUCCESS [1.228s] searchModelIdOrProps .............................. SUCCESS [0.784s] searchView ........................................ SUCCESS [1.979s] sequenceViewer .................................... SUCCESS [0.926s] survey ............................................ SUCCESS [1.163s] thresholding ...................................... SUCCESS [1.375s] update ............................................ SUCCESS [0.941s] Plugin Manager .................................... SUCCESS [3.749s] Paired End Data Preprocessor ...................... SUCCESS [0.621s] Console Gui ....................................... SUCCESS [1.530s] Session Manager ................................... SUCCESS [0.667s] Keybinding-registry ............................... SUCCESS [2.119s] org.lorainelab.igb.error.reporter.manager ......... SUCCESS [2.009s] feedback .......................................... SUCCESS [0.795s] narrowpeak ........................................ SUCCESS [1.692s] bed ............................................... SUCCESS [6.489s] bam ............................................... SUCCESS [5.026s] main .............................................. SUCCESS [13.935s] ------------------------------------------------------------------------ BUILD SUCCESS ------------------------------------------------------------------------ Total time: 1:34.606s (Wall Clock) Finished at: Wed May 31 12:39:08 EDT 2017 Final Memory: 54M/796M ------------------------------------------------------------------------ Issue is ready for a pull request.
            Hide
            mason Mason Meyer (Inactive) added a comment -

            I tested this issue on the master branch (since the description says that the problem relates to developers running from the master branch) and I tested this issue on the IGBF-1142 branch of Jenny's fork. I realize that the fix for this issue must not be incorporated into the master branch yet, because I am experiencing different behavior on the branches when building from Netbeans.

            On the IGBF-1142 branch, I am able to successfully build IGB from Netbeans without skipping tests. The build is successful on my Windows and Mac. For this reason, I believe this JIRA story to be resolved.

            On the master branch I am running into something odd: The build is successful on my Windows machine but fails on my Mac. I am trying to look into why this is but do not have any answers yet. I have reached out to Jenny to see why this may be, but I don't think any fix for this issue has been merged into the master branch yet, so I still believe this JIRA story to be resolved although I do think the behavior is odd.

            Since the build is successful on the IGBF-1142 branch on both my Windows and Mac, I am marking this issue as resolved. If I gain new information that the fix should have been merged into the master branch already, I will re-open the issue and continue looking into it.

            Show
            mason Mason Meyer (Inactive) added a comment - I tested this issue on the master branch (since the description says that the problem relates to developers running from the master branch) and I tested this issue on the IGBF-1142 branch of Jenny's fork. I realize that the fix for this issue must not be incorporated into the master branch yet, because I am experiencing different behavior on the branches when building from Netbeans. On the IGBF-1142 branch, I am able to successfully build IGB from Netbeans without skipping tests. The build is successful on my Windows and Mac. For this reason, I believe this JIRA story to be resolved. On the master branch I am running into something odd: The build is successful on my Windows machine but fails on my Mac. I am trying to look into why this is but do not have any answers yet. I have reached out to Jenny to see why this may be, but I don't think any fix for this issue has been merged into the master branch yet, so I still believe this JIRA story to be resolved although I do think the behavior is odd. Since the build is successful on the IGBF-1142 branch on both my Windows and Mac, I am marking this issue as resolved. If I gain new information that the fix should have been merged into the master branch already, I will re-open the issue and continue looking into it.

              People

              • Assignee:
                mason Mason Meyer (Inactive)
                Reporter:
                ann.loraine Ann Loraine
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: