Opened on 11/08/2018 at 06:49:11 PM

Closed on 11/15/2018 at 11:22:07 AM

#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

Attachments (0)

Change History (3)

comment:1 Changed on 11/08/2018 at 07:40:31 PM by atudor

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

comment:2 Changed on 11/14/2018 at 05:26:36 PM by abpbot

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

comment:3 Changed on 11/15/2018 at 11:22:07 AM by atudor

  • Resolution set to fixed
  • Status changed from reviewing to closed

Add Comment

Modify Ticket

Change Properties
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from atudor.
Note: See TracTickets for help on using tickets.