Skip to content

Expand CacheDependencyHelper with new methods and tests#7

Merged
brandonhenricks merged 1 commit intomainfrom
feature/tests-added
Jan 28, 2025
Merged

Expand CacheDependencyHelper with new methods and tests#7
brandonhenricks merged 1 commit intomainfrom
feature/tests-added

Conversation

@brandonhenricks
Copy link
Owner

The CacheDependencyHelper class has been significantly expanded with new methods for creating cache keys and dependencies for content and web page items. Detailed XML documentation has been added for each method.

The CONTENT_TYPE_NAME field in TestContentItemFieldsSource and TestWebPageFieldsSource classes has been changed from static to const in multiple test files.

In TypeExtensionsTests.cs, the Assert.That method call has been updated to use square brackets for array initialization.

A new test class CacheDependencyHelperTests has been added, including various unit tests for the methods in CacheDependencyHelper, with mock implementations of IContentItemFieldsSource and IWebPageFieldsSource for testing purposes.

The `CacheDependencyHelper` class has been significantly expanded with new methods for creating cache keys and dependencies for content and web page items. Detailed XML documentation has been added for each method.

The `CONTENT_TYPE_NAME` field in `TestContentItemFieldsSource` and `TestWebPageFieldsSource` classes has been changed from `static` to `const` in multiple test files.

In `TypeExtensionsTests.cs`, the `Assert.That` method call has been updated to use square brackets for array initialization.

A new test class `CacheDependencyHelperTests` has been added, including various unit tests for the methods in `CacheDependencyHelper`, with mock implementations of `IContentItemFieldsSource` and `IWebPageFieldsSource` for testing purposes.
@brandonhenricks brandonhenricks merged commit 78d6073 into main Jan 28, 2025
5 checks passed
@brandonhenricks brandonhenricks deleted the feature/tests-added branch January 28, 2025 19:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant