Skip to content

Add tests for openPathUpdateAll #5

@camdroid

Description

@camdroid

Kim Jones #7442, active Ceramics - Half Shelf membership, WaiverDate and FacilityTourDate fields populated, did not propagate to Alta Open. I’m going to manually add them to the “Subscribers” and “Ceramics Subscribers” access group, just wanted to notify this team.

Slack link
Dependent on this PR.

Add tests so that we have a more readable way to determine what accounts are being filtered out. It would be great to have tests read something like:

regular_user = createMockRegularUser()
openPathUpdateSingle(regular_user['neonId'])
assert user_has_facility_access(regular_user)

OR

expired_user = createMockExpiredUser()
uid = expired_user['neonId']
openPathUpdateSingle(uid)
assert not user_has_facility_access(uid)

That way, we could easily add tests to make sure that different types of users (e.g. ceramics, regular, staff) have the correct access to the facility under the correct circumstances.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions