Uploaded image for project: 'Yetus'
  1. Yetus
  2. YETUS-995

vote table should have access to the log filename

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 0.13.0
    • Precommit
    • None
    • Hide
      <!-- markdown -->
      Users:

      The log files for tests have been popped up into the voting table for many of the generated reports.

      Developers:

      This is a breaking change. It reworks how tests should be registering the logfiles that they present to users and how bugsystem reporting engines should retrieve the filenames for those logfiles:

      * add_vote_table() has been deprecated and replaced with add_vote_table_v2() that now takes an additional parameter. Code that uses add_vote_table()/add_footer_table() combo will NOT have their log file entries reported with some bugsystem reporting engines.
      * The TP_VOTE_TABLE has been re-arranged a bit to add in the log filename. Custom reporting engines will need to modified.
      Show
      <!-- markdown --> Users: The log files for tests have been popped up into the voting table for many of the generated reports. Developers: This is a breaking change. It reworks how tests should be registering the logfiles that they present to users and how bugsystem reporting engines should retrieve the filenames for those logfiles: * add_vote_table() has been deprecated and replaced with add_vote_table_v2() that now takes an additional parameter. Code that uses add_vote_table()/add_footer_table() combo will NOT have their log file entries reported with some bugsystem reporting engines. * The TP_VOTE_TABLE has been re-arranged a bit to add in the log filename. Custom reporting engines will need to modified.
    • Important

    Description

      A big design from early on in test-patch history was that the log entry was separate from the vote. This setup was likely done for a variety of aesthetic and practical reasons for JIRA. However, that design is flawed in that it is letting display dictate internals. With GitHub Status (YETUS-994), this flaw is now very very obvious.

      Fixing this flaw is going to be a MAJOR compatibility break with no particularly easy way to make existing code work. Details below.

      Attachments

        Issue Links

          Activity

            People

              aw Allen Wittenauer
              aw Allen Wittenauer
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 0.5h
                  0.5h