Skip to content

Commit

Permalink
Make spec tests use analyzers fully qualified name (#953)
Browse files Browse the repository at this point in the history
This will help split this repo up into separate packages with embedded
analyzers.
  • Loading branch information
j3parker authored Aug 7, 2024
1 parent aaa1ced commit 907b8cc
Show file tree
Hide file tree
Showing 34 changed files with 34 additions and 37 deletions.
2 changes: 1 addition & 1 deletion src/D2L.CodeStyle.SpecTests/SpecTestFixtureBase.cs
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ private DiagnosticAnalyzer GetAnalyzerNameFromSpec( string source ) {

string analyzerName = headerContents.Substring( "analyzer: ".Length ).Trim();

var type = Type.GetType( analyzerName + ", D2L.CodeStyle.Analyzers" );
var type = Type.GetType( analyzerName );

Assert.NotNull( type, "couldn't get type for analyzer {0}", analyzerName );

Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.Language.AttributeAliasesAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.Language.AttributeAliasesAnalyzer, D2L.CodeStyle.Analyzers

using System;

Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.Language.AwaitedTasksAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.Language.AwaitedTasksAnalyzer, D2L.CodeStyle.Analyzers

using System.Threading.Tasks;
using System.Runtime.CompilerServices;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.Async.BlockingAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.Async.BlockingAnalyzer, D2L.CodeStyle.Analyzers

using System;
using System.Threading.Tasks;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.Language.ClassShouldBeSealedAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.Language.ClassShouldBeSealedAnalyzer, D2L.CodeStyle.Analyzers

namespace D2L.CodeStyle.Analyzers.Specs {
// no diagnostic because they're public
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.Configs.ConfigViewerAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.Configs.ConfigViewerAnalyzer, D2L.CodeStyle.Analyzers

namespace D2L.LP.Configuration.Config.Domain {
public interface IConfigViewer {
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.ConsistentParameterAttributesAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.ConsistentParameterAttributesAnalyzer, D2L.CodeStyle.Analyzers

using System;
using D2L.CodeStyle.Annotations.Contract
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.ConstantAttributeAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.ConstantAttributeAnalyzer, D2L.CodeStyle.Analyzers

using System;

Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.D2LPageAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.D2LPageAnalyzer, D2L.CodeStyle.Analyzers

namespace D2L.Web
{
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.DangerousMemberUsages.DangerousMemberUsagesAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.DangerousMemberUsages.DangerousMemberUsagesAnalyzer, D2L.CodeStyle.Analyzers

using System;
using System.Reflection;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.DangerousMemberUsages.DangerousMemberUsagesAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.DangerousMemberUsages.DangerousMemberUsagesAnalyzer, D2L.CodeStyle.Analyzers

using System;

Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.Language.DefaultValueConsistencyAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.Language.DefaultValueConsistencyAnalyzer, D2L.CodeStyle.Analyzers

namespace ThingThatShouldntCrash {
public void FunctionOutOfPlace( int arg = 3 );
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@

// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.Events.EventHandlerLoaderTypesAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.Events.EventHandlerLoaderTypesAnalyzer, D2L.CodeStyle.Analyzers

namespace D2L.LP.Distributed.Events.Domain {

Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.Events.EventHandlersDisallowedListAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.Events.EventHandlersDisallowedListAnalyzer, D2L.CodeStyle.Analyzers

namespace D2L.LP.Distributed.Events.ExternalPublish.UserInteraction {
public sealed class UserInteractionEvent { }
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@

// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.Events.EventPublisherEventTypesAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.Events.EventPublisherEventTypesAnalyzer, D2L.CodeStyle.Analyzers

namespace D2L.LP.Distributed.Events.Domain {

Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@

// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.Events.EventTypesAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.Events.EventTypesAnalyzer, D2L.CodeStyle.Analyzers

namespace D2L.LP.Distributed.Events.Domain {

Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.Async.ExplicitCancellationTokenArgumentAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.Async.ExplicitCancellationTokenArgumentAnalyzer, D2L.CodeStyle.Analyzers

using System.Threading;
using System.Threading.Tasks;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.LaunchDarkly.FeatureDefinitionAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.LaunchDarkly.FeatureDefinitionAnalyzer, D2L.CodeStyle.Analyzers

namespace D2L.LP.LaunchDarkly {
public abstract class FeatureDefinition<TValue> { }
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.Immutability.ImmutabilityAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.Immutability.ImmutabilityAnalyzer, D2L.CodeStyle.Analyzers

using System;
using System.Collections.Generic;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.Immutability.ImmutabilityAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.Immutability.ImmutabilityAnalyzer, D2L.CodeStyle.Analyzers

using System;
using System.Collections.Generic;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.SystemCollectionsImmutable.ImmutableCollectionsAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.SystemCollectionsImmutable.ImmutableCollectionsAnalyzer, D2L.CodeStyle.Analyzers

using System;
using System.Collections.Generic;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.DataRecordConverters.InterfaceBinderAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.DataRecordConverters.InterfaceBinderAnalyzer, D2L.CodeStyle.Analyzers

using System;
using D2L.LP.Serialization;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.JsonParamBinderAttribute.JsonParamBinderAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.JsonParamBinderAttribute.JsonParamBinderAnalyzer, D2L.CodeStyle.Analyzers

using System;
using D2L.CodeStyle.Analyzers.ApiUsage.JsonParamBinderAttribute;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.NoAsyncFuncsAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.NoAsyncFuncsAnalyzer, D2L.CodeStyle.Analyzers

using System;
using System.Runtime.CompilerServices;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.ServiceLocator.OldAndBrokenServiceLocatorAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.ServiceLocator.OldAndBrokenServiceLocatorAnalyzer, D2L.CodeStyle.Analyzers

using System;
using D2L.LP.Extensibility.Activation.Domain;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.Language.OnlyVisibleToAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.Language.OnlyVisibleToAnalyzer, D2L.CodeStyle.Analyzers

using System;

Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.Immutability.ReadOnlyParameterAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.Immutability.ReadOnlyParameterAnalyzer, D2L.CodeStyle.Analyzers

using System;

Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.Serialization.ReflectionSerializerAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.Serialization.ReflectionSerializerAnalyzer, D2L.CodeStyle.Analyzers

using System;
using D2L.LP.Serialization;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.Language.RequireNamedArgumentsAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.Language.RequireNamedArgumentsAnalyzer, D2L.CodeStyle.Analyzers

using D2L.CodeStyle.Annotations.Contract;

Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.RpcAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.RpcAnalyzer, D2L.CodeStyle.Analyzers

using System;
namespace D2L.Web {
Expand Down
2 changes: 1 addition & 1 deletion tests/D2L.CodeStyle.Analyzers.Test/Specs/RpcAnalyzer.cs
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.RpcAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.RpcAnalyzer, D2L.CodeStyle.Analyzers

using System;
using System.Collections.Generic;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.Serialization.SerializerAttributeAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.ApiUsage.Serialization.SerializerAttributeAnalyzer, D2L.CodeStyle.Analyzers

using System;
using D2L.LP.Serialization;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.Immutability.StatelessFuncAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.Immutability.StatelessFuncAnalyzer, D2L.CodeStyle.Analyzers

using System;

Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// analyzer: D2L.CodeStyle.Analyzers.Language.StructShouldBeReadonlyAnalyzer
// analyzer: D2L.CodeStyle.Analyzers.Language.StructShouldBeReadonlyAnalyzer, D2L.CodeStyle.Analyzers

namespace D2L.CodeStyle.Analyzers.Specs.Good {

Expand Down

0 comments on commit 907b8cc

Please sign in to comment.