hedgehogqa
Add more detailed state for property.
This PR aggregates all the "check time" state of a property into a single structure. This will hopefully make adding features like #341 easier, and I think it's much easier to see at a glance what's happening internally by looking at the new PropertyState module.
/cc @TysonMN @dharmaturtle
Can we merge #336 first? I don't want to fix merge conflicts.
Absolutely. Are we still waiting on a Fable issue there?
I like this a lot.
:+1:
Can we merge #336 first? I don't want to fix merge conflicts.
Absolutely. Are we still waiting on a Fable issue there?
We are primarily waiting on PR #364 to be reviewed and completed. After that, we could merge PR #336 with understanding that the API needs some work, but maybe that is ok since improving the API around Property is what issue #365, PR #346, and this PR are intended to do.
And the issue with Fable is minor. The simple fix is to comment out one of the assertions. I think it would be better to have that assertion, but it isn't the only assertion, and we can wait to comment back in this second assertion once issue with Fable is resolved.