v6 : Tests / Refactorisation

.gitattributes

@@ -1,3 +1,5 @@
+/tests export-ignore
+/phpunit.xml.dist export-ignore
 .github export-ignore
 .gitattributes export-ignore
 .gitignore export-ignore

.github/ISSUE_TEMPLATE/bug_report.yaml

@@ -0,0 +1,54 @@
+name: 🐛 Bug report
+description: Create a report to help us improve
+labels: ["bug"]
+body:
+  - type: markdown
+    attributes:
+      value: |
+        Thanks for using this bundle and taking the time to fill out this bug report !
+  - type: input
+    id: vite-bundle-version
+    attributes:
+      label: vite-bundle version
+      description: composer show pentatrion/vite-bundle | grep versions
+      placeholder: x.y.z
+    validations:
+      required: true
+  - type: input
+    id: vite-plugin-symfony-version
+    attributes:
+      label: vite-plugin-symfony version
+      description: npm list vite-plugin-symfony
+      placeholder: x.y.z
+    validations:
+      required: true
+  - type: input
+    id: context
+    attributes:
+      label: Symfony version, PHP version, your OS (node version ?)
+      placeholder: "symfony 7.3, PHP 8.2, node 20.5, OS Linux"
+  - type: textarea
+    id: description
+    attributes:
+      label: Description
+      description: A clear and concise description of the problem
+    validations:
+      required: true
+  - type: textarea
+    id: how-to-reproduce
+    attributes:
+      label: How to reproduce
+      description: |
+        ⚠️  This is the most important part of the report ⚠️
+        Without a way to easily reproduce your issue, there is little chance we will be able to help you and work on a fix.
+        Please, take the time to show us some code and/or config that is needed for others to reproduce the problem easily.
+    validations:
+      required: true
+  - type: textarea
+    id: possible-solution
+    attributes:
+      label: Possible Solution
+      description: |
+        Optional: only if you have suggestions on a fix/reason for the bug
+        Don't hesitate to create a pull request with your solution, it helps get faster feedback. You should open a pull request on the main symfony-vite-dev monorepo : https://github.com/lhapaipai/symfony-vite-dev.
+        This repository is a "subtree split": a read-only subset of that main repository.

.github/ISSUE_TEMPLATE/config.yaml

@@ -0,0 +1 @@
+blank_issues_enabled: true

.github/ISSUE_TEMPLATE/feature_request.yaml

@@ -0,0 +1,17 @@
+name: 🚀 Feature Request
+description: RFC and ideas for new features and improvements
+body:
+  - type: textarea
+    id: description
+    attributes:
+      label: Description
+      description: A clear and concise description of the new feature
+    validations:
+      required: true
+  - type: textarea
+    id: example
+    attributes:
+      label: Example
+      description: |
+        A simple example of the new feature in action (include PHP code, YAML config, etc.)
+        If the new feature changes an existing feature, include a simple before/after comparison.

.gitignore

@@ -2,4 +2,6 @@
 /.local
 /.idea
 /vendor
-/.php-cs-fixer.cache
+/.php-cs-fixer.cache
+/.phpunit.result.cache
+/composer.lock

CHANGELOG.md

@@ -1,3 +1,9 @@
+## Next
+
+- make services privates.
+- add tests for EntrypointRenderer, EntrypointsLookup and TagRenderer.
+- add preload option (symfony/web-link)
+
 ## v5.0.1
 
 - remove deprecated options

MiGRATION.md

@@ -0,0 +1,5 @@
+<!-- 
+```diff
+- Pentatrion\ViteBundle\Asset\ViteAssetVersionStrategy
++ Pentatrion\ViteBundle\Asset\VersionStrategy\ViteAssetVersionStrategy
+``` -->

README.md

@@ -7,6 +7,8 @@
 This bundle helping you render all of the dynamic `script` and `link` tags needed.
 Essentially, he provides two twig functions to load the correct scripts into your templates.
 
+⚠️ This repository is a "subtree split": a read-only subset of that main repository [symfony-vite-dev](https://github.com/lhapaipai/symfony-vite-dev).
+
 ## Installation
 
 Install the bundle with :
@@ -45,7 +47,6 @@ Add this twig functions in any template or base layout where you need to include
 | ----------------------------------------------------------------------- | :------------------------ |
 | [vite-bundle](https://github.com/lhapaipai/vite-bundle)                 | Symfony Bundle            |
 | [vite-plugin-symfony](https://github.com/lhapaipai/vite-plugin-symfony) | Vite plugin               |
-| [symfony-vite-docs](https://github.com/lhapaipai/symfony-vite-docs)     | Documentation             |
 | [symfony-vite-dev](https://github.com/lhapaipai/symfony-vite-dev)       | Package for contributors  |
 
 ## License

bin/phpunit

@@ -0,0 +1,19 @@
+#!/usr/bin/env php
+<?php
+
+if (!ini_get('date.timezone')) {
+    ini_set('date.timezone', 'UTC');
+}
+
+if (is_file(dirname(__DIR__).'/vendor/phpunit/phpunit/phpunit')) {
+    define('PHPUNIT_COMPOSER_INSTALL', dirname(__DIR__).'/vendor/autoload.php');
+    require PHPUNIT_COMPOSER_INSTALL;
+    PHPUnit\TextUI\Command::main();
+} else {
+    if (!is_file(dirname(__DIR__).'/vendor/symfony/phpunit-bridge/bin/simple-phpunit.php')) {
+        echo "Unable to find the `simple-phpunit.php` script in `vendor/symfony/phpunit-bridge/bin/`.\n";
+        exit(1);
+    }
+
+    require dirname(__DIR__).'/vendor/symfony/phpunit-bridge/bin/simple-phpunit.php';
+}

composer.json

@@ -17,29 +17,38 @@
     ],
     "require": {
         "php": "^7.4 || ^8.0",
-        "symfony/asset": "^4.4 || ^5.0 || ^6.0",
-        "symfony/config": "^4.4 || ^5.0 || ^6.0",
-        "symfony/dependency-injection": "^4.4 || ^5.0 || ^6.0",
-        "symfony/http-client": "^4.4 || ^5.0 || ^6.0",
-        "symfony/http-kernel": "^4.4 || ^5.0 || ^6.0",
-        "symfony/framework-bundle": "^4.4 || ^5.0 || ^6.0",
-        "symfony/twig-bundle": "^4.4 || ^5.0 || ^6.0"
+        "symfony/asset": "^4.4 || ^5.0 || ^6.0 || ^7.0",
+        "symfony/config": "^4.4 || ^5.0 || ^6.0 || ^7.0",
+        "symfony/dependency-injection": "^4.4 || ^5.0 || ^6.0 || ^7.0",
+        "symfony/http-client": "^4.4 || ^5.0 || ^6.0 || ^7.0",
+        "symfony/http-kernel": "^4.4 || ^5.0 || ^6.0 || ^7.0",
+        "symfony/framework-bundle": "^4.4 || ^5.0 || ^6.0 || ^7.0",
+        "symfony/twig-bundle": "^4.4 || ^5.0 || ^6.0 || ^7.0"
     },
     "autoload": {
         "psr-4": {
             "Pentatrion\\ViteBundle\\": "src/"
         }
     },
+    "autoload-dev": {
+        "psr-4": {
+            "Pentatrion\\ViteBundle\\Tests\\": "tests/"
+        }
+    },
     "require-dev": {
         "friendsofphp/php-cs-fixer": "^3.9",
-        "phpstan/phpstan": "^1.8"
+        "phpstan/phpstan": "^1.8",
+        "phpunit/phpunit": "^9.5",
+        "symfony/phpunit-bridge": "^4.4 || ^5.0 || ^6.0 || ^7.0",
+        "symfony/web-link": "^4.4 || ^5.0 || ^6.0 || ^7.0"
     },
     "scripts": {
-        "cs-fix": "php8.1 vendor/bin/php-cs-fixer fix --config=.php-cs-fixer.dist.php",
+        "cs-fix": "php8.2 vendor/bin/php-cs-fixer fix --config=.php-cs-fixer.dist.php",
         "ci-check": [
-            "find -L . -path ./vendor -prune -o -type f -name '*.php' -print0 | xargs -0 -n 1 -P $(nproc) php -l",
-            "php8.1 vendor/bin/php-cs-fixer fix --config=.php-cs-fixer.dist.php --diff --dry-run --stop-on-violation --using-cache=no",
-            "php8.1 vendor/bin/phpstan analyse --configuration=phpstan.neon"
+            "find -L . -path ./vendor -prune -o -type f -name '*.php' -print0 | xargs -0 -n 1 -P $(nproc) php7.4 -l",
+            "find -L . -path ./vendor -prune -o -type f -name '*.php' -print0 | xargs -0 -n 1 -P $(nproc) php8.2 -l",
+            "php8.2 vendor/bin/php-cs-fixer fix --config=.php-cs-fixer.dist.php --diff --dry-run --stop-on-violation --using-cache=no",
+            "php8.2 vendor/bin/phpstan analyse --configuration=phpstan.neon"
         ]
     }
 }