Fixes: clean database and case 0 bugs #47
Conversation
|
I have tested the changes in Staging first to be sure they work. |
| df['modification_date'] = pd.to_datetime(df['modification_date'], format='%Y%m%dT%H:%M:%S') # noqa | ||
| df['creation_date'] = pd.to_datetime(df['creation_date'], format='%Y%m%dT%H:%M:%S') # noqa | ||
| if not rows: | ||
| print("There are no bugs to verify today") |
There was a problem hiding this comment.
I assume this is diagnostic for the github action logs 👍🏼
There was a problem hiding this comment.
yes, to know why did not have data if that's the case and not due to any other error / situation ..
| @@ -146,6 +152,13 @@ def __init__(self): | |||
| super().__init__() | |||
| self.db = Database() | |||
|
|
|||
| def qa_needed_delete(self): | |||
There was a problem hiding this comment.
if I remember correctly, this is be cause we're using the DB as temp cache for this data so wiping it before rehydrating it is the way to go. true?
There was a problem hiding this comment.
That's the case for TestRail test suites. In this case I want to clean the table so that it shows daily the active bugs that require qa attention
| qe_needed_count = self.db.report_bugzilla_qa_needed_count(data_frame) | ||
| self.db.report_bugzilla_qa_needed_count_insert(qe_needed_count) | ||
| # Drop the columns 'type_id' and 'id' | ||
| df_cleaned = df.drop(columns=["type_id", "id"]) |
There was a problem hiding this comment.
just curious, are you trimming these columns from just the dataframe because ... not needed / housekeeping?
There was a problem hiding this comment.
yes, those are not needed, those are the id for the flag and its type which do not give much information
There are two issues identified:
In order to list the current bugs to be verified, we need to clean the table first.
In Android due to the low number of qe-verify+, it could be the case that there are no bugs to verify. This is considered now.