Add constructor overloads to override AllowMultipleDisposeCalls and AllowMultipleDisposeAttempts

[konard]

Summary

This PR implements the enhancement requested in issue #35 by adding constructor overloads to the Disposable class that allow directly setting the AllowMultipleDisposeCalls and AllowMultipleDisposeAttempts properties without requiring inheritance.

Changes Made

• New Constructor Overloads: Added optional parameters allowMultipleDisposeCalls and allowMultipleDisposeAttempts to all existing constructors:

  • Disposable(Action action, bool allowMultipleDisposeCalls = false, bool allowMultipleDisposeAttempts = false)
  • Disposable(Disposal disposal, bool allowMultipleDisposeCalls = false, bool allowMultipleDisposeAttempts = false)
  • Disposable(bool allowMultipleDisposeCalls, bool allowMultipleDisposeAttempts = false)

• Property Overrides: Override the virtual properties AllowMultipleDisposeCalls and AllowMultipleDisposeAttempts to return constructor-provided values when set, falling back to base implementation when not.

• Comprehensive Tests: Added 6 new unit tests covering all constructor scenarios:

  • Tests for new constructor functionality with both properties enabled
  • Tests for backward compatibility (existing constructors still work as before)
  • Demonstration test showing how the new constructors replace the inheritance workaround

• Bug Fix: Updated DisposalOrderTest to use .NET 8 instead of .NET 7 target framework

Before (Required Inheritance Workaround)

public class DisposableWithMultipleCallsAllowed : Disposable
{
   public DisposableWithMultipleCallsAllowed(Disposal disposal) : base(disposal) { }
   protected override bool AllowMultipleDisposeCalls => true;
}

After (Direct Constructor Usage)

var disposable = new Disposable(disposal, allowMultipleDisposeCalls: true);

Test Results

All tests pass (11/11):

• 6 new constructor tests
• 5 existing tests (including the previously failing DisposalOrderTest)

Backward Compatibility

This change is fully backward compatible. All existing code will continue to work exactly as before, with the new parameters defaulting to false (maintaining current behavior).

🤖 Generated with Claude Code

---

Resolves #35

[konard]

Hi! 👋

I'm investigating issue linksplatform/Data.Doublets#122, which depends on this PR being merged.

This PR looks excellent - all tests pass, it's fully backward compatible, and it provides exactly the functionality requested in issue #35.

Status check: Is there anything blocking this PR from being merged? Any changes needed, or is it just waiting for review?

The Data.Doublets repository is ready to remove its workaround inner class as soon as:

1. This PR is merged ✅ (pending)
2. A new version is released to NuGet 📦 (pending)

Looking forward to seeing this feature available! Thanks for your work on this. 🙏