some minor tweaks and turn off debug logging for spiff step details w/ burnettk

This commit is contained in:
jasquat 2023-03-06 16:16:55 -05:00
parent 9705244f6c
commit 94776a78b5
4 changed files with 17 additions and 8 deletions

View File

@ -19,6 +19,11 @@ RUN apt-get update \
# Setup image for installing Python dependencies. # Setup image for installing Python dependencies.
FROM base AS setup FROM base AS setup
# poetry 1.4 seems to cause an issue where it errors with
# This error originates from the build backend, and is likely not a
# problem with poetry but with lazy-object-proxy (1.7.1) not supporting PEP 517 builds.
# You can verify this by running 'pip wheel --use-pep517 "lazy-object-proxy (==1.7.1) ; python_version >= "3.6""'.
# Pinnning to 1.3.2 to attempt to avoid it.
RUN pip install poetry==1.3.2 RUN pip install poetry==1.3.2
RUN useradd _gunicorn --no-create-home --user-group RUN useradd _gunicorn --no-create-home --user-group

View File

@ -1218,7 +1218,7 @@ class ProcessInstanceProcessor:
) )
db.session.add(spiff_step_detail) db.session.add(spiff_step_detail)
db.session.commit() db.session.commit()
self.log_spiff_step_details(spiff_step_detail_mapping) # self.log_spiff_step_details(spiff_step_detail_mapping)
if len(human_tasks) > 0: if len(human_tasks) > 0:
for at in human_tasks: for at in human_tasks:
@ -1255,7 +1255,7 @@ class ProcessInstanceProcessor:
spiff_step_detail = SpiffStepDetailsModel(**step) spiff_step_detail = SpiffStepDetailsModel(**step)
db.session.add(spiff_step_detail) db.session.add(spiff_step_detail)
db.session.commit() db.session.commit()
self.log_spiff_step_details(step) # self.log_spiff_step_details(step)
def manual_complete_task(self, task_id: str, execute: bool) -> None: def manual_complete_task(self, task_id: str, execute: bool) -> None:
"""Mark the task complete optionally executing it.""" """Mark the task complete optionally executing it."""
@ -1720,7 +1720,7 @@ class ProcessInstanceProcessor:
raise ApiError.from_workflow_exception("task_error", str(swe), swe) from swe raise ApiError.from_workflow_exception("task_error", str(swe), swe) from swe
finally: finally:
self.log_spiff_step_details(step_details) # self.log_spiff_step_details(step_details)
db.session.bulk_insert_mappings(SpiffStepDetailsModel, step_details) db.session.bulk_insert_mappings(SpiffStepDetailsModel, step_details)
spiff_logger = logging.getLogger("spiff") spiff_logger = logging.getLogger("spiff")
for handler in spiff_logger.handlers: for handler in spiff_logger.handlers:

View File

@ -31,14 +31,17 @@ const approveWithUser = (
}; };
describe('pp1', () => { describe('pp1', () => {
it.skip('can run PP1', () => { it('can run PP1', () => {
cy.login('core5.contributor', 'core5.contributor'); cy.login('core5.contributor', 'core5.contributor');
cy.visit('/'); cy.visit('/');
cy.contains('Start New +').click(); cy.contains('Start New +').click();
cy.contains('Raise New Demand Request'); cy.contains('Raise New Demand Request');
cy.runPrimaryBpmnFile(true); cy.runPrimaryBpmnFile(true);
cy.contains('Procurement').click(); cy.contains('Please select the type of request to Start the process.');
// cy.contains('Submit').click(); // wait a second to ensure we can click the radio button
cy.wait(1000);
cy.get('input#root-procurement').click();
cy.wait(1000);
cy.get('button') cy.get('button')
.contains(/^Submit$/) .contains(/^Submit$/)
.click(); .click();
@ -77,7 +80,8 @@ describe('pp1', () => {
.click(); .click();
cy.contains( cy.contains(
'Review and provide any supporting information or files for your request.' 'Review and provide any supporting information or files for your request.',
{ timeout: 20000 }
); );
cy.contains('Submit the Request').click(); cy.contains('Submit the Request').click();
cy.get('input[value="Submit the Request"]').click(); cy.get('input[value="Submit the Request"]').click();

View File

@ -109,7 +109,7 @@ Cypress.Commands.add(
if (expectAutoRedirectToHumanTask) { if (expectAutoRedirectToHumanTask) {
// the url changes immediately, so also make sure we get some content from the next page, "Task:", or else when we try to interact with the page, it'll re-render and we'll get an error with cypress. // the url changes immediately, so also make sure we get some content from the next page, "Task:", or else when we try to interact with the page, it'll re-render and we'll get an error with cypress.
cy.url().should('include', `/tasks/`); cy.url().should('include', `/tasks/`);
cy.contains('Task: '); cy.contains('Task: ', { timeout: 10000 });
} else { } else {
cy.contains(/Process Instance.*[kK]icked [oO]ff/); cy.contains(/Process Instance.*[kK]icked [oO]ff/);
cy.reload(true); cy.reload(true);