Skip to content

Commit d58e360

Browse files
committed
fix test
1 parent 7e8d22d commit d58e360

File tree

2 files changed

+10
-3
lines changed

2 files changed

+10
-3
lines changed

.claude/CLAUDE.md

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@ This file is organized in tiers:
3434
- .NET 10.0 SDK (as specified in global.json)
3535
- C# 14.0 (as specified in Directory.Build.props)
3636
- Multiple .NET versions (8.0, 9.0, 10.0) for framework compatibility testing
37+
- IDE: The user primarily works in JetBrains Rider, but all IDE features (refactorings, code fixes) must work in both Rider and Visual Studio
3738

3839
---
3940

@@ -205,6 +206,11 @@ The following documentation files in `docs/` are read by both humans and AI agen
205206
- `Empty-....md`, `SingleItem.md`, `ToReadOnlyCollection.md`, `TrimOrNullify.md` -- Utility/helper documentation
206207
- `articles/` -- In-depth articles on Smart Enums, Value Objects, and Discriminated Unions
207208

209+
**Documentation conventions:**
210+
211+
- Use `Console.WriteLine` in doc examples, not logger injection -- unless the example specifically demonstrates closure/capture patterns
212+
- Keep example items consistent across documentation pages -- reuse the same enum items (e.g., Electronics, Clothing, Food) rather than switching between different items within or across pages
213+
208214
**Legacy documentation is frozen.** The `docs/version-7/`, `docs/version-8/` directories and `docs/Version-7.x.x.md`, `docs/Version-8.x.x.md` files are archived and must NEVER be read, updated, or modified. Ignore them completely -- they do not exist for the purposes of any task.
209215

210216
---
@@ -290,6 +296,7 @@ Skipping step 3 is a policy violation, even if you are confident you know the AP
290296
- XML documentation required for all publicly visible types and members in `src/Thinktecture.Runtime.Extensions` and framework integration projects
291297
- XML documentation NOT required in source generator, analyzer, test, and sample projects
292298
- Do not use `#region`/`#endregion`
299+
- In XML documentation, wrap code tokens like `true`, `false`, `null`, `default(T)` in `<c>` tags
293300

294301
**Source Generator Code** (treat as a checklist -- verify all items before presenting code):
295302

@@ -302,6 +309,7 @@ Skipping step 3 is a policy violation, even if you are confident you know the AP
302309
- Use Fully Qualified Names: Use `global::` prefix for type references (e.g., `global::System.ArgumentException`) -- do not generate using directives. Exception: C# language keywords (`string`, `int`, `bool`, etc.) are used as-is
303310
- Use `#if NET9_0_OR_GREATER` conditional compilation in generated code for NET9+-only features
304311
- Generated files use file-scoped namespaces
312+
- Use constants from `Thinktecture.CodeAnalysis.Constants` for well-known names (method names, type names, etc.) -- do not use hardcoded strings or local constants for values already defined there
305313

306314
**Runtime Library Code**:
307315

test/Thinktecture.Runtime.Extensions.SourceGenerator.Tests/Extensions/EventSymbolExtensionsTests/GetEventLocation.cs

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -780,14 +780,13 @@ public void Should_handle_event_with_attributes()
780780
{
781781
var src = @"
782782
using System;
783-
using System.ComponentModel;
784783
785784
namespace Test;
786785
787786
public class MyClass
788787
{
789-
[Browsable(false)]
790-
[Description(""This is a test event"")]
788+
[Obsolete(""test"")]
789+
[CLSCompliant(false)]
791790
public event EventHandler AttributedEvent;
792791
}
793792
";

0 commit comments

Comments
 (0)