Skip to content
Merged
Show file tree
Hide file tree
Changes from 5 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
99 changes: 99 additions & 0 deletions PowerKit.Tests/CellTests.cs
Original file line number Diff line number Diff line change
@@ -0,0 +1,99 @@
using FluentAssertions;
using PowerKit;
using Xunit;

namespace PowerKit.Tests;

public class CellTests
{
[Fact]
public void TryOpen_Test()
{
// Arrange
var cell = new Cell<int?>();
cell.Store(42);

// Act
var result = cell.TryOpen(out var value);

// Assert
result.Should().BeTrue();
value.Should().Be(42);
}

[Fact]
public void TryOpen_Unset_Test()
{
// Arrange
var cell = new Cell<int?>();

// Act
var result = cell.TryOpen(out var value);

// Assert
result.Should().BeFalse();
value.Should().BeNull();
}

[Fact]
public void TryOpen_Null_Test()
{
// Arrange
var cell = new Cell<int?>();
cell.Store(null);

// Act
var result = cell.TryOpen(out var value);

// Assert
result.Should().BeTrue();
value.Should().BeNull();
}

[Fact]
public void OpenOrDefault_Test()
{
// Arrange
var cell = new Cell<int?>();
cell.Store(42);

// Act
var value = cell.OpenOrDefault();
var valueOrFallback = cell.OpenOrDefault(99);

// Assert
value.Should().Be(42);
valueOrFallback.Should().Be(42);
}

[Fact]
public void OpenOrDefault_Unset_Test()
{
// Arrange
var cell = new Cell<int?>();

// Act
var value = cell.OpenOrDefault();
var valueOrFallback = cell.OpenOrDefault(99);

// Act & assert
value.Should().BeNull();
valueOrFallback.Should().Be(99);
}

[Fact]
public void OpenOrDefault_Unset_Test()

Check failure on line 85 in PowerKit.Tests/CellTests.cs

View workflow job for this annotation

GitHub Actions / main / test (windows-latest)

Type 'CellTests' already defines a member called 'OpenOrDefault_Unset_Test' with the same parameter types

Copilot AI Apr 12, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are two test methods with the same name/signature OpenOrDefault_Unset_Test in this file, which will cause a compile-time error. Rename the second one (the one that stores null) to a distinct name that reflects the scenario (e.g., OpenOrDefault_Null_Test).

Suggested change
public void OpenOrDefault_Unset_Test()
public void OpenOrDefault_Null_Test()

Copilot uses AI. Check for mistakes.
{
// Arrange
var cell = new Cell<int?>();
cell.Store(null);

// Act
var value = cell.OpenOrDefault();
var valueOrFallback = cell.OpenOrDefault(99);

// Act & assert
value.Should().BeNull();
valueOrFallback.Should().BeNull();
}
}
42 changes: 42 additions & 0 deletions PowerKit/Cell.cs
Original file line number Diff line number Diff line change
@@ -0,0 +1,42 @@
namespace PowerKit;

/// <summary>
/// Container for a value that may or may not be set.
/// Essentially <see cref="System.Nullable{T}" />, but for cases where null is also a valid value.
/// </summary>
internal class Cell<T>
{
private bool _isValueSet;
private T _value = default!;

/// <summary>
/// Stores the specified value in the cell.
/// </summary>
public void Store(T value)
{
_value = value;
_isValueSet = true;
}

/// <summary>
/// Tries to retrieve the value stored in the cell.
/// Returns <see langword="true" /> if a value has been stored, <see langword="false" /> otherwise.
/// </summary>
public bool TryOpen(out T value)
Comment thread
Tyrrrz marked this conversation as resolved.
{
if (_isValueSet)
{
value = _value;
return true;
}

value = default!;
return false;
}

/// <summary>
/// Retrieves the value stored in the cell, or <paramref name="defaultValue" /> if no value has been stored.
/// </summary>
public T OpenOrDefault(T defaultValue = default!) =>
TryOpen(out var value) ? value : defaultValue;
}
Loading