zydronium/manage.get.gov
1
0
Fork 0
mirror of https://github.com/cisagov/manage.get.gov.git synced 2025-05-27 22:59:54 +02:00
Code Issues Projects Releases Packages Wiki Activity

(no changes, but git detects a change)

Browse source 
Signed-off-by: CocoByte <nicolle.leclair@gmail.com>
This commit is contained in:
CocoByte 2023-10-08 20:00:16 -06:00
parent c8a8d0c34a
commit c971d838fe
No known key found for this signature in database
GPG key ID: BBFAA2526384C97F
1 changed files with 28 additions and 28 deletions
Download patch file Download diff file Expand all files Collapse all files

56
src/registrar/management/commands/load_transition_domain.py
View file

@ -30,7 +30,7 @@ class termColors:
BackgroundLightYellow = "\033[103m"
def query_yes_no(question: str, default="yes") -> dict[str, bool]:
def query_yes_no(question: str, default="yes") -> dict[str,bool]:
"""Ask a yes/no question via raw_input() and return their answer.
"question" is a string that is presented to the user.
@ -81,14 +81,16 @@ class Command(BaseCommand):
for testing purposes, but USE WITH CAUTION
"""
parser.add_argument(
"domain_contacts_filename", help="Data file with domain contact information"
"domain_contacts_filename",
help="Data file with domain contact information"
)
parser.add_argument(
"contacts_filename",
help="Data file with contact information",
)
parser.add_argument(
"domain_statuses_filename", help="Data file with domain status information"
"domain_statuses_filename",
help="Data file with domain status information"
)
parser.add_argument("--sep", default="|", help="Delimiter character")
@ -105,9 +107,7 @@ class Command(BaseCommand):
action=argparse.BooleanOptionalAction,
)
def print_debug_mode_statements(
self, debug_on: bool, debug_max_entries_to_parse: int
):
def print_debug_mode_statements(self, debug_on: bool, debug_max_entries_to_parse: int):
"""Prints additional terminal statements to indicate if --debug
or --limitParse are in use"""
if debug_on:
@ -117,7 +117,7 @@ class Command(BaseCommand):
Detailed print statements activated.
{termColors.ENDC}
"""
)
)
if debug_max_entries_to_parse > 0:
logger.info(
f"""{termColors.OKCYAN}
@ -127,11 +127,9 @@ class Command(BaseCommand):
Detailed print statements activated.
{termColors.ENDC}
"""
)
)
def get_domain_user_dict(
self, domain_statuses_filename: str, sep: str
) -> defaultdict(str):
def get_domain_user_dict(self, domain_statuses_filename: str, sep: str) -> defaultdict(str):
"""Creates a mapping of domain name -> status"""
domain_status_dictionary = defaultdict(str)
logger.info("Reading domain statuses data file %s", domain_statuses_filename)
@ -165,21 +163,21 @@ class Command(BaseCommand):
and any ok state should map to Ready.
"""
status_maps = {
"hold": TransitionDomain.StatusChoices.HOLD,
"serverhold": TransitionDomain.StatusChoices.HOLD,
"clienthold": TransitionDomain.StatusChoices.HOLD,
"hold": TransitionDomain.StatusChoices.ON_HOLD,
"serverhold": TransitionDomain.StatusChoices.ON_HOLD,
"clienthold": TransitionDomain.StatusChoices.ON_HOLD,
"created": TransitionDomain.StatusChoices.READY,
"ok": TransitionDomain.StatusChoices.READY,
}
return status_maps.get(status_to_map)
def print_summary_duplications(
self,
duplicate_domain_user_combos: list[TransitionDomain],
duplicate_domains: list[TransitionDomain],
users_without_email: list[str],
self,
duplicate_domain_user_combos: list[TransitionDomain],
duplicate_domains: list[TransitionDomain],
users_without_email: list[str]
):
"""Called at the end of the script execution to print out a summary of
"""Called at the end of the script execution to print out a summary of
data anomalies in the imported Verisign data. Currently, we check for:
- duplicate domains
- duplicate domain - user pairs
@ -219,10 +217,11 @@ class Command(BaseCommand):
{termColors.ENDC}"""
)
def print_summary_status_findings(
self, domains_without_status: list[str], outlier_statuses: list[str]
):
"""Called at the end of the script execution to print out a summary of
def print_summary_status_findings(self,
domains_without_status: list[str],
outlier_statuses: list[str]
):
"""Called at the end of the script execution to print out a summary of
status anomolies in the imported Verisign data. Currently, we check for:
- domains without a status
- any statuses not accounted for in our status mappings (see
@ -265,6 +264,7 @@ class Command(BaseCommand):
{termColors.ENDC}"""
)
def handle( # noqa: C901
self,
domain_contacts_filename,
@ -275,7 +275,7 @@ class Command(BaseCommand):
"""Parse the data files and create TransitionDomains."""
sep = options.get("sep")
# If --resetTable was used, prompt user to confirm
# If --resetTable was used, prompt user to confirm
# deletion of table data
if options.get("resetTable"):
confirm_reset = query_yes_no(
@ -356,7 +356,7 @@ class Command(BaseCommand):
new_entry_status = TransitionDomain.StatusChoices.READY
new_entry_email = ""
new_entry_emailSent = False # set to False by default
new_entry_emailSent = False # set to False by default
if new_entry_domainName not in domain_status_dictionary:
# this domain has no status...default to "Create"
@ -463,7 +463,7 @@ class Command(BaseCommand):
# DEBUG:
if debug_on or debug_max_entries_to_parse > 0:
if (
total_rows_parsed > debug_max_entries_to_parse
total_rows_parsed >= debug_max_entries_to_parse
and debug_max_entries_to_parse != 0
):
logger.info(
@ -482,11 +482,11 @@ class Command(BaseCommand):
updated {total_updated_domain_entries} transition domain entries
{termColors.ENDC}
"""
)
)
# Print a summary of findings (duplicate entries,
# missing data..etc.)
self.print_summary_duplications(
duplicate_domain_user_combos, duplicate_domains, users_without_email
)
self.print_summary_status_findings(domains_without_status, outlier_statuses)
self.print_summary_status_findings(domains_without_status, outlier_statuses)