Comments (2)
There's only a single test failure in this output. The other output is due to the unit tests comprehensive negative condition testing. It's not easy to extract relevant information for real failures, we should try to clean it up. If you examine the final test summary, you'll see the actual # of failures. Search on [FAIL] to get the real thing.
Microsoft.CodeAnalysis.Sarif.Driver.Sdk.AnalyzeCommandBaseTests.UnauthorizedAccessExceptionCreatingSarifLog [FAIL]
Sarif.Driver.UnitTests Total: 17, Errors: 0, Failed: 1, Skipped: 0, Time: 1.107s
The problem here is likely that you are running with admin privileges, which is allowing you to create log files where tests expect to get blocked. We can change the test code to provide a more reliable mechanism to raise UnauthorizedAccessException
from sarif-sdk.
@michaelcfanning fixed this in #85.
(Michael, GitHub didn't auto-resolve it because you typoed the title of the PR ("Resolves #181" instead of "Resolves #81").
from sarif-sdk.
Related Issues (20)
- Validating Visitor should not break on name changes
- [RULE REQUEST] Concise description of new analysis
- CPD conversion support
- HDF->SARIF converters missing location, making invalid for use by GitHub HOT 1
- HDF->SARIF converter output doesn't map status to kind/level/rank correctly HOT 14
- SARIF Multitool query OR logic doesn't work
- Support .NET 8 target
- [REQUEST] Allow users of the SARIF SDK to emit SARIF results with both ruleId and rule.id
- In HDF->SARIF, conversion will fail if no version is specified
- Multitool convert documentation typo HOT 1
- HDF converter crashes on "code" with null value
- Microsoft.CodeAnalysis.Sarif.Query.QueryParseException: Expected compare operator but found <End>
- [FEATURE REQUEST] Converter for dotnet-format report
- Declare Microsoft.Diagnostics.Tracing references as PrivateAssets
- Upgrade Sarif.Multitool to a supported .NET runtime
- Valid SARIF file being rejected by GitHub with locationFromSarifResult error
- SARIF1010.RuleIdMustBeConsistent falsely claims ruleId property is absent
- [Feature Request] Nodes with default objects have null instead of the default object
- Configurable size of preview text for region in ConstructMultilineContextSnippet
- Null Reference Exception in SarifLogger when Extensions are specified. HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from sarif-sdk.