728 mail aliases needs a new test suite

uchicago-library / library_website

The University of Chicago Library Website

5 stars 5 forks source link

Closed bluairis closed 1 week ago

bluairis commented 3 months ago

Fixes #728

Changes in this request

I added a few tests for the mail aliases page (I did not test each_list because it needs to be fixed pretty soon)
I made one change to the filter function because the test made me realize that it was removing aliases like "MAILER-DAEMON" from "m" filter value because it was only looking for lowercase first values
Tests are passing.

bufordrat commented 1 week ago

These changes were incorporated into the code by way of PR #766. Thank you @bluairis for all your work on this.

bufordrat commented 1 week ago

Closing the PR...