Skip to content

Conversation

@Paliak
Copy link
Contributor

@Paliak Paliak commented Nov 28, 2022

Description of the problem being solved:

Tests fail if they're implemented in the same pr as parsing for a mod that they require due to the github tests runner using outdated mod cache.

This pr implements a headless flag that prevent loading of mod cache when running using the headless wrapper and also separates ModCache rebuilding code into a separate function.

Steps taken to verify a working solution:

  • Ran tests with different versions of mod cache.

Currently only causes ModCache to not load on start which allows
writing tests that involve mod parsing.
The CI envirement variable should always be true when the headless
wrapper is ran from github workflows but should not be set when other
tools use it. Should hopefully prevent slowing things down for 3rd
parties interfacing with pob.
@QuickStick123 QuickStick123 added bug Something isn't working enhancement New feature, calculation, or mod technical Hidden from release notes and removed bug Something isn't working labels Nov 28, 2022
@LocalIdentity LocalIdentity merged commit 4c3862b into PathOfBuildingCommunity:dev Dec 8, 2022
@Paliak Paliak deleted the modParserTests branch January 20, 2024 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature, calculation, or mod technical Hidden from release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants