Skip to content

Commit

Permalink
Merge remote-tracking branch 'origin/dev' into issue/NEST-464
Browse files Browse the repository at this point in the history
  • Loading branch information
wAsnk committed Aug 7, 2023
2 parents dc9c680 + ab9a656 commit 54584e6
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 6 deletions.
2 changes: 1 addition & 1 deletion Lombiq.Tests.UI.Shortcuts/Lombiq.Tests.UI.Shortcuts.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@
</ItemGroup>

<ItemGroup Condition="'$(NuGetBuild)' == 'true'">
<PackageReference Include="Lombiq.HelpfulLibraries.OrchardCore" Version="6.0.0" />
<PackageReference Include="Lombiq.HelpfulLibraries.OrchardCore" Version="7.0.0" />
</ItemGroup>

</Project>
2 changes: 1 addition & 1 deletion Lombiq.Tests.UI/Docs/TestableOrchardCoreApps.md
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,7 @@ Tips on making specific features testable are under the ["Creating tests" page](
- Some features send out e-mails. You can test them with the Lombiq UI Testing Toolbox's built-in feature to run an isolated local SMTP server with a web UI. The `OrchardCore.Email` feature will be automatically enabled, as well as the rest of the configuration applied.
- If you want the site to use Azure Blob Storage then you have to do the following:
- The `OrchardCore.Media.Azure` feature will be automatically enabled, as well as the rest of the configuration applied.
- It's recommended that you use the [Azurite emulator](https://docs.microsoft.com/en-us/azure/storage/common/storage-use-azurite) as the storage for tests, not a real Azure Blob Storage resource. This is used by the UI Testing Toolbox by default. Be sure that it's running when the tests are executing.
- It's recommended that you use the [Azurite emulator](https://docs.microsoft.com/en-us/azure/storage/common/storage-use-azurite) as the storage for tests, not a real Azure Blob Storage resource. This is used by the UI Testing Toolbox by default. Be sure that it's running when the tests are executing. You'll need at least v3.25.0 of Azurite (you can check this with `azurite -v`, otherwise you'll get "The API version 2022-11-02 is not supported by Azurite." exceptions).
- If you want to use Blob Storage during local development then you can also configure it in your app's _appsettings.json_ or _appsettings.Development.json_ file like below but this is not necessary for UI testing:

```json
Expand Down
6 changes: 3 additions & 3 deletions Lombiq.Tests.UI/Lombiq.Tests.UI.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -93,9 +93,9 @@

<ItemGroup Condition="'$(NuGetBuild)' == 'true'">
<PackageReference Include="Lombiq.Tests" Version="2.2.2" />
<PackageReference Include="Lombiq.HelpfulLibraries.Cli" Version="6.0.0" />
<PackageReference Include="Lombiq.HelpfulLibraries.OrchardCore" Version="6.0.0" />
<PackageReference Include="Lombiq.HelpfulLibraries.Refit" Version="6.0.0" />
<PackageReference Include="Lombiq.HelpfulLibraries.Cli" Version="7.0.0" />
<PackageReference Include="Lombiq.HelpfulLibraries.OrchardCore" Version="7.0.0" />
<PackageReference Include="Lombiq.HelpfulLibraries.Refit" Version="7.0.0" />
<PackageReference Include="Lombiq.Npm.Targets" Version="1.4.0" />
<!--The reference for NodeJs Extensions is for the reason to prevent NuGet build
to use a PNPM version higher than 7.x.x. We need to upgrade solutions to use PNPM 8, also see
Expand Down
4 changes: 3 additions & 1 deletion Lombiq.Tests.UI/Services/WebDriverFactory.cs
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,8 @@ namespace Lombiq.Tests.UI.Services;

public static class WebDriverFactory
{
private static readonly object _setupLock = new();

public static ChromeDriver CreateChromeDriver(BrowserConfiguration configuration, TimeSpan pageLoadTimeout)
{
ChromeDriver CreateDriverInner(ChromeDriverService service)
Expand Down Expand Up @@ -163,7 +165,7 @@ private static TDriver CreateDriver<TDriver>(string browserName, Func<TDriver> d
{
try
{
DriverSetup.AutoSetUp(browserName);
lock (_setupLock) DriverSetup.AutoSetUp(browserName);
return driverFactory();
}
catch (Exception ex)
Expand Down

0 comments on commit 54584e6

Please sign in to comment.