Add tests #28

Open
opened 2020-06-16 15:29:02 +00:00 by phryk · 0 comments
phryk commented 2020-06-16 15:29:02 +00:00 (Migrated from rnd.phryk.net)

Originally from github issue:

As poobrains matures, it's sensible to finally add some tests.

Flask has some testing stuff built-in - use it!

What can even be tested?

  • CRUD (directly via ORM, via web and via CLI)
    • This should be possible to do automagically for every Storable, i.e. to supply some automated tests for developers using poobrains
  • Form/CLI validation and coercion
  • ForeignKey cascade on update/delete (when #\2 is fixed)# ADD LINK TO GITEA ISSUE
  • Whether permissions work as intended
  • Whether Renderable.expose properly works
  • HTTP status codes
    • Do nonexistant pages actually throw a 404?
  • ?

Issue from 2017 that was never updated. A good chunk of this should be implemented by the testing module, which at least tests installation, CRUD and permissions.

Originally from github issue: > As poobrains matures, it's sensible to finally add some tests. > > Flask has some testing stuff built-in - use it! > > What can even be tested? > * CRUD (directly via ORM, via web and via CLI) > * This should be possible to do automagically for every `Storable`, i.e. to supply some automated tests for developers using poobrains > * Form/CLI validation and coercion > * ForeignKey cascade on update/delete (when #\2 is fixed)# ADD LINK TO GITEA ISSUE > * Whether permissions work as intended > * Whether `Renderable.expose` properly works > * HTTP status codes > * Do nonexistant pages actually throw a 404? > * ? Issue from 2017 that was never updated. A good chunk of this should be implemented by the testing module, which at least tests installation, CRUD and permissions.
Sign in to join this conversation.
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: phryk-evil-mad-sciences-llc/poobrains#28
No description provided.