tests are now working, there were a few typos as to why it wasn't testing everything. They all pass, but currently the functionality to delete a line_item has been removed as this was causing issues with the email view rendering the _line_item partial. This will be changed to use a separate partial