Bugfix webmaster queue

[benne238]

No description provided.

[campb303]

Attempted to test test the current live queue with:

if __name__ == "__main__":
    loadQueues()

but I got an error:

This will need to be fixed before I can merge the code.

[campb303]

Attempted to test test the current live queue with:

if __name__ == "__main__":
    loadQueues()

but I got an error in graddb414:

[campb303]

Cannot merge at this time. Breaks on aae55 in the live queue. A copy of aae55 is below:

CC: "Chynoweth, Brandon C" <bchynowe@purdue.edu>
Date: Wed, 27 Jan 2021 15:09:57 +0000
From: "Wunderlich, Daren K" <dwunder@purdue.edu>
Message-ID: <CH2PR22MB1781788936122DB6D972650CD8BB9@CH2PR22MB1781.namprd22.prod.outlook.com>
Subject: RE: Requirement for Wind Tunnel Computer
To: "aaesite@ecn.purdue.edu" <aaesite@ecn.purdue.edu>
Content-Type: text/plain; charset="utf-8"
X-ECN-Queue-Original-Path: /home/pier/e/queue/Attachments/inbox/2021-01-27/108-original.txt
X-ECN-Queue-Original-URL: https://engineering.purdue.edu/webqueue/Attachments/inbox/2021-01-27/108-original.txt
X-ECN-Queue-Attachment-1-Path: /home/pier/e/queue/Attachments/inbox/2021-01-27/108-attachment-1.bin
X-ECN-Queue-Attachment-1-URL: https://engineering.purdue.edu/webqueue/Attachments/inbox/2021-01-27/108-attachment-1.bin

[benne238]

Empty Content

The parser attempts to find the ending line and starting line of content within and item, storing the line numbers in respective variables: contentStart and contentEnd. When the content of an item is empty, the contentStart and contentEnd variables will either be equal to eachother, or sometimes, contentStart will be greater than contentEnd, which isn't logically possible, but does indicate empty content in an item. A simple fix just involves comparing the values of the variables with the following code:

        if contentEnd <= contentStart:
            blankInitialMessage = self.__initialMessageParsing([""])
            sections.append(blankInitialMessage)
            return sections

This snipet will call the __initialMessageParsing() function, passing a list with a single empty string (signifying empty content), which will then return an initial message dictionary with the content key having a value of [""].

[benne238]

Testing

The Bugfix-Webmaster-Queue branch was locally merged into staging and the loadQueues() function was tested against q-snapshot and the live queue and no errors were encountered. Furthermore the api calls were also tested on the locally merged branch against the live queue and no errors were encountered there either.

[campb303]

See comments about explicit exception handling.

[campb303]

Be more explicit here. What error are you anticipating? Catch it by name. For example:

# Catching a divide by zero error.

try:
    num = 1 / 0
catch DivideByZeroError as e:
    print("Cannot divide by 0. Try again.")