You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Consolidate cryptography tests to System.Security.Cryptography #66338
With #55690 merged, the implementations of the S.S.Cryptography assemblies have been consolidated in to a single in-box assembly, where possible.
The unit tests for these are still in their separate projects however, such as S.S.C.Algorithms, S.S.C.Cng, etc.
This issue is to track consolidating them under System.Security.Cryptography. Note that this is not as simple as moving the files and adjusting namespaces. We made heavy use of partials to test different implementations of algorithms (internal vs public CNG for example). The partial approach won't work. The current solution is to refactor to a "driver" base type and test each implementation as a derived type.
I don't think we want to disable all of S.S.Cryptography tests on Mariner. But do we actually need to disable X509Certificates on Mariner? According to #57810 (comment), that ActiveIssue didn't stop the Mariner crashes.
Put ActiveIssue on every single X509Certificates class, and future classes.
Move all of the tests anyway and remove the assembly-level ActiveIssue since prior comment indicated it did not help. If that is true, then what is being done for Mariner? Are tests running, or did we disable them some other way?
cc @ViktorHofer in case you have any insight on the X509Certificate tests in Mariner.
Activity
bartonjs commentedon Mar 8, 2022
Huh, thought I'd opened this already. But, I hadn't. Thanks 😄.
https://github.com/bartonjs/runtime/commits/crypto_test_model has my first-draft thoughts here. They seemed to be fruitful, but certainly aren't a commitment.
vcsjones commentedon Mar 14, 2022
We could move X509Certificates pretty easily, if not for:
runtime/src/libraries/System.Security.Cryptography.X509Certificates/tests/AssemblyInfo.cs
Line 8 in c3cc9fd
I don't think we want to disable all of S.S.Cryptography tests on Mariner. But do we actually need to disable X509Certificates on Mariner? According to #57810 (comment), that
ActiveIssuedidn't stop the Mariner crashes.I think the given options are:
ActiveIssueon every single X509Certificates class, and future classes.ActiveIssuesince prior comment indicated it did not help. If that is true, then what is being done for Mariner? Are tests running, or did we disable them some other way?cc @ViktorHofer in case you have any insight on the X509Certificate tests in Mariner.
bartonjs commentedon Jul 6, 2022
Moving to the Future/Indefinite milestone because it's just code cleanup and our own pain, nothing impacting customers.
Move DSA tests into System.Security.Cryptography (#129320)
Consolidate EC tests into S.S.C. (#129552)
Move DSA tests into System.Security.Cryptography (#129320)
Consolidate EC tests into S.S.C. (#129552)
Consolidate RSA tests into S.S.C. (#129924)
Move DSA tests into System.Security.Cryptography (dotnet#129320)