Merge pull request #504 from dod-ccpo/jrn_no_branch_coverage
JRN no branch coverage
This commit is contained in:
commit
92f425ac79
@ -51,7 +51,7 @@ def get_current_user():
|
|||||||
|
|
||||||
|
|
||||||
def logout():
|
def logout():
|
||||||
if session.get("user_id"):
|
if session.get("user_id"): # pragma: no branch
|
||||||
del (session["user_id"])
|
del (session["user_id"])
|
||||||
|
|
||||||
|
|
||||||
|
@ -84,7 +84,7 @@ class Invitations(object):
|
|||||||
elif invite.is_accepted or invite.is_revoked or invite.is_rejected:
|
elif invite.is_accepted or invite.is_revoked or invite.is_rejected:
|
||||||
raise InvitationError(invite)
|
raise InvitationError(invite)
|
||||||
|
|
||||||
elif invite.is_pending:
|
elif invite.is_pending: # pragma: no branch
|
||||||
Invitations._update_status(invite, InvitationStatus.ACCEPTED)
|
Invitations._update_status(invite, InvitationStatus.ACCEPTED)
|
||||||
WorkspaceRoles.enable(invite.workspace_role)
|
WorkspaceRoles.enable(invite.workspace_role)
|
||||||
return invite
|
return invite
|
||||||
|
@ -81,7 +81,7 @@ class Invitation(Base, TimestampsMixin, AuditableMixin):
|
|||||||
|
|
||||||
@property
|
@property
|
||||||
def workspace(self):
|
def workspace(self):
|
||||||
if self.workspace_role:
|
if self.workspace_role: # pragma: no branch
|
||||||
return self.workspace_role.workspace
|
return self.workspace_role.workspace
|
||||||
|
|
||||||
@property
|
@property
|
||||||
|
@ -200,7 +200,7 @@ class Request(Base, mixins.TimestampsMixin, mixins.AuditableMixin):
|
|||||||
or self.status == RequestStatus.CHANGES_REQUESTED_TO_FINVER
|
or self.status == RequestStatus.CHANGES_REQUESTED_TO_FINVER
|
||||||
):
|
):
|
||||||
review = self.latest_status.review
|
review = self.latest_status.review
|
||||||
if review:
|
if review: # pragma: no branch
|
||||||
return review.comment
|
return review.comment
|
||||||
|
|
||||||
@property
|
@property
|
||||||
|
Loading…
x
Reference in New Issue
Block a user