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

Remove unneeded logging and println statements from tests

        Details

        • Type: Improvement
        • Status: Closed (View Workflow)
        • Priority: Major
        • Resolution: Done
        • Affects Version/s: None
        • Fix Version/s: None
        • Labels:

          Description

          Many of IGB's test cases are printing unnecessary stuff to console.

          This is unnecessary. If a test fails, we will notice. If it passes, we don't want to hear about it.

          Here is an example of gratuitous printing of stuff to the console:

          -------------------------------------------------------
          T E S T S
          -------------------------------------------------------
          Running apollo.analysis.RemoteBlastNCBITest
          GAAGTTTCCTTATCATCATTAATCATGCCTATGAAAACCCTAACCGGCCT
          https://blast.ncbi.nlm.nih.gov/Blast.cgi?RID=NJHUT165015&CMD=Get
          RID=NJHUT165015
          GAAGTTTCCTTATCATCATTAATCATGCCTATGAAAACCCTAACCGGCCTACATATTTGCAGTCTCTTTTAGGTCAGGAATCTAATGAGAATGAATTATTTTATTTTATTTAACAATAATTGGAGCATGTCATTAATTCTCTTTATTCTTCTTACACAACTAATCATTTAGATGTGTTACAATATTATTTCCTTTAGTCATTTTCATAATTTTAATACCTCCGTACTTTTCACTAATACCTCCCCTTTTAATTTTCATTATTTCTTCTTTTCTATCAGTCTATGCATGCATTCTTTTGAATATTAAAATGCATTTTATATTCTTTTGACAACTATGCACAAGCCTTTTGAGACACATCTACACAATATAATAGCACAAGCCTTTATGAGACATATCTACACAATATCATTGCTACTTGTAGACTATTTGGAATACGTATTTACATATTCCATTGTATCATCCCTTTGCAAATGTTTTATACATATAACTAACATATACATTATTCGTAACTTTGTTCCCCTATATCGAAAAATGTGGGCTACACATATAACTAACATATATATATATATGTATGTTTATATGATATAGTCTCCATGTCTATATATCTTATATATTACATGTTTCATGTTTACGGTCAAGGGAGTATTTTTATACGCATACACAATCATACACACTTAACCCTACTTATAATGATGTAGGTTCATATATTTATCTTATTTTAGGATCATTCGATCACAAATTATACGGACCCTCATACTCTCTAAAGATATACAAAATCCGCTATGTCATATCCGATCCGAATTAGCAGCTAAAGAAAAACAAACACATGCATCTACTGAAGATTTGAGTCTCGAGTGCTTAGTTACATGAACTATCACAAAGGATATGGATAATATAAGGTGTACTGAAGTATGTCTATGCAATGGGAGGGAAATACATTCTGTTAAATGACTTGTCGATTTGATCTTTCATGCCAAAGATTAAAAATTTAACACTTAATTAACGCAATCTTACCATATATTCATGGACTACATGCAGAATAGTAATTCTCCCAACCTTTCTAGTTATTTACCTGAATGTGTTTATGTACATGGACCGGTAACCTCATGTATATATGCACATACTGACAATCTGACATACATATATATAGTAGATATGACAACAACAACAAAAAAAAAAAAAATTCCTTGTTCGTGAAGCATGATCTGAGAGTTCCTAGTTAGCATGTTGTGTGGGATCATACTTTTAATATGCTGCAAGTACCAGTCAATTTTAGTATGGGAAACTATAAACATGTATAATCAACCAATGAACACGTCAATAACCTATTGAACAGCTTAGGGTGAAAATTATGATCCGTAGAGACAGCATTTAAAAGTTCCTTACGTCCACGTAAAATAATATATCAATTTATACATATACATGTGTAAACTGTGTATATATAGGGTAGGTATATGTGTATATATATAGTAATTGACAAATGATTTAGGTTCTAACATATATTCTAAAAGTACTCATGAGTTTGTGAGATCTACACAAGATACCTGATTTGATAAAAATGGCTTCAACTTGCAATCCAAACCAAACCAAACAAAGTTAATAACCAAGGGTTAATAACAAAAACAAGAATCTAGAATTAGTAAAAAAATGAGAAATTAATGAACCTGTGATCATAAAAAAAGTCAAACAATGTGAAAACATATCATACCTTTTGTTCTTTTTAATATAATAATTGAATTACTAAATGGATGGATCAGTCCTTCTTCCATAGCTAGCTTCTCTTTATTTTCTCTGCCCATAACCTGCAGAAAATCTCTTTAACCGAGCAAAATTACAAGAGTAACCAAACAAACAAAATAGGACCATCAGAGAGAGAGAAAAGAGTGCCTTTTTTTGGACCTGCCCATTAGCTTAGAATGTACCATGAAACACTTTGTCAGTGTAGGGAGATAGGCACAGAGAGTACAATTCATGAAATTTATAAGCTTTTTTCCCACTCATCAATT
          https://blast.ncbi.nlm.nih.gov/Blast.cgi?RID=NJHUWRD7015&CMD=Get
          RID=NJHUWRD7015
          Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 8.481 sec - in apollo.analysis.RemoteBlastNCBITest

          For this task, please inspect the testing code and remove println and logging statements.

            Attachments

              Issue Links

              relates to

              Bug - A problem which impairs or prevents the functions of the product. IGBF-1981 Investigate and fix test errors in BookMarkTest.java (BookMark Plugin)

              • Minor - Minor loss of function, or other problem where easy workaround is present.
              • Closed

                Activity

                Descending order - Click to sort in ascending order
                Hide
                Permalink
                ann.loraine Ann Loraine added a comment -

                Some notes:

                • Tests in core/das package declaration did not match directory structure. Fixed.

                Pushed new IGBF-1977 branch to my fork and merged with lorainelab/master.

                Show
                ann.loraine Ann Loraine added a comment - Some notes: Tests in core/das package declaration did not match directory structure. Fixed. Pushed new IGBF-1977 branch to my fork and merged with lorainelab/master.
                Hide
                Permalink
                ann.loraine Ann Loraine added a comment -

                Tests are exposing places where tested code prints to stdout instead of using the logging system.
                For example: core/genometry/src/main/java/com/affymetrix/genometry/parsers/graph/BgrParser.java.

                Show
                ann.loraine Ann Loraine added a comment - Tests are exposing places where tested code prints to stdout instead of using the logging system. For example: core/genometry/src/main/java/com/affymetrix/genometry/parsers/graph/BgrParser.java.
                Hide
                Permalink
                ann.loraine Ann Loraine added a comment -

                Looks there are still some unnecessary things being printed during build. Looking into this a bit more.

                Show
                ann.loraine Ann Loraine added a comment - Looks there are still some unnecessary things being printed during build. Looking into this a bit more.
                Hide
                Permalink
                noor91zahara Noor Zahara (Inactive) added a comment -

                Logs have been removed.

                Show
                noor91zahara Noor Zahara (Inactive) added a comment - Logs have been removed.
                Hide
                Permalink
                svallapu Sai Charan Reddy Vallapureddy (Inactive) added a comment -

                Created new ticket for bookmarktest.java(IGBF-1981) and linked to this ticket.

                Pull Request Submitted

                Show
                svallapu Sai Charan Reddy Vallapureddy (Inactive) added a comment - Created new ticket for bookmarktest.java( IGBF-1981 ) and linked to this ticket. Pull Request Submitted
                Hide
                Permalink
                ann.loraine Ann Loraine added a comment -

                Please create new issue for the BookMarkTest.java issue and link it to IGBF-1977 (this Jira ticket.)
                Once that is done, please submit pull request and move this ticket forward on the board.

                Show
                ann.loraine Ann Loraine added a comment - Please create new issue for the BookMarkTest.java issue and link it to IGBF-1977 (this Jira ticket.) Once that is done, please submit pull request and move this ticket forward on the board.
                Hide
                Permalink
                svallapu Sai Charan Reddy Vallapureddy (Inactive) added a comment -

                Branch: https://bitbucket.org/svallapu/charan_igb/branch/IGBF-1977

                Checked all projects test files. Removed unnecessary print and logger statements.

                Did not touch:
                1. Necessary log statements
                2. ignored part of the code (using "ignored" annotation)
                3. commented print and log statements.

                Note: While going through the projects, I found that BookMarkTest.java (BookMark Plugin) dependencies are not resolved. I will fix this in another ticket.

                Show
                svallapu Sai Charan Reddy Vallapureddy (Inactive) added a comment - Branch: https://bitbucket.org/svallapu/charan_igb/branch/IGBF-1977 Checked all projects test files. Removed unnecessary print and logger statements. Did not touch: 1. Necessary log statements 2. ignored part of the code (using "ignored" annotation) 3. commented print and log statements. Note: While going through the projects, I found that BookMarkTest.java (BookMark Plugin) dependencies are not resolved. I will fix this in another ticket.

                  People

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

                    Dates

                    • Created:
                      Updated:
                      Resolved: