Replace unnecesary calls to SaveChangesAsync by FlushAsync (#17084) #595
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| name: Main - CI | |
| on: | |
| # Run it on main and release pushes too, in case we merge from a branch that's not up-to-date with the target branch | |
| # and breaks something after merge (or if we push to main). | |
| push: | |
| paths-ignore: | |
| - '**/*.md' | |
| - 'mkdocs.yml' | |
| - 'src/docs/**/*' | |
| branches: [ main, release/** ] | |
| env: | |
| DOTNET_SKIP_FIRST_TIME_EXPERIENCE: true | |
| DOTNET_CLI_TELEMETRY_OPTOUT: true | |
| jobs: | |
| test: | |
| runs-on: ${{ matrix.os }} | |
| name: Build & Test | |
| strategy: | |
| fail-fast: false | |
| matrix: | |
| os: [ubuntu-latest, windows-latest] | |
| steps: | |
| - uses: actions/checkout@v4 | |
| - uses: actions/setup-node@v4 | |
| with: | |
| node-version: "15" | |
| - uses: actions/setup-dotnet@v4 | |
| with: | |
| dotnet-version: '8.0.x' | |
| - name: Build | |
| # See pr_ci.yml for the reason why we disable NuGet audit warnings. | |
| run: | | |
| dotnet build -c Release -warnaserror /p:TreatWarningsAsErrors=true /p:RunAnalyzers=true /p:NuGetAudit=false | |
| - name: Unit Tests | |
| run: | | |
| dotnet test -c Release --no-build ./test/OrchardCore.Tests/OrchardCore.Tests.csproj | |
| - name: Functional Tests | |
| if: matrix.os == 'ubuntu-latest' | |
| run: | | |
| cd test/OrchardCore.Tests.Functional | |
| npm install | |
| npm run cms:test | |
| npm run mvc:test | |
| - uses: actions/upload-artifact@v4 | |
| if: matrix.os == 'ubuntu-latest' && failure() | |
| with: | |
| name: Functional Test failure | |
| path: | | |
| test/OrchardCore.Tests.Functional/cms-tests/cypress/screenshots | |
| src/OrchardCore.Cms.Web/App_Data_Tests/logs |