BETA 11

BETA 11

Author: egil

.github/workflows/deploy-docs.yml

@@ -32,7 +32,7 @@ jobs:
 
       - uses: actions/setup-dotnet@v1
         with:
-          dotnet-version: '3.1.x'
+          dotnet-version: '3.1.x'          
       - uses: actions/setup-dotnet@v1
         with:
           dotnet-version: '5.0.100-rc.1.20452.10'

.github/workflows/main.yml

@@ -117,63 +117,45 @@ jobs:
           dotnet restore ${GITHUB_WORKSPACE}/TemplateTest --source ${GITHUB_WORKSPACE}/packages --source https://api.nuget.org/v3/index.json
           dotnet test ${GITHUB_WORKSPACE}/TemplateTest
 
-  dotnet-format:
-    if: github.event_name == 'pull_request'
-    runs-on: ubuntu-latest
-    steps:
-      - name: Checkout repository
-        uses: actions/checkout@v2
-        
-      - uses: jwalton/gh-find-current-pr@v1
-        id: findPr
-        with:
-          github-token: ${{ secrets.GITHUB_TOKEN }}
-
-      - name: Checkout PR
-        uses: dawidd6/action-checkout-pr@v1
-        with:
-          pr: steps.findPr.outputs.number
-
-      - uses: actions/setup-dotnet@v1
-        with:
-          dotnet-version: '3.1.x'
-      - uses: actions/setup-dotnet@v1
-        with:
-          dotnet-version: '5.0.100-rc.1.20452.10'
-
-      - name: Install dotnet-format
-        run: dotnet tool install -g dotnet-format
-
-      - name: Run dotnet-format
-        run: dotnet format ./bunit.sln --verbosity diag
-
-      - name: Commit formatted files
-        uses: EndBug/[email protected]
-        with:
-          add: '.'
-          author_name: Github Actions
-          author_email: [email protected]
-          message: Automated dotnet-format update
-        env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-
   sonarcloud:
     if: github.event_name != 'schedule' || github.event_name != 'pull_request'
     runs-on: ubuntu-latest
     steps:
     - name: Checkout repository
       uses: actions/checkout@v2
       with:
-        fetch-depth: 0
+        fetch-depth: 0w
 
-    - name: Create sonar-project.properties
-      run: echo -n -e "sonar.organization=egil\nsonar.projectKey=egil_bUnit\nsonar.sources=./src\nsonar.tests=./tests\n" > sonar-project.properties
+    - uses: dotnet/nbgv@master
+      with:
+        setAllVars: true
+
+    - name: Setting VERSION and BRANCH env
+      run: |
+        echo "::set-env name=VERSION::$NBGV_NuGetPackageVersion"
+        echo "::set-env name=BRANCH::$(echo ${GITHUB_REF#refs/heads/} | sed 's/\//_/g')"
 
-    - name: SonarCloud Scan
-      uses: sonarsource/sonarcloud-github-action@master
+    - uses: actions/setup-dotnet@v1
+      with:
+        dotnet-version: '3.1.x'
+    - uses: actions/setup-dotnet@v1
+      with:
+        dotnet-version: '5.0.100-rc.1.20452.10'
+
+    - name: Install dotnet-sonarscanner
+      run: dotnet tool install --global dotnet-sonarscanner
+    - name: Running dotnet-sonarscanner
+      shell: bash
       env:
-        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
+        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+      run: |
+        dotnet sonarscanner begin /k:"egil_bUnit" /o:"egil" /n:"bUnit" /v:"$NBGV_NuGetPackageVersion" /d:sonar.login="$SONAR_TOKEN" /d:sonar.host.url=https://sonarcloud.io /d:sonar.branch.name="$BRANCH" /d:sonar.verbose="true" /d:sonar.cs.opencover.reportsPaths="tests/**/coverage/*.opencover.xml"
+        dotnet build
+        dotnet test tests/bunit.core.tests/ /p:CollectCoverage=true /p:CoverletOutput=./coverage/ /p:CoverletOutputFormat=opencover /p:ExcludeByAttribute=\"Obsolete,GeneratedCodeAttribute,CompilerGeneratedAttribute\"
+        dotnet test tests/bunit.web.tests/ /p:CollectCoverage=true /p:CoverletOutput=./coverage/ /p:CoverletOutputFormat=opencover /p:ExcludeByAttribute=\"Obsolete,GeneratedCodeAttribute,CompilerGeneratedAttribute\"
+        dotnet test tests/bunit.xunit.tests/ /p:CollectCoverage=true /p:CoverletOutput=./coverage/ /p:CoverletOutputFormat=opencover /p:ExcludeByAttribute=\"Obsolete,GeneratedCodeAttribute,CompilerGeneratedAttribute\"
+        dotnet sonarscanner end /d:sonar.login="$SONAR_TOKEN"
 
   code-ql:
     if: github.event_name != 'pull_request'

.gitignore

@@ -334,3 +334,5 @@ bunit.docs/log.txt
 
 .store
 *coverage*.info
+.sonarqube
+tests/*/coverage

CHANGELOG.md

@@ -2,9 +2,98 @@
 
 All notable changes to **bUnit** will be documented in this file. The project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+<!-- The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/) -->
+
+## [UNRELEASED BETA 11]
+
+The following section list all changes in beta-11.
+
+### Added
+List of new features.
+
+- Two new overloads to the `RenderFragment()` and `ChildContent()` component parameter factory methods have been added that takes a `RenderFragment` as input. By [@egil](https://github.com/egil) in [#203](https://github.com/egil/bUnit/pull/203).
+
+- Added a `ComponentParameterCollection` type. The `ComponentParameterCollection` is a collection of component parameters, that knows how to turn those components parameters into a `RenderFragment`, which will render a component and pass any parameters inside the collection to that component. That logic was spread out over multiple places in bUnit, and is now owned by the `ComponentParameterCollection` type. By [@egil](https://github.com/egil) in [#203](https://github.com/egil/bUnit/pull/203).
+
+- Added additional placeholder services for `NavigationManager`, `HttpClient`, and `IStringLocalizer`, to make it easier for users to figure out why a test is failing due to missing service registration before rendering a component. By [@joro550](https://github.com/joro550) in [#223](https://github.com/egil/bUnit/pull/223).
+
+- Added `Key` class that represents a keyboard key and helps to avoid constructing `KeyboardEventArgs` object manually. The key can be passed to `KeyPress`, `KeyDown`, or `KeyUp` helper methods to raise keyboard events. The `Key` class provides static special keys or can be obtained from character or string. Keys can be combined with key modifiers: `Key.Enter + Key.Alt`.
+
+  For example, this makes it easier to trigger keyboard events on an element:
+  
+  ```csharp
+  var cut = ctx.RenderComponent<ComponentWithKeyboardEvents>();
+  var element = cut.Find("input");
+  
+  element.KeyDown(Key.Enter + Key.Control); // Triggers onkeydown event with Ctrl + Enter
+  element.KeyUp(Key.Control + Key.Shift + 'B'); // Triggers onkeyup event with Ctrl + Shift  + B
+  element.KeyPress('1'); // Triggers onkeypress event with key 1
+  element.KeyDown(Key.Alt + "<"); // Triggers onkeydown event with Alt + <
+  ```
+
+  By [@duracellko](https://github.com/duracellko) in [#101](https://github.com/egil/bUnit/issues/101).
+
+- Added support for registering/adding components to a test context root render tree, which components under test is rendered inside. This allows you to simplify the "arrange" step of a test when a component under test requires a certain render tree as its parent, e.g. a cascading value.
+  
+  For example, to pass a cascading string value `foo` to all components rendered with the test context, do the following:
+
+  ```csharp
+  ctx.RenderTree<CascadingValue<string>>(parameters => parameters.Add(p => p.Value, "foo"));
+  var cut = ctx.RenderComponent<ComponentReceivingFoo>();
+  ```
+
+  By [@egil](https://github.com/egil) in [#236](https://github.com/egil/bUnit/pull/236).
+
+- Added "catch-all" `Setup` method to bUnit's mock JS runtime, that allows you to specify only the type when setting up a planned invocation. By [@nemesv](https://github.com/nemesv) in [#234](https://github.com/egil/bUnit/issues/234).
+
+### Changed
+List of changes in existing functionality.
+
+- The `ComponentParameterBuilder` has been renamed to `ComponentParameterCollectionBuilder`, since it now builds the `ComponentParameterCollection` type, introduced in this release of bUnit. By [@egil](https://github.com/egil) in [#203](https://github.com/egil/bUnit/pull/203).
+
+- `ComponentParameterCollectionBuilder` now allows adding cascading values that is not directly used by the component type it targets. This makes it possible to add cascading values to children of the target component. By [@egil](https://github.com/egil) in [#203](https://github.com/egil/bUnit/pull/203).
+
+- The `Add(object)` has been replaced by `AddCascadingValue(object)` in `ComponentParameterCollectionBuilder`, to make it more clear that an unnamed cascading value is being passed to the target component or one of its child components. It is also possible to pass unnamed cascading values using the `Add(parameterSelector, value)` method, which now correctly detect if the selected cascading value parameter is named or unnamed. By [@egil](https://github.com/egil) in [#203](https://github.com/egil/bUnit/pull/203).
+
+- It is now possible to call the `Add()`, `AddChildContent()` methods on `ComponentParameterCollectionBuilder`, and the factory methods `RenderFragment()`, `ChildContent()`, and `Template()`, _**multiple times**_ for the same parameter, if it is of type `RenderFragment` or `RenderFragment<TValue>`. Doing so previously would either result in an exception or just the last passed `RenderFragment` to be used. Now all the provided `RenderFragment` or `RenderFragment<TValue>` will be combined at runtime into a single `RenderFragment` or `RenderFragment<TValue>`.
+
+  For example, this makes it easier to pass e.g. both a markup string and a component to a `ChildContent` parameter:
+   
+  ```csharp
+  var cut = ctx.RenderComponent<Component>(parameters => parameters
+    .AddChildContent("<h1>Below you will find a most interesting alert!</h1>")
+    .AddChildContent<Alert>(childParams => childParams
+      .Add(p => p.Heading, "Alert heading")
+      .Add(p => p.Type, AlertType.Warning)
+      .AddChildContent("<p>Hello World</p>")
+    )
+  );
+  ```
+  By [@egil](https://github.com/egil) in [#203](https://github.com/egil/bUnit/pull/203).
+
+- All test doubles are now in the same namespace, `Bunit.TestDoubles`. So all import statements for `Bunit.TestDoubles.JSInterop` and `Bunit.TestDoubles.Authorization` must be changed to `Bunit.TestDoubles`. By [@egil](https://github.com/egil) in [#223](https://github.com/egil/bUnit/pull/223).
+
+- Marked MarkupMatches methods as assertion methods to stop SonarSource analyzers complaining about missing assertions in tests. By [@egil](https://github.com/egil) in [#229](https://github.com/egil/bUnit/pull/229).
+
+- `AddTestAuthorization` now extends `TestContext` instead of `TestServiceProvider`, and also automatically adds the `CascadingAuthenticationState` component to the root render tree. [@egil](https://github.com/egil) in [#237](https://github.com/egil/bUnit/pull/367).
+
+### Deprecated
+List of soon-to-be removed features.
+
+### Removed
+List of now removed features.
+
+- The async event dispatcher helper methods have been removed (e.g. `ClickAsync()`), as they do not provide any benefit. If you have an event that triggers async operations in the component under test, instead use `cut.WaitForState()` or `cut.WaitForAssertion()` to await the expected state in the component.  
+
+### Fixed
+List of any bug fixes.
+
+- Using the ComponentParameterCollectionBuilder's `Add(p => p.Param, value)` method to add a unnamed cascading value didn't create an unnnamed cascading value parameter. By [@egil](https://github.com/egil) in [#203](https://github.com/egil/bUnit/pull/203). Credits to [Ben Sampica (@benjaminsampica)](https://github.com/benjaminsampica) for reporting and helping investigate this issue.
+- Triggered events now bubble correctly up the DOM tree and triggers other events of the same type. This is a **potentially breaking change,** since this changes the behaviour of event triggering and thus you might see tests start breaking as a result hereof. By [@egil](https://github.com/egil) in [#119](https://github.com/egil/bUnit/issues/119).
+
 ## [1.0.0-beta 10] - 2020-09-15
 
-The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
+The following section list all changes in beta-10.
 
 ### Added
 List of new features.

docs/.editorconfig

@@ -10,6 +10,7 @@ trim_trailing_whitespace = false
 
 [*.{cs,razor}]
 tab_size = 2
+indent_style = space
 dotnet_diagnostic.BL0001.severity = none
 dotnet_diagnostic.BL0002.severity = none
 dotnet_diagnostic.BL0003.severity = none

docs/samples/components/PrintCascadingValue.razor

@@ -0,0 +1,6 @@
+Cascading value: @Value
+@code 
+{
+  [CascadingParameter]
+  public string Value { get; set; }
+}

docs/samples/components/UserInfo.razor

@@ -9,20 +9,17 @@
 {
   <h1>Please log in!</h1>
 }
-<CascadingAuthenticationState>
-  <AuthorizeView>
-    <Authorized>
-      <p>State: Authorized</p>
-    </Authorized>
-    <Authorizing>
-      <p>State: Authorizing</p>
-    </Authorizing>
-    <NotAuthorized>
-      <p>State: Not authorized</p>
-    </NotAuthorized>
-  </AuthorizeView>
-</CascadingAuthenticationState>
-
+<AuthorizeView>
+  <Authorized>
+    <p>State: Authorized</p>
+  </Authorized>
+  <Authorizing>
+    <p>State: Authorizing</p>
+  </Authorizing>
+  <NotAuthorized>
+    <p>State: Not authorized</p>
+  </NotAuthorized>
+</AuthorizeView>
 @code
 {
   bool isAuthenticated = false;

docs/samples/components/UserRights.razor

@@ -2,28 +2,26 @@
 @using System.Security.Claims
 @using System.Globalization
 
-<CascadingAuthenticationState>
+<AuthorizeView>
+  <h1>Hi @context.User.Identity.Name, you have these claims and rights:</h1>
+</AuthorizeView>
+<ul>
   <AuthorizeView>
-    <h1>Hi @context.User.Identity.Name, you have these claims and rights:</h1>
+    @foreach (var claim in @context.User.FindAll(x => x.Type != ClaimTypes.Name))
+    {
+      <li>@GetClaimName(claim): @claim.Value</li>
+    }
   </AuthorizeView>
-  <ul>
-    <AuthorizeView>
-      @foreach (var claim in @context.User.FindAll(x => x.Type != ClaimTypes.Name))
-      {
-        <li>@GetClaimName(claim): @claim.Value</li>
-      }
-    </AuthorizeView>
-    <AuthorizeView Roles="superuser">
-      <li>You have the role SUPER USER</li>
-    </AuthorizeView>  
-    <AuthorizeView Roles="admin">
-      <li>You have the role ADMIN</li>
-    </AuthorizeView>
-    <AuthorizeView Policy="content-editor">
-      <li>You are a CONTENT EDITOR</li>
-    </AuthorizeView>
-  </ul>
-</CascadingAuthenticationState>
+  <AuthorizeView Roles="superuser">
+    <li>You have the role SUPER USER</li>
+  </AuthorizeView>  
+  <AuthorizeView Roles="admin">
+    <li>You have the role ADMIN</li>
+  </AuthorizeView>
+  <AuthorizeView Policy="content-editor">
+    <li>You are a CONTENT EDITOR</li>
+  </AuthorizeView>
+</ul>
 
 @code 
 {

docs/samples/tests/Directory.Build.props

@@ -2,7 +2,7 @@
 	<PropertyGroup>
 		<TargetFrameworks>netcoreapp3.1;net5.0</TargetFrameworks>
 		<IsPackable>false</IsPackable>
-		<LangVersion>8.0</LangVersion>
+		<LangVersion>9.0</LangVersion>
     	<RazorLangVersion>3.0</RazorLangVersion>
 	</PropertyGroup>
 </Project>

docs/samples/tests/mstest/BunitTestContext.cs

@@ -6,7 +6,7 @@
 
 namespace Bunit.Docs.Samples
 {
-  public abstract class BunitTestContext : ITestContext, IDisposable
+  public abstract class BunitTestContext : IDisposable
   {
     private Bunit.TestContext _context;
 
@@ -29,7 +29,7 @@ public void Dispose()
     public IRenderedComponent<TComponent> RenderComponent<TComponent>(params ComponentParameter[] parameters) where TComponent : IComponent
         => _context?.RenderComponent<TComponent>(parameters) ?? throw new InvalidOperationException("MSTest has not started executing tests yet");
 
-    public IRenderedComponent<TComponent> RenderComponent<TComponent>(Action<ComponentParameterBuilder<TComponent>> parameterBuilder) where TComponent : IComponent
+    public IRenderedComponent<TComponent> RenderComponent<TComponent>(Action<ComponentParameterCollectionBuilder<TComponent>> parameterBuilder) where TComponent : IComponent
         => _context?.RenderComponent<TComponent>(parameterBuilder) ?? throw new InvalidOperationException("MSTest has not started executing tests yet");
   }
 }