Merge pull request #344 from creative-commoners/pulls/4.12/frameworktest

FIX Move test requirements out of require-dev

.gitattributes

@@ -1,3 +1,4 @@
+.github export-ignore
 .cow.json export-ignore
 .cow export-ignore
 .gitattributes export-ignore

.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

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": [