Re-enable dumpmt tests in OtherCommands.script (#4304)
authorMike McLaughlin <mikem@microsoft.com>
Tue, 10 Oct 2023 04:49:51 +0000 (21:49 -0700)
committerGitHub <noreply@github.com>
Tue, 10 Oct 2023 04:49:51 +0000 (21:49 -0700)
Addresses issue #https://github.com/dotnet/diagnostics/issues/3516

src/SOS/SOS.UnitTests/Scripts/OtherCommands.script

index 8274219dcd379034b6ba82634292c0a7caf9cc04..b3473d5b366ec2799b67e20dfd6a73aff8fa3d07 100644 (file)
@@ -109,18 +109,9 @@ VERIFY:\s*Class Name:\s+SymbolTestApp.Program\s+
 VERIFY:\s*File:\s+.*SymbolTestApp\.(dll|exe)\s+
 
 # Verify DumpMT
-!IFDEF:MAJOR_RUNTIME_VERSION_GE_7
-# https://github.com/dotnet/diagnostics/issues/3516
 SOSCOMMAND:DumpMT <POUT>\s*Method Table:\s+(<HEXVAL>)\s+<POUT>
 VERIFY:\s*Name:\s+SymbolTestApp.Program\s+
 VERIFY:\s*File:\s+.*SymbolTestApp\.(dll|exe)\s+
-ENDIF:MAJOR_RUNTIME_VERSION_GE_7
-
-IFDEF:MAJOR_RUNTIME_VERSION_GE_8
-SOSCOMMAND:DumpMT <POUT>\s*Method Table:\s+(<HEXVAL>)\s+<POUT>
-VERIFY:\s*Name:\s+SymbolTestApp.Program\s+
-VERIFY:\s*File:\s+.*SymbolTestApp\.(dll|exe)\s+
-ENDIF:MAJOR_RUNTIME_VERSION_GE_8
 
 SOSCOMMAND:FinalizeQueue
 VERIFY:\s*SyncBlocks to be cleaned up:\s+<DECVAL>\s+