Skip to content

[pull] master from tienvx:master #96

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 124 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
124 commits
Select commit Hold shift + click to select a range
5889c35
Wait for video container
tienvx Apr 8, 2022
5f3ecfe
Fix unit tests coverage not updated
tienvx Apr 9, 2022
b4ea83b
Set video frame rate to 60
tienvx Apr 9, 2022
92cb4ef
Separate @covers and @uses
tienvx Apr 9, 2022
06210dd
Merge pull request #601 from tienvx/wait-for-video-container
tienvx Apr 9, 2022
3ddf246
Add cs step in readme
tienvx Apr 9, 2022
6c367fb
Test invalid ranges
tienvx Apr 9, 2022
e6d18db
Rename parameter to make it easier to understand
tienvx Apr 9, 2022
3a6eaf1
Test PetrinetDomainLogic
tienvx Apr 9, 2022
776e509
Test reducer isSupported()
tienvx Apr 9, 2022
520186d
Refactor plugin manager tests
tienvx Apr 9, 2022
9fe2547
Test message along with message handler
tienvx Apr 9, 2022
facfc35
Test validate revision
tienvx Apr 9, 2022
fd6bdb1
Test set place label
tienvx Apr 9, 2022
82ab298
Test set transition label
tienvx Apr 9, 2022
2eaada8
Test get single place in model's revision
tienvx Apr 9, 2022
adf5e45
Test AbstractChannel
tienvx Apr 9, 2022
35cba33
Merge pull request #602 from tienvx/increase-code-coverage
tienvx Apr 9, 2022
d9c5fc4
Test Step value object
tienvx Apr 10, 2022
0ca4e20
Test State
tienvx Apr 10, 2022
b4c2964
At least 1 place and 1 transition in model
tienvx Apr 10, 2022
da678a8
Fix cs
tienvx Apr 10, 2022
c717dca
Test Browser
tienvx Apr 10, 2022
420f1f5
Covers Model model
tienvx Apr 10, 2022
90d9cf8
Test Bug
tienvx Apr 10, 2022
04ffa8d
Test Task
tienvx Apr 10, 2022
59eaaa2
Test model value objects
tienvx Apr 10, 2022
9311832
Reuse services
tienvx Apr 10, 2022
c4044cb
Test dispatch too short steps
tienvx Apr 10, 2022
a1487f8
Change to abstract method
tienvx Apr 10, 2022
31bdf06
Merge pull request #603 from tienvx/increase-more-code-coverage
tienvx Apr 10, 2022
89d0bf0
Allow record video for bug manually
tienvx Apr 11, 2022
1490ff3
Merge pull request #604 from tienvx/record-video-for-bug-manually
tienvx Apr 11, 2022
1adae72
Capture error message on failed recording
tienvx Apr 12, 2022
85b7628
Merge pull request #605 from tienvx/capture-error-message-on-failed-r…
tienvx Apr 12, 2022
9c0b8fb
More tests for CommandPreprocessor
tienvx Apr 12, 2022
3c6c3a5
Merge pull request #606 from tienvx/more-tests-for-command-pre-processor
tienvx Apr 12, 2022
418709e
Use last step color when running bug steps
tienvx Apr 16, 2022
f028e90
Test debug model
tienvx Apr 16, 2022
24619fd
Fix service definitions
tienvx Apr 16, 2022
76802f8
Merge pull request #607 from tienvx/use-last-step-color-when-running-…
tienvx Apr 16, 2022
2b7e054
Only store different error message on recording bug
tienvx Apr 16, 2022
9c25575
Merge pull request #608 from tienvx/only-store-different-error-messag…
tienvx Apr 16, 2022
6cd4602
Fix missing constructor parameter for TaskStepsRunner
tienvx Apr 17, 2022
e991fca
Merge pull request #609 from tienvx/fix-missing-constructor-parameter
tienvx Apr 17, 2022
99c2167
Reuse start transition checking code
tienvx Apr 17, 2022
f4faea8
Merge pull request #610 from tienvx/reuse-start-transition-checking-code
tienvx Apr 17, 2022
0027d46
Set min pair length (for dispatchers)
tienvx Apr 17, 2022
554ee9f
Merge pull request #611 from tienvx/set-min-pair-length
tienvx Apr 18, 2022
e8efdee
Prevent updating steps after recording
tienvx Apr 18, 2022
be28c20
Merge pull request #612 from tienvx/prevent-updating-steps-after-reco…
tienvx Apr 18, 2022
a1438b4
Prevent updating bug's steps after running
tienvx Apr 23, 2022
acc68e4
Test clone invalid steps
tienvx Apr 23, 2022
329b312
Merge pull request #613 from tienvx/prevent-updating-bug-steps-after-…
tienvx Apr 23, 2022
0aba8c8
Fix service id
tienvx Apr 23, 2022
dc81000
Merge pull request #614 from tienvx/fix-service-id
tienvx Apr 23, 2022
f4520c8
Check can run step while reducing steps
tienvx Apr 24, 2022
fe84f38
Merge pull request #615 from tienvx/check-can-run-step-while-reducing…
tienvx Apr 24, 2022
5651459
Isolate transition expression from commands values
tienvx Apr 27, 2022
827514b
Merge pull request #616 from tienvx/isolate-transition-expression-fro…
tienvx Apr 27, 2022
016d868
Stop running task or recording bug if something wrong happen
tienvx Apr 27, 2022
1eda551
Merge pull request #617 from tienvx/stop-running-task-or-recording-bu…
tienvx Apr 27, 2022
952ed75
Fire start transition
tienvx Apr 28, 2022
d514d50
Merge pull request #618 from tienvx/fire-start-transition
tienvx Apr 28, 2022
91e1b37
Fix missing expression on import
tienvx Apr 29, 2022
e37d5e6
Merge pull request #619 from tienvx/fix-missing-expression-on-import
tienvx Apr 29, 2022
3f5731d
Revert refresh bug to store bug error message
tienvx May 2, 2022
a3d9de4
Merge pull request #620 from tienvx/revert-refresh-bug-to-store-bug-e…
tienvx May 2, 2022
d8c2bc2
Fix running commands in old places while reducing steps
tienvx May 4, 2022
ed92b9a
Merge pull request #621 from tienvx/fix-running-commands-in-old-place…
tienvx May 4, 2022
48904e4
Revert code because step's color is separated from command values
tienvx May 4, 2022
9690e85
Merge pull request #622 from tienvx/revert-code-because-step-color-is…
tienvx May 4, 2022
6088b49
Shortest path without color
tienvx May 4, 2022
831712c
Merge pull request #623 from tienvx/shortest-path-without-color
tienvx May 5, 2022
d5158e8
Don't show error can't connect steps while reducing it
tienvx May 6, 2022
f83e41d
Merge pull request #624 from tienvx/dont-show-error-cant-connect-step…
tienvx May 6, 2022
187063e
Do nothing if steps not connected during reducing steps
tienvx May 6, 2022
918b594
Merge pull request #625 from tienvx/do-nothing-if-steps-not-connected…
tienvx May 6, 2022
017a4b5
Use assignments syntax for transition's expression
tienvx Jul 12, 2022
763ca30
Fix cs with latest php cs fixer
tienvx Jul 12, 2022
06ad0b9
Use correct assignments syntax in test
tienvx Jul 12, 2022
2f3b5e4
Merge pull request #626 from tienvx/use-assignments-syntax-for-transi…
tienvx Jul 12, 2022
c73b6eb
Fix expression syntax error
tienvx Jul 13, 2022
b3175d4
Merge pull request #627 from tienvx/fix-expression-syntax-error
tienvx Jul 13, 2022
08a31a2
Override expression language
tienvx Jul 13, 2022
0cdea1e
Merge pull request #628 from tienvx/override-expression-language
tienvx Jul 13, 2022
3beed74
Upload command
tienvx Jul 14, 2022
92bacf7
Support Symfony 6
tienvx Jul 14, 2022
3a423cd
Fix return type is not covariant
tienvx Jul 14, 2022
24a7401
Limit to PHP 8.1 and use nested attributes to fix validation tests fa…
tienvx Jul 14, 2022
91fcc22
Remove Symfony 5.4
tienvx Jul 14, 2022
0d1b5aa
Fix unit tests failed with lowest assignments-evaluator-bundle version
tienvx Jul 15, 2022
f126209
Revert deleting new line
tienvx Jul 15, 2022
41e8328
Fix code coverage not uploaded
tienvx Jul 15, 2022
18d7da6
Test upload command
tienvx Jul 15, 2022
47e45b3
Test commands require target and value
tienvx Jul 15, 2022
c0a830b
Merge pull request #629 from tienvx/upload-command
tienvx Jul 15, 2022
3c89217
Assert file downloaded
tienvx Jul 17, 2022
7e7141e
Merge pull request #631 from tienvx/assert-file-downloaded
tienvx Jul 17, 2022
a3e3514
Assert clipboard
tienvx Jul 17, 2022
ad7601e
Merge pull request #632 from tienvx/assert-clipboard
tienvx Jul 17, 2022
18b4aa8
Update clipboard
tienvx Jul 17, 2022
29537b5
Merge pull request #633 from tienvx/update-clipboard
tienvx Jul 17, 2022
e06cebd
Use new bundle structure. Use new attributes from Symfony 6.1
tienvx Jul 18, 2022
bd958ee
Remove un-used variable
tienvx Jul 19, 2022
c23190b
Update changelog and readme
tienvx Jul 19, 2022
69ec5d9
Only push code coverage on push
tienvx Jul 19, 2022
73946f7
Only upload code coverage if lowest dependency
tienvx Jul 19, 2022
a0b29e1
Merge pull request #634 from tienvx/use-symfony-6.1-features
tienvx Jul 19, 2022
329a50b
Refactor commands
tienvx Jul 21, 2022
9b39302
Merge pull request #635 from tienvx/refactor-commands
tienvx Jul 24, 2022
ffe2ad5
CS for config
tienvx Jul 24, 2022
99fadfa
Merge pull request #636 from tienvx/cs-for-config
tienvx Jul 24, 2022
2f87635
Fix service not found
tienvx Jul 24, 2022
cbe7a79
Merge pull request #637 from tienvx/fix-service-not-found
tienvx Jul 24, 2022
386a182
Update commands in readme
tienvx Jul 25, 2022
99fcef1
Promote properties in constructors
tienvx Jul 26, 2022
1c2b8e0
Create new bug while reducing. Allow config default bug title
tienvx Jul 26, 2022
a04a649
Type hint mock object
tienvx Jul 26, 2022
91eb056
Merge pull request #638 from tienvx/log-new-bug
tienvx Jul 26, 2022
6c6f998
Fix missing service arguments
tienvx Jul 26, 2022
ecefec2
Merge pull request #639 from tienvx/fix-missing-service-arguments
tienvx Jul 26, 2022
71f565e
Revert auto configuration
tienvx Jul 26, 2022
e50699f
Merge pull request #640 from tienvx/revert-auto-configuration
tienvx Jul 26, 2022
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
20 changes: 11 additions & 9 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,39 +7,41 @@ jobs:
runs-on: ubuntu-latest
strategy:
matrix:
php-versions: ['8.0']
name: PHP ${{ matrix.php-versions }}
php-versions: ['8.1']
dependency-versions: ['lowest', 'highest']
name: PHP ${{ matrix.php-versions }} with ${{ matrix.dependency-versions }} versions of Composer dependencies
steps:
- name: Setup PHP
uses: shivammathur/setup-php@v2
with:
php-version: ${{ matrix.php-versions }}
coverage: pcov
tools: phpstan, phpcs, php-cs-fixer:3, phpunit, composer:v2
#extensions: mbstring, intl
tools: phpstan, phpcs, php-cs-fixer:3

- name: Checkout
uses: actions/checkout@v2

- name: Install
run: composer install
uses: "ramsey/composer-install@v2"
with:
dependency-versions: ${{ matrix.dependency-versions }}

- name: Run PHP CS
run: phpcs --standard=PSR12 src tests
run: phpcs --standard=PSR12 src tests config

- name: Run PHP CS Fixer
run: php-cs-fixer fix --diff --dry-run

- name: Run PHPStan
run: phpstan analyse src tests
run: phpstan analyse src tests config

- name: Test & Generate Code Coverage
run: phpunit
run: ./vendor/bin/phpunit

- name: Upload coverage results to Coveralls
env:
COVERALLS_REPO_TOKEN: ${{ secrets.GITHUB_TOKEN }}
run: |
composer global require php-coveralls/php-coveralls
php-coveralls --coverage_clover=clover.xml -v
if: matrix.php-versions == '7.4'
if: ${{ github.event_name == 'push' && matrix.php-versions == '8.1' && matrix.dependency-versions == 'lowest' }}
1 change: 1 addition & 0 deletions .php-cs-fixer.php
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
$finder = PhpCsFixer\Finder::create()
->in(__DIR__.'/src')
->in(__DIR__.'/tests')
->in(__DIR__.'/config')
;

$config = new PhpCsFixer\Config();
Expand Down
9 changes: 9 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,15 @@
CHANGELOG
=========

3.0.0
-----

* Only support Symfony ^6.1
* Only support PHP ^8.1
* Isolate model variables <> place variables <> transition variables on runtime
* Support transition expression, make it user friendly
* Add custom commands (upload, download, update/assert clipboard)

2.0.0
-----

Expand Down
11 changes: 6 additions & 5 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,8 @@ This Bundle is a core library for [Sicope Model][sicope-model], a Model-Based Te

## Requirements

* PHP 8.0
* Symfony 5.3
* PHP 8.1
* Symfony 6.1
* See also the `require` section of [composer.json](composer.json)

## Installation
Expand All @@ -24,8 +24,9 @@ vendor/bin/phpunit
## Validate code with coding standards

```shell
phpcs --standard=PSR12 src tests
phpstan analyse src tests
phpcs --standard=PSR12 src tests config
php-cs-fixer fix --diff --dry-run
phpstan analyse src tests config
```

## Built With
Expand Down Expand Up @@ -65,7 +66,7 @@ This package is available under the [MIT license](LICENSE).
[version-image]: https://img.shields.io/github/v/release/tienvx/mbt-bundle?include_prereleases

[php-version-url]: https://packagist.org/packages/tienvx/mbt-bundle
[php-version-image]: http://img.shields.io/badge/php-8.0.0+-ff69b4.svg
[php-version-image]: http://img.shields.io/badge/php-8.1.0+-ff69b4.svg

[contributors]: https://github.com/tienvx/mbt-bundle/graphs/contributors
[pulls]: https://github.com/tienvx/mbt-bundle/pulls
Expand Down
21 changes: 11 additions & 10 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,20 +16,21 @@
}
],
"require": {
"php": "^8.0",
"php": "^8.1",
"ext-json": "*",
"doctrine/annotations": "^1.13",
"doctrine/doctrine-bundle": "^2.6",
"doctrine/orm": "^2.8",
"florianv/petrinet": "^2.1",
"jmgq/a-star": "^2.1",
"php-webdriver/webdriver": "^1.11",
"symfony/config": "^5.4",
"symfony/dependency-injection": "^5.4",
"symfony/expression-language": "^5.4",
"symfony/http-kernel": "^5.4",
"symfony/messenger": "^5.4",
"symfony/validator": "^5.4",
"tienvx/single-color-petrinet": "^1.3"
"symfony/config": "^6.1",
"symfony/dependency-injection": "^6.1",
"symfony/expression-language": "^6.1",
"symfony/http-client": "^6.1",
"symfony/http-kernel": "^6.1",
"symfony/messenger": "^6.1",
"symfony/validator": "^6.1",
"tienvx/single-color-petrinet": "^1.5",
"tienvx/assignments-evaluator-bundle": "^1.0.1"
},
"require-dev": {
"phpunit/phpunit": "^9.5"
Expand Down
22 changes: 22 additions & 0 deletions config/definition.php
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
<?php

use Symfony\Component\Config\Definition\Configurator\DefinitionConfigurator;
use Tienvx\Bundle\MbtBundle\TienvxMbtBundle;

return static function (DefinitionConfigurator $definition) {
$definition->rootNode()
->children()
->scalarNode(TienvxMbtBundle::WEBDRIVER_URI)
->isRequired()
->cannotBeEmpty()
->defaultValue('http://localhost:4444')
->end()
->end()
->children()
->scalarNode(TienvxMbtBundle::UPLOAD_DIR)
->isRequired()
->cannotBeEmpty()
->end()
->end()
;
};
Loading