Test infrastructure maintainability: refactor FixtureManipulatorTest into more test methods, add extra validation

Created on 3 February 2023, almost 2 years ago
Updated 17 April 2023, over 1 year ago

Problem/Motivation

I had to alter FixtureManipulatorTest for ๐Ÿ› The 'fake_site' fixture cannot be using with `composer show` because the packages are not installed Fixed I found it very confusing.

Proposed resolution

  1. split up testAddPackage() and testModifyPackage() into multiple methods
    have each with 1 call to addPackage or modifyPackage

    and the assert that it changed or didn't change the fixture

  2. consider moving the adding of packages out of setUp() since these aren't needed for all tests and into something like addTestPackages
๐Ÿ“Œ Task
Status

RTBC

Version

3.0

Component

Code

Created by

๐Ÿ‡บ๐Ÿ‡ธUnited States tedbow Ithaca, NY, USA

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Production build 0.71.5 2024