fix: this is an alternative approach to get the test suite working. I…

[benjam-es]

First

Please check #92 first as I believe this prevents future problems for other users

What:

• Bug Fix

Description:

When running the test suite locally, my home directory is atum which is seen as a mis-spelling. As we usually exclude that part of the directory elsewhere, this issue only arrises on the constant check within the Cache class.

In #92, I add ignore words to the spell checker for the home directory to prevent this happening. In this PR, we simply extract it out.

Related:

This is an alternative approach to the PR: #92

[c0nst4ntin]

@benjam-es I don't really understand what this fixes? You simply moved the __DIR__ to a new function? What is the benefit?

[benjam-es]

@c0nst4ntin please see #92

On my machine, it is finding a spelling error within my home directory (which is the project path)

/Users/atum/Code/packages/peck

I can't recreate this in a test, but I imagine #92 is the best fix and future proofs for other users that will face the same issue.

I hit the problem simply by running the test suite locally with the new Preset files

[c0nst4ntin]

@benjam-es Okay, I didn't get why this also changes it. But I guess it's the fact that the __DIR__ in the constant gets evaluated before the constant gets checked by reflection so we check the path? It was unclear to me, that this was the problem.

In this case I think this is an easy way to solve this.

[benjam-es]

Closing as duplicate for now