Opened 10 months ago

Closed 9 months ago

#7112 closed change (fixed)

Add centralised conftest

Reported by: atudor Assignee: atudor
Priority: Unknown Milestone:
Module: Sitescripts Keywords:
Cc: kvas Blocked By:
Blocking: #7090 Platform: Unknown / Cross platform
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description (last modified by atudor)


Currently, sitescripts has one file in the tests for sitescripts/extensions. This, however, prevents us from adding other files when testing other modules.

What to change

In this environment, I suggest we create a dedicated conftest directory for the entire sitescripts, define specific fixtures for every module in separate files and then put them all together using the technique from here:


  1. Create a /conftest directory, with a file inside
  2. Move the contents of /sitescripts/extensions/test/ into a new file, /conftest/sitescripts/
  3. Add the fixtures from the new file in /conftest/, using the techique linked above.
  4. Add conftest at the head of the directories parsed by

Change History (3)

comment:1 Changed 10 months ago by atudor

  • Description modified (diff)
  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:2 Changed 9 months ago by abpbot

A commit referencing this issue has landed:
Issue 7112 - Add centralised conftest

comment:3 Changed 9 months ago by atudor

  • Resolution set to fixed
  • Status changed from reviewing to closed
Note: See TracTickets for help on using tickets.