-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
OSOE-751: Upgrade to Orchard Core 1.8 #156
Conversation
Lombiq.DataTables.Samples/Services/SampleIndexBasedDataTableDataProvider.cs
Outdated
Show resolved
Hide resolved
Lombiq.DataTables/Tests/Lombiq.DataTables.Tests/UnitTests/Services/LiquidTests.cs
Outdated
Show resolved
Hide resolved
This reverts commit f9c267b. # Conflicts: # Lombiq.DataTables.Samples/Services/SampleIndexBasedDataTableDataProvider.cs # Lombiq.DataTables.Samples/Services/SampleJsonResultDataTableDataProvider.cs # Lombiq.DataTables/Liquid/ActionsLiquidFilter.cs
Even on the latest dev with OC 1.7 if I am running the NuGet solution with its test recipe and I execute the Data Tables Sample recipe and I navigate to the HomePage the DataTables dropdown that the UI Test is trying to find is missing. I get a failing CI on OSOE-751 because of this and I don't know whether I am missing something here. So I know that using the BaseTheme.Samples recipe it's on the UI but this Nuget test recipe I have never tested this exact functionality. For me it seems to be missing. Both on dev both on my OC 1.8 branch. Well atleast the selector I am also unable to run tests locally due to |
Seems like it needed a whole |
{ | ||
await context.SignInDirectlyAsync(); | ||
await context.ExecuteDataTablesSampleRecipeDirectlyAsync(); | ||
|
||
if (!isNugetTest) | ||
if (!checkMainMenu) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Negation should be removed here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I made some changes to this because now you have to set checkMainMenu to true if you don't want it to be checked.
OSOE-751