Fix GitHub actions (#1013)

* wip * check output test * Fix workflow
spatie · Dec 14, 2019 · 87d332c · 87d332c
1 parent 2304ee7
commit 87d332c
Show file tree

Hide file tree

Showing 2 changed files with 45 additions and 40 deletions.
diff --git a/.github/workflows/run-tests.yml b/.github/workflows/run-tests.yml
@@ -3,42 +3,47 @@ name: run-tests
 on: [push]
 
 jobs:
-  tests:
-    runs-on: ubuntu-latest
-
-    strategy:
-      fail-fast: true
-      matrix:
-        php: [7.2, 7.3, 7.4]
-        laravel: [6.*]
-        dependency-version: [prefer-lowest, prefer-stable]
-        include:
-          - laravel: 6.*
-            testbench: 4.*
-
-    name: P${{ matrix.php }} - L${{ matrix.laravel }} - ${{ matrix.dependency-version }}
-
-    steps:
-      - name: Checkout code
-        uses: actions/checkout@v1
-
-      - name: Cache dependencies
-        uses: actions/cache@v1
-        with:
-          path: ~/.composer/cache/files
-          key: dependencies-laravel-${{ matrix.laravel }}-php-${{ matrix.php }}-composer-${{ hashFiles('composer.json') }}
-
-      - name: Setup PHP
-        uses: shivammathur/setup-php@v1
-        with:
-          php-version: ${{ matrix.php }}
-          extension-csv: curl, mbstring, zip, pcntl, pdo, sqlite, pdo_sqlite, iconv
-          coverage: none
-
-      - name: Install dependencies
-        run: |
-          composer require "laravel/framework:${{ matrix.laravel }}" "orchestra/testbench:${{ matrix.testbench }}" --no-interaction --no-update
-          composer update --${{ matrix.dependency-version }} --prefer-dist --no-interaction --no-suggest
-
-      - name: Execute tests
-        run: vendor/bin/phpunit
+    tests:
+        runs-on: ubuntu-latest
+
+        strategy:
+            fail-fast: true
+            matrix:
+                php: [7.2, 7.3, 7.4]
+                laravel: [6.*]
+                dependency-version: [prefer-lowest, prefer-stable]
+                include:
+                    -   laravel: 6.*
+                        testbench: 4.*
+
+        name: P${{ matrix.php }} - L${{ matrix.laravel }} - ${{ matrix.dependency-version }}
+
+        steps:
+            -   name: Checkout code
+                uses: actions/checkout@v1
+
+            -   name: Install SQLite 3
+                run: |
+                    sudo apt-get update
+                    sudo apt-get install sqlite3
+
+            -   name: Cache dependencies
+                uses: actions/cache@v1
+                with:
+                    path: ~/.composer/cache/files
+                    key: dependencies-laravel-${{ matrix.laravel }}-php-${{ matrix.php }}-composer-${{ hashFiles('composer.json') }}
+
+            -   name: Setup PHP
+                uses: shivammathur/setup-php@v1
+                with:
+                    php-version: ${{ matrix.php }}
+                    extensions: curl, mbstring, zip, pcntl, pdo, sqlite, pdo_sqlite, iconv
+                    coverage: none
+
+            -   name: Install dependencies
+                run: |
+                    composer require "laravel/framework:${{ matrix.laravel }}" "orchestra/testbench:${{ matrix.testbench }}" --no-interaction --no-update
+                    composer update --${{ matrix.dependency-version }} --prefer-dist --no-interaction --no-suggest
+
+            -   name: Execute tests
+                run: vendor/bin/phpunit
diff --git a/tests/Commands/BackupCommandTest.php b/tests/Commands/BackupCommandTest.php
@@ -138,6 +138,7 @@ public function it_can_selectively_backup_db()
         $this
             ->artisan('backup:run --only-db --db-name=db1')
             ->assertExitCode(0);
+
         Storage::disk('local')->assertExists($this->expectedZipPath);
 
         $this
@@ -154,7 +155,6 @@ public function it_can_selectively_backup_db()
 
         $this
             ->artisan('backup:run --only-db --db-name=wrongName')
-
             ->assertExitCode(1);
     }