TrustStore: Disable unit tests #3311
Conversation
There was a problem hiding this comment.
Reviewed 7 of 7 files at r1.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @oncilla)
a discussion (no related file):
Please open an issue mentioning which tests are disabled and why, and that they must be enabled again.
Also include a summary of the minimal requirements that need to be implemented before we can enable them again.
Please reference the issue in this PR.
d847d72 to
5d94c8a
Compare
There was a problem hiding this comment.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @scrye)
a discussion (no related file):
Previously, scrye (Sergiu Costea) wrote…
Please open an issue mentioning which tests are disabled and why, and that they must be enabled again.
Also include a summary of the minimal requirements that need to be implemented before we can enable them again.
Please reference the issue in this PR.
Done.
The unit tests rely on generated trust material. The scion-pki tool that generates the material is currently under rework and having tests that require full operability is time consuming during the rewrite. This change temporarily disables the unit tests to allow faster rework.
5d94c8a to
238f29a
Compare
There was a problem hiding this comment.
Reviewable status:
complete! all files reviewed, all discussions resolved
The unit tests rely on generated trust material. The scion-pki tool
that generates the material is currently under rework and having tests
that require full operability is time consuming during the rewrite.
This change temporarily disables the unit tests to allow faster rework.
Tracking issue #3317
This change is