FIX Move test requirements out of require-dev

silverstripe · Dec 20, 2022 · 62c49ca · 62c49ca
1 parent 07e1b32
commit 62c49ca
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/.gitattributes b/.gitattributes
@@ -1,3 +1,4 @@
+.github export-ignore
 .cow.json export-ignore
 .cow export-ignore
 .gitattributes export-ignore

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -17,6 +17,9 @@ jobs:
     with:
       # installer contains a sample behat.yml file, though there are no behat tests to run
       endtoend: false
+      # require the following in ci.yml rather than composer.json require-dev specifically for installer because unlike
+      # regular modules, require-dev will get installed to projects during `composer create-project silverstripe/installer`
+      composer_require_extra: silverstripe/recipe-testing:^2 silverstripe/frameworktest:^0.4.2 mikey179/vfsstream:^1.6.10
       extra_jobs: |
         - endtoend: true
           endtoend_suite: admin

diff --git a/composer.json b/composer.json
@@ -10,9 +10,7 @@
         "silverstripe/login-forms": "4.8.x-dev"
     },
     "require-dev": {
-        "silverstripe/recipe-testing": "^2",
-        "silverstripe/frameworktest": "^0.4.2",
-        "mikey179/vfsstream": "^1.6.10"
+        "phpunit/phpunit": "^9.5"
     },
     "extra": {
         "project-files": [