Skip to content

Commit

Permalink
Merge pull request #777 from microsoft/fixFullGeneration
Browse files Browse the repository at this point in the history
Fix full generation tests
  • Loading branch information
AArnott committed Nov 16, 2022
2 parents 6ae12d1 + a2d686a commit b7584ae
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
6 changes: 3 additions & 3 deletions azure-pipelines/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ parameters:

jobs:
- job: Windows
timeoutInMinutes: 120
timeoutInMinutes: 240
pool: ${{ parameters.windowsPool }}
variables:
TestFilter: ""
Expand All @@ -30,7 +30,7 @@ jobs:
RunTests: ${{ parameters.RunTests }}

- job: Linux
timeoutInMinutes: 120
timeoutInMinutes: 240
pool:
vmImage: Ubuntu 20.04
variables:
Expand All @@ -48,7 +48,7 @@ jobs:

- job: macOS
condition: ${{ parameters.includeMacOS }}
timeoutInMinutes: 120
timeoutInMinutes: 240
pool:
vmImage: macOS-12
variables:
Expand Down
2 changes: 1 addition & 1 deletion azure-pipelines/dotnet-test-cloud.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ if ($x86) {
--filter "TestCategory!=FailsInCloudTest$env:TESTFILTER" `
--collect "Code Coverage;Format=cobertura" `
--settings "$PSScriptRoot/test.runsettings" `
--blame-hang-timeout 600s `
--blame-hang-timeout 1500s `
--blame-crash `
-bl:"$ArtifactStagingFolder/build_logs/test.binlog" `
--diag "$ArtifactStagingFolder/test_logs/diag.log;TraceLevel=info" `
Expand Down
2 changes: 1 addition & 1 deletion src/Microsoft.Windows.CsWin32/Generator.cs
Original file line number Diff line number Diff line change
Expand Up @@ -1473,7 +1473,7 @@ internal void RequestAllInteropTypes(CancellationToken cancellationToken)
{
cancellationToken.ThrowIfCancellationRequested();
TypeDefinition typeDef = this.Reader.GetTypeDefinition(typeDefinitionHandle);
if (typeDef.BaseType.IsNil)
if (typeDef.BaseType.IsNil && (typeDef.Attributes & TypeAttributes.Interface) != TypeAttributes.Interface)
{
continue;
}
Expand Down

0 comments on commit b7584ae

Please sign in to comment.