Skip to content

Commit

Permalink
🎨 Remove redundant step to build tests
Browse files Browse the repository at this point in the history
`dotnet test`, when run without the `--no-build` flag, will build the
tests automatically. Also, it seems like including the `--no-build` flag
also causes the test suite not to run at all. Yeah.
  • Loading branch information
canterberry committed May 29, 2019
1 parent f6c3276 commit 821f21b
Showing 1 changed file with 4 additions and 20 deletions.
24 changes: 4 additions & 20 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,16 +9,13 @@ workflows:
version: 2
default:
jobs:
- build-sources:
- build:
requires:
- initialize
- build-tests:
requires:
- build-sources
- initialize
- test:
requires:
- build-tests
- build

jobs:
initialize:
Expand All @@ -29,7 +26,7 @@ jobs:
root: /docker
paths:
- project
build-sources:
build:
<<: *defaults
steps:
- attach_workspace:
Expand All @@ -42,19 +39,6 @@ jobs:
paths:
- project/Blockmason.Link/obj
- project/Blockmason.Link/bin
build-tests:
<<: *defaults
steps:
- attach_workspace:
at: /docker
- run:
name: Building tests
command: dotnet publish --configuration Release Blockmason.Link.Tests/Blockmason.Link.Tests.csproj
- persist_to_workspace:
root: /docker
paths:
- project/Blockmason.Link.Tests/obj
- project/Blockmason.Link.Tests/bin
test:
<<: *defaults
steps:
Expand All @@ -66,4 +50,4 @@ jobs:
dotnet test --logger trx Blockmason.Link.Tests/Blockmason.Link.Tests.csproj
find . -type f -name '*.trx' | xargs /docker/.dotnet/tools/trx2junit
- store_test_results:
path: TestResults
path: Blockmason.Link.Tests/TestResults

0 comments on commit 821f21b

Please sign in to comment.