Code

Move mailboxes code to separate app#295

Code

Open

kevincarrogan wants to merge 38 commits intomasteruktrade/lite-hmrc:masterfrom LTD-5750-mailboxesuktrade/lite-hmrc:LTD-5750-mailboxesCopy head branch name to clipboard

+2,059-2,138

Commits

Commits on Jan 6, 2025

• Remove unused mail service functions

  kevincarrogan

  committed
  75e48cd

• Rename test file to match file under test

  kevincarrogan

  committed
  99e116d

• Use context manager for managing pop3 connection

  kevincarrogan

  committed
  b53bc4a

• Remove default settings for mail server initialisation

  kevincarrogan

  committed
  5b35c4c

• Allow configuration of mailservers to be defined in config

  kevincarrogan

  committed
  5431d0a

• Fix management command to check mailserver connections

  kevincarrogan

  committed
  2cb3741

• Move mailboxes code to separate app

  kevincarrogan

  committed
  3ad7b34

• Add str methods for mailbox models

  kevincarrogan

  committed
  f19405f

• Move get_message_id function to mailboxes

  kevincarrogan

  committed
  2c32798

• Extract out utils for getting headers and ids from emails

  kevincarrogan

  committed
  8901195

• Change API for mailbox functions to separate out responsibilities

  kevincarrogan

  committed
  4588b5e

• Pull out utility for getting message number

  kevincarrogan

  committed
  6da0cd4

• Move getting read message function into utils

  kevincarrogan

  committed
  87de06c

• Return a set of message ids

  kevincarrogan

  committed
  994ab1e

• Set reverse relation name for mail read status

  kevincarrogan

  committed
  d3cf6b3

• Jump out of email loop earlier if message id has already been read

  kevincarrogan

  committed
  73740b3

• Split out getting unread messages into separate function

  kevincarrogan

  committed
  1822cac

• Move out message iterator function

  kevincarrogan

  committed
  e2c2b8d

• Rewrite getting message iterator

  kevincarrogan

  committed
  d46f598

• Save email data to object

  kevincarrogan

  committed
  fb747f2

• Ensure message iterator runs with active pop connection

  kevincarrogan

  committed
  8a2a4e9

• Update to correctly match output from real mail servers

  kevincarrogan

  committed
  a40ceef

• Handle cases where the payload may have slightly changed when getting data

  kevincarrogan

  committed
  428dc5d

• By default only run unit tests

  kevincarrogan

  committed
  8236a8b

• Add integration tests

  kevincarrogan

  committed
  444a1d7

• Split out tests into separate jobs

  kevincarrogan

  committed
  401efa8

• Add integration tests to circleci

  kevincarrogan

  committed
  afccf54

• Add integration tests for message iterator

  kevincarrogan

  committed
  6b8252c

• Set better default value for database url

  kevincarrogan

  committed
  c5be73e

• Set distinct URL values in env and docker compose

  kevincarrogan

  committed
  ad95ace

• Use docker.env file when running integration tests

  kevincarrogan

  committed
  6353d46

• Run all tests locally and ensure circleci doesn't run integration tests with unit tests

  kevincarrogan

  committed
  3551c71

• Remove unused mock_hmrc app

  kevincarrogan

  committed
  1b51caa

• Remove untested and unused management command

  kevincarrogan

  committed
  310693f

• Add tests for find_mail_of

  kevincarrogan

  committed
  3fce8fb

• Swap out mocking mailservers for overriding settings

  kevincarrogan

  committed
  efb1856

Commits on Jan 7, 2025

• Add comment for update_or_create on read statuses

  kevincarrogan

  committed
  6b2ac82

• Add code comments and annotations to mailbox utils

  kevincarrogan

  committed
  8135a5d